WIP

Signed-off-by: Andrew Stein <steinlink@gmail.com>

Author: texodus

.github/workflows/build.yaml

@@ -646,6 +646,7 @@ jobs:
                   path: .
 
             - name: Run Tests
+              id: run_tests
               run: pnpm run test -- --fetch-snapshots
               env:
                   PACKAGE: "server,client,viewer,viewer-datagrid,viewer-charts,viewer-openlayers,workspace,react"
@@ -654,6 +655,15 @@ jobs:
                   PSP_SNAPSHOT_REF: ${{ github.head_ref || github.ref_name }}
                   # PSP_USE_CCACHE: 1
 
+            - name: Upload Test Failures
+              if: ${{ failure() && steps.run_tests.outcome == 'failure' }}
+              uses: actions/upload-artifact@v4
+              with:
+                  name: perspective-js-test-results
+                  path: tools/test/dist/results
+                  if-no-files-found: ignore
+                  overwrite: true
+
     # ,--,--'      .   .-,--.     .  .
     # `- | ,-. ,-. |-   '|__/ . . |- |-. ,-. ,-.
     #  , | |-' `-. |    ,|    | | |  | | | | | |

packages/viewer-charts/test/js/helpers.ts

@@ -19,7 +19,7 @@ import type { ViewerConfigUpdate } from "@perspective-dev/viewer";
  * deterministic on a given machine, but a handful of sub-pixel AA
  * decisions still wiggle across Chromium versions.
  */
-const DEFAULT_MAX_DIFF_PIXEL_RATIO = 0.02;
+const DEFAULT_MAX_DIFF_PIXEL_RATIO = 0.0;
 const DEFAULT_THRESHOLD = 0;
 
 /**

tools/scripts/lint.mjs

@@ -18,13 +18,23 @@ import "zx/globals";
 
 export async function lint_js(is_fix = false) {
     const prettier_flags = is_fix ? "--write" : "--check";
-    await $`prettier ${prettier_flags} "examples/**/*.js" "examples/**/*.tsx" "tools/scripts/*.mjs" "rust/**/*.ts" "rust/**/*.js" "packages/**/*.js" "packages/**/*.ts"`.verbose();
+    await $`prettier ${prettier_flags} \
+        "examples/**/*.js" \
+        "examples/**/*.tsx" \
+        "tools/scripts/*.mjs" \
+        "rust/**/*.ts" \
+        "rust/**/*.js" \
+        "packages/**/*.js" \
+        "packages/**/*.ts" \
+        "**/css/**/*.css" \
+        "**/themes/**/*.css" \
+        "**/html/*.html" \
+        "packages/**/package.json" \
+        "rust/**/package.json" \
+        "examples/**/package.json" \
+        "docs/package.json" \
+        `.verbose();
     await $`prettier --prose-wrap=always ${prettier_flags} "rust/*/docs/**/*.md"`.verbose();
-    // cmd.sh`prettier ${prettier_flags} "**/*.yaml"`;
-    await $`prettier ${prettier_flags} "**/css/**/*.css"`.verbose();
-    await $`prettier ${prettier_flags} "**/themes/**/*.css"`.verbose();
-    await $`prettier ${prettier_flags} "**/html/*.html"`.verbose();
-    await $`prettier ${prettier_flags} "packages/**/package.json" "rust/**/package.json" "examples/**/package.json" "docs/package.json"`.verbose();
 
     const check = is_fix ? [] : ["--check"];
     const dirty = is_fix ? ["--allow-dirty"] : [];