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
chore(smart-contracts): cleanup tests from remains of Truffle #13365
Conversation
Ok, this is a big PR, so hard to review! But since this is mostly refactors, we can just check that the same amount of tests are running. This is the output on a successful recent build on
|
29d657b
to
17b254a
Compare
Currently we got 939 passing (8m) Before refactoring:
|
97eca88
to
acf8b21
Compare
@julien51 we should be good to go here ! |
ok, I am looking at the coverage also,
I have been trying to run this locally from
|
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.
Let's do it!
beforeEach(async () => { | ||
;[, , , { address: someAccount }] = await ethers.getSigners() |
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.
If we don't really care what that is maybe we can rather just create a random
one?
I was in fact able to run it after cleaning
|
Description
There are still places in the tests where we use patterns like
artifacts.require
that originated in using web3/Truffle some years ago. This PR removes them.Issues
Refs #12843
Checklist:
Release Note Draft Snippet