lesson 7: SmartContract Lottery #1836

sundaepromix · 2023-02-23T21:48:50Z

sundaepromix
Feb 23, 2023

here is the error in my terminal

Running 'scripts/deploy_lottery.py::main'...
Transaction sent: 0xfd99566180954fef0805cea66c6a3011ca99d86475a901d34267d0eae2c109e3
Gas price: 0.0 gwei Gas limit: 12000000 Nonce: 0
MockV3Aggregator.constructor confirmed Block: 1 Gas used: 430659 (3.59%)
MockV3Aggregator deployed at: 0x3194cBDC3dbcd3E11a07892e7bA5c3394048Cc87

Transaction sent: 0x8ef2772ab557cf6e7bb5088682c154788b79e379341c296acc3572470547e073
Gas price: 0.0 gwei Gas limit: 12000000 Nonce: 1
LinkToken.constructor confirmed Block: 2 Gas used: 669136 (5.58%)
LinkToken deployed at: 0x602C71e4DAC47a042Ee7f46E0aee17F94A3bA0B6

Transaction sent: 0x737d009e9827546c81bb7567a7dd980f9b4c36ba10eae9579866d60c656851b7
Gas price: 0.0 gwei Gas limit: 12000000 Nonce: 2
VRFCoordinatorMock.constructor confirmed Block: 3 Gas used: 276395 (2.30%)
VRFCoordinatorMock deployed at: 0xE7eD6747FaC5360f88a2EFC03E00d25789F69291

