cosmjs icon indicating copy to clipboard operation
cosmjs copied to clipboard
verified publisher icon cosmos

Migrating off Jasmine

Open dynst opened this issue 6 months ago • 3 comments

Completing what #1809 started. Probably replace jasmine with vitest in #1803

To do:

  • Remove any uses of the fail() global like

https://github.com/cosmos/cosmjs/blob/8fb7421c573252f794c6de830c86c2aca20a6bb6/packages/tendermint-rpc/src/tendermint34/tendermint34client.spec.ts#L701

  • And replace the numerous pending() calls in stargate and cosmwasm-stargate with xdescribe()

https://github.com/cosmos/cosmjs/blob/8fb7421c573252f794c6de830c86c2aca20a6bb6/packages/cosmwasm-stargate/src/cosmwasmclient.spec.ts#L45

Related: #1688

dynst avatar Oct 08 '25 23:10 dynst

What's the motivation for migrating away from Jasmine? Looks like a significant endeavor with unclear reasoning right now.

webmaster128 avatar Oct 15 '25 09:10 webmaster128

Was looking at replacing karma #1688 and Vitest looked like the way to go, it's one simple tool for running all of the tests, in node and in browser.

It also doesn't require a separate TS-to-JS transpile step, it runs *.spec.ts files directly.

dynst avatar Oct 15 '25 18:10 dynst

Ah, nice, thanks, understood. Sounds good.

webmaster128 avatar Oct 16 '25 07:10 webmaster128