123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456 |
- const { ethers } = require('hardhat');
- const { expect } = require('chai');
- const { setStorageAt } = require('@nomicfoundation/hardhat-network-helpers');
- const { EXECUTION_ID_STORAGE_SLOT, EXPIRATION, prepareOperation } = require('../../helpers/access-manager');
- const { impersonate } = require('../../helpers/account');
- const time = require('../../helpers/time');
- // ============ COMMON PREDICATES ============
- const LIKE_COMMON_IS_EXECUTING = {
- executing() {
- it('succeeds', async function () {
- await this.caller.sendTransaction({ to: this.target, data: this.calldata });
- });
- },
- notExecuting() {
- it('reverts as AccessManagerUnauthorizedAccount', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerUnauthorizedAccount')
- .withArgs(this.caller, this.role.id);
- });
- },
- };
- const LIKE_COMMON_GET_ACCESS = {
- requiredRoleIsGranted: {
- roleGrantingIsDelayed: {
- callerHasAnExecutionDelay: {
- beforeGrantDelay() {
- it('reverts as AccessManagerUnauthorizedAccount', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerUnauthorizedAccount')
- .withArgs(this.caller, this.role.id);
- });
- },
- afterGrantDelay: undefined, // Diverges if there's an operation delay or not
- },
- callerHasNoExecutionDelay: {
- beforeGrantDelay() {
- it('reverts as AccessManagerUnauthorizedAccount', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerUnauthorizedAccount')
- .withArgs(this.caller, this.role.id);
- });
- },
- afterGrantDelay() {
- it('succeeds called directly', async function () {
- await this.caller.sendTransaction({ to: this.target, data: this.calldata });
- });
- it('succeeds via execute', async function () {
- await this.manager.connect(this.caller).execute(this.target, this.calldata);
- });
- },
- },
- },
- roleGrantingIsNotDelayed: {
- callerHasAnExecutionDelay: undefined, // Diverges if there's an operation to schedule or not
- callerHasNoExecutionDelay() {
- it('succeeds called directly', async function () {
- await this.caller.sendTransaction({ to: this.target, data: this.calldata });
- });
- it('succeeds via execute', async function () {
- await this.manager.connect(this.caller).execute(this.target, this.calldata);
- });
- },
- },
- },
- requiredRoleIsNotGranted() {
- it('reverts as AccessManagerUnauthorizedAccount', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerUnauthorizedAccount')
- .withArgs(this.caller, this.role.id);
- });
- },
- };
- const LIKE_COMMON_SCHEDULABLE = {
- scheduled: {
- before() {
- it('reverts as AccessManagerNotReady', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerNotReady')
- .withArgs(this.operationId);
- });
- },
- after() {
- it('succeeds called directly', async function () {
- await this.caller.sendTransaction({ to: this.target, data: this.calldata });
- });
- it('succeeds via execute', async function () {
- await this.manager.connect(this.caller).execute(this.target, this.calldata);
- });
- },
- expired() {
- it('reverts as AccessManagerExpired', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerExpired')
- .withArgs(this.operationId);
- });
- },
- },
- notScheduled() {
- it('reverts as AccessManagerNotScheduled', async function () {
- await expect(this.caller.sendTransaction({ to: this.target, data: this.calldata }))
- .to.be.revertedWithCustomError(this.manager, 'AccessManagerNotScheduled')
- .withArgs(this.operationId);
- });
- },
- };
- // ============ MODE ============
- /**
- * @requires this.{manager,target}
- */
- function testAsClosable({ closed, open }) {
- describe('when the manager is closed', function () {
- beforeEach('close', async function () {
- await this.manager.$_setTargetClosed(this.target, true);
- });
- closed();
- });
- describe('when the manager is open', function () {
- beforeEach('open', async function () {
- await this.manager.$_setTargetClosed(this.target, false);
- });
- open();
- });
- }
- // ============ DELAY ============
- /**
- * @requires this.{delay}
- */
- function testAsDelay(type, { before, after }) {
- beforeEach('define timestamp when delay takes effect', async function () {
- const timestamp = await time.clock.timestamp();
- this.delayEffect = timestamp + this.delay;
- });
- describe(`when ${type} delay has not taken effect yet`, function () {
- beforeEach(`set next block timestamp before ${type} takes effect`, async function () {
- await time.increaseTo.timestamp(this.delayEffect - 1n, !!before.mineDelay);
- });
- before();
- });
- describe(`when ${type} delay has taken effect`, function () {
- beforeEach(`set next block timestamp when ${type} takes effect`, async function () {
- await time.increaseTo.timestamp(this.delayEffect, !!after.mineDelay);
- });
- after();
- });
- }
- // ============ OPERATION ============
- /**
- * @requires this.{manager,scheduleIn,caller,target,calldata}
- */
- function testAsSchedulableOperation({ scheduled: { before, after, expired }, notScheduled }) {
- describe('when operation is scheduled', function () {
- beforeEach('schedule operation', async function () {
- if (this.caller.target) {
- await impersonate(this.caller.target);
- this.caller = await ethers.getSigner(this.caller.target);
- }
- const { operationId, schedule } = await prepareOperation(this.manager, {
- caller: this.caller,
- target: this.target,
- calldata: this.calldata,
- delay: this.scheduleIn,
- });
- await schedule();
- this.operationId = operationId;
- });
- describe('when operation is not ready for execution', function () {
- beforeEach('set next block time before operation is ready', async function () {
- this.scheduledAt = await time.clock.timestamp();
- const schedule = await this.manager.getSchedule(this.operationId);
- await time.increaseTo.timestamp(schedule - 1n, !!before.mineDelay);
- });
- before();
- });
- describe('when operation is ready for execution', function () {
- beforeEach('set next block time when operation is ready for execution', async function () {
- this.scheduledAt = await time.clock.timestamp();
- const schedule = await this.manager.getSchedule(this.operationId);
- await time.increaseTo.timestamp(schedule, !!after.mineDelay);
- });
- after();
- });
- describe('when operation has expired', function () {
- beforeEach('set next block time when operation expired', async function () {
- this.scheduledAt = await time.clock.timestamp();
- const schedule = await this.manager.getSchedule(this.operationId);
- await time.increaseTo.timestamp(schedule + EXPIRATION, !!expired.mineDelay);
- });
- expired();
- });
- });
- describe('when operation is not scheduled', function () {
- beforeEach('set expected operationId', async function () {
- this.operationId = await this.manager.hashOperation(this.caller, this.target, this.calldata);
- // Assert operation is not scheduled
- expect(await this.manager.getSchedule(this.operationId)).to.equal(0n);
- });
- notScheduled();
- });
- }
- /**
- * @requires this.{manager,roles,target,calldata}
- */
- function testAsRestrictedOperation({ callerIsTheManager: { executing, notExecuting }, callerIsNotTheManager }) {
- describe('when the call comes from the manager (msg.sender == manager)', function () {
- beforeEach('define caller as manager', async function () {
- this.caller = this.manager;
- if (this.caller.target) {
- await impersonate(this.caller.target);
- this.caller = await ethers.getSigner(this.caller.target);
- }
- });
- describe('when _executionId is in storage for target and selector', function () {
- beforeEach('set _executionId flag from calldata and target', async function () {
- const executionId = ethers.keccak256(
- ethers.AbiCoder.defaultAbiCoder().encode(
- ['address', 'bytes4'],
- [this.target.target, this.calldata.substring(0, 10)],
- ),
- );
- await setStorageAt(this.manager.target, EXECUTION_ID_STORAGE_SLOT, executionId);
- });
- executing();
- });
- describe('when _executionId does not match target and selector', notExecuting);
- });
- describe('when the call does not come from the manager (msg.sender != manager)', function () {
- beforeEach('define non manager caller', function () {
- this.caller = this.roles.SOME.members[0];
- });
- callerIsNotTheManager();
- });
- }
- /**
- * @requires this.{manager,scheduleIn,caller,target,calldata,executionDelay}
- */
- function testAsDelayedOperation() {
- describe('with operation delay', function () {
- describe('when operation delay is greater than execution delay', function () {
- beforeEach('set operation delay', async function () {
- this.operationDelay = this.executionDelay + time.duration.hours(1);
- await this.manager.$_setTargetAdminDelay(this.target, this.operationDelay);
- this.scheduleIn = this.operationDelay; // For testAsSchedulableOperation
- });
- testAsSchedulableOperation(LIKE_COMMON_SCHEDULABLE);
- });
- describe('when operation delay is shorter than execution delay', function () {
- beforeEach('set operation delay', async function () {
- this.operationDelay = this.executionDelay - time.duration.hours(1);
- await this.manager.$_setTargetAdminDelay(this.target, this.operationDelay);
- this.scheduleIn = this.executionDelay; // For testAsSchedulableOperation
- });
- testAsSchedulableOperation(LIKE_COMMON_SCHEDULABLE);
- });
- });
- describe('without operation delay', function () {
- beforeEach('set operation delay', async function () {
- this.operationDelay = 0n;
- await this.manager.$_setTargetAdminDelay(this.target, this.operationDelay);
- this.scheduleIn = this.executionDelay; // For testAsSchedulableOperation
- });
- testAsSchedulableOperation(LIKE_COMMON_SCHEDULABLE);
- });
- }
- // ============ METHOD ============
- /**
- * @requires this.{manager,roles,role,target,calldata}
- */
- function testAsCanCall({
- closed,
- open: {
- callerIsTheManager,
- callerIsNotTheManager: { publicRoleIsRequired, specificRoleIsRequired },
- },
- }) {
- testAsClosable({
- closed,
- open() {
- testAsRestrictedOperation({
- callerIsTheManager,
- callerIsNotTheManager() {
- testAsHasRole({
- publicRoleIsRequired,
- specificRoleIsRequired,
- });
- },
- });
- },
- });
- }
- /**
- * @requires this.{target,calldata,roles,role}
- */
- function testAsHasRole({ publicRoleIsRequired, specificRoleIsRequired }) {
- describe('when the function requires the caller to be granted with the PUBLIC_ROLE', function () {
- beforeEach('set target function role as PUBLIC_ROLE', async function () {
- this.role = this.roles.PUBLIC;
- await this.manager
- .connect(this.roles.ADMIN.members[0])
- .$_setTargetFunctionRole(this.target, this.calldata.substring(0, 10), this.role.id);
- });
- publicRoleIsRequired();
- });
- describe('when the function requires the caller to be granted with a role other than PUBLIC_ROLE', function () {
- beforeEach('set target function role as PUBLIC_ROLE', async function () {
- await this.manager
- .connect(this.roles.ADMIN.members[0])
- .$_setTargetFunctionRole(this.target, this.calldata.substring(0, 10), this.role.id);
- });
- testAsGetAccess(specificRoleIsRequired);
- });
- }
- /**
- * @requires this.{manager,role,caller}
- */
- function testAsGetAccess({
- requiredRoleIsGranted: {
- roleGrantingIsDelayed: {
- // Because both grant and execution delay are set within the same $_grantRole call
- // it's not possible to create a set of tests that diverge between grant and execution delay.
- // Therefore, the testAsDelay arguments are renamed for clarity:
- // before => beforeGrantDelay
- // after => afterGrantDelay
- callerHasAnExecutionDelay: { beforeGrantDelay: case1, afterGrantDelay: case2 },
- callerHasNoExecutionDelay: { beforeGrantDelay: case3, afterGrantDelay: case4 },
- },
- roleGrantingIsNotDelayed: { callerHasAnExecutionDelay: case5, callerHasNoExecutionDelay: case6 },
- },
- requiredRoleIsNotGranted,
- }) {
- describe('when the required role is granted to the caller', function () {
- describe('when role granting is delayed', function () {
- beforeEach('define delay', function () {
- this.grantDelay = time.duration.minutes(3);
- this.delay = this.grantDelay; // For testAsDelay
- });
- describe('when caller has an execution delay', function () {
- beforeEach('set role and delay', async function () {
- this.executionDelay = time.duration.hours(10);
- this.delay = this.grantDelay;
- await this.manager.$_grantRole(this.role.id, this.caller, this.grantDelay, this.executionDelay);
- });
- testAsDelay('grant', { before: case1, after: case2 });
- });
- describe('when caller has no execution delay', function () {
- beforeEach('set role and delay', async function () {
- this.executionDelay = 0n;
- await this.manager.$_grantRole(this.role.id, this.caller, this.grantDelay, this.executionDelay);
- });
- testAsDelay('grant', { before: case3, after: case4 });
- });
- });
- describe('when role granting is not delayed', function () {
- beforeEach('define delay', function () {
- this.grantDelay = 0n;
- });
- describe('when caller has an execution delay', function () {
- beforeEach('set role and delay', async function () {
- this.executionDelay = time.duration.hours(10);
- await this.manager.$_grantRole(this.role.id, this.caller, this.grantDelay, this.executionDelay);
- });
- case5();
- });
- describe('when caller has no execution delay', function () {
- beforeEach('set role and delay', async function () {
- this.executionDelay = 0n;
- await this.manager.$_grantRole(this.role.id, this.caller, this.grantDelay, this.executionDelay);
- });
- case6();
- });
- });
- });
- describe('when role is not granted', function () {
- // Because this helper can be composed with other helpers, it's possible
- // that role has been set already by another helper.
- // Although this is highly unlikely, we check for it here to avoid false positives.
- beforeEach('assert role is unset', async function () {
- const { since } = await this.manager.getAccess(this.role.id, this.caller);
- expect(since).to.equal(0n);
- });
- requiredRoleIsNotGranted();
- });
- }
- module.exports = {
- LIKE_COMMON_IS_EXECUTING,
- LIKE_COMMON_GET_ACCESS,
- LIKE_COMMON_SCHEDULABLE,
- testAsClosable,
- testAsDelay,
- testAsSchedulableOperation,
- testAsRestrictedOperation,
- testAsDelayedOperation,
- testAsCanCall,
- testAsHasRole,
- testAsGetAccess,
- };
|