123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133 |
- const { shouldFail, constants, expectEvent } = require('openzeppelin-test-helpers');
- const { ZERO_ADDRESS } = constants;
- function capitalize (str) {
- return str.replace(/\b\w/g, l => l.toUpperCase());
- }
- // Tests that a role complies with the standard role interface, that is:
- // * an onlyRole modifier
- // * an isRole function
- // * an addRole function, accessible to role havers
- // * a renounceRole function
- // * roleAdded and roleRemoved events
- // Both the modifier and an additional internal remove function are tested through a mock contract that exposes them.
- // This mock contract should be stored in this.contract.
- //
- // @param authorized an account that has the role
- // @param otherAuthorized another account that also has the role
- // @param other an account that doesn't have the role, passed inside an array for ergonomics
- // @param rolename a string with the name of the role
- // @param manager undefined for regular roles, or a manager account for managed roles. In these, only the manager
- // account can create and remove new role bearers.
- function shouldBehaveLikePublicRole (authorized, otherAuthorized, [other], rolename, manager) {
- rolename = capitalize(rolename);
- describe('should behave like public role', function () {
- beforeEach('check preconditions', async function () {
- (await this.contract[`is${rolename}`](authorized)).should.equal(true);
- (await this.contract[`is${rolename}`](otherAuthorized)).should.equal(true);
- (await this.contract[`is${rolename}`](other)).should.equal(false);
- });
- if (manager === undefined) { // Managed roles are only assigned by the manager, and none are set at construction
- it('emits events during construction', async function () {
- await expectEvent.inConstruction(this.contract, `${rolename}Added`, {
- account: authorized,
- });
- });
- }
- it('reverts when querying roles for the null account', async function () {
- await shouldFail.reverting(this.contract[`is${rolename}`](ZERO_ADDRESS));
- });
- describe('access control', function () {
- context('from authorized account', function () {
- const from = authorized;
- it('allows access', async function () {
- await this.contract[`only${rolename}Mock`]({ from });
- });
- });
- context('from unauthorized account', function () {
- const from = other;
- it('reverts', async function () {
- await shouldFail.reverting(this.contract[`only${rolename}Mock`]({ from }));
- });
- });
- });
- describe('add', function () {
- const from = manager === undefined ? authorized : manager;
- context(`from ${manager ? 'the manager' : 'a role-haver'} account`, function () {
- it('adds role to a new account', async function () {
- await this.contract[`add${rolename}`](other, { from });
- (await this.contract[`is${rolename}`](other)).should.equal(true);
- });
- it(`emits a ${rolename}Added event`, async function () {
- const { logs } = await this.contract[`add${rolename}`](other, { from });
- expectEvent.inLogs(logs, `${rolename}Added`, { account: other });
- });
- it('reverts when adding role to an already assigned account', async function () {
- await shouldFail.reverting(this.contract[`add${rolename}`](authorized, { from }));
- });
- it('reverts when adding role to the null account', async function () {
- await shouldFail.reverting(this.contract[`add${rolename}`](ZERO_ADDRESS, { from }));
- });
- });
- });
- describe('remove', function () {
- // Non-managed roles have no restrictions on the mocked '_remove' function (exposed via 'remove').
- const from = manager || other;
- context(`from ${manager ? 'the manager' : 'any'} account`, function () {
- it('removes role from an already assigned account', async function () {
- await this.contract[`remove${rolename}`](authorized, { from });
- (await this.contract[`is${rolename}`](authorized)).should.equal(false);
- (await this.contract[`is${rolename}`](otherAuthorized)).should.equal(true);
- });
- it(`emits a ${rolename}Removed event`, async function () {
- const { logs } = await this.contract[`remove${rolename}`](authorized, { from });
- expectEvent.inLogs(logs, `${rolename}Removed`, { account: authorized });
- });
- it('reverts when removing from an unassigned account', async function () {
- await shouldFail.reverting(this.contract[`remove${rolename}`](other, { from }));
- });
- it('reverts when removing role from the null account', async function () {
- await shouldFail.reverting(this.contract[`remove${rolename}`](ZERO_ADDRESS, { from }));
- });
- });
- });
- describe('renouncing roles', function () {
- it('renounces an assigned role', async function () {
- await this.contract[`renounce${rolename}`]({ from: authorized });
- (await this.contract[`is${rolename}`](authorized)).should.equal(false);
- });
- it(`emits a ${rolename}Removed event`, async function () {
- const { logs } = await this.contract[`renounce${rolename}`]({ from: authorized });
- expectEvent.inLogs(logs, `${rolename}Removed`, { account: authorized });
- });
- it('reverts when renouncing unassigned role', async function () {
- await shouldFail.reverting(this.contract[`renounce${rolename}`]({ from: other }));
- });
- });
- });
- }
- module.exports = {
- shouldBehaveLikePublicRole,
- };
|