Lesson 6: Brownie FundMe

[ManiBharathiSS]

I get this error when i run test with the network ganache-local. But i am not getting the error when i run the test with development network

[Niferu]

Hello @ManiBharathiSS

The error pops because your brownie-config.yaml is probably setup wrong. Double check all addresses you have added under network you are testing it on, because price feed you have pasted there for eth_usd might be the problem here as it should be following: 0x5f4eC3Df9cbd43714FE2740f5E3616155c5b8419 (you can get that on mainnet here: https://docs.chain.link/docs/data-feeds/price-feeds/addresses/?network=ethereum)

Hope this might help.

[ManiBharathiSS]

@Neftyr thanks for the reply,
ganache-local is a local blockchain network. So the mocks are deployed and it doesn't need eth_usd_price_feed. Only for the test nets and mainnet fork, eth_usd_price_feed needs to be added in brownie-config. So it doesn't solve the problem.

Correct me if I am wrong.

This error pops up only when i run tests on ganache-local.

[Niferu]

Hello @ManiBharathiSS
Sorry I have misunderstood your question then and I thought it is not working for you on mainnet fork network. In this case please share your deploy() and deploy_mocks() functions as your test code looks good. I'm definitely not pro and I'm still beginner but as far as I know most of the "Gas Estimation Fails" comes from incorrect price feeds.

[cromewar]

As @Neftyr
Please share your deploy script, as bad_actor might not be getting correctly the account.

[ManiBharathiSS]

@cromewar @Neftyr

[cromewar]

ok you have bad actor as bad_actor = accounts.add() you can change it to accounts[2] and try again.
Does this works?
Or you can change the get_account() function to be able to receive the index like this:

def get_account(index=None, id=None):
    # accounts[0]
    # accounts.add("env")
    # accounts.load("id")
    if index:
        return accounts[index]
    if id:
        return accounts.load(id)
    if (
        network.show_active() in LOCAL_BLOCKCHAIN_ENVIRONMENTS
        or network.show_active() in FORKED_LOCAL_ENVIRONMENTS
    ):
        return accounts[0]
    return accounts.add(config["wallets"]["from_key"])

[ManiBharathiSS]

I get the same error

[Niferu]

Hi @ManiBharathiSS

Your codes looks good and when I applied them to my code it works for me, so I would suggest you copying repository and running that code to see if issue is solved, so we can confirm there is everything ok with brownie. Please also send full consol message after running brownie test -s -k test_only_owner_can_withdraw command.

[cromewar]

Also on my side, I cloned the repo and the code runs fine... please can you follow @Neftyr suggestion @ManiBharathiSS?