LESSON 9: Error Running Test [15:35:49]

[DorianDaSilva]

| Making reference to this function in VRFCoordinatorV2Mock.sol

  modifier onlyValidConsumer(uint64 _subId, address _consumer) {
    if (!consumerIsAdded(_subId, _consumer)) {
      revert InvalidConsumer();
    }
    _;
  }

======================================================================================

There is no apparent reason for the test to break as my setup, dependencies and script are in theory identical to the one in the video besides from running all tests succesfully until now.

| Below is my function

              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"
                  )

======================================================================================

| Here is the setup

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

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

          beforeEach(async function () {
              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()
          })

======================================================================================

Any ideas of what I may be missing here, please? Should I define, import or declare a consumer()?

Thank you!

[alymurtazamemon]

@PatrickAlphaC

[PatrickAlphaC]

Which line is erroring? If you can't find out from the stack trace, can you do some console.logs to find out?

Also, please don't take screenshots of errors, please copy-paste them.

[PatrickAlphaC]

What's your fixture look like? (your deploy script)

[DorianDaSilva]

@PatrickAlphaC
Below 01-deploy-raffle.js script

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

const VRF_SUB_FUND_AMOUNT = ethers.utils.parseEther("3");

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

  if (developmentChains.includes(network.name)) {

    const VRFCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock");
    vrfCoordinatorV2Address = VRFCoordinatorV2Mock.address;
    const transactionResponse = await VRFCoordinatorV2Mock.createSubscription();
    const transactionReceipt = await transactionResponse.wait(1);
    subscriptionId = transactionReceipt.events[0].args.subId;
    await VRFCoordinatorV2Mock.fundSubscription(
      subscriptionId,
      VRF_SUB_FUND_AMOUNT
    );
  } else {
    vrfCoordinatorV2Address = networkConfig[chainId]["vrfCoordinatorV2"];
    subscriptionId = networkConfig[chainId]["subscritpionId"];
  }

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

  const args = [
    vrfCoordinatorV2Address,
    entranceFee,
    gasLane,
    subscriptionId,
    callbackGasLimit,
    interval,
  ];

  const raffle = await deploy("Raffle", {
    from: deployer,
    args: args,
    log: true,
    waitConfirmations: network.config.blockConfirmations || 1,
  });

  if (
    !developmentChains.includes(network.name) &&
    process.env.ETHERSCAN_API_KEY
  ) {
    console.log("verifying...");
    await verify(raffle.address, args);
  }

  console.log(
    "--------------------------------------------------------------------------"
  );
};

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

============================================================================

Additionally, the same error repeats when running:

it("returns false if raffle isn't open", async function () {}

=================================================================================

| Code breaks once again at:

await raffle.performUpkeep("0x")

Issue is definitely linked to performUpkeep() although Raffle.sol compiles & deploys properly

[PatrickAlphaC]

What happens when you git clone from the scratch? Do you still get the error?

(make a new folder, run git clone https://github..., yarn and whatever other setup to get to where you are)

It could be an issue with your code.

[DorianDaSilva]

Do you mean git cloning your repo?

[alymurtazamemon]

@DorianDaSilva Yes

[krakxn]

@DorianDaSilva Try doing this:

beforeEach(async () => {
              accounts = await ethers.getSigners() 
              //   deployer = accounts[0]
              player = accounts[1]
              await deployments.fixture(["mocks", "raffle"])
              vrfCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock")
              raffleContract = await ethers.getContract("Raffle")
              raffle = raffleContract.connect(player)
              raffleEntranceFee = await raffle.getEntranceFee()
              interval = await raffle.getInterval()
          })

[DorianDaSilva]

@krakxn Thank you for the input! I get the following errors when running tests using your beforeEach() function

  1) Raffle
       enterRaffle
         records players when they enter:
     AssertionError: expected '0x70997970C51812dc3A010C7d01b....…' to equal SignerWithAddress{ …(4) }
      at Context.<anonymous> (test/unit/Raffle.test.js:52:26)

  2) Raffle
       enterRaffle
         doesn't allow entrance when raffle is calculating:
     Error: VM Exception while processing transaction: reverted with custom error 'InvalidConsumer()'

[DorianDaSilva]

@PatrickAlphaC @ali-thegilfoyle @krakxn

Additionally I attempted to remove the await keyword from the problem function

                   raffle.performUpkeep([])
                   expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.be.revertedWith(
                       "Raffle__NotOpen"
                   )

=====================================================================================

It passes since it skips this section as seen below

| However, I still get the same error but this time it is pointing to a code line in provider/node.ts

/home/user/hh-fcc/hardhat-smartcontract-lottery-fcc/node_modules/hardhat/src/internal/hardhat-network/provider/node.ts:1773
          traces.push(await this._gatherTraces(txResult.execResult));
                      ^
Error: VM Exception while processing transaction: reverted with custom error 'InvalidConsumer()'

Could it be related to a provider issue or is this irrelevant to the issue?

[alymurtazamemon]

@DorianDaSilva is it your error name in a contract "Raffle__NotOpen"?

[DorianDaSilva]

@ali-thegilfoyle Yes, it is in Raffle.sol along with the other error declarations

error Raffle__NotEnoughETHEntered();
error Raffle__TransferFailed();
error Raffle__NotOpen();
error Raffle__UpkeepNotNeeded(uint256 currentBalance, uint256 numPlayers, uint256 raffleState);

| The issue is definitely with performUpkeep([]) which seems to be working fine in Raffle.sol

Should I add a catch method to handle VM custom errors in 01-raffle-deploy.js script or Raffle.sol?

[alymurtazamemon]

@PatrickAlphaC

[DorianDaSilva]

Below coverage result => VRFCoordinatorV2Mock.sol is at 100% in video which is not the case here.

[g01xm]

Hi, I running to the same issue today, searched on google and found "https://ethereum.stackexchange.com/questions/131426/chainlink-keepers-getting-invalidconsumer", I tried "yarn add --dev @chainlink/contracts@0.4.0" and it resolves the issue.

[IamGalexing]

This solution helped me as well!
I had the same issue and after @chainlink/contracts@0.4.0 installation the issue disappeared.
I had @chainlink/contracts@0.4.2 in the beginning.

[letallan]

This helped me too, thx, mate!

[cah311]

Same here. Ran into the exact same error but this fixed it.

[ManuWeb3]

Ran into same issue just now and yarn add --dev @chainlink/contracts@0.4.0 helped.
Thanks everyone.