Lesson 9: Error ... 'InvalidConsumer()'

[paulcoffee85]

Just outputs 'InvalidConsumer()' even after several corrections . Is this possibly a file issue ?

Error:

Error: VM Exception while processing transaction: reverted with custom error 'InvalidConsumer()'
    at VRFCoordinatorV2Mock.onlyValidConsumer (@chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:72)
    at VRFCoordinatorV2Mock.requestRandomWords (@chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:147)

github repo => https://github.com/paulcoffee85/hh-fcc-harhat-smartcontract-lottery-fcc-TEST

Raffle.test.js:

const { getNamedAccounts, deployments, ethers, network } = require("hardhat")
const { developmentChains, networkConfig } = require("../../helper-hardhat-config")
const { assert, expect } = require("chai")

!developmentChains.includes(network.name)
    ? describe.skip
    : describe("Raffle Unit Tests", async function () {
          let raffle, vrfCoordinatorV2Mock, raffleEntranceFee, deployer, interval
          const chainId = network.config.chainId

          beforeEach(async function () {
              //   const { deployer } = await getNamedAccounts()
              // new line to get deployer
              deployer = (await getNamedAccounts()).deployer
              await deployments.fixture(["all"])
              raffle = await ethers.getContract("Raffle", deployer)
              vrfCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock", deployer)
              raffleEntranceFee = await raffle.getEntranceFee()
              interval = await raffle.getInterval()
          })

          describe("constructor", async function () {
              it("initalizes the raffle correctly", async function () {
                  // Ideally we make our tests have just 1 assert per"it"
                  const raffleState = await raffle.getRaffleState()
                  //   const interval = await raffle.getInterval()
                  //   => changed getInterval() to Global variable in beforeEach() above
                  assert.equal(raffleState.toString(), 0)
                  assert.equal(interval.toString(), networkConfig[chainId]["interval"])
              })
          })
          describe("enterRaffle", async function () {
              it("reverts when you don't pay enough", async function () {
                  await expect(raffle.enterRaffle()).to.be.revertedWith(
                      "Raffle__NotEnoughETHEntered"
                  )
              })
              it("records players when they enter", async function () {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  const playerFromContract = await raffle.getPlayer(0)
                  assert.equal(playerFromContract, deployer)
              })
              // Make sure a function emits an event
              it("emits event on enter", async function () {
                  await expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.emit(
                      raffle,
                      "RaffleEnter"
                  )
              })
              it("doesn't allow entrance when raffle is calculating", async function () {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.send("evm_mine", [])
                  // We pretend to be a Chainlink Keeper
                  await raffle.performUpkeep([])
                  await expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.be.revertedWith(
                      "Raffle__NotOpen"
                  )
              })
          })
      })

[Sunil-Reddy-Gummalla]

Hi @paulcoffee85
Invalid Consumer is Mostly because if we didn't add our smart contract address in VRF Chainlink Consumer
Go back to the part how to use Chainlink VRF in the Course
Pls also go through this link on how to ask Question guide
@alymurtazamemon can you post that link here

[alymurtazamemon]

@paulcoffee85 run npm install --save-dev @chainlink/contracts@0.4.1

[hoahamho]

Thanks mate!!! This helped me save so much time.

[satishnvrn]

I liked this answer @MohmayaGit I'm searching for a solution that doesn't downgrade the version and this helped me a lot.

[Ebenezary]

Thanks, the solution is too hidden.
I have searched and searched, almost gotten tired of searching...lol

[Vinhnv0901]

Thanks!!!

[siddharth-chauhan-it]

Thanks a ton! I was looking for a solution without downgrading the @chainlink/contracts version.

[AGMASO]

Super!!

[codersharma2001]

Thanks , I love this community

[nortln]

To solve this issue input the code on your CLI / terminal
yarn add --dev @chainlink/contracts@0.4.1

[hariprasad9899]

Thanks! That worked like a charm. Just want to add for the people in the future, to use (in Raffle.sol)

import "@chainlink/contracts/src/v0.8/interfaces/KeeperCompatibleInterface.sol";

instead of

import "@chainlink/contracts/src/v0.8/interfaces/AutomationCompatibleInterface.sol";

[AKIN-THOMAS]

Thank you

[jtsaich]

So as the error suggest, we did not add our Lottery contract as consumer to VRFCoordinator. If you do want to use the latest version of @chainlink/contracts (in my case is ^0.6.1). I updated my 01-deploy-lottery.js, after Lottery contract is deployed. I added this:

