Closed hl662 closed 1 day ago
This pull request is now in conflicts. Could you fix it @hl662? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/
BTW, just pulled and tried out vitest, and the multithreading is so nice. All core-geometry tests done in 50s. Wow.
- @dassaf4
just in case anyone comes back to this pr, dont want you to miss this comment :)
Continuing off of the work from #6890
For
core-bentley
,core-geometry
,core-common
:@vitest/coverage-istanbul
(I'm open to switching to node's native code coveragev8
too, no opinions on either)chai
,chai-as-promised
,mocha
,nyc
dependenciesOneAtATime tests in core-bentley are skipped for now, haven't finished refactoring them.
Switching the integration tests to vitest is out of the scope of this PR, so just looking at the unit tests within each sub repo instead. We'll investigate this more after.
A couple more issues to circle back to later on as a result of this: #7200 #7201