Lesson 6: FAILED tests/test_fund_me.py::test_can_fund_and_withdraw - AttributeError: 'int' objec...

[BrysonHall]

I was doing the Patrick Collins tutorial link and got this error and I got this error: FAILED tests/test_fund_me.py::test_can_fund_and_withdraw - AttributeError: 'int' objec.. when I was running brownie test
Any help is appreciated.

My files:

test_fund_me.py

from scripts.helpful_scripts import get_account
from scripts.deploy import deploy_fund_me

def test_can_fund_and_withdraw():
    account = get_account()
    fund_me = deploy_fund_me()
    entrance_fee = fund_me.getEntranceFee()
    tx = fund_me.fund({"from": account, "value": entrance_fee})
    tx.wait(1)
    assert fund_me.addressToAmountFunded(account.address) == entrance_fee
    tx2 = fund_me.withdraw({"from": account})
    tx2.wait(1)
    assert fund_me.addressToAmountFunded(account.address) == 0

fund_and_withdraw.py

from brownie import FundMe
from scripts.helpful_scripts import get_account


def fund():
    fund_me = FundMe[-1]
    account = get_account()
    entrance_fee = fund_me.getEntranceFee()
    print(entrance_fee)
    print(f"The current entry fee is {entrance_fee}")
    print("Funding")
    fund_me.fund({"from": account, "value": entrance_fee})


def withdraw():
    fund_me = FundMe[-1]
    account = get_account()
    fund_me.withdraw({"from": account})


def main():
    fund()
    withdraw()

deploy.py

from brownie import FundMe, MockV3Aggregator, network, config
from scripts.helpful_scripts import (
    get_account,
    deploy_mocks,
    LOCAL_BLOCKCHAIN_ENVIRONMENTS,
)


def deploy_fund_me():
    account = get_account()
    if network.show_active() not in LOCAL_BLOCKCHAIN_ENVIRONMENTS:
        price_feed_address = config["networks"][network.show_active()][
            "eth_usd_price_feed"
        ]
    else:
        deploy_mocks()
        price_feed_address = MockV3Aggregator[-1].address

    fund_me = FundMe.deploy(
        price_feed_address,
        {"from": account},
        publish_source=config["networks"][network.show_active()].get("verify"),
    )
    print(f"Contract deployed to {fund_me.address}")
    return fund_me


def main():
    deploy_fund_me()

helpfull_scripts.py

from brownie import network, config, accounts, MockV3Aggregator
from web3 import Web3

LOCAL_BLOCKCHAIN_ENVIRONMENTS = ["development", "ganache-local"]


DECIMALS = 8
STARTING_PRICE = 200000000


def get_account():
    if network.show_active() in LOCAL_BLOCKCHAIN_ENVIRONMENTS:
        return accounts[0]
    else:
        return accounts.add(config["wallets"]["from_key"])


def deploy_mocks():
    print(f"The active network is {network.show_active()}")
    print(f"Deploying Mocks...")
    if len(MockV3Aggregator) <= 0:
        MockV3Aggregator.deploy(
            DECIMALS, Web3.toWei(DECIMALS, STARTING_PRICE), {"from": get_account()}
        )
    print("Mocks Deployed!")

FundMe.sol

// SPDX-License-Identifier: MIT

pragma solidity ^0.6.6;

import "@chainlink/contracts/src/v0.6/interfaces/AggregatorV3Interface.sol";
import "@chainlink/contracts/src/v0.6/vendor/SafeMathChainlink.sol";

contract FundMe {
    using SafeMathChainlink for uint256;

    mapping(address => uint256) public addressToAmountFunded;
    address[] public funders;
    address public owner;
    AggregatorV3Interface public priceFeed;

    constructor(address _priceFeed) public {
        priceFeed = AggregatorV3Interface(_priceFeed);
        owner = msg.sender;
    }

    function fund() public payable {
        uint256 mimimumUSD = 50 * 10**18;
        require(
            getConversionRate(msg.value) >= mimimumUSD,
            "You need to spend more ETH!"
        );
        addressToAmountFunded[msg.sender] += msg.value;
        funders.push(msg.sender);
    }

    function getVersion() public view returns (uint256) {
        return priceFeed.version();
    }

    function getPrice() public view returns (uint256) {
        (, int256 answer, , , ) = priceFeed.latestRoundData();
        return uint256(answer * 10000000000);
    }

    // 1000000000
    function getConversionRate(uint256 ethAmount)
        public
        view
        returns (uint256)
    {
        uint256 ethPrice = getPrice();
        uint256 ethAmountInUsd = (ethPrice * ethAmount) / 1000000000000000000;
        return ethAmountInUsd;
    }

    function getEntranceFee() public view returns (uint256) {
        // minimumUSD
        uint256 minimumUSD = 50 * 10**18;
        uint256 price = getPrice();
        uint256 precision = 1 * 10**18;
        // return (minimumUSD * precision) / price;
        // We fixed a rounding error found in the video by adding one!
        return ((minimumUSD * precision) / price) + 1;
    }

    modifier onlyOwner() {
        require(msg.sender == owner);
        _;
    }

    function withdraw() public payable onlyOwner {
        msg.sender.transfer(address(this).balance);

        for (
            uint256 funderIndex = 0;
            funderIndex < funders.length;
            funderIndex++
        ) {
            address funder = funders[funderIndex];
            addressToAmountFunded[funder] = 0;
        }
        funders = new address[](0);
    }
}