deployed!
File "brownie/_cli/run.py", line 51, in main
return_value, frame = run(
File "brownie/project/scripts.py", line 110, in run
return_value = f_locals[method_name](*args, **kwargs)
File "./scripts/deploy_lottery.py", line 19, in main
deploy_lottery()
File "./scripts/deploy_lottery.py", line 8, in deploy_lottery
get_contract("eth_usd_price_feed").address,
AttributeError: 'NoneType' object has no attribute 'address'
Terminating local RPC client...

Answered by cromewar

Feb 24, 2023

Hello @sundaepromix
Additionally to what @Neftyr said, the none type is saying that is not able to get the contract itself.
The reasons could be various:

if you deploy a contract o a script, and the try to use it in another script for local development on ganache. It won't find the contract as when you deploy to local you wont have a build folder with the addresses of the last deployments. Try to run everything on the same execution
Check out your are deploying the mocks as @Neftyr suggested.
Try to find any typo on your imports.

View full answer

sundaepromix · 2023-02-23T21:50:12Z

sundaepromix
Feb 23, 2023
Author

deploy_lottery.py

from scripts.helpful_scripts import get_account, get_contract
from brownie import Lottery, network, config

def deploy_lottery():
    account = get_account()
    lottery = Lottery.deploy(
        get_contract("eth_usd_price_feed").address,
        get_contract("vrf_coordinator").address,
        get_contract("link_token").address,
        config["networks"][network.show_active()]["fee"],
        config["networks"][network.show_active()]["keyhash"],
        {"from": account},
        publish_source=config["networks"][network.show_active()].get("verify", False),)
    print("Deployed lottery!")
    return lottery

def main():
    deploy_lottery()

0 replies

sundaepromix · 2023-02-23T21:51:25Z

sundaepromix
Feb 23, 2023
Author

brownie-config.yaml

dependencies:
  - smartcontractkit/chainlink-brownie-contracts@1.1.1
  - OpenZeppelin/openzeppelin-contracts@3.4.0
compiler:
  solc:
    remappings:
      - '@chainlink=smartcontractkit/chainlink-brownie-contracts@1.1.1'
      - '@openzeppelin=OpenZeppelin/openzeppelin-contracts@3.4.0'
networks:
  default: development
  development: 
    keyHash: "0x2ed0feb3e7fd2022120aa84fab1945545a9f2ffc9076fd6156fa96eaff4c1311"
    fee: 100000000000000000
  goerli: 
    vrf_coordinator: "0x2bce784e69d2Ff36c71edcB9F88358dB0DfB55b4"
    eth_usd_price_feed: "0xD4a33860578De61DBAbDc8BFdb98FD742fA7028e"
    link_token: "0x326C977E6efc84E512bB9C30f76E30c160eD06FB"
    keyHash: "0x0476f9a745b61ea5c0ab224d3a6e4c99f0b02fce4da01143a4f70aa80ae76e8a"
    fee: 100000000000000000
    verify: True
  mainnet-fork:
   eth_usd_price_feed: "0x5f4eC3Df9cbd43714FE2740f5E3616155c5b8419"
   verify: False
wallets: 
  from key: $(PRIVATE_KEY)

1 reply

sundaepromix Feb 23, 2023
Author

please i need help

Nlferu · 2023-02-24T15:11:23Z

Nlferu
Feb 24, 2023

Hello @sundaepromix

Can you pass link to your repository if you have one? I think there is error in your helpful_scripts.py or you just need to add eth_usd_price_feed: in your development section in brownie-config.yaml file as for now you have price feed only for goerli network. Check if your get_contract() fucntion returns contract

contract = contract_type[-1]
or
contract = Contract.from_abi(contract_type._name, contract_address, contract_type.abi)

And check if you have mapping implemented in helpful_scripts.py like below:

contract_to_mock = {
    "eth_usd_price_feed": MockV3Aggregator,
    "vrf_coordinator_v2": VRFCoordinatorV2Mock,
    "link_token": LinkToken
}

As the problem is that your get_contract() function doesn't return object with address argument.

1 reply

sundaepromix Feb 24, 2023
Author

like what @cromewar said, there was a typo error in my helpful_scripts.
thanks @Neftyr and @cromewar you guys are the best

cromewar · 2023-02-24T15:52:08Z

cromewar
Feb 24, 2023

Hello @sundaepromix
Additionally to what @Neftyr said, the none type is saying that is not able to get the contract itself.
The reasons could be various:

if you deploy a contract o a script, and the try to use it in another script for local development on ganache. It won't find the contract as when you deploy to local you wont have a build folder with the addresses of the last deployments. Try to run everything on the same execution
Check out your are deploying the mocks as @Neftyr suggested.
Try to find any typo on your imports.

0 replies

sundaepromix · 2023-02-24T20:20:33Z

sundaepromix
Feb 24, 2023
Author

Thanks so much for this, your the best.. Please is there any I can get your contact for mentorship.. I'm new in the world of Blockchain

…

On Fri, Feb 24, 2023, 4:52 PM Vasiliy Gualoto ***@***.***> wrote: Hello @sundaepromix <https://github.com/sundaepromix> Additionally to what @Neftyr <https://github.com/Neftyr> said, the none type is saying that is not able to get the contract itself. The reasons could be various: 1. if you deploy a contract o a script, and the try to use it in another script for local development on ganache. It won't find the contract as when you deploy to local you wont have a build folder with the addresses of the last deployments. Try to run everything on the same execution 2. Check out your are deploying the mocks as @Neftyr <https://github.com/Neftyr> suggested. 3. Try to find any typo on your imports. — Reply to this email directly, view it on GitHub <#1836 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A5IM265LCD2EF4CZZINAJITWZDKLHANCNFSM6AAAAAAVGFFI3U> . You are receiving this because you were mentioned.Message ID: <smartcontractkit/full-blockchain-solidity-course-py/repo-discussions/1836/comments/5101773 @github.com>

1 reply

cromewar Feb 28, 2023

you can follow me on twitter "@cromewar" I always have the DMs open. Also you can keep posting your questions here.

Aishwaria8786 · 2023-04-23T15:12:04Z

Aishwaria8786
Apr 23, 2023

@cromewar Can you please help me in solving this problem?

tests/test_lottery_integration.py:39: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

    def deploy_lottery():
        account = get_account()
>       lottery = Lottery.deploy(
            get_contract("eth_usd_price_feed").address,
            get_contract("vrf_coordinator").address,
            config["networks"][network.show_active()]["fee"],
            get_contract("link_token").address,
            config["networks"][network.show_active()]["keyhash"],
            {"from": account},
            publish_source=config["networks"][network.show_active()].get("verify", False),
        )
E       ValueError: insufficient funds for gas * price + value

scripts/deploy_lottery.py:8: ValueError
========================================================== short test summary info ==========================================================
FAILED tests/test_lottery_integration.py::test_can_pick_winner - ValueError: insufficient funds for gas * price + value
================================================ 1 failed, 1 skipped, 4 deselected in 11.67s ================================================

1 reply

cromewar Apr 23, 2023

Be sure you are suing the correct account @Aishwaria8786

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

lesson 7: SmartContract Lottery #1836

{{title}}

Replies: 6 comments 4 replies

{{title}}

{{editor}}'s edit

{{editor}}'s edit

{{title}}

{{editor}}'s edit

{{editor}}'s edit

{{title}}

{{title}}

{{editor}}'s edit

{{editor}}'s edit

{{title}}

{{title}}

{{title}}

{{title}}

{{title}}

{{title}}

Select a reply

lesson 7: SmartContract Lottery #1836

sundaepromix Feb 23, 2023

Replies: 6 comments · 4 replies

sundaepromix Feb 23, 2023 Author

sundaepromix Feb 23, 2023 Author

sundaepromix Feb 23, 2023 Author

Nlferu Feb 24, 2023

sundaepromix Feb 24, 2023 Author

cromewar Feb 24, 2023

sundaepromix Feb 24, 2023 Author

cromewar Feb 28, 2023

Aishwaria8786 Apr 23, 2023

cromewar Apr 23, 2023

sundaepromix
Feb 23, 2023

Replies: 6 comments 4 replies

sundaepromix
Feb 23, 2023
Author

sundaepromix
Feb 23, 2023
Author

sundaepromix Feb 23, 2023
Author

Nlferu
Feb 24, 2023

sundaepromix Feb 24, 2023
Author

cromewar
Feb 24, 2023

sundaepromix
Feb 24, 2023
Author

Aishwaria8786
Apr 23, 2023