Lesson 7: TypeError: Cannot read properties of undefined (reading 'priceFeed')

[ManuWeb3]

Hi everyone,

Not sure why I'm getting this error below when I run yarn hardhat test

FundMe.test.js code:

Fundme.sol (priceFeed code):

Directory Structure:

Help please.

Thanks
Manu

[krakxn]

Push the repo to GitHub and share link - much easier to diagnose

[krakxn]

The following fixes your issue: Okay, I am done - you can merge the pull requests and pull them to your local repo now. The tests will work fine. (Also, your test file's name has a typo, please correct it to FundMe from FuneMe)

[ManuWeb3]

Hi krakxn, THANKS A LOT man, I'm on it.
It's my first time dealing with Git, so might take a bit of effort :)
By the way, what precisely went wrong? Just the crux of it.

[krakxn]

You're welcome!! We all learn at one point. As for what was wrong, I implemented a getter function for price feed in the main contract. That (priceFeed()) was incorrect in your test file, you can check the changes via my PR in your repo.

[ManuWeb3]

Ok.
Well, with hit and trial to dig deep, I again ran the test by changing getPriceFeed() to priceFeed() (after making priceFeed public), it is still passing.
So, I'm not really sure what went wrong back then and what got fixed in between as I've been following Ali's suggested changes as well.
Anyway, thanks to both of you, I'm finally able to continue :)

[krakxn]

You did not merge the PRs though to your repo though; then a git pull to your local repo - which fixes your problem.

[zeroxvee]

We can get priceFeed address by 2 ways:

1. If s_priceFeed variable in FundMe.sol is public, then await fundMe.s_priceFeed() would work and will return the address, because solidity compiler creates getter functions for every public variables.

2. Or you should create a getter function yourself for priceFeed, which we did in the course by definining it as

   function getPriceFeed() public view returns (AggregatorV3Interface) {
       return s_priceFeed;
   }
   

[alymurtazamemon]

@ManuWeb3 Copy my package.json file in it and install dependencies again.

[ManuWeb3]

Hi Ali, THANKS A LOT buddy, I'm on it.
Just going through ton of changes being shown on my codes and trying to make sense of those.
By the way, what precisely went wrong? Just the crux of it.

[ManuWeb3]

So far I found 1 silly mistake that I did in declaring fundMe and mockV3Aggregator inside beforeEach that restricted their scope for it() to access these 2.

The test finally passed.

Any other mistake that you spotted?
Because I'm still not able to understand why it wasn't able to read "priceFeed" and why it did not threw the error "undefined fundMe" in the first place.

[alymurtazamemon]

@ManuWeb3 You can read the changes, I have updated many projects in 2 days so I do not remember your projects' mistakes. It is good your tests are passed. Mark, it a answered if you got the solution.

[ManuWeb3]

I understand buddy.
Highly appreciate your effort in correcting projects that don't really belong to you and helping others out.
By the way, I don't see "Mark as Answer" tag to your posts. Am I missing something here?