Lesson 10: Upkeep not needed when runn MockOffchain Script

[Avelous]

I have copied the mockOffChain.js code snippet from Patrick's github and when I deploy it, it runs perfectly. But when I try to run the scripts through, yarn hardhat run scripts/mockOffChain.js

The result is "Upkeep Not Needed" even though I've made entered the lottery with my wallets

[pyromaus]

I'm getting the same problem when I run the script the second time and on.

The first time I run mockOffchain.js, terminal returns "Performed upkeep with RequestId: 1" and it ends there. The rest of the mockVrf function never fires ("let's pretend", "responded", etc..) and doesn't return a winner.
The second time I run the script, I get "no upkeep needed" as well.

THEN, at this point when I go to the front-end, hit Enter Raffle, send the tx, I get a Raffle__NotOpen error in the console.

If we check Raffle.sol, we can see this error means that s_raffleState is CALCULATING

So why is it stuck in that state? The only situation its calculating is in the performUpkeep function.

s_raffleState = RaffleState.CALCULATING;
uint256 requestId = i_vrfCoordinator.requestRandomWords(
      i_gasLane, // gasLane
      i_subscriptionId,
      REQUEST_CONFIRMATIONS,
      i_callbackGasLimit,
      NUM_WORDS
    );
emit RequestedRaffleWinner(requestId);

I believe for some reason the function fulfillRandomWords is not getting called. That's where the winner is picked and the raffle state is changed back to OPEN
Also the function reverts the number of players to 0 but after running mockOffchain.js the number on my front-end is unchanged

If anyone has a tip on this please share TY

[Avelous]

@pyromaus I managed to solve mine. The chainId was undefined and mockvrf function wasn't called. So I went to add this to my networks in network.config

localhost: { chainId: 1337 || 31337, blockConfirmations: 1, },

Check here also, metamask seems to have an issue reading localhost/hardhat https://hardhat.org/hardhat-network/docs/metamask-issue

Let me know if this helps you

[pyromaus]

Yoo thats awesome that you got it workin bro!
I tried adding that line to my hardhat.config.js:

defaultNetwork: "hardhat",
    networks: {
        hardhat: {
            chainId: 31337,
            blockConfirmations: 1,
        },
        localhost: {
            chainId: 1337 || 31337,
            blockConfirmations: 1,
        },
        rinkeby: {
            chainId: 4,
            blockConfirmations: 6,
            url: RINKEBY_RPC_URL,
            accounts: [PRIVATE_KEY],
        },
    }

Sadly it's now throwing me another error when I run mockOffchain.js
HardhatError: HH101: Hardhat was set to use chain id 1337, but connected to a chain with id 31337.

[SADFrancis]

For the next person that gets hit with that error as this question was asked months ago, I got a stack overflow answer.

The gist of it is that you create an extra hardhat configuration file with the same content but in the networks section of the JSON, only include the info for the chain you want to deploy to and nothing else. Name the catered configuration chainname.config.js (ex: localhost.config.js) and when you run hh node or yarn hardhat node command on the command line, run below (in this case, for running a localhost node ):

$ hh node --config localhost.config.js node

It'll spin up a node with no chainId conflicts and you can run deploy scripts as normal.

[Niferu]

Hello guys,

First of all check your MetaMask localhost setup if it has correct chainId as mine was 1337 somehow...

Then you can add below in hardhat.config.js to fix this issue:

networks: {
        localhost: {
            chainId: 31337,
            blockConfirmations: 1,
        },

There is no need to add 1337 anywhere in code as it is invalid chainId

[sumit180301]

Yeah, I too just added the following to hardhat.config.js: and it worked:
networks: { localhost: { chainId: 31337, blockConfirmations: 1 },

[pyromaus]

After leaving this part alone for a while I went back and tried to run mockOffchain.js for shits and giggs.

Performed upkeep with RequestId: 1
We on a local network? Ok let's pretend...
Responded!
The winner is: 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266
✨  Done in 1.32s.

I didn't change anything in the code. I did restart my comp tho
I was pulling my hair out trying to fix this but.. glad it works now!
Cheers

[Avelous]

awesome buddy