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

* Added address of pauser/unpauser in events * Added the account to the Pausable tests.
OpenZeppelin · Oct 18, 2018 · fcab9c8 · fcab9c8
1 parent cbe4148
commit fcab9c8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/contracts/lifecycle/Pausable.sol b/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,14 +44,14 @@ contract Pausable is PauserRole {
    */
   function pause() public onlyPauser whenNotPaused {
     _paused = true;
-    emit Paused();
+    emit Paused(msg.sender);
   }
 
   /**
    * @dev called by the owner to unpause, returns to normal state
    */
   function unpause() public onlyPauser whenPaused {
     _paused = false;
-    emit Unpaused();
+    emit Unpaused(msg.sender);
   }
 }
diff --git a/test/lifecycle/Pausable.test.js b/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 () {