if (developmentChains.includes(network.name)) {
    const vrfCoordinatorV2Mock: VRFCoordinatorV2Mock = await ethers.getContract(
      "VRFCoordinatorV2Mock"
    );
    await vrfCoordinatorV2Mock.addConsumer(subscriptionId, lottery.address);
  }

[sonhd91]

So as the error suggest, we did not add our Lottery contract as consumer to VRFCoordinator. If you do want to use the latest version of @chainlink/contracts (in my case is ^0.6.1). I updated my 01-deploy-lottery.js, after Lottery contract is deployed. I added this:

if (developmentChains.includes(network.name)) {
    const vrfCoordinatorV2Mock: VRFCoordinatorV2Mock = await ethers.getContract(
      "VRFCoordinatorV2Mock"
    );
    await vrfCoordinatorV2Mock.addConsumer(subscriptionId, lottery.address);
  }

I wonder why we need to declare vrfCoordinatorV2Mock again. Didn't we already declare it above?

[mouchegmouradian]

Above it was declared in the if block so that variable isn't available outside of the if. You don't need to declare it again if you do something like this:

module.exports = async (hre: HardhatRuntimeEnvironment) => {
    const { getNamedAccounts, deployments, network } = hre
    const { deploy, log } = deployments
    const { deployer } = await getNamedAccounts()

    let vrfCoordinatorV2Mock: Contract | undefined
    let vrfCoordinatorV2Address
    let subscriptionId
    const entranceFee = networkConfig[network.config.chainId!]["entranceFee"]
    const gasLane = networkConfig[network.config.chainId!]["gasLane"]
    const callbackGasLimit = networkConfig[network.config.chainId!]["callbackGasLimit"]
    const interval = networkConfig[network.config.chainId!]["interval"]

    if (developmentChains.includes(network.name)) {
        vrfCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock")
        vrfCoordinatorV2Address = vrfCoordinatorV2Mock.address
        const transactionResponse = await vrfCoordinatorV2Mock.createSubscription()
        const transactionReceipt = await transactionResponse.wait()
        subscriptionId = transactionReceipt.events[0].args.subId
    //...
    } else {
    // ... 
    }

    const args = [
        vrfCoordinatorV2Address,
        entranceFee,
        gasLane,
        subscriptionId,
        callbackGasLimit,
        interval,
    ]
    const raffle = await deploy("Raffle", {
        from: deployer,
        args: args,
        log: true,
        waitConfirmations: networkConfig[network.config.chainId!].blockConfirmations || 1,
    })

    if (developmentChains.includes(network.name)) {
        vrfCoordinatorV2Mock!.addConsumer(subscriptionId, raffle.address)
    }
    // ...
}

[Plycedes]

Hello, I cannot seem to access the methods of VRFCoordinatorV2Mock

When I try to call createSubscription() it says it is not a function same goes for addConsumer() method

Here's the full error message : TypeError: vrfCoordinatorV2Mock.addConsumer is not a function

My solidity file:

//SPDX-License-Identifier: MIT

pragma solidity ^0.8.7;

import "@chainlink/contracts/src/v0.8/VRFConsumerBaseV2.sol";
import "@chainlink/contracts/src/v0.8/interfaces/VRFCoordinatorV2Interface.sol";
import "@chainlink/contracts/src/v0.8/interfaces/KeeperCompatibleInterface.sol";

error OnePiece__UpkeepNotNeeded(uint256 currentBalance, uint256 playerNum, uint256 treasureState);
error OnePiece__NotEnoughETHEntered();
error OnePiece__TransferFailed();
error OnePiece__NotOpen();

