Skip to content

Commit

Permalink
chore(cleanup): remove legacy visual regression tests (#19527)
Browse files Browse the repository at this point in the history
  • Loading branch information
thmsobrmlr authored Jan 9, 2024
1 parent 5f0aee8 commit fec25c9
Show file tree
Hide file tree
Showing 14 changed files with 7 additions and 247 deletions.
10 changes: 1 addition & 9 deletions .github/workflows/storybook-chromatic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -146,15 +146,7 @@ jobs:
VARIANT: ${{ github.event.pull_request.head.repo.full_name == github.repository && 'update' || 'verify' }}
STORYBOOK_SKIP_TAGS: 'test-skip,test-skip-${{ matrix.browser }}'
run: |
pnpm test:visual-regression:stories:ci:$VARIANT --browsers ${{ matrix.browser }} --shard ${{ matrix.shard }}/$SHARD_COUNT
- name: Run @playwright/test (legacy, Chromium-only)
if: matrix.browser == 'chromium' && matrix.shard == 1
env:
# Update snapshots for PRs on the main repo, verify on forks, which don't have access to PostHog Bot
VARIANT: ${{ github.event.pull_request.head.repo.full_name == github.repository && 'update' || 'verify' }}
run: |
pnpm test:visual-regression:legacy:ci:$VARIANT
pnpm test:visual-regression:ci:$VARIANT --browsers ${{ matrix.browser }} --shard ${{ matrix.shard }}/$SHARD_COUNT
- name: Archive failure screenshots
if: ${{ failure() }}
Expand Down
16 changes: 6 additions & 10 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,16 +21,12 @@
"test": "pnpm test:unit && pnpm test:visual-regression",
"test:unit": "jest --testPathPattern=frontend/",
"jest": "jest",
"test:visual-regression": "docker compose -f docker-compose.playwright.yml run --rm -it --build playwright pnpm test:visual-regression:legacy:docker && pnpm test:visual-regression:stories:docker",
"test:visual-regression:legacy": "docker compose -f docker-compose.playwright.yml run --rm -it --build playwright pnpm test:visual-regression:legacy:docker",
"test:visual-regression:legacy:docker": "STORYBOOK_URL=http://host.docker.internal:6006 playwright test -u",
"test:visual-regression:legacy:ci:update": "playwright test -u",
"test:visual-regression:legacy:ci:verify": "playwright test",
"test:visual-regression:stories": "rm -rf frontend/__snapshots__/__failures__/ && docker compose -f docker-compose.playwright.yml run --rm -it --build playwright pnpm test:visual-regression:stories:docker",
"test:visual-regression:stories:docker": "NODE_OPTIONS=--max-old-space-size=6144 test-storybook -u --no-index-json --browsers chromium webkit --url http://host.docker.internal:6006",
"test:visual-regression:stories:local": "NODE_OPTIONS=--max-old-space-size=6144 test-storybook -u --no-index-json --browsers chromium webkit --url http://localhost:6006",
"test:visual-regression:stories:ci:update": "test-storybook -u --no-index-json --maxWorkers=2",
"test:visual-regression:stories:ci:verify": "test-storybook --ci --no-index-json --maxWorkers=2",
"test:visual-regression": "pnpm test:visual-regression:docker",
"test:visual-regression": "rm -rf frontend/__snapshots__/__failures__/ && docker compose -f docker-compose.playwright.yml run --rm -it --build playwright pnpm test:visual-regression:docker",
"test:visual-regression:docker": "NODE_OPTIONS=--max-old-space-size=6144 test-storybook -u --no-index-json --browsers chromium webkit --url http://host.docker.internal:6006",
"test:visual-regression:local": "NODE_OPTIONS=--max-old-space-size=6144 test-storybook -u --no-index-json --browsers chromium webkit --url http://localhost:6006",
"test:visual-regression:ci:update": "test-storybook -u --no-index-json --maxWorkers=2",
"test:visual-regression:ci:verify": "test-storybook --ci --no-index-json --maxWorkers=2",
"start": "concurrently -n ESBUILD,TYPEGEN -c yellow,green \"pnpm start-http\" \"pnpm run typegen:watch\"",
"start-http": "pnpm clean && pnpm copy-scripts && node frontend/build.mjs --dev",
"start-docker": "pnpm start-http --host 0.0.0.0",
Expand Down
66 changes: 0 additions & 66 deletions playwright/e2e-vrt/README.md

This file was deleted.

17 changes: 0 additions & 17 deletions playwright/e2e-vrt/lemon-ui/LemonButton.spec.ts

This file was deleted.

Binary file not shown.
12 changes: 0 additions & 12 deletions playwright/e2e-vrt/scenes-app/PersonsModal.spec.ts

This file was deleted.

Binary file not shown.
Binary file not shown.
33 changes: 0 additions & 33 deletions playwright/e2e-vrt/scenes-app/insights.spec.ts

This file was deleted.

Binary file not shown.
Binary file not shown.
16 changes: 0 additions & 16 deletions playwright/fixtures/storybook.ts

This file was deleted.

12 changes: 0 additions & 12 deletions playwright/helpers/storybook.ts

This file was deleted.

72 changes: 0 additions & 72 deletions playwright/pages/storybook.ts

This file was deleted.

0 comments on commit fec25c9

Please sign in to comment.