Skip to content

chore: migrate from jest to vitest#983

Open
luipe wants to merge 4 commits intomainfrom
migrate-to-vitest
Open

chore: migrate from jest to vitest#983
luipe wants to merge 4 commits intomainfrom
migrate-to-vitest

Conversation

@luipe
Copy link
Copy Markdown
Contributor

@luipe luipe commented Aug 22, 2025

Test Execution Times (locally on my machine)

  • for this branch: 73.91s
  • for main: 117.541s

@luipe luipe force-pushed the migrate-to-vitest branch 2 times, most recently from 37a4c2b to 632cac8 Compare August 22, 2025 15:05
@NiklasEi
Copy link
Copy Markdown
Member

This is a good idea, I much prefer vitest over jest. 👍
Then we can also finally get rid of #957 and #963.

@NiklasEi
Copy link
Copy Markdown
Member

While we are moving to a more modern tool chain, we could see if we can replace prettier and eslint with biome.

@weissu42
Copy link
Copy Markdown
Member

weissu42 commented Sep 5, 2025

Nice initiative. Regarding test execution times, maybe we can also have a look again at the "improve-tests" branch.
main...improve-tests
I started looking into reducing the "sleep" times of our tests. Never finished that one, though

Signed-off-by: Luisa Peter <luisa.peter@tngtech.com>
@luipe luipe force-pushed the migrate-to-vitest branch from 632cac8 to 99791e1 Compare April 24, 2026 16:28
weissu42 and others added 2 commits April 24, 2026 18:38
…ecution by over 20% on my machine

Signed-off-by: Ute Weiß <ute.weiss@tngtech.com>
Signed-off-by: Luisa Peter <luisa.peter@tngtech.com>
Signed-off-by: Luisa Peter <luisa.peter@tngtech.com>
@luipe luipe force-pushed the migrate-to-vitest branch from c976a99 to 62f41d6 Compare April 24, 2026 17:34
@luipe
Copy link
Copy Markdown
Contributor Author

luipe commented Apr 24, 2026

@weissu42 I included your commit for test improvements in this PR.

"long running interval jobs does not start a long running job twice that should not run in parallel" had a race condition due to the duration being a multiple of the interval.

Signed-off-by: Luisa Peter <luisa.peter@tngtech.com>
@luipe
Copy link
Copy Markdown
Contributor Author

luipe commented Apr 24, 2026

Adresses #937
Final result for running all tests on my machine is 48s. A flaky test was fixed as well.

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.

3 participants