contract OnePiece is VRFConsumerBaseV2, KeeperCompatibleInterface {

    // Type Declarations
    enum TreasureState {
        OPEN,
        CALCULATING
    }

    //state variables
    address payable[] private s_players;
    address private s_recentFinder;
    TreasureState private s_treasureState;
    uint256 private s_lastTimeStamp;    

    VRFCoordinatorV2Interface private immutable i_vrfCoordinator;
    uint256 private immutable i_entranceFee;    
    bytes32 private immutable i_gasLane;
    uint64 private immutable i_subscriptionId;
    uint32 private immutable i_callbackGasLimit;
    uint256 private i_interval;

    uint16 private constant REQUEST_CONFIRMATIONS = 3;
    uint16 private constant NUM_WORDS = 1;

    event OnePieceRaceEnter(address indexed player);
    event RequestedOnePieceFinder(uint256 indexed requestId);
    event FinderPicked(address indexed finder);

    constructor(
         address vrfCoordinatorV2,
         uint256 entranceFee,
         bytes32 gasLane,
         uint64 subscriptionId,
         uint32 callbackGasLimit,
         uint256 interval
        ) VRFConsumerBaseV2(vrfCoordinatorV2){ 
        i_entranceFee = entranceFee;
        i_vrfCoordinator = VRFCoordinatorV2Interface(vrfCoordinatorV2);
        i_gasLane = gasLane;
        i_interval = interval;
        i_subscriptionId = subscriptionId;
        i_callbackGasLimit = callbackGasLimit;
        s_treasureState = TreasureState.OPEN;
        s_lastTimeStamp = block.timestamp;        
    }

    function enterOnePieceRace() public payable {
        if(msg.value < i_entranceFee){
            revert OnePiece__NotEnoughETHEntered();
        }
        if(s_treasureState != TreasureState.OPEN){
            revert OnePiece__NotOpen();
        }
        s_players.push(payable(msg.sender));

        // Emit an event when we update a dynamic array or mapping
        emit OnePieceRaceEnter(msg.sender);
    }

    // This is the function that the Chainlink Keeper nodes call
    // They look for the `upkeepNeeded` to return true. 

    function checkUpkeep(bytes memory /*checkData*/)public override
    returns (bool upkeepNeeded, bytes memory /* performData */ ) {
        bool isOpen = (TreasureState.OPEN == s_treasureState);
        bool timePassed = ((block.timestamp - s_lastTimeStamp) > i_interval); 
        bool hasPlayers = (s_players.length > 0);
        bool hasBalance = (address(this).balance > 0);
        upkeepNeeded = (isOpen && timePassed && hasPlayers && hasBalance);
        
     }

    function performUpkeep(bytes calldata /* performData */) external override{
        (bool upkeepNeeded, ) = checkUpkeep("");
        if(!upkeepNeeded){
            revert OnePiece__UpkeepNotNeeded(
                address(this).balance, 
                s_players.length, 
                uint256(s_treasureState)
            );
        }
        s_treasureState = TreasureState.CALCULATING;
        uint256 requestId = i_vrfCoordinator.requestRandomWords(
            i_gasLane,
            i_subscriptionId,
            REQUEST_CONFIRMATIONS,
            i_callbackGasLimit,
            NUM_WORDS
        );
        emit RequestedOnePieceFinder(requestId);
    }

    function fulfillRandomWords(uint256 /*requestId*/, uint256[] memory randomWords) internal override {
        uint256 finderIndex = randomWords[0] % s_players.length;
        address payable recentFinder = s_players[finderIndex];        
        s_recentFinder = recentFinder;

        s_treasureState = TreasureState.OPEN;
        s_players = new address payable[](0);
        s_lastTimeStamp = block.timestamp;

       (bool callSuccess, ) = recentFinder.call{value: address(this).balance}("");
        //require(callSuccess, "Call Failed");
        if(!callSuccess){
            revert OnePiece__TransferFailed();
        }        
        emit FinderPicked(recentFinder);
    }    

    // View functions
    function getEntranceFee () public view returns (uint256){
        return i_entranceFee;
    }

    function getPlayer(uint256 index) public view returns (address){
        return s_players[index];
    }

    function getRecentFinder() public view returns(address){
        return s_recentFinder;
    }

    function getTreasureState() public view returns (TreasureState){
        return s_treasureState;
    }

    function getNumWords() public pure returns (uint256){
        return NUM_WORDS;
    }

    function getNumOfPlayers() public view returns (uint256){
        return s_players.length;
    }
    
    function getLatestTimeStamp() public view returns (uint256){
        return s_lastTimeStamp;
    }

    function getRequestConfirmations() public pure returns (uint256){
        return REQUEST_CONFIRMATIONS;
    }

    function getInterval() public view returns(uint256){
        return i_interval;
    }
    
}

My deploy script:

const { network, ethers } = require("hardhat");
const {
    developmentChains,
    networkConfig,
} = require("../helper-hardhat-config");
const { verify } = require("../utils/verify");

const VRF_SUB_FUND_AMOUNT = ethers.parseEther("0.02");

