-
Notifications
You must be signed in to change notification settings - Fork 138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update tests for Crypto
contract to the latest testing framework API
#2856
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2856 +/- ##
=======================================
Coverage 79.43% 79.43%
=======================================
Files 332 332
Lines 79111 79111
=======================================
Hits 62842 62842
Misses 13958 13958
Partials 2311 2311
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏 👏
d3a24e1
to
8037ea1
Compare
I have updated the CI workflow to also run the Cadence test file for the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, great work! 👏
It's so nice how the test functions can now directly test the code and the testing code does not have to be in separate scripts 👌
Work towards: onflow/developer-grants#216
Description
Utilizes the latest API additions from the Cadence testing API, to provide a working example of unit testing.
Note: This was tested locally with a custom
flow-cli
build.master
branchFiles changed
in the Github PR explorer