Lesson 9 : Error for last unit test . Uncaught RuntimeError

[Lavishq]

error

              
              1) Raffle Unit Tests
       fulfillRandomWords
         picks a winner, resets the raffle, and sends money:
     Uncaught RuntimeError: abort(ReferenceError: i is not defined). Build with -s ASSERTIONS=1 for more info.
      at process.abort (node_modules/@ethereum-waffle/compiler/node_modules/solc/soljson.js:1:13938)
      at process.emit (node:events:539:35)
      at process.emit (node_modules/source-map-support/source-map-support.js:516:21)
      at emit (node:internal/process/promises:140:20)
      at processPromiseRejections (node:internal/process/promises:274:27)
      at processTicksAndRejections (node:internal/process/task_queues:97:32)
      at runNextTicks (node:internal/process/task_queues:65:3)
      at listOnTimeout (node:internal/timers:528:9)
      at processTimers (node:internal/timers:502:7)

code

it("picks a winner, resets the raffle, and sends money", async function () {
                  const additionalEntrants = 3
                  const startingAccountIndex = 1 // deployer is 0
                  const accounts = await ethers.getSigners()
                  for (
                      let i = startingAccountIndex;
                      i < startingAccountIndex + additionalEntrants;
                      i++
                  ) {
                      const accountConnectedRaffle = raffle.connect(accounts[i])
                      await accountConnectedRaffle.enterRaffle({ value: raffleEntranceFee })
                  }
                  const startingTimeStamp = await raffle.getLatestTimestamp()

                  // performUpkeep (mock being chainlink keepers)
                  // fulfillRandomWords (mock being the chainlink vrf)
                  // we will have to wait for the fulfillRandomWords to be called
                  await new Promise(async (resolve, reject) => {
                      raffle.once("WinnerPicked", async () => {
                          console.log("found the event!")
                          try {
                              const recentWinner = await raffle.getRecentWinner()
                      
                              const raffleState = await raffle.getRaffleState()
                              const endingTimeStamp = await raffle.getLatestTimestamp()
                              const numPlayers = await raffle.getNumberOfPlayers()
                              const winnerEndingBalance = await accounts[i].getBalance()

                              assert.equal(numPlayers.toString(), "0")
                              assert.equal(raffleState.toString(), "0")
                              assert(endingTimeStamp > startingTimeStamp)

                              assert.equal(
                                  winnerEndingBalance.toString(),
                                  winnerStartingBalance.add(
                                      raffleEntranceFee
                                          .mul(additionalEntrants)
                                          .add(raffleEntranceFee)
                                          .toString()
                                  )
                              )
                          } catch (e) {
                              reject(e)
                          }
                          resolve()
                      })

                      // setting up the listerner
                      // below, we wil fire the event, and the listeneerr will pick it up, and resolve
                      const tx = await raffle.performUpkeep([])
                      const txReceipt = await tx.wait(1)
                      const winnerStartingBalance = await accounts[i].getBalance()
                      await vrfCoordinatorV2Mock.fulfillRandomWords(
                          txReceipt.events[1].args.requestId,
                          raffle.address
                      )
                  })
              })

[alymurtazamemon]

@Lavishq Here const winnerStartingBalance = await accounts[i].getBalance() you use i variable that is out of loop scope

Also here const winnerEndingBalance = await accounts[i].getBalance()

And in unit tests because we are using mocks so the winner will always be same, you use 0 instead of i

[Lavishq]

thanks, i will quickly finish the last test got me stuck for a hour (excluding time to write) i rechecked the video and looked on internet and github repo. But a really silly i was invisible to me :)