Merge pull request #864 from tradingstrategy-ai/troubleshoot-build

Troubleshoot build
tradingstrategy-ai · Dec 16, 2024 · 7572f70 · 7572f70
2 parents 51ba4b4 + 6bcc6fb
commit 7572f70
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/.github/workflows/javascript.yml b/.github/workflows/javascript.yml
@@ -56,6 +56,16 @@ jobs:
       # skip build in test step
       - name: Run Tests
         run: npm run test --skip-build
+      - name: Upload Test Results
+        if: failure()
+        uses: actions/upload-artifact@v4
+        with:
+          name: playwright-test-results
+          path: |
+            test-results/
+            tests/integration/**/*-snapshots/
+          retention-days: 7
+          compression-level: 0 # Skip compression since PNGs are already compressed
 
   build:
     runs-on: ubuntu-latest

diff --git a/tests/integration/index.test.ts b/tests/integration/index.test.ts
@@ -10,7 +10,7 @@ test.describe('home page', () => {
 	// $ ./scripts/update-test-screenshots.sh
 	test('hero banner looks correct', async ({ page }) => {
 		const header = page.getByTestId('home-hero-banner');
-		await expect(header).toHaveScreenshot();
+		await expect(header).toHaveScreenshot({ maxDiffPixels: 150 });
 	});
 
 	test('home page has impressive numbers', async ({ page }) => {