brownie-config.yalm

dependencies:
   # - <organization/repo>@<version>
   - smartcontractkit/chainlink-brownie-contracts@1.1.1
compiler:
  solc:
    remappings:
        - '@chainlink=smartcontractkit/chainlink-brownie-contracts@1.1.1'
dotenv: .env
networks:
  rinkeby:
    eth_usd_price_feed: '0x8A753747A1Fa494EC906cE90E9f37563A8AF630e'
    verify: True
  development:
    verify: False 
  ganache-local:
    verify: False
wallets:
  from_key: ${PRIVATE_KEY}

I also have .env file with export PRIVATE_KEY and export WEB3_INFURA_PROJECT_ID

[n4n0b1t3]

Can you give us the whole terminal error output with sensitive data removed and formatted as codeblock with "rb" as language identifier Like this:

This will create something like this here:

 {
  require 'something'

  def markdown(*args, &block)
    @markdown ||=
      Redcarpet::Markdown.new(
        SyntaxHighlighting,

[n4n0b1t3]

Just a hunch: I am wondering if you are really getting the exact value you are expecting here,
assert fund_me.addressToAmountFunded(account.address) == entrance_fee
Shouldn't this be more in a range like >= or <=?

and by the way:

so cute!

[cromewar]

@n4n0b1t3 is right should be the test >=?

Anyway, could you please share the detailed error message? as for me is related to a non well declared variable.

[BrysonHall]

========================================================================================== FAILURES =============================================================================================== 
______________________________________________________________________________________ test_can_fund_and_withdraw ______________________________________________________________________________________ 

    def test_can_fund_and_withdraw():
        account = get_account()
>       fund_me = deploy_fund_me()

tests\test_fund_me.py:7:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _  
scripts\deploy.py:16: in deploy_fund_me
    deploy_mocks()
scripts\helpful_scripts.py:23: in deploy_mocks
    DECIMALS, Web3.toWei(DECIMALS, STARTING_PRICE), {"from": get_account()}
..\..\AppData\Local\Programs\Python\Python310\lib\site-packages\web3\main.py:201: in toWei
    return cast(Wei, to_wei(number, unit))
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _  

number = 8, unit = 200000000

    def to_wei(number: Union[int, float, str, decimal.Decimal], unit: str) -> int:
        """
        Takes a number of a unit and converts it to wei.
        """
>       if unit.lower() not in units:
E       AttributeError: 'int' object has no attribute 'lower'

..\..\AppData\Local\Programs\Python\Python310\lib\site-packages\eth_utils\currency.py:68: AttributeError
----------------------------------------------------------------------------------------- Captured stdout call ----------------------------------------------------------------------------------------- 
The active network is development
Deploying Mocks...
======================================================================================= short test summary info ======================================================================================== 
FAILED tests/test_fund_me.py::test_can_fund_and_withdraw - AttributeError: 'int' object has no attribute 'lower'
========================================================================================== 1 failed in 1.22s =========================================================================================== 

[cromewar]

Based on the error for me seems to be a problem with web3 library, which python version are you using?

[BrysonHall]

Python 3.10.2

[n4n0b1t3]

this is nonsense here in your helpfull_scripts.py
Web3.toWei(DECIMALS, STARTING_PRICE)

Just compare it with the original: https://github.com/PatrickAlphaC/brownie_fund_me/blob/main/scripts/helpful_scripts.py

Pythons toWei() is used like this: https://web3py.readthedocs.io/en/stable/examples.html?highlight=towei#converting-currency-denominations

If you'd like to understand the details of what Decimals are used for, read this post: #1153

[n4n0b1t3]

Here is a post about how to better spot errors and differences from the original repo: #1267

This article might be of interest if you would like to sharpen your debugging skills: #1187

[BrysonHall]

This worked! Thanks for the resources.

[cromewar]

nice catch!