ソースを参照

Added address of pauser/unpauser in events (#1410)

* Added address of pauser/unpauser in events

* Added the account to the Pausable tests.
Christopher Glisch 7 年 前
コミット
fcab9c89f2
2 ファイル変更6 行追加6 行削除
  1. 4 4
      contracts/lifecycle/Pausable.sol
  2. 2 2
      test/lifecycle/Pausable.test.js

+ 4 - 4
contracts/lifecycle/Pausable.sol

@@ -7,8 +7,8 @@ import "../access/roles/PauserRole.sol";
  * @dev Base contract which allows children to implement an emergency stop mechanism.
  */
 contract Pausable is PauserRole {
-  event Paused();
-  event Unpaused();
+  event Paused(address account);
+  event Unpaused(address account);
 
   bool private _paused;
 
@@ -44,7 +44,7 @@ contract Pausable is PauserRole {
    */
   function pause() public onlyPauser whenNotPaused {
     _paused = true;
-    emit Paused();
+    emit Paused(msg.sender);
   }
 
   /**
@@ -52,6 +52,6 @@ contract Pausable is PauserRole {
    */
   function unpause() public onlyPauser whenPaused {
     _paused = false;
-    emit Unpaused();
+    emit Unpaused(msg.sender);
   }
 }

+ 2 - 2
test/lifecycle/Pausable.test.js

@@ -57,7 +57,7 @@ contract('Pausable', function ([_, pauser, otherPauser, anyone, ...otherAccounts
         });
 
         it('emits a Paused event', function () {
-          expectEvent.inLogs(this.logs, 'Paused');
+          expectEvent.inLogs(this.logs, 'Paused', { account: pauser });
         });
 
         it('cannot perform normal process in pause', async function () {
@@ -89,7 +89,7 @@ contract('Pausable', function ([_, pauser, otherPauser, anyone, ...otherAccounts
             });
 
             it('emits an Unpaused event', function () {
-              expectEvent.inLogs(this.logs, 'Unpaused');
+              expectEvent.inLogs(this.logs, 'Unpaused', { account: pauser });
             });
 
             it('should resume allowing normal process', async function () {