-
Notifications
You must be signed in to change notification settings - Fork 14
Migrate 400_contributeAndCallback.js test file to Hardhat and Typescript #176
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
Migrate 400_contributeAndCallback.js test file to Hardhat and Typescript #176
Conversation
test/byContract/IexecPoco/IexecPoco2-contribute-and-finalize.test.ts
Outdated
Show resolved
Hide resolved
).to.be.revertedWithoutReason(); // require#2 | ||
}); | ||
|
||
it('Should not contributeAndFinalize when someone else has already contributed', async () => { |
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.
note for later this would require a trust > 1, but that's also the require #4
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #176 +/- ##
========================================
Coverage 84.87% 84.87%
========================================
Files 35 35
Lines 1084 1084
Branches 221 221
========================================
Hits 920 920
Misses 164 164 ☔ 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.
Good thanks!
test/byContract/IexecPoco/IexecPoco2-contribute-and-finalize.test.ts
Outdated
Show resolved
Hide resolved
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.
Great, thank you!
No description provided.