Lesson 7: FundMe.test.js => On executing <yarn hardhat test> i get following error , (I have gone through all the related discussions).

[skayp]

Link to Repo https://github.com/skayp/hh-FundMe
i have my FundMe.test.js as:

const { deployments, ethers, getNamedAccounts } = require("hardhat")
const { assert } = require("chai")

describe("FundMe", function () {
    let fundMe
    let deployer
    let mockV3Aggregator
    beforeEach(async function () {
        //deploy fundme contract using hardhat deploy
        // const accounts =awwait ethers.getSigners()
        // const accountZero = accounts[0]
        deployer = (await getNamedAccounts()).deployer
        await deployments.fixture(["all"])
        fundMe = await ethers.getContract("FundMe", deployer)//Gets the fundme deployed contract
        mockV3Aggregator = await ethers.getContract(
            "MockV3Aggregator",
            deployer
        )
    })
    describe("constructor testing", function () {
        it("sets the aggregator address correctly", async function () {
            const response = await fundMe.priceFeed()
            assert.equal(response, mockV3Aggregator.address)
        })
    })
})

On executing following command:

yarn hardhat test

I get Error in the terminal as:

       "before each" hook for "sets the aggregator address correctly":
     TypeError: Cannot read properties of undefined (reading 'getContract') 
at Context.<anonymous> (test/unit/FundMe.test.js:14:31

NOTE

Upto this command all the commands work as expected for example FundMe can be deployed at testnet

What i have tried:

1. Deleting npm_modules and reinstalling dependencies using yarn keeping my package.json intact
2. Deleting npm_modules and copying package.json from patrick's repo and installing using yarn
3. Checking if name of Contract matches the file name FundMe.
4. Deleting ethers and reinstalling using yarn add --dev @nomiclabs/hardhat-ethers@npm:hardhat-deploy-ethers ethers

I have gone through discussions. Some help would be appreciated.

01-deploy-fundMe.js :

// function deployFunc() {
//     console.log("hi")
// }
// module.exports.default = deployFunc
const { networkConfig, developmentChains } = require("../helper-hardhat-config")
const { network } = require("hardhat")
const { verify } = require("../utils/verify")
module.exports = async (hre) => {
    const { getNamedAccounts, deployments } = hre
    //similar to
    //hre.getNamedAccounts
    //hre.deployments
    const { deploy, log } = deployments //pulling two functions deploy, log out of deployments
    const { deployer } = await getNamedAccounts()
    const chainId = network.config.chainId
    //when going for localhost or hardhat we want to use mocks
    //const ethUsdPriceFeedAddress = networkConfig[chainId]["ethUsdPriceFeed"]
    let ethUsdPriceFeedAddress

    if (developmentChains.includes(network.name)) {
        const ethUsdAggregator = await deployments.get("MockV3Aggregator")
        ethUsdPriceFeedAddress = ethUsdAggregator.address
    } else {
        ethUsdPriceFeedAddress = networkConfig[chainId]["ethUsdPriceFeed"]
    }
    const fundMe = await deploy("FundMe", {
        from: deployer,
        args: [ethUsdPriceFeedAddress],
        log: true, //so that we dont have to do console log
        waitConfirmations: network.config.blockConfirmations || 1,
    })
    if (
        !developmentChains.includes(network.name) &&
        process.env.ETHERSCAN_API_KEY
    ) {
        //verify
        await verify(fundMe.address, [ethUsdPriceFeedAddress])
    }
    log("----------------------------------------------")
    //log(ethUsdPriceFeedAddress)
}
//similar to module.exports = async ({ getNamedAccounts, deployments }) => {}
module.exports.tags = ["all", "fundme"]

00-deploy-mocks.js

const { network } = require("hardhat")
const {
    developmentChains,
    DECIMALS,
    INITIAL_ANSWER,
} = require("../helper-hardhat-config")
module.exports = async (hre) => {
    const { getNamedAccounts, deployments } = hre
    const { deploy, log } = deployments //pulling two functions deploy, log out of deployments
    const { deployer } = await getNamedAccounts()
    const chainId = network.config.chainId

    if (developmentChains.includes(network.name)) {
        log("Local network detected Deploying mocs.. ")
        await deploy("MockV3Aggregator", {
            contract: "MockV3Aggregator",
            from: deployer,
            log: true,
            args: [
                DECIMALS,
                INITIAL_ANSWER,
            ] /*Constructor parameters for Mockv3aggregator*/,
        })
        log("Mocks Deployed")
        log("---------------------------------------------")
    }
}
module.exports.tags = ["all", "mocks"]

[ejmorian]

  "before each" hook for "sets the aggregator address correctly":
     TypeError: Cannot read properties of undefined (reading 'getContract') 

This error indicates that are is something wrong in your "before each" section, and more specifically the somewhere along the "getContract()" line.

This may be due to syntax errors if found in your beforeEach():

1. The await deployments.fixture(["all"]) should be await deployments.fixture("all") without the []

2. I'm not sure if this (**_fundMe = await ethers.getContract("FundMe", deployer)_** was a typo.. but this should be fundMe = await ethers.getContract("FundMe", deployer) without the ** and _

[skayp]

Thanks for the reply,

1. I have removed [ ] but still receiving the same error.
2. I was trying to make that line bold by using _ _ , my bad. The actual code is without them

Still thanks for your efforts.

[alymurtazamemon]

@skayp Make sure you have hardhat-deploy dependency installed, if not then run this command;

npm install -D hardhat-deploy @nomiclabs/hardhat-ethers@npm:hardhat-deploy-ethers ethers@^5

and add this in hardhat.config.js file;

require('hardhat-deploy');

[alymurtazamemon]

@skayp please leave your repo link.

[skayp]

ok let me upload it

[skayp]

https://github.com/skayp/hh-FundMe @alymurtazamemon

[alymurtazamemon]

@skayp Update these in hardhat.config.js file;

// require("@nomiclabs/hardhat-waffle")
// require("hardhat-gas-reporter")
//require("@nomiclabs/hardhat-ethers")
require("@nomicfoundation/hardhat-toolbox")
// require("@nomiclabs/hardhat-etherscan")
require("dotenv").config()
// require("solidity-coverage")
require("hardhat-deploy")

[skayp]

Thanks a lot @alymurtazamemon , the solution worked.
I added

require("@nomicfoundation/hardhat-toolbox")

Is there a function in the above library that is used in code because in the patrick's repo this statement is missing but everything seems to be fine there?

[alymurtazamemon]

@skayp not actually, the getContract function comes from hardhat-deploy dependency, may be the hardhat-waffle is now old and not compatible with hardhat-deploy.

[neerajnerlekar]

After adding the require("@nomicfoundation/hardhat-toolbox")
I was getting this warning

warning package.json: No license field
$ /home/lotus/Downloads/patrickCollins/hh-fcc/hh-fund-me/node_modules/.bin/hardhat test
An unexpected error occurred:

Error: You are using both @nomicfoundation/hardhat-chai-matchers and @nomiclabs/hardhat-waffle. They don't work correctly together, so please make sure you only use one.

We recommend you migrate to @nomicfoundation/hardhat-chai-matchers. Learn how to do it here: https://hardhat.org/migrate-from-waffle
    at hardhatWaffleIncompatibilityCheck (/home/lotus/Downloads/patrickCollins/hh-fcc/hh-fund-me/node_modules/@nomicfoundation/hardhat-chai-matchers/src/internal/hardhatWaffleIncompatibilityCheck.ts:3:11)

I updated the

require("@nomiclabs/hardhat-waffle")

to

require("@nomicfoundation/hardhat-chai-matchers")

and the test worked

[alymurtazamemon]

@neerajnerlekar and for others who will see this in the figure.

Follow this migration guide and you will not face any issue Migrating away from hardhat-waffle