Lesson9: TypeError: Cannot read properties of undefined (reading 'toNumber')

[Arpit7707]

While testing performUpkeep() I got this error:

 1) Raffle
       performUpkeep
         updates the raffleState and emits the event, and calls the vrfCoordinator:
     TypeError: Cannot read properties of undefined (reading 'toNumber')
      at Context.<anonymous> (test/unit/Raffle.test.js:115:36)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at runNextTicks (node:internal/process/task_queues:65:3)
      at listOnTimeout (node:internal/timers:528:9)
      at processTimers (node:internal/timers:502:7)

code for testing

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

!developementChains.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"]) //to deploy all contracts with tag "all"
              raffle = await ethers.getContract("Raffle", deployer) // Returns a new connection to the Raffle contract
              vrfCoordinatorV2Mock = await ethers.getContract("VRFCoordinatorV2Mock") // Returns a new connection to the VRFCoordinatorV2Mock contract
              raffleEntranceFee = await raffle.getEntranceFee()
              interval = await raffle.getInterval()
          })

          describe("constructor", async function () {
              it("initialize the raffle contract correctly", async function () {
                  //Ideally we make out tests have just 1 assert per "it"
                  const raffleState = await raffle.getRaffleState()
                  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 player when they enter", async function () {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  const playerFromContract = await raffle.getPlayers(0)
                  assert.equal(playerFromContract, deployer)
              })
              it("emits event when enter", async function () {
                  await expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.emit(
                      raffle,
                      "RaffleEnter"
                  )
              })
              it("dosen't allow entrance when raffle is calculating", async function () {
                  //following three lines are used to make checkUpKeep true
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1]) //increase the time of our blockchain by the interval
                  await network.provider.send("evm_mine", []) //to mine one block
                  //we pretend to be chainlink keeper
                  await raffle.performUpkeep([]) //now rafflestate is: calculating
                  await expect(raffle.enterRaffle({ value: raffleEntranceFee })).to.be.revertedWith(
                      "Raffle_NotOpen"
                  )
              })
          })

          describe("checkUpkeep", async function () {
              it("returns false if people haven't send any ETH", async function () {
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.send("evm_mine", [])
                  const { upKeepNeeded } = await raffle.callStatic.checkUpkeep([])
                  //callstatic is used to simulate sending transaction to any public function, checkUpkeep in this case
                  assert(!upKeepNeeded)
              })
              it("returns false if raffle isn't open", async function () {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.send("evm_mine", [])
                  await raffle.performUpkeep([]) //"0x0" == [] , passing empty byte string //now rafflestate is: calculating
                  const raffleState = await raffle.getRaffleState()
                  const { upKeepNeeded } = await raffle.callStatic.checkUpkeep([])
                  assert.equal(raffleState.toString(), "1")
                  assert.equal(upKeepNeeded, false)
              })
              it("returns false if enough time hasn't passed", async () => {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() - 5]) // use a higher number here if this test fails
                  await network.provider.request({ method: "evm_mine", params: [] })
                  const { upkeepNeeded } = await raffle.callStatic.checkUpkeep("0x") // upkeepNeeded = (timePassed && isOpen && hasBalance && hasPlayers)
                  assert(!upkeepNeeded)
              })
              it("returns true if enough time has passed, has players, eth, and is open", async () => {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.request({ method: "evm_mine", params: [] })
                  const { upKeepNeeded } = await raffle.callStatic.checkUpkeep([]) // upkeepNeeded = (timePassed && isOpen && hasBalance && hasPlayers)
                  assert(upKeepNeeded)
              })
          })
          describe("performUpkeep", async function () {
              it("it can only run is checkUpKeep is true", async function () {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.request({ method: "evm_mine", params: [] })
                  const tx = await raffle.performUpkeep([])
                  assert(tx)
              })
              it("reverts when checkUpKeep is false", async function () {
                  await expect(raffle.performUpkeep([])).to.be.revertedWith(
                      "Raffle_UpKeepNotNeeded"
                  )
              })
              it("updates the raffleState and emits the event, and calls the vrfCoordinator", async function () {
                  await raffle.enterRaffle({ value: raffleEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.request({ method: "evm_mine", params: [] })
                  //getting requestId from transaction receipt
                  const txResponse = await raffle.performUpkeep([])
                  const txReceipt = await txResponse.wait(1)
                  const requestId = txReceipt.events[1].args.requestId

                  const raffleState = await raffle.getRaffleState()
                  assert(requestId.toNumber() > 0) //checking for emittion of events
                  assert.equal(raffleState.toString(), "1") //checking for updation of  raffleState
              })
          })
      })

Can anyone help?

[alymurtazamemon]

@Arpit7707 Here they made a litter update;

-const requestId = txReceipt.events[1].args.requestId
+const requestId = txReceipt.events[1].args.requestID

[alymurtazamemon]

@Arpit7707 try to log the txReceipt and see what is inside it.

[alymurtazamemon]

@Arpit7707 It should contains two object and second one will have the requestId

[Arpit7707]

Well it is actually "RequestId"....Error is resolved....Thanks

[alymurtazamemon]

@Arpit7707 ok so now it is updated even more.

[mulugeta-89]

You should change your lines of code like this

const requestId = await transactionReceipt.events[1].args[0]
assert(requestId.toNumber() > 0)