Lesson 9: Unit Test - Timeout 500s Error. (on localhost)

[RohitKS7]

Hey, Guys!
I know this error is pretty common and have so many answers in discussion section But unfortunately I can't find a solution for mine code. I think I'm doing something wrong but still can't find the problem. That's Why I'm hoping if someone can Help me to get the solution.

ERROR: Timeout of 500000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves

Lottery.test.js

describe("fulfillRandomWords", () => {
              beforeEach(async () => {
                  await lottery.enterLottery({ value: lotteryEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.send("evm_mine", [])
              })

              it("picks a winner, resets the lottery, and sends money", async () => {
                  const accounts = await ethers.getSigners()
                  const startingAccountIndex = 1 // since deployer is 0. New accounts will starts from index 1
                  const additionalEntrances = 3
                  for (let i = startingAccountIndex; i <= additionalEntrances; i++) {
                      const accountConnectedLottery = lottery.connect(accounts[i])
                      await accountConnectedLottery.enterLottery({ value: lotteryEntranceFee })
                  }

                  const startingTimeStamp = await lottery.getLastTimeStamp()

                  await new Promise(async (resolve, reject) => {
                      lottery.once("WinnerPicked", async () => {
                          console.log("Found the events")
                          try {
                              const recentWinner = await lottery.getRecentwinner()
                              console.log(recentWinner)
                              console.log(accounts[0].address)
                              console.log(accounts[1].address)
                              console.log(accounts[2].address)
                              console.log(accounts[3].address)
                              const lotteryState = await lottery.getLotteryState()
                              const playersCount = await lottery.getNumberOfPlayers()
                              const endingTimeStamp = await lottery.getLastTimeStamp()
                              assert.equal(playersCount.toString(), "0")
                              assert.equal(lotteryState.toString(), "0")
                              assert(endingTimeStamp > startingTimeStamp)
                          } catch (e) {
                              reject(e)
                          }
                          resolve()
                      })
                      const tx = await lottery.performUpkeep([])
                      const txReceipt = tx.wait(1)
                      await vrfCoordinatorV2Mock.fulfillRandomWords(
                          txReceipt.events[1].args.requestId,
                          lottery.address
                      )
                  })
              })
          })

fulfillRandomWords Function

function fulfillRandomWords(
        uint256, /*requestId*/
        uint256[] memory randomWords
    ) internal override {
        uint256 indexOfWinner = randomWords[0] % s_players.length;
        address payable recentWinner = s_players[indexOfWinner];
        s_recentWinner = recentWinner;
        s_lotteryState = LotteryState.OPEN;
        s_players = new address payable[](0);
        s_lastTimeStamp = block.timestamp;
        (bool success, ) = recentWinner.call{value: address(this).balance}("");
        if (!success) {
            revert Lottery__TransferFailed();
        }
        emit WinnerPicked(recentWinner);
    }

If you need more code then here is my repo link
https://github.com/RohitKS7/hardhat-lottery-smartcontract

Thanks In Advance!

[othaime-en]

Hey @RohitKS7 you Lotter.test.js file doesn't have the code above. Please push all the updates and notify me.

[adityabhattad2021]

Hey @othaime-en, I don't know If this is the correct way to ask for help but I am really struggling with this #2068 I did checked moralis forum but I did not solve my problem.

[othaime-en]

Hey @adityabhattad2021 Sorry but I'm more of a backend/blockend guy. I haven't really focused on frontend at the moment. Try asking @PatrickAlphaC and @alymurtazamemon. They might help.

[RohitKS7]

@othaime-en Bro, I've updated the repo.

[RohitKS7]

@othaime-en

I have tried to do this test without any promise and It worked. But this might never work on testnet.

 it("picks a winner, resets the lottery, and sends money", async () => {
                  await lottery.enterLottery({ value: lotteryEntranceFee })
                  await network.provider.send("evm_increaseTime", [interval.toNumber() + 1])
                  await network.provider.send("evm_mine", [])
                  const accounts = await ethers.getSigners()
                  const startingAccountIndex = 1
                  const additionalEntrances = 3
                  for (let i = startingAccountIndex; i <= additionalEntrances; i++) {
                      const accountConnectedLottery = lottery.connect(accounts[i])
                      await accountConnectedLottery.enterLottery({ value: lotteryEntranceFee })
                  }

                  const startingTimeStamp = await lottery.getLastTimeStamp()
                  const tx = await lottery.performUpkeep([])
                  const txReceipt = await tx.wait(1)
                  await vrfCoordinatorV2Mock.fulfillRandomWords(
                      txReceipt.events[1].args.requestId,
                      lottery.address
                  )

                  const recentWinner = await lottery.getRecentwinner()
                  console.log(recentWinner)
                  console.log(accounts[0].address)
                  console.log(accounts[1].address)
                  console.log(accounts[2].address)
                  console.log(accounts[3].address)
                  const lotteryState = await lottery.getLotteryState()
                  const playersCount = await lottery.getNumberOfPlayers()
                  const endingTimeStamp = await lottery.getLastTimeStamp()
                  assert.equal(playersCount.toString(), "0")
                  assert.equal(lotteryState.toString(), "0")
                  assert(endingTimeStamp > startingTimeStamp)
              })

and I have a question if you know-

Why Everytime account 1 wins?
It should be random, NO?

What I think is That mocks are already set with a particular Number!

[adityabhattad2021]

Hey, put this code:

const tx = await lottery.performUpkeep([])
                      const txReceipt = tx.wait(1)
                      await vrfCoordinatorV2Mock.fulfillRandomWords(
                          txReceipt.events[1].args.requestId,
                          lottery.address
                      )

In a try and catch block, Like this

try{
const tx = await lottery.performUpkeep([])
                      const txReceipt = tx.wait(1)
                      await vrfCoordinatorV2Mock.fulfillRandomWords(
                          txReceipt.events[1].args.requestId,
                          lottery.address
                      )
} catch (error) {
console.log(error);
} 

This will help you to debug accordingly.

[adityabhattad2021]

I guess I found the problem as well
Here:

const txReceipt = tx.wait(1)

This should be,

const txReceipt = await tx.wait(1)

[adityabhattad2021]

But try and catch blocks are still helpful for debugging.

[RohitKS7]

Try & Catch Idea is Nice. But the same problem still persist. And Yes! This might be strange to hear but I didn't get anything new except same "Timeout"

[adityabhattad2021]

Did you add await tx.wait(1) as I said above?

[RohitKS7]

yes bro!

[alymurtazamemon]

@RohitKS7 is this repo updated?

[RohitKS7]

Yes!

[othaime-en]

Hey @RohitKS7 checkout the PR i've made, that should fix your issue.
There was an uncommented NOTE on line 146 that was weirdly resulting in the promise not being resolved. I also added the await tx.wait(1) as @adityabhattad2021 suggested.

[RohitKS7]

Seriously!? Oh! man, sorry for the trouble and thanks for the help!