-
Notifications
You must be signed in to change notification settings - Fork 2.2k
chore(test): vitest migration #9969
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+19,760
−7,285
Merged
Changes from 17 commits
Commits
Show all changes
146 commits
Select commit
Hold shift + click to select a range
9109505
RFC: Migrate from tape to vitest
chrisgervang 659c49f
RFC: Add multi-environment architecture and expanded implementation plan
chrisgervang ee46a47
RFC: Add @deck.gl/test-utils deprecation timeline
chrisgervang 04b8519
chore(test): add vitest infrastructure
chrisgervang 718a052
chore(test): add tape-to-vitest migration script
chrisgervang 4949e46
chore(test): convert test files from tape to vitest
chrisgervang d632932
chore(test): add TODO for typed array equality tester removal
chrisgervang 541f0c1
chore(test): remove obsolete gpu-grid-layer test
chrisgervang 3b187e2
chore(test): convert spy.called to vitest toHaveBeenCalled()
chrisgervang fd05149
chore: ignore yarn engine checks for vitest dependencies
chrisgervang 8c1954a
chore(test): remove unused describe imports from vitest conversions
chrisgervang 47e1d61
chore: ignore vite.config.local.mjs in eslint
chrisgervang 14d3c7c
chore(test): convert t.pass() to console.log() in vitest migration
chrisgervang f14ccae
chore(test): handle test.skip and test.only in vitest migration
chrisgervang 9e75ff6
fix(test): preserve assertion message in assert callback conversion
chrisgervang c2955bb
chore(test): handle expression body arrow functions in migration
chrisgervang 2f77513
fix(ci): install Playwright browsers and use headless project
chrisgervang c8eef73
fix(test): exclude tests that were not in original CI suite
chrisgervang cfffba1
chore(test): restructure test commands with vitest projects
chrisgervang b0280a0
docs(rfc): update vitest migration RFC with implementation details
chrisgervang 84ff56c
fix(test): convert makeSpy to vi.spyOn for browser compatibility
chrisgervang 4a35838
fix(test): convert spy.restore/reset to mockRestore/mockReset
chrisgervang f2402ce
fix(test): convert spy.called to toHaveBeenCalled matcher
chrisgervang e95ae27
fix(test): handle nested t.test() conversion in migration script
chrisgervang d616d74
fix(test): address diff comment issues
chrisgervang cc69f73
chore(test): add manual fix skip list to migration script
chrisgervang 6161023
fix(test): convert spy.callCount to toHaveBeenCalledTimes
chrisgervang fd9930c
fix(test): pass message to expect assertions for better error messages
chrisgervang 8fce5f5
fix(test): convert spy.callCount to toHaveBeenCalledTimes in terrain …
chrisgervang 4766b53
chore(test): exclude tests that were commented out on master
chrisgervang 7054272
feat(test): add browser commands for vitest test runner
chrisgervang a9edc69
feat(test): add interaction and render test specs for vitest
chrisgervang b1e947f
docs: update CONTRIBUTING and RFC with vitest migration progress
chrisgervang a4ffad0
chore(deps): add image comparison dependencies for vitest
chrisgervang facc654
fix(scripts): remove unused fullCall variable in migration script
chrisgervang 354485a
fix(test): add await detection for testLayerAsync and exclude failing…
chrisgervang a14cd26
fix(test): rename jupyter-widget utility file to not have .spec suffix
chrisgervang ed053cc
fix(test-utils): use mockRestore instead of mockClear for spy cleanup
chrisgervang cea4d17
chore(test): reduce verbose logging and use TAP for CI
chrisgervang 8bcb3ee
RFC: Migrate from tape to vitest
chrisgervang d44d9d5
RFC: Add multi-environment architecture and expanded implementation plan
chrisgervang 237a9f6
RFC: Add @deck.gl/test-utils deprecation timeline
chrisgervang 7750fed
RFC: Update CLI commands and finalize Phase 4 discovery outcome
chrisgervang a30500c
fix(test-utils): enable es2022 for top-level await support
chrisgervang 5e4683f
chore: merge chr/tape-to-vitest into chr/vitest-setup
chrisgervang d5a9bbf
feat(test-utils): add backward compatibility for tape/probe.gl users
chrisgervang c711bb8
fix(vitest): pre-bundle dependencies to prevent CI flakiness
chrisgervang 6c04566
ci: split test steps for better visibility
chrisgervang ad34130
ci: use npx for vitest commands
chrisgervang 16b3e9d
fix(test-utils): make spy framework initialization lazy
chrisgervang 305668e
fix(test-utils): enable test-utils import in Node with NullDevice fal…
chrisgervang beb5176
feat(test-utils): add Injectable Spy API for framework-agnostic testing
chrisgervang ae7e43f
RFC: Migrate from tape to vitest
chrisgervang b4d166d
RFC: Add multi-environment architecture and expanded implementation plan
chrisgervang 8c8ed22
RFC: Add @deck.gl/test-utils deprecation timeline
chrisgervang ec43806
RFC: Update CLI commands and finalize Phase 4 discovery outcome
chrisgervang bb7a6a5
RFC: Add injectable spy API, Phase 5 outcome, and Phase 7
chrisgervang 5443914
Merge branch 'origin/chr/tape-to-vitest' into chr/vitest-setup
chrisgervang e8aa70c
chore(test): add createSpy to all testLayer calls (migration script o…
chrisgervang 2ab06f0
chore(test): separate render tests into dedicated vitest project
chrisgervang a6acf0c
chore(test): fix test-render script and add separate entry points
chrisgervang 4f7be42
chore(test-utils): add vi.spyOn default to vitest entry point
chrisgervang 1f8ff88
chore(test): update migration script to use @deck.gl/test-utils/vitest
chrisgervang 5002217
chore(test): migrate unit tests to @deck.gl/test-utils/vitest
chrisgervang 329e6cc
chore(test): update core-layers.node.spec.ts to use vitest entry
chrisgervang 8250c0f
chore(test): remove old tape/probe.gl test entry points
chrisgervang f067e09
chore(test): convert interaction tests to native vitest browser mode
chrisgervang de68ae4
chore(test): include interaction tests in headless project
chrisgervang 875b0c4
chore(test): convert render tests to native vitest with test.each
chrisgervang c406bc3
chore(test): fix render test layer loading and use tap reporter
chrisgervang 0eb3d30
chore(test): add views, effects, and useDevicePixels support to rende…
chrisgervang c84c896
chore(test): fix render test timeouts with absolute URLs and new Deck…
chrisgervang 959390b
chore(test): add skip support for render tests and skip timeout tests
chrisgervang 4b88f59
RFC: Migrate from tape to vitest
chrisgervang 16437f1
RFC: Add multi-environment architecture and expanded implementation plan
chrisgervang 4579dfb
RFC: Add @deck.gl/test-utils deprecation timeline
chrisgervang f860847
RFC: Update CLI commands and finalize Phase 4 discovery outcome
chrisgervang 7119e0f
RFC: Add injectable spy API, Phase 5 outcome, and Phase 7
chrisgervang 0a646af
Merge remote-tracking branch 'origin/chr/tape-to-vitest' into chr/vit…
chrisgervang f8a9a0b
fix(test): fix collision-filter-effect test expectation
chrisgervang d127bda
fix(test-utils): make @probe.gl/test-utils a required peer dependency
chrisgervang ac1293a
fix(test): use mockClear instead of mockReset for spy.reset conversion
chrisgervang 119252b
chore(test): use @deck.gl/test-utils/vitest entry point for all spec …
chrisgervang 0c3fa68
fix(test): correctly convert t.throws regex matchers to toThrow()
chrisgervang 972be68
chore(test): add tap-spec formatter for prettier test output
chrisgervang 2b7bf5e
chore(ci): upload render test failure images as artifacts
chrisgervang 394fe75
chore(test): update golden images for mvt-layer and post-process-effects
chrisgervang 6becbcd
RFC: Migrate from tape to vitest
chrisgervang d2f2f23
RFC: Add multi-environment architecture and expanded implementation plan
chrisgervang 8a60964
RFC: Add @deck.gl/test-utils deprecation timeline
chrisgervang c8e9bd9
RFC: Update CLI commands and finalize Phase 4 discovery outcome
chrisgervang ed2dece
RFC: Add injectable spy API, Phase 5 outcome, and Phase 7
chrisgervang a999c4b
Merge branch 'chr/tape-to-vitest' into chr/vitest-setup
chrisgervang 2763739
chore(test): restore coverage TODO and run migration script
chrisgervang 754737e
fix(test): add dblclick support to browser emulateInput command
chrisgervang 907474f
chore(test): replace tap-spec with vitest default reporter
chrisgervang 6b9de2f
fix(test): improve modifier key handling for dblclick events
chrisgervang 33ec6d6
RFC: Migrate from tape to vitest
chrisgervang 7437439
RFC: Add multi-environment architecture and expanded implementation plan
chrisgervang 1121bc5
RFC: Add @deck.gl/test-utils deprecation timeline
chrisgervang 1bcda33
RFC: Update CLI commands and finalize Phase 4 discovery outcome
chrisgervang d2c9f78
RFC: Add injectable spy API, Phase 5 outcome, and Phase 7
chrisgervang b2970ad
refactor(test): split render tests into individual spec files
chrisgervang 3ab80db
fix(test): fix timeline render tests by reusing Deck instance
chrisgervang fe58fe9
fix(test): unskip scatterplot-transition-3 and transition-4 tests
chrisgervang 7f083a7
revert(test): restore original arc-lnglat and gridcell-lnglat golden …
chrisgervang d81014e
Merge branch 'origin/chr/tape-to-vitest' into chr/vitest-setup
chrisgervang 8619d9f
chore: remove --ignore-engines and add playwright postinstall
chrisgervang 39e5275
feat(test): upgrade vitest from 2.x to 4.x
chrisgervang 3cc0944
fix(test): disable isolation and file parallelism for browser tests
chrisgervang 5fdc63a
fix(test): configure viewport via playwright contextOptions for vites…
chrisgervang fb0c44a
fix(test): exclude flaky tests that fail with isolate: false in CI
chrisgervang 411746c
chore(ci): simplify CI workflow and re-enable coverage
chrisgervang df2f530
fix(test-utils): update vitest peer dependency to ^4.0.18
chrisgervang 5a537d4
fix(test-utils): mark @probe.gl/test-utils peer dependency as optional
chrisgervang 66d9959
feat(scripts): add jscodeshift-based tape-to-vitest codemod
chrisgervang ec78fd2
fix(scripts): improve tape-to-vitest codemod transform
chrisgervang a752446
fix(test): correctly pass expected error to toThrow() assertions
chrisgervang 9f43b37
docs(test): add comprehensive TEST-STATUS.md and fix test scripts
chrisgervang 35a33c7
fix(scripts): detect 3-arg t.ok pattern as equality assertion
chrisgervang 03b09b9
fix(test): re-enable mvt-layer tests after await fix
chrisgervang 4bd8dea
docs(test): update TEST-STATUS.md after mvt-layer fix
chrisgervang 02d1726
fix(test-utils): require @probe.gl/test-utils peer dependency
chrisgervang 82f9dcb
fix(test-utils): add @probe.gl/test-utils to root devDependencies
chrisgervang 05614bf
refactor(test): DRY up screenshot logic in deck-test-utils
chrisgervang 51b411d
chore(test-utils): remove completed TODO comment
chrisgervang 701afe0
Merge branch 'master' into chr/vitest-setup
chrisgervang fe8ea26
refactor(test-utils): Add resetSpy callback and extract shared utilities
chrisgervang ce9016e
fix(test-utils): Remove unused imports from setup-gl.ts
chrisgervang 9584d2e
fix(test-utils): Add async cleanup to prevent luma.gl unhandled rejec…
chrisgervang 0eb0a50
fix(test): Re-enable carto-raster-tile tests and document excluded tests
chrisgervang 93bde3e
fix(test): Exclude render tests from browser project and fix vitest d…
chrisgervang 3131d3a
fix(codemod): Use vitest entry point for utility file fixture
chrisgervang e70e2a3
refactor(test): Use import.meta.dirname for modern Node.js
chrisgervang 79bc238
chore: Merge master into chr/vitest-setup
chrisgervang e2add2c
Merge branch 'master' into chr/vitest-setup
ibgreen 01efe7c
rebase-cleanup
ibgreen c6d7b1b
fixes
ibgreen 7eb0a2f
tape-compat
ibgreen b5eb8bd
Enabled skipped test cases and files
ibgreen 6eccf27
partial-test-restore
ibgreen f5fb11f
fixes
ibgreen b03f384
fixes
ibgreen 8535d8d
wip
ibgreen 632f5a7
fix
ibgreen 2625a74
disable
ibgreen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| --ignore-engines true | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Engine checks disabled for all installs
Medium Severity
Adding
--ignore-engines truedisables dependency engine validation globally. This allows unsupported Node versions to installvitestand related tooling, then fail later at runtime instead of failing fast during install, which makes setup and CI/local behavior inconsistent and harder to diagnose.