Lesson 7: Error: Transaction reverted: function returned an unexpected amount of data

[DorianDaSilva]

| I am getting this error when running the test on addressTo AmountFunded() (11:23:25)

Below is my code (I ran it with & without describe line)

  describe("amount funded", async function () {
    it("Updated the amount funded data structure", async function () {
      await fundMe.fund({ value: sendValue });
      const response = await fundMe.addressToAmountFunded(deployer);
      assert.equal(response.toString(), sendValue.toString());
    });
  });

========================================================================================
|I tried checking online but no solution available on any of the forums

Does anyone have an idea of what maybe wrong? Been on this for hours and nothing. Thank you!

[dibakarsutradhar]

You have to pass the deployer address in the fundMe.addressToAmountFunded() function, right now you are only passing the deployer, it should be deployer.address instead.

Here's the full code for this test -

it('update the amount funded data structure', async () => {
      await fundMe.fund({ value: sendValue });
      const response = await fundMe.getAddressToAmountFunded(deployer.address);
      assert.equal(response.toString(), sendValue.toString());
    });

Hope this will solve your issue.

[DorianDaSilva]

@dibakarsutradhar

| beforeEach() -> FundMe.test.js

  beforeEach(async function () {
    //deploy FundMe contract using hardhat deploy
    //const accounts = await ethers.getSigners();
    //const accountZero = accounts[0];

    deployer = (await getNamedAccounts()).deployer;
    await deployments.fixture(["all"]);
    fundMe = await ethers.getContract("FundMe", deployer);
    MockV3Aggregator = await ethers.getContract("MockV3Aggregator", deployer);
  });

======================================================================================

| Constructor -> FundMe.sol

    constructor(address priceFeedAddress){
        i_owner = msg.sender;
        priceFeed = AggregatorV3Interface(priceFeedAddress);
    }

======================================================================================

Thank you!

[dibakarsutradhar]

@DorianDaSilva the beforeEach() looks good and also you shared the constructor from the Contra ct, instead can you share the describe("constructor") () please?

[DorianDaSilva]

@dibakarsutradhar Sure thing!

| describe("constructor")

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

===============================================================================

| The issue seems to be with this line
const sendValue = ethers.utils.parseEther("1");

I tried all possible combinations from the ethers docs with no success.

[DorianDaSilva]

@dibakarsutradhar

===============================================================================
As you can see when I run all 3 tests, both constructor and fundme pass but "amount funded" fails.

[dibakarsutradhar]

Could you also share your PriceConverter.sol contract, could be an issue from getPrice() and getConversionRate().

[mattjaf]

console.log(sendValue) && console.log(response)

console.log(typeof sendValue) && console.log(typeof response)

lets see why they arent equal

[mattjaf]

ohh, try changing

const response = await fundMe.addressToAmountFunded(deployer);

to

const response = await fundMe.getAddressToAmountFunded( deployer )

[DorianDaSilva]

@mattjaf Thank you for the input.

I tried modifying const response() as shown above but issue persisted

| const response = await fundMe.getAddressToAmountFunded( deployer )

======================================================================================

Also, tested the suggested console log statements but I still get the same error and it is not printing (Maybe I am doing wrong)

  describe("amount funded", async function () {
    it("Updated the amount funded data structure", async function () {
      await fundMe.fund({ value: sendValue });
      console.log(sendValue)
      const response = await fundMe.addressToAmountFunded(
        "amount funded",
        deployer
      );
      console.log(response)
      assert.equal(response.toString(), sendValue.toString());
    });
  });

======================================================================================

[DorianDaSilva]

@dibakarsutradhar I have identified the source of the error

FundMe.sol (Line 47)

| require(msg.value.getConversionRate(priceFeed) >= MINIMUM_USD, "Didn't send enough");

In the video it is commented out of the function ( // )

| When I comment it out, it passes 2 tests but fails at

  describe("fund", async function () {
    it("Fails if you don't send enough ETH", async function () {
      await expect(fundMe.fund()).to.be.reverted;
    });

======================================================================================

| When I leave it in passes 2 of the 3 test but fails at

    it("updated the amount funded data structure", async function () {
      await fundMe.fund({ value: sendValue });
      const response = await fundMe.addressToAmountFunded(deployer);
    });

=======================================================================================

Still trying to figure out the reason for this. Any thoughts?

@PatrickAlphaC Do you have any suggestions?

[DorianDaSilva]

@dibakarsutradhar As I continued with the lesson I managed to fix the initial error as stated above but now I am getting the following error

======================================================================================

| Note that "updated the amount funded data structure" function was passing previously but not anymore because I commented out
| FundMe.sol line 47 in my code.

Any thoughts on what modification I could do to my fundMe contract to avoid previous error while fixing assertion error?

[DorianDaSilva]

@dibakarsutradhar @mattjaf
Fixed by removing await from expect(fundMe.fund()).to.be.reverted;

====================================================================================

[dibakarsutradhar]

@DorianDaSilva Amazing work!