Lesson 8: I am getting error while unit testing #5327
-
|
Cannot read properties of undefined (reading 'JsonRpcProvider') |
Beta Was this translation helpful? Give feedback.
Replies: 5 comments 7 replies
-
|
Hello replace your fundMe and mockV3Aggregator to this instead don't use getContractAt |
Beta Was this translation helpful? Give feedback.
-
|
@Jay-6957 For this error npm install --save-dev ethers@^5or yarn add --dev ethers@^5and as @pratokko said do not use npm install -D hardhat-deploy @nomiclabs/hardhat-ethers@npm:hardhat-deploy-ethers ethers@^5or yarn add --dev hardhat-deploy @nomiclabs/hardhat-ethers@npm:hardhat-deploy-ethers ethers@^5and add this inside require('hardhat-deploy'); |
Beta Was this translation helpful? Give feedback.
-
|
How it happens that some line of code is compatiable with specific ethers version ? before getting this error, i had ethers ^6.3.0, and it worked properly. And it should, because this is the latest version. But still it shows the error which is often fixed by downgrading the version, |
Beta Was this translation helpful? Give feedback.
-
|
@prakhar9015 hardhat-deploy uses the ethers dependency but they do not have upgraded it to ethers 6. So if you just use the ethers then it works perfectly but when you use it with hardhat-deploy you need to downgrade it to ethers 5. ethers 6 is in beta, this could be the reason why hardhat-deploy team is not upgrading it. |
Beta Was this translation helpful? Give feedback.
-
|
Hi @alymurtazamemon , |
Beta Was this translation helpful? Give feedback.
-
|
@mahziarwoas You should always use the latest version of any dependency but some dependencies are dependent on another dependency and due to that, we need to use the specific version which that dependency support. For example, you should always use the updated version of ethers.js but because we are using the hardhat-deploy dependency, it still uses the ethers version 5. So in order to use hardhat-deploy we need to use ethers.js version 5. |
Beta Was this translation helpful? Give feedback.
-
|
thanks @alymurtazamemon !! I was stuck on this error. Your response helped me :) |
Beta Was this translation helpful? Give feedback.
-
|
Hi guys, I tried all the steps that you suggested still getting errors. please help me i am stuck here form last 1 day. my test file my pakage.json errror |
Beta Was this translation helpful? Give feedback.
-
|
@gradder7 I have already answered it in another discussion, please checkout this #5574 |
Beta Was this translation helpful? Give feedback.
-
|
Hi guys, I tried all the steps that you suggested still getting errors. please help me i am stuck here form last 1 day. my test file my pakage.json my error |
Beta Was this translation helpful? Give feedback.
-
|
me too. I don not know how to solve it |
Beta Was this translation helpful? Give feedback.
-
|
i have correct the error for the latest version of ethers and it's working correctly then you should have the same code in hardhat-config.js require("@nomiclabs/hardhat-waffle");
require("hardhat-gas-reporter");
require("@nomiclabs/hardhat-etherscan");
require("dotenv").config();
require("solidity-coverage");
require("hardhat-deploy");
const SEPOLIA_RPC_URL = process.env.SEPOLIA_RPC_URL || "https://eth-sepolia";
const PRIVATE_KEY = process.env.PRIVATE_KEY || "0x..";
const ETHERSCAN_API_KEY = process.env.ETHERSCAN_API_KEY || "key";
const COINMARKET_API_KEY = process.env.COINMARKET_API_KEY || "key";
module.exports = {
defaultNetwork: "hardhat",
networks: {
sepolia: {
url: SEPOLIA_RPC_URL,
accounts: [PRIVATE_KEY],
chainId: 11155111,
blockConfirmations: 6,
},
localhost: {
url: "http://127.0.0.1:8545/",
chainId: 31337,
},
},
// solidity: "0.8.8",
solidity: {
compilers: [{ version: "0.8.8" }, { version: "0.6.6" }],
},
etherscan: {
apiKey: ETHERSCAN_API_KEY,
},
gasReporter: {
enabled: false,
outputFile: "gas-report.txt",
noColors: true,
currency: "USD",
coinmarketcap: COINMARKET_API_KEY,
token: "MATIC",
},
namedAccounts: {
deployer: {
default: 0, // here this will by default take the first account as deployer
1: 0, // similarly on mainnet it will take the first account as deployer. Note though that depending on how hardhat network are configured, the account 0 on one network can be different than on another
},
},
};then we have FundMe.test.js should be like mine and remember to replace mockV3Aggregator.address by await mockV3Aggregator.getAddress() const { deployments, getNamedAccounts, ethers } = require("hardhat");
const { assert } = require("chai");
describe("FundMe", () => {
let fundMe;
let deployer;
let mockV3Aggregator;
beforeEach(async () => {
// deploy fundMe
// using Hardhat-deploy
deployer = (await getNamedAccounts()).deployer;
await deployments.fixture(["all"]);
fundMe = await ethers.getContract("FundMe", deployer);
mockV3Aggregator = await ethers.getContract("MockV3Aggregator", deployer);
});
describe("constructor", () => {
it("sets the aggregator addresses correctly", async ()=> {
const response = await fundMe.priceFeed();
assert.equal(response, await mockV3Aggregator.getAddress());
});
});
})Thank you and good luck <3 |
Beta Was this translation helpful? Give feedback.
@Jay-6957 For this error
Cannot read properties of undefined (reading 'JsonRpcProvider')run this command;or
and as @pratokko said do not use
getContractAtfunction instead usegetContractfunction and if you havecannot find getContract functionerror, then run this command;or
and add this inside
hardhat.config.jsfile;