Skip to content

Commit 0874a54

Browse files
authored
fix(hardhat-polkadot-node): ensure correct env setup for tests (#378)
* fix(hardhat-polkadot-node): ensure correct env setup for tests * linting * typo
1 parent 3e8b5ef commit 0874a54

File tree

2 files changed

+18
-17
lines changed

2 files changed

+18
-17
lines changed

packages/hardhat-polkadot-node/src/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -181,8 +181,6 @@ task(
181181

182182
const files = await run(TASK_TEST_GET_TEST_FILES, { testFiles })
183183

184-
await run(TASK_TEST_SETUP_TEST_ENVIRONMENT)
185-
186184
let nodePort = userConfig.networks?.hardhat?.nodeConfig?.rpcPort || NODE_START_PORT
187185
let adapterPort =
188186
userConfig.networks?.hardhat?.adapterConfig?.adapterPort || ETH_RPC_ADAPTER_START_PORT
@@ -231,6 +229,8 @@ task(
231229
useAnvil ? adapterPort : adapterPort || nodePort,
232230
)
233231

232+
await run(TASK_TEST_SETUP_TEST_ENVIRONMENT)
233+
234234
let testFailures = 0
235235
try {
236236
await handleFactoryDependencies(

packages/hardhat-polkadot-node/src/utils.ts

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,13 @@ import os from "os"
88
import path from "path"
99
import type { HardhatNetworkConfig, HardhatNetworkUserConfig } from "hardhat/types/config"
1010

11+
import { HARDHAT_NETWORK_NAME } from "hardhat/internal/constants"
1112
import { createRpcServer } from "./rpc-server"
1213
import type { CommandArguments, SplitCommands } from "./types"
1314
import { PolkadotNodePluginError } from "./errors"
1415
import {
1516
BASE_URL,
1617
MAX_PORT_ATTEMPTS,
17-
NETWORK_ETH,
18-
NETWORK_GAS,
19-
NETWORK_GAS_PRICE,
2018
NODE_START_PORT,
2119
ETH_RPC_ADAPTER_START_PORT,
2220
POLKADOT_TEST_NODE_NETWORK_NAME,
@@ -165,17 +163,15 @@ export function adjustTaskArgsForPort(taskArgs: string[], currentPort: number):
165163
return taskArgs
166164
}
167165

168-
export function getNetworkConfig(url: string, chainId?: number) {
166+
export function getNetworkConfig(url: string) {
169167
return {
170-
accounts: "remote",
171-
gas: NETWORK_GAS.AUTO,
172-
gasPrice: NETWORK_GAS_PRICE.AUTO,
168+
accounts: "remote" as const,
169+
gas: "auto" as const,
170+
gasPrice: "auto" as const,
173171
gasMultiplier: 1,
174172
httpHeaders: {},
175173
timeout: 20000,
176174
url,
177-
ethNetwork: NETWORK_ETH.LOCALHOST,
178-
chainId: chainId || 420420421,
179175
}
180176
}
181177

@@ -192,22 +188,27 @@ export async function configureNetwork(
192188
params: [],
193189
id: 1,
194190
}
195-
let chainId = 0
191+
let _chainId = 0
196192
try {
197193
const response = await axios.post(url, payload)
198194

199195
if (response.status == 200) {
200-
chainId = parseInt(response.data.result)
196+
_chainId = parseInt(response.data.result)
201197
}
202198
// eslint-disable-next-line @typescript-eslint/no-explicit-any
203199
} catch (_e: any) {
204200
// If it fails, it will just try again
205201
}
206202

207-
network.name = POLKADOT_TEST_NODE_NETWORK_NAME
208-
network.config = getNetworkConfig(url, chainId)
209-
config.networks[network.name] = network.config
210-
network.provider = await createProvider(config, network.name)
203+
const networkName =
204+
network.name === HARDHAT_NETWORK_NAME ? POLKADOT_TEST_NODE_NETWORK_NAME : network.name
205+
206+
const networkConfig = getNetworkConfig(url)
207+
208+
network.name = networkName
209+
network.config = networkConfig
210+
config.networks[networkName] = networkConfig
211+
network.provider = await createProvider(config, networkName)
211212
}
212213

213214
export async function startServer(

0 commit comments

Comments
 (0)