Skip to content

Commit

Permalink
Merge pull request #102 from lidofinance/fix/sealable-calls-resume
Browse files Browse the repository at this point in the history
fix: sealable calls resume return value fix
  • Loading branch information
bulbozaur authored Aug 20, 2024
2 parents 7e1094c + 3c3faa8 commit 39283d8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion contracts/libraries/SealableCalls.sol
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ library SealableCalls {
function callResume(ISealable sealable) internal returns (bool success, bytes memory lowLevelError) {
try sealable.resume() {
(bool isPausedCallSuccess, bytes memory isPausedLowLevelError, bool isPaused) = callIsPaused(sealable);
success = isPausedCallSuccess && isPaused;
success = isPausedCallSuccess && !isPaused;
lowLevelError = isPausedLowLevelError;
} catch (bytes memory resumeLowLevelError) {
success = false;
Expand Down
2 changes: 1 addition & 1 deletion test/unit/libraries/SealableCalls.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ contract SealableCallsUnitTests is UnitTest {

(bool success, bytes memory lowLevelError) = SealableCalls.callResume(_sealableMock);

assertFalse(success);
assertTrue(success);
assertEq(lowLevelError.length, 0);

(bool isPausedSuccess,, bool isPaused) = SealableCalls.callIsPaused(_sealableMock);
Expand Down

0 comments on commit 39283d8

Please sign in to comment.