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

[SC-288] Add pause support and external authorized contract #5

Merged
merged 10 commits into from
Dec 6, 2023

Conversation

hexonaut
Copy link
Contributor

@hexonaut hexonaut commented Dec 2, 2023

Add the ability to pause/unpause + unfreeze as per requirements change. Also allow external contracts to do so (for multisig use case).

@hexonaut hexonaut marked this pull request as ready for review December 2, 2023 16:30
src/interfaces/ISparkLendFreezerMom.sol Outdated Show resolved Hide resolved
src/interfaces/ISparkLendFreezerMom.sol Outdated Show resolved Hide resolved
src/interfaces/ISparkLendFreezerMom.sol Outdated Show resolved Hide resolved
src/SparkLendFreezerMom.sol Show resolved Hide resolved
test/IntegrationTests.t.sol Show resolved Hide resolved
test/IntegrationTests.t.sol Outdated Show resolved Hide resolved
test/IntegrationTests.t.sol Show resolved Hide resolved
test/IntegrationTests.t.sol Show resolved Hide resolved
test/IntegrationTests.t.sol Show resolved Hide resolved
test/IntegrationTests.t.sol Outdated Show resolved Hide resolved
Copy link
Contributor

@lucas-manuel lucas-manuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice tests

test/IntegrationTests.t.sol Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Dec 6, 2023

Coverage after merging SC-288-add-pause into master will be

100.00%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   SparkLendFreezerMom.sol100%100%100%100%
src/spells
   EmergencySpell_SparkLend_FreezeAllAssets.sol100%100%100%100%
   EmergencySpell_SparkLend_FreezeSingleAsset.sol100%100%100%100%
   EmergencySpell_SparkLend_PauseAllAssets.sol100%100%100%100%
   EmergencySpell_SparkLend_PauseSingleAsset.sol100%100%100%100%

@hexonaut hexonaut merged commit 1c480e1 into master Dec 6, 2023
3 checks passed
@hexonaut hexonaut deleted the SC-288-add-pause branch December 6, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants