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.

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

Epoch manager enabler tests #11213

Merged
merged 8 commits into from
Sep 20, 2024

Conversation

soloseng
Copy link
Contributor

@soloseng soloseng commented Sep 18, 2024

Description

Added basic unit test for epochManagerEnabler contract

Other changes

removed extra native token balance check in initializeSystem of epochManager contract.

Tested

unit tested

Related issues

Backwards compatibility

Brief explanation of why these changes are/are not backwards compatible.

Documentation

The set of community facing docs that have been added/modified because of this change

@soloseng soloseng requested review from a team as code owners September 18, 2024 22:03
Copy link

gitguardian bot commented Sep 18, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@soloseng soloseng changed the base branch from master to feat/l2-epoch-system September 18, 2024 22:03
@soloseng soloseng self-assigned this Sep 20, 2024
@soloseng soloseng merged commit 032f543 into feat/l2-epoch-system Sep 20, 2024
38 checks passed
@soloseng soloseng deleted the soloseng/epoch-manager-enabler-test branch September 20, 2024 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants