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

* Added address of pauser/unpauser in events

* Added the account to the Pausable tests.

(cherry picked from commit fcab9c89f2)
pull/1474/head
Christopher Glisch 6 years ago committed by Leo Arias
parent c126e3e81a
commit 5f3ecd9c64
  1. 8
      contracts/lifecycle/Pausable.sol
  2. 4
      test/lifecycle/Pausable.test.js

@ -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);
}
}

@ -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 () {

Loading…
Cancel
Save