Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test: RegistryCoordinator unit tests #91

Closed
4 tasks done
Tracked by #22
8sunyuan opened this issue Dec 5, 2023 · 3 comments
Closed
4 tasks done
Tracked by #22

Test: RegistryCoordinator unit tests #91

8sunyuan opened this issue Dec 5, 2023 · 3 comments
Assignees
@ChaoticWalrus

Comments

@8sunyuan
Copy link
Collaborator

8sunyuan commented Dec 5, 2023 •
edited by ChaoticWalrus
Loading

Description

Lot of refactoring with this contract in particular, could also implement these tests after changes involving removal of the PubkeyCompendium and new operator registration flow. Mock contracts for the registry contracts should also be setup for testing functions here such as _registerOperator, _deregisterOperator, updateOperators

Action Items
@8sunyuan 8sunyuan mentioned this issue Dec 5, 2023
Closed
@8sunyuan 8sunyuan self-assigned this Dec 5, 2023
@8sunyuan 8sunyuan changed the title Test: BLSRegistryCoordinatorWithIndices unit tests Test: RegistryCoordinator unit tests Dec 13, 2023
@8sunyuan 8sunyuan removed their assignment Dec 19, 2023
@ChaoticWalrus ChaoticWalrus self-assigned this Dec 20, 2023
@ChaoticWalrus
Copy link
Contributor

starting some work on this. putting together a draft tree diagram first

@ChaoticWalrus
Copy link
Contributor

PR is here (currently marked as a draft): #121

@ChaoticWalrus
Copy link
Contributor

The relevant PR was merged!
Closing this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChaoticWalrus ChaoticWalrus

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@8sunyuan @ChaoticWalrus