From 39924452566a611af17debea091d30c868eca595 Mon Sep 17 00:00:00 2001 From: Adam Gall Date: Wed, 6 Nov 2024 12:18:06 -0500 Subject: [PATCH] Already had these tests! --- ...RC20VotingWithHatsProposalCreation.test.ts | 19 +------------------ ...C721VotingWithHatsProposalCreation.test.ts | 19 +------------------ 2 files changed, 2 insertions(+), 36 deletions(-) diff --git a/test/Azorius-LinearERC20VotingWithHatsProposalCreation.test.ts b/test/Azorius-LinearERC20VotingWithHatsProposalCreation.test.ts index 7b16b60..bad3bf9 100644 --- a/test/Azorius-LinearERC20VotingWithHatsProposalCreation.test.ts +++ b/test/Azorius-LinearERC20VotingWithHatsProposalCreation.test.ts @@ -45,7 +45,6 @@ describe('LinearERC20VotingWithHatsProposalCreation', () => { // Wallets let deployer: SignerWithAddress; let gnosisSafeOwner: SignerWithAddress; - let rando: SignerWithAddress; // Gnosis let createGnosisSetupCalldata: string; @@ -59,7 +58,7 @@ describe('LinearERC20VotingWithHatsProposalCreation', () => { const abiCoder = new ethers.AbiCoder(); - [deployer, gnosisSafeOwner, rando] = await hre.ethers.getSigners(); + [deployer, gnosisSafeOwner] = await hre.ethers.getSigners(); createGnosisSetupCalldata = // eslint-disable-next-line camelcase @@ -247,22 +246,6 @@ describe('LinearERC20VotingWithHatsProposalCreation', () => { ); }); - it('Non-owner cannot whitelist a hat', async () => { - const hatId = 1; // Example hat ID - - await expect(linearERC20VotingWithHats.connect(rando).whitelistHat(hatId)).to.be.revertedWith( - 'Ownable: caller is not the owner', - ); - }); - - it('Non-owner cannot remove a hat from the whitelist', async () => { - const hatId = 1; // Example hat ID - - await expect( - linearERC20VotingWithHats.connect(rando).removeHatFromWhitelist(hatId), - ).to.be.revertedWith('Ownable: caller is not the owner'); - }); - it('Cannot call setUp function again', async () => { const setupParams = ethers.AbiCoder.defaultAbiCoder().encode( ['address', 'address', 'address', 'uint32', 'uint256', 'uint256', 'address', 'uint256[]'], diff --git a/test/Azorius-LinearERC721VotingWithHatsProposalCreation.test.ts b/test/Azorius-LinearERC721VotingWithHatsProposalCreation.test.ts index dec15eb..4bb6427 100644 --- a/test/Azorius-LinearERC721VotingWithHatsProposalCreation.test.ts +++ b/test/Azorius-LinearERC721VotingWithHatsProposalCreation.test.ts @@ -44,7 +44,6 @@ describe('LinearERC721VotingWithHatsProposalCreation', () => { // Wallets let deployer: SignerWithAddress; let gnosisSafeOwner: SignerWithAddress; - let rando: SignerWithAddress; // Gnosis let createGnosisSetupCalldata: string; @@ -58,7 +57,7 @@ describe('LinearERC721VotingWithHatsProposalCreation', () => { const abiCoder = new ethers.AbiCoder(); - [deployer, gnosisSafeOwner, rando] = await hre.ethers.getSigners(); + [deployer, gnosisSafeOwner] = await hre.ethers.getSigners(); createGnosisSetupCalldata = // eslint-disable-next-line camelcase @@ -229,22 +228,6 @@ describe('LinearERC721VotingWithHatsProposalCreation', () => { ); }); - it('Non-owner cannot whitelist a hat', async () => { - const hatId = 1; // Example hat ID - - await expect(linearERC721VotingWithHats.connect(rando).whitelistHat(hatId)).to.be.revertedWith( - 'Ownable: caller is not the owner', - ); - }); - - it('Non-owner cannot remove a hat from the whitelist', async () => { - const hatId = 1; // Example hat ID - - await expect( - linearERC721VotingWithHats.connect(rando).removeHatFromWhitelist(hatId), - ).to.be.revertedWith('Ownable: caller is not the owner'); - }); - it('Cannot call setUp function again', async () => { const setupParams = ethers.AbiCoder.defaultAbiCoder().encode( [