✅ Better ReentrancyGuard test

transmissions11 · May 31, 2022 · eaaccf8 · eaaccf8
1 parent 1681dc5
commit eaaccf8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.gas-snapshot b/.gas-snapshot
@@ -155,9 +155,9 @@ MultiRolesAuthorityTest:testSetTargetCustomAuthority() (gas: 27976)
 OwnedTest:testCallFunctionAsNonOwner() (gas: 11311)
 OwnedTest:testCallFunctionAsOwner() (gas: 10479)
 OwnedTest:testSetOwner() (gas: 13035)
-ReentrancyGuardTest:testFailUnprotectedCall() (gas: 45665)
+ReentrancyGuardTest:testFailUnprotectedCall() (gas: 46167)
 ReentrancyGuardTest:testNoReentrancy() (gas: 7515)
-ReentrancyGuardTest:testProtectedCall() (gas: 32949)
+ReentrancyGuardTest:testProtectedCall() (gas: 33470)
 RolesAuthorityTest:testCanCallPublicCapability() (gas: 33336)
 RolesAuthorityTest:testCanCallWithAuthorizedRole() (gas: 79601)
 RolesAuthorityTest:testSetPublicCapabilities() (gas: 29183)

diff --git a/src/test/ReentrancyGuard.t.sol b/src/test/ReentrancyGuard.t.sol
@@ -13,15 +13,15 @@ contract RiskyContract is ReentrancyGuard {
 
         if (enterTimes > 1) return;
 
-        protectedCall();
+        this.protectedCall();
     }
 
     function protectedCall() public nonReentrant {
         enterTimes++;
 
         if (enterTimes > 1) return;
 
-        protectedCall();
+        this.protectedCall();
     }
 
     function overprotectedCall() public nonReentrant {}