module.exports = async function ({ getNamedAccounts, deployments }) {
    const { deploy, log } = deployments;
    const { deployer } = await getNamedAccounts();
    const chainId = network.config.chainId;

    let vrfCoordinatorv2Address, subscriptionId, vrfCoordinatorV2Mock;
    if (developmentChains.includes(network.name)) {
        vrfCoordinatorV2Mock = await deployments.get("VRFCoordinatorV2Mock");
        vrfCoordinatorv2Address = vrfCoordinatorV2Mock.address;
        /*
        const transactionResponse =
            await vrfCoordinatorV2Mock.createSubscription();
        const transactionReciept = await transactionResponse.wait(1);
        log(transactionReciept);
        subscritptionId = transactionReciept.events[1].args.subId;
        await vrfCoordinatorV2Mock.fundSubscription(
            subscritptionId,
            VRF_SUB_FUND_AMOUNT
        );*/
        subscriptionId = 0;
    } else {
        vrfCoordinatorv2Address = networkConfig[chainId]["vrfCoordinatorV2"];
        subscriptionId = networkConfig[chainId]["subscriptionId"];
    }

    const entranceFee = networkConfig[chainId]["entranceFee"];
    const gasLane = networkConfig[chainId]["gasLane"];
    const callbackGasLimit = networkConfig[chainId]["callbackGasLimit"];
    const interval = networkConfig[chainId]["interval"];

    args = [
        vrfCoordinatorv2Address,
        entranceFee,
        gasLane,
        subscriptionId,
        callbackGasLimit,
        interval,
    ];
    const onepiece = await deploy("OnePiece", {
        from: deployer,
        args: args,
        log: true,
        waitConfirmations: network.config.blockConfirmations || 1,
    });

    if (developmentChains.includes(network.name)) {
        vrfCoordinatorV2Mock.addConsumer(subscriptionId, onepiece.address);
    }

    if (
        !developmentChains.includes(network.name) &&
        process.env.ETHERSCAN_API_KEY
    ) {
        log("Verifying...");
        await verify(onepiece.address, args);
    }
    log("---------------------------------------------------------");
};

module.exports.tags = ["all", "raffle"];

I commented out the createSubscription part and hardcoded 0 to the subId to work around like a pro coder XD

Please help me out

[Jusoter]

: Use 'yarn hardhat test' with the hard-coded 'subId=1'. If you use 'yarn hardhat test --network localhost', it will work correctly.

[krish3957]

write this affter you deploy your contract which is in 01-deploy-raffle.js
if (chainId == 31337) {
await vrfCoordinatorV2Mock.addConsumer(subscriptionId.toNumber(), raffle.address)
}

//Deploying contract
const raffle = await deploy("Raffle", {
from: deployer,
args: arguments,
log: true,
waitConfirmations: waitBlockConfirmations,
})
//After deploying write this
if (chainId == 31337) {
await vrfCoordinatorV2Mock.addConsumer(subscriptionId.toNumber(), raffle.address)
}

[haritchowdhury]

Hi, I am stuck exactly at this point, I have added consumer but still get this error . It points to @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:89
which is error InsufficientGasForConsumer(uint256 have, uint256 want);
But we have already added gas. I don't want to downgrade chainlink. Here's my ssh link: git@github.com:iminparallel/SolidityLottery.git

     Error: VM Exception while processing transaction: reverted with custom error 'InvalidConsumer()'
    at VRFCoordinatorV2Mock.onlyValidConsumer (@chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:89)
    at VRFCoordinatorV2Mock.requestRandomWords (@chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:162)
    at Lottery.performUpkeep (contracts/Lottery.sol:106)
    at EdrProviderWrapper.request (node_modules/hardhat/src/internal/hardhat-network/provider/provider.ts:433:41)
    at HardhatEthersProvider.estimateGas (node_modules/@nomicfoundation/hardhat-ethers/src/internal/hardhat-ethers-provider.ts:246:27)
    at /Users/haritchowdhury/lottery/node_modules/@nomicfoundation/hardhat-ethers/src/signers.ts:235:35
    at async Promise.all (index 0)
    at HardhatEthersSigner._sendUncheckedTransaction (node_modules/@nomicfoundation/hardhat-ethers/src/signers.ts:256:7)
    at HardhatEthersSigner.sendTransaction (node_modules/@nomicfoundation/hardhat-ethers/src/signers.ts:125:18)
    at send (node_modules/ethers/src.ts/contract/contract.ts:313:20)
    at Proxy.performUpkeep (node_modules/ethers/src.ts/contract/contract.ts:352:16)
    at Context.<anonymous> (test/unit/Lottery.test.js:102:11)

[haritchowdhury]

I found a way to solve it inside solidity itself, you can even have to pass subscription id when deploying the contract. Check out this answer. smartcontractkit/chainlink#7982 (comment)