Lesson 7: could not decode result data (value="0x", info={ "method": "priceFeed", "signature": "priceFeed()" }, code=BAD_DATA, version=6.6.4)

[TrudyCyns]

import { assert, expect } from 'chai';
import { ethers, deployments, getNamedAccounts } from 'hardhat';
import { FundMe, MockV3Aggregator } from '../../typechain-types';
import "hardhat-deploy"
import '@nomiclabs/hardhat-ethers';
import { Address } from 'hardhat-deploy/dist/types';

describe('FundMe', async () => {
  let fundMe: FundMe;
  let deployer: Address;
  let mockV3Aggregator;
  const sendValue = ethers.parseEther('1');

  beforeEach(async () => {
    //Deploy contract with hardhat-deploy
    deployer = (await getNamedAccounts()).deployer;
    await deployments.fixture(['all']);
    fundMe = await ethers.getContractAt('FundMe', deployer) as unknown as FundMe;
    mockV3Aggregator = await ethers.getContractAt('MockV3Aggregator', deployer);
  });

  describe('constructor', async () => {
    it('Should set the Aggregator addresses correctly', async () => {
      const response = await fundMe.priceFeed();
      console.log('Response: ', response);
      assert.equal(response, await mockV3Aggregator.getAddress());
    });
  });
});

Above is my FundMe.test.ts. I have tried solutions from issue #5749 but I still get the following error:
Error: could not decode result data (value="0x", info={ "method": "priceFeed", "signature": "priceFeed()" }, code=BAD_DATA, version=6.6.4)

Here is package.json:

{
  "name": "hardhat-fund-me",
  "version": "1.0.0",
  "license": "MIT",
  "devDependencies": {
    "@chainlink/contracts": "^0.6.1",
    "@nomicfoundation/hardhat-chai-matchers": "^2.0.0",
    "@nomicfoundation/hardhat-ethers": "^3.0.0",
    "@nomicfoundation/hardhat-network-helpers": "^1.0.0",
    "@nomicfoundation/hardhat-toolbox": "^3.0.0",
    "@nomicfoundation/hardhat-verify": "^1.0.0",
    "@nomiclabs/hardhat-ethers": "^2.2.3",
    "@typechain/ethers-v6": "^0.4.0",
    "@typechain/hardhat": "^8.0.0",
    "@types/chai": "^4.2.0",
    "@types/mocha": ">=9.1.0",
    "@types/node": ">=12.0.0",
    "chai": "^4.2.0",
    "dotenv": "^16.3.1",
    "ethers": "^6.6.1",
    "hardhat": "^2.16.0",
    "hardhat-deploy": "^0.11.34",
    "hardhat-deploy-ethers": "^0.3.0-beta.13",
    "hardhat-gas-reporter": "^1.0.8",
    "prettier": "^2.8.8",
    "prettier-plugin-solidity": "^1.1.3",
    "solhint": "^3.4.1",
    "solidity-coverage": "^0.8.0",
    "ts-node": ">=8.0.0",
    "typechain": "^8.1.0",
    "typescript": ">=4.5.0"
  }
}

[Nicolaswg]

I got the same error I tried fundme.priceFeed without the () , it outputs the function but I can't access to target or any other key of the object

[Nicolaswg]

this should be reverted await expect(fundMe.fund({ value: 0 })).to.be.reverted but it pass 🫠🫠🫠

[TrudyCyns]

That happens for me too. Despite having the require statement, the test passes without the expect to be reverted...

[fgvhjm]

did anyone found out the solution, i have the same problem

[mscurtescu]

I managed to get it working, see:
https://github.com/mscurtescu-blockchain-solidity-course/hardhat-fund-me-fcc/blob/main/test/unit/FundMe.test.js

[fgvhjm]

thanks brooo appreciate your help.

[Nicolaswg]

@alymurtazamemon, it would be good if you can leave us a short answer when you have free time to resolve these issues

