test: disable interval mining in hardhat network

Signed-off-by: Tomás Migone <[email protected]>
graphprotocol · Nov 27, 2023 · e00c6f9 · e00c6f9
1 parent 149635b
commit e00c6f9
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 31 deletions.
diff --git a/hardhat.config.ts b/hardhat.config.ts
@@ -157,10 +157,6 @@ const config: HardhatUserConfig = {
       accounts: {
         mnemonic: DEFAULT_TEST_MNEMONIC,
       },
-      mining: {
-        auto: false,
-        interval: 13000,
-      },
       hardfork: 'london',
     },
     localhost: {

diff --git a/tasks/deployment/deploy.ts b/tasks/deployment/deploy.ts
@@ -26,16 +26,6 @@ task('migrate', 'Migrate contracts')
   .setAction(async (taskArgs, hre) => {
     const graph = hre.graph(taskArgs)
 
-    // Set automine before deploying protocol
-    // Not all local nodes support it though
-    if (isGraphChainL1Localhost(graph.chainId)) {
-      try {
-        await helpers.setAutoMine(true)
-      } catch (error) {
-        console.error('Could not set automine to true, node might not support it')
-      }
-    }
-
     await deployGraphNetwork(
       taskArgs.addressBook,
       taskArgs.graphConfig,

diff --git a/test/epochs.test.ts b/test/epochs.test.ts
@@ -18,8 +18,6 @@ describe('EpochManager', () => {
   const epochLength: BigNumber = toBN('3')
 
   before(async function () {
-    await helpers.setIntervalMining(0)
-    await helpers.setAutoMine(true)
     ;[me, governor] = await graph.getTestAccounts()
     ;({ governor } = await graph.getNamedAccounts())
   })

diff --git a/test/governance/governed.test.ts b/test/governance/governed.test.ts
@@ -19,9 +19,6 @@ describe('Governed', () => {
   beforeEach(async function () {
     ;[me, governor] = await graph.getTestAccounts()
 
-    await helpers.setAutoMine(true)
-    await helpers.setIntervalMining(0)
-
     const factory = await ethers.getContractFactory('GovernedMock')
     governed = (await factory.connect(governor).deploy()) as unknown as Governed
   })

diff --git a/test/l2/l2GraphToken.test.ts b/test/l2/l2GraphToken.test.ts
@@ -24,8 +24,6 @@ describe('L2GraphToken', () => {
     let grt: L2GraphToken
 
     before(async function () {
-      await helpers.setIntervalMining(0)
-      await helpers.setAutoMine(true)
       ;[mockL1GRT, mockL2Gateway, user] = await graph.getTestAccounts()
       ;({ governor } = await graph.getNamedAccounts())
       fixture = new NetworkFixture(graph.provider)

diff --git a/test/lib/fixtures.ts b/test/lib/fixtures.ts
@@ -25,14 +25,10 @@ import { LibExponential } from '../../build/types/LibExponential'
 import {
   DeployType,
   GraphNetworkContracts,
-  acceptOwnership,
   deploy,
   deployGraphNetwork,
   helpers,
-  isGraphL1ChainId,
   loadGraphNetworkContracts,
-  toBN,
-  toGRT,
 } from '@graphprotocol/sdk'
 import { SignerWithAddress } from '@nomiclabs/hardhat-ethers/signers'
 
@@ -78,6 +74,7 @@ export class NetworkFixture {
   constructor(public provider: providers.Provider) {}
 
   async load(deployer: SignerWithAddress, l2Deploy?: boolean): Promise<GraphNetworkContracts> {
+    // Ensure we are auto mining
     await helpers.setIntervalMining(0)
     await helpers.setAutoMine(true)
 
@@ -206,8 +203,6 @@ export class NetworkFixture {
 
   async setUp(): Promise<void> {
     this.lastSnapshot = await helpers.takeSnapshot()
-    await helpers.setIntervalMining(0)
-    await helpers.setAutoMine(true)
   }
 
   async tearDown(): Promise<void> {

diff --git a/test/lib/graphTokenTests.ts b/test/lib/graphTokenTests.ts
@@ -72,8 +72,6 @@ export function grtTests(isL2: boolean): void {
   }
 
   before(async function () {
-    await helpers.setIntervalMining(0)
-    await helpers.setAutoMine(true)
     ;({ governor } = await graph.getNamedAccounts())
     me = new ethers.Wallet(mePrivateKey, graph.provider)
     other = new ethers.Wallet(otherPrivateKey, graph.provider)

diff --git a/test/staking/rebate.test.ts b/test/staking/rebate.test.ts
@@ -209,8 +209,6 @@ describe('Staking:rebates', () => {
   }
 
   before(async function () {
-    await helpers.setIntervalMining(0)
-    await helpers.setAutoMine(true)
     ;({ governor } = await graph.getNamedAccounts())
     fixture = new NetworkFixture(graph.provider)
     await fixture.load(governor)