-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: remove maci-start-block and modify related codes #121
feat: remove maci-start-block and modify related codes #121
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Using the service, assumes that we won't deploy MACI in another way right? As we removed the MACI Address |
Yes, I think so, but the user could also input existing maci address and register it, this is an opened issue: #112 |
d76d986
to
3533ec4
Compare
3533ec4
to
8bcc349
Compare
7c8a362
to
af5f972
Compare
8bcc349
to
768784b
Compare
768784b
to
46d9f4f
Compare
46d9f4f
to
788e5de
Compare
expect(maciAddress).toBe(ZeroAddress); | ||
const maciData = await service.getMaciData(); | ||
expect(maciData.address).toBe(ZeroAddress); | ||
expect(maciData.startBlock).toBe(0); |
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.
Actually not sure how to test this case, since the outputs are all mocked? or we just wanna make sure it does call those functions?
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.
You can mock it with different data and test it. For instance, if there is no provider or transaction hash.
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.
@kittybest thanks, just left some comments
expect(maciAddress).toBe(ZeroAddress); | ||
const maciData = await service.getMaciData(); | ||
expect(maciData.address).toBe(ZeroAddress); | ||
expect(maciData.startBlock).toBe(0); |
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.
You can mock it with different data and test it. For instance, if there is no provider or transaction hash.
788e5de
to
ef198f6
Compare
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.
@kittybest thanks!
ef198f6
to
f35c145
Compare
Description
NEXT_PUBIC_MACI_START_BLOCK
in environment variables and config filesMaciService.getMaciData
would return{ maciAddress, maciStartBlock }
Related Issue
close issue #115