[Grapeve]

It's so great, I solved this problem when I use ethers v6, of course, switching to v5 is the simplest solution. The following is my code using ethers v6.

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

describe("FundMe", async () => {
  let FundMe
  let signer
  let MockV3Aggregator
  beforeEach(async () => {
     // don't use (await getNamedAccounts()).deployer, as a parameter to the getContractAt function, it will report an error !!!
    const accounts = await ethers.getSigners()
    signer = accounts[0]   
    await deployments.fixture(["all"])
   
    // there is no getContract function in ethers, so using getContractAt
    const FundMeDeployment = await deployments.get("FundMe")
    FundMe = await ethers.getContractAt(
      FundMeDeployment.abi,
      FundMeDeployment.address,
      signer,
    )
    const MockV3AggregatorDeployment = await deployments.get(
      "MockV3Aggregator",
    )
    MockV3Aggregator = await ethers.getContractAt(
      MockV3AggregatorDeployment.abi,
      MockV3AggregatorDeployment.address,
      signer,
    )
  })

  describe("constructor", async () => {
    it("sets the aggregator address correctly", async () => {
      const response = await FundMe.priceFeed()
      assert.equal(response, MockV3Aggregator.target) // get address using target instead of address property
    })
  })
})

[Nicolaswg]

What about when testing the fund function ? It always return a passing test beside you send 0 funds

[Grapeve]

I don't encounter any problems when testing the fund function. There are some differences, I wrote in the comments

  ...
  const sendValue = ethers.parseEther("0.001")   // The minimum funding I assume here is 0.001 ETH，everyone's settings may be different
  ...
  describe("fund", async () => {
    it("Fails if you don't send enough ETH", async () => {
      await expect(FundMe.fund()).to.be.reverted  // using revertedWith ("xxx") may report an error, it seems that there is a conflict between the two thrown errors, an error thrown in the fund function, another is this. 
     //If you want to use revertedWith ("xxx"), they should have the same error message.
    })
    it("updated the amount funded data structure", async () => {
      await FundMe.fund({ value: sendValue })
      const response = await FundMe.addressToAmountFunder(signer.address)
      assert.equal(response.toString(), sendValue.toString())
    })
    it("Adds funder to array of funders", async () => {
      await FundMe.fund({ value: sendValue })
      const funder = await FundMe.funders(0)
      assert.equal(funder, signer.address)
    })
  })

[Grapeve]

1.you said 'this should be reverted await expect(fundMe.fund({ value: 0 })).to.be.reverted but it pass 🫠🫠🫠 ', fundMe.fund({ value: 0 } will throw a error, this is exact, you don't send enough ETH, but now we just need to tell the program that the error is ok, this precisely indicates that the fund function we wrote is correct, and an error will be reported when sendValue is empty or less than the critical value, so passing test is right.
2. fundMe.fund({ value: 0 }, the unit of this 0 is wei, if your sendValue is 0.001 ETH, you should wirte 0.001 * 1e18 = "1000000000000000", write ethers.parseEther("0.001") is better, if you miss a zero, it will report an error, you don't send enough ETH.

[mscurtescu]

await deployments.get("FundMe") fails with "Error: No deployment found for: FundMe"

[dominicjin]

I got it.
The key point is that function getContract becomes getContractAt in ethers@V6, and the parameter also changes.

  getContractAt: (
    nameOrAbi: string | any[],
    address: string | ethers.Addressable,
    signer?: ethers.Signer
  ) => Promise<ethers.Contract>;

The new version needs signer as its option parameter, but in the original code getNameAccounts only returns address instead of signer. Only getSigner returns signer struct, which explains that if you use (await getNamedAccounts()).deployer as a parameter, it failed.

If you try the following code

accounts = await ethers.getSigners()
deployer = accounts[0]
console.log("getSigners:", deployer)

And it returns

getSigners: HardhatEthersSigner {
 _gasLimit: 30000000,
 address: '0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266',
 provider: HardhatEthersProvider {
   _hardhatProvider: LazyInitializationProviderAdapter {
     _providerFactory: [AsyncFunction (anonymous)],
     _emitter: [EventEmitter],
     _initializingPromise: [Promise],
     provider: [BackwardsCompatibilityProviderAdapter]
   },
   _networkName: 'hardhat',
   _blockListeners: [],
   _transactionHashListeners: Map(0) {},
   _eventListeners: []
 }
}

[RMTKroos]

maybe this code can solve your problem.
https://github.com/mscurtescu-blockchain-solidity-course/hardhat-fund-me-fcc/blob/main/test/unit/FundMe.test.js

    beforeEach(async function(){

        deployer = (await getNamedAccounts()).deployer;
        await deployments.fixture(["all"]);
        const fundMeDeployment = await deployments.get("FundMe");
        fundMe = await ethers.getContractAt(fundMeDeployment.abi, fundMeDeployment.address);
        const mockV3AggregatorDeployment = await deployments.get("MockV3Aggregator");
        mockV3Aggregator = await ethers.getContractAt(mockV3AggregatorDeployment.abi, mockV3AggregatorDeployment.address);
        
    })

[icanci]

hello. I also encountered this problem today. Now I have solved this problem.
The solution code is as follows.
First: remove the ()
Second: the return of await fundMe.priceFeed is a object ,so we need get the address to run equal

describe("constructor", async function () {
   it("sets the aggregator addresses correctly", async function () {
       // AggregatorV3Interface = priceFeed
       const response = await fundMe.priceFeed
       assert.equal(response.address, mockV3Aggregator.address)
    })
})

[murilloramos]

IT WORKED FOR ME! Thank you so much, i spent two days searching for the answer...

[icanci]

这是来自QQ邮箱的假期自动回复邮件。您好，您的邮件我已经收到，我会尽快回复，谢谢。

[icanci]

这是来自QQ邮箱的假期自动回复邮件。您好，您的邮件我已经收到，我会尽快回复，谢谢。

[icanci]

这是来自QQ邮箱的假期自动回复邮件。您好，您的邮件我已经收到，我会尽快回复，谢谢。

[leleleocc]

I have the same problem like you guys,my solution code is as follows

const { deployments, ethers } = require("hardhat");
const { assert, expect } = require("chai");
describe("FundMe", async function () {
let FundMe;
let signer;
let MockV3Aggregator;
beforeEach(async function () {
const accounts = await ethers.getSigners();
signer = accounts[0];
await deployments.fixture(["all"]); //Run deploying scripts whose tags include 'all'
const allDeployments = await deployments.all();
console.log("All deployments:", Object.keys(allDeployments));
const FundMeDeployment = await deployments.get("FundMe");

FundMe = await ethers.getContractAt(
    FundMeDeployment.abi,
    FundMeDeployment.address,
    signer
);
console.log("FundMe contract address:", FundMe.address);
const MockV3AggregatorDeployment = await deployments.get(
    "MockV3Aggregator"
);
MockV3Aggregator = await ethers.getContractAt(
    MockV3AggregatorDeployment.abi,
    MockV3AggregatorDeployment.address,
    signer
);
console.log(MockV3Aggregator);
});
describe("constructor", async function () {
it("sets the aggregator addresses correctly", async function () {
const response = await FundMe.priceFeed();
assert.equal(response, await MockV3Aggregator.getAddress());
});
});
}); 

Notice that deployer = (await getNamedAccounts()).deployer; is a address of hardhat account,but const accounts = await ethers.getSigners(); signer = accounts[0]; is something like signer obj.Plus, you could check getContractAt function and you could see the differences. But my problem acctually is not this，but the two deploying javascripts.Because I used Code auto-completion ,so module.exports was mistakenly written as module.exports.default that would lead to Error:No deployment found for: FundMe.