diff --git a/contracts/test/executors/executor.ts b/contracts/test/executors/executor.ts
index 2a304468..8ded5e0e 100644
--- a/contracts/test/executors/executor.ts
+++ b/contracts/test/executors/executor.ts
@@ -21,7 +21,7 @@ const EventExecuted = 'Executed';
 export async function getActions() {
   const signers = await ethers.getSigners();
   const ActionExecuteFactory = new ActionExecute__factory(signers[0]);
-  let ActionExecute = await ActionExecuteFactory.deploy();
+  const ActionExecute = await ActionExecuteFactory.deploy();
   const iface = new ethers.utils.Interface(ActionExecute__factory.abi);
 
   const num = 20;
@@ -63,7 +63,7 @@ describe('Executor', async () => {
   });
 
   it('reverts if array of actions is too big', async () => {
-    let actions = [];
+    const actions = [];
     for (let i = 0; i < MAX_ACTIONS; i++) {
       actions[i] = data.succeedAction;
     }
@@ -88,9 +88,9 @@ describe('Executor', async () => {
     // Allow the call to fail so we can get the error message
     // in the `execResults`, otherwise with allowFailureMap = 0,
     // it fails with `ActionFailed` even though reentrancy worked correctly.
-    let allowFailureMap = flipBit(0, ethers.BigNumber.from(0));
+    const allowFailureMap = flipBit(0, ethers.BigNumber.from(0));
 
-    let tx = await executor.execute(
+    const tx = await executor.execute(
       ZERO_BYTES32,
       [data.reentrancyAction],
       allowFailureMap
@@ -129,7 +129,7 @@ describe('Executor', async () => {
 
   it('succeeds and correctly constructs failureMap results ', async () => {
     let allowFailureMap = ethers.BigNumber.from(0);
-    let actions = [];
+    const actions = [];
 
     // First 3 actions will fail
     actions[0] = data.failAction;
@@ -148,8 +148,8 @@ describe('Executor', async () => {
     }
 
     // If the below call not fails, means allowFailureMap is correct.
-    let tx = await executor.execute(ZERO_BYTES32, actions, allowFailureMap);
-    let event = findEvent<ExecutedEvent>(await tx.wait(), EventExecuted);
+    const tx = await executor.execute(ZERO_BYTES32, actions, allowFailureMap);
+    const event = findEvent<ExecutedEvent>(await tx.wait(), EventExecuted);
 
     expect(event.args.actor).to.equal(ownerAddress);
     expect(event.args.callId).to.equal(ZERO_BYTES32);
diff --git a/contracts/test/permission/condition/powerful-condition.ts b/contracts/test/permission/condition/powerful-condition.ts
index eeae9dc8..dc28ba2d 100644
--- a/contracts/test/permission/condition/powerful-condition.ts
+++ b/contracts/test/permission/condition/powerful-condition.ts
@@ -9,7 +9,6 @@ import {
 import {
   BLOCK_NUMBER_RULE_ID,
   TIMESTAMP_RULE_ID,
-  RULE_VALUE_RULE_ID,
   CONDITION_RULE_ID,
   LOGIC_OP_RULE_ID,
   DUMMY_PERMISSION_ID,