Lesson #9 Error: VM Exception while processing transaction: reverted with custom error 'InvalidConsumer()'

[Night-Prime]

I have been trying to solve this error by trying the suggestions made in similar discussions around it, but it seems to lead to another error and it's like i'm circling back.

First i had this error, after running test :

Raffle
    constructor
      ✔ initializes the raffle correctly
    enter Raffle
      ✔ reverts when you don't pay enough
      ✔ records players when they enter
      ✔ emits events on enter
      1) doesn't allow entrance when raffle is calculating


  4 passing (985ms)
  1 failing

  1) Raffle
       enter Raffle
         doesn't allow entrance when raffle is calculating:
     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)
    at Raffle.performUpkeep (contracts/Raffle.sol:114)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at runNextTicks (node:internal/process/task_queues:64:3)
    at listOnTimeout (node:internal/timers:533:9)

On the similar discussions, i saw that i had to install a specific version of @chainlinks/contracts : npm i --save-dev @chainlink/contracts@0.4.0 (of which i had the latest version at the time), after the installation, i got this error:

Error HH404: File @chainlink/contracts/src/v0.8/interfaces/AutomationCompatibleInterface.sol, imported from contracts/Raffle.sol, not found.

The suggestions for similar discussions is that i had to upgrade back to the latest version of @chainlink/contracts i was using.

Here's my Raffle.test.js code :
`
const { network, getNamedAccounts, deployments, ethers } = require("hardhat");
const { developmentChains, networkConfig } = require("../../helper-hardhat-config");
const { assert, expect } = require("chai");

!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);
          //   const subscriptionId = raffle.getSubscriptionId();
          //   await vrfCoordinatorV2Mock.addConsumer(subscriptionId, raffle.address);
          raffleEntranceFee = await raffle.getEntranceFee();
          interval = await raffle.getInterval();
      });

      describe("constructor", async function () {
          it("initializes the raffle correctly", async function () {
              const raffleState = await raffle.getRaffleState();
              assert.equal(raffleState.toString(), "0");
              assert.equal(interval.toString(), networkConfig[chainId]["interval"]);
          });
      });

      describe("enter Raffle", async function () {
          it("reverts when you don't pay enough", async function () {
              await expect(raffle.enterRaffle()).to.be.revertedWith("Raffle__NotEnoughFee");
          });
          it("records players when they enter", async function () {
              await raffle.enterRaffle({ value: raffleEntranceFee });
              const playerFromContract = await raffle.getPlayer(0);
              assert.equal(playerFromContract, deployer);
          });
          it("emits events on enter", async function () {
              await expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.emit(
                  raffle,
                  "RaffleEnter"
              );
          });
          it("doesn't allow entrance when raffle is calculating", async () => {
              await raffle.enterRaffle({ value: raffleEntranceFee });
              // for a documentation of the methods below, go here: https://hardhat.org/hardhat-network/reference
              await network.provider.send("evm_increaseTime", [interval.toNumber() + 1]);
              await network.provider.request({ method: "evm_mine", params: [] });
              // we pretend to be a keeper for a second
              await raffle.performUpkeep([]); // changes the state to calculating for our comparison below
              await expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.be.revertedWith(
                  "Raffle__NotOpen"
              );
          });
      });
  });

`

my hardhat.config.js :

/** @type import('hardhat/config').HardhatUserConfig */
require("@nomiclabs/hardhat-waffle");
require("@nomiclabs/hardhat-etherscan");
require("hardhat-deploy");
require("solidity-coverage");
require("hardhat-gas-reporter");
require("hardhat-contract-sizer");
require("dotenv").config();

const SEPOLIA = process.env.SEPOLIA_RPC_URL;
const PRIVATE_KEY = process.env.PRIVATE_KEY;
const COINMARKET_KEY = process.env.REPORT_GAS;
const ETHERSCAN_API_KEY = process.env.ETHERSCAN_API_KEY;

module.exports = {
    solidity: "0.8.18",
    defaultNetwork: "hardhat",
    networks: {
        hardhat: {
            chainId: 31337,
            blockConfirmations: 1,
        },
        sepolia: {
            chainId: 11155111,
            blockConfirmations: 6,
            url: SEPOLIA,
        },
    },
    gasReporter: {
        enabled: false,
        currency: "USD",
        outputFile: "gas-report.txt",
        noColors: true,
    },
    namedAccounts: {
        deployer: {
            default: 0,
        },
        player: {
            default: 1,
        },
    },
};

and my package.json:

{
  "devDependencies": {
    "@chainlink/contracts": "^0.4.0",
    "@nomiclabs/hardhat-ethers": "npm:hardhat-deploy-ethers@^0.3.0-beta.13",
    "@nomiclabs/hardhat-etherscan": "^3.1.7",
    "@nomiclabs/hardhat-waffle": "^2.0.6",
    "chai": "^4.3.7",
    "dotenv": "^16.1.3",
    "ethereum-waffle": "^4.0.10",
    "ethers": "^5.7.2",
    "hardhat": "^2.14.1",
    "hardhat-contract-sizer": "^2.9.0",
    "hardhat-deploy": "^0.11.30",
    "hardhat-gas-reporter": "^1.0.9",
    "prettier": "^2.8.8",
    "prettier-plugin-solidity": "^1.1.3",
    "solhint": "^3.4.1",
    "solidity-coverage": "^0.8.2"
  }
}

[alfaqi]

Did you add Consumer while deploying the contract?, like this:
await vrfCoordinatorV2Mock.addConsumer(subscriptionId, raffle.address)
see here https://github.com/PatrickAlphaC/hardhat-smartcontract-lottery-fcc/blob/main/deploy/01-deploy-raffle.js

[Night-Prime]

Did that and now i'm having this :

"before each" hook for "initializes the raffle correctly":
     ERROR processing /Users/mac/Documents/Codes/blockchain-projects/Lottery-web3/deploy/01-deploy-raffle.js:
ReferenceError: vrfCoordinatorV2Mock is not defined

Here's my github repo: https://github.com/Night-Prime/Lottery-web3.git

[alymurtazamemon]

@Night-Prime here;

const vrfCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock");

you should remove the const keyword and define vrfCoordinatorV2Mock with above let variables so later you can use it for addConsumer

[Night-Prime]

I was able to fix the problem, thank you very much

[aaankeet]

@Night-Prime

In you Raffle contract why are you inheriting from AutomationCompatibleInterface you don't need that use KeeperCompatibleInterface instead. In your 01-deploy-raffle.js remove
await vrfCoordinatorV2Mock.addConsumer(subscriptionId, raffle.address); on line 21. and paste this code after deploying raffle
(maybe on line 49).

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

In your Raffle-test.js on line 178 raffle is missing to call preformUpkeep. There are still some failing test cases Good Luck with that :)