Skip to content

test: migrate to Vitest - #2280

Merged
damienbod merged 3 commits into
damienbod:mainfrom
timdeschryver:vitest
Aug 23, 2026
Merged

test: migrate to Vitest#2280
damienbod merged 3 commits into
damienbod:mainfrom
timdeschryver:vitest

Conversation

@timdeschryver

@timdeschryver timdeschryver commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sorry it got soo big, most of it include:

  • obviously, migrating to vitest including the code coverage
  • migrate to Vitest spies
  • migrate waitForAsync (deprecated) => this also means replacing subscribes to firstValueFrom
  • remove zone.js dependency
  • add import statements for the Vitest methods

@damienbod
damienbod merged commit 7366743 into damienbod:main Aug 23, 2026
5 checks passed
@damienbod

Copy link
Copy Markdown
Owner

@timdeschryver Thanks, looks great. When I run, I have 8 tests failing. Same for you?

Cheers Damien

@timdeschryver

Copy link
Copy Markdown
Contributor Author

@damienbod I noticed that the PR got merged. Did you resolve the issue?
It could be required to clear the .angular cache to resolve this.
If that's not the case, feel free to share the test cases and I'll take another look (mine are all passing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants