strudel icon indicating copy to clipboard operation
strudel copied to clipboard

Published 20 hours ago • verified publisher icon tidalcycles

replace mocha with vitest

Open felixroos opened this issue 3 years ago • 3 comments

  • refactored all the tests to vitest syntax
  • tune test not yet refactored
  • blocked by https://github.com/vitest-dev/vitest/issues/1807

felixroos avatar Aug 06 '22 10:08 felixroos

fixes https://github.com/tidalcycles/strudel/issues/145

felixroos avatar Aug 06 '22 10:08 felixroos

waiting for [email protected] https://www.npmjs.com/package/vitest

felixroos avatar Aug 06 '22 21:08 felixroos

i have now refactored all tests to vitest + now using the snapshot feature instead of the custom solution. unfortunately, it's actually slower than mocha by ~50% so I am a bit disappointed.. it seems vitest has had a performance drop lately, so hopefully this will get better soon

felixroos avatar Aug 09 '22 22:08 felixroos

i think it's still better, because the dev mode seems much quicker + the snapshot feature alone will be of great use. also, if we decide to move to typescript later, this will automatically work. I'll merge this

felixroos avatar Aug 12 '22 21:08 felixroos