zkapp-cli
zkapp-cli copied to clipboard
Additional unit tests and coverage.
Closes #645 Closes #650
Codecovproject can be found here: https://app.codecov.io/gh/o1-labs/zkapp-cli- From now on we will run other CI jobs on the success of
Unittests only.- This will allow us to fail faster and save resources.
- CI dependencies were refactored.
- Publishing on NPM will now be triggered on the success of
UnitandSmoketests. - CI dependencies:
Common unit tests←Examples unit tests←Smoke E2E tests←Publish to NPM.
- Publishing on NPM will now be triggered on the success of
- From now on
Jestwill fail theUnittests execution in case if resultingcoveragewill be less thanconfigured threshold.- This will force us to write additional
Unittests onmaincodebase changes.
- This will force us to write additional
- Do we want the
Unittests to be configured as mandatory pre-commit hook (alongside with the currently configured linter)?
Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:
Note for assignee: address TODO in CI config: https://github.com/o1-labs/zkapp-cli/pull/662.
UPD: Done.
https://app.codecov.io/gh/o1-labs/zkapp-cli/tree/feat%2Funit-tests
Smoke testing run against this branch: https://github.com/o1-labs/zkapp-cli/actions/runs/9970821097 Please refer to PR's description for the information about why it is not picked up.