-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
write happy path tests for automation23 registrar registerUpkeep() #12474
Conversation
I see you updated files related to |
Go solidity wrappers are out-of-date, regenerate them via 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.
LGTM
@@ -14,7 +15,116 @@ contract SetUp is BaseTest { | |||
function setUp() public override { | |||
super.setUp(); | |||
(registry, registrar) = deployAndConfigureAll(); | |||
vm.stopPrank(); // reset identity at the start of each test |
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 catch
Go solidity wrappers are out-of-date, regenerate them via the |
aca91d8
to
97a9dfe
Compare
97a9dfe
to
2efb55c
Compare
Quality Gate passedIssues Measures |
@@ -60,6 +65,9 @@ contract BaseTest is Test { | |||
USDTOKEN_USD_FEED = new MockV3Aggregator(8, 100_000_000); // $1 | |||
FAST_GAS_FEED = new MockV3Aggregator(0, 1_000_000_000); // 1 gwei | |||
|
|||
TARGET1 = new MockUpkeep(); | |||
TARGET2 = new MockUpkeep(); |
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.
nit, TARGET2 is not used anywhere.
Code LGTM. let me know when you are ready, will stamp again.
…12474) * write happy path tests for automation23 registrar registerUpkeep() * regerate wrappers * rename UpkeepMock --> MockUpkeep
No description provided.