Fix bypassing target admin delay using execute

test/access/manager/AccessManager.test.js

@@ -1677,6 +1677,30 @@ describe('AccessManager', function () {
         });
       });
     });
+
+    it('BUG: execute can bypass target admin delay to update authority', async function () {
+      const newAuthority = await ethers.deployContract('$AccessManager', [this.admin]);
+
+      // set a delay on the target - and wait for it to be active
+      await this.manager.$_setTargetAdminDelay(this.target, 10n);
+      await time.increaseBy.timestamp(5n * 86400n, true); // minSetBack is 5 days
+
+      await expect(this.manager.getTargetAdminDelay(this.target)).to.eventually.equal(10n);
+
+      // cannot update directly - there is a delay
+      await expect(
+        this.manager.connect(this.admin).updateAuthority(this.target, newAuthority),
+      ).to.be.revertedWithCustomError(this.manager, 'AccessManagerNotScheduled');
+
+      // bypass the check by just doing execute
+      await expect(
+        this.manager
+          .connect(this.admin)
+          .execute(this.target, this.target.interface.encodeFunctionData('setAuthority', [newAuthority.target])),
+      )
+        .to.emit(this.target, 'AuthorityUpdated')
+        .withArgs(newAuthority);
+    });
   });
 
   describe('access managed self operations', function () {