Lesson 9: Contract not deploying successfully

[Tosinkoa]

Hello pls my contract is not deploying successfully, I was getting this error message
Error: ERROR processing C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\deploy\00-deploy-mocks.js: TypeError: from.toLowerCase is not a function

And I'm also getting warning messages from the VRF Mock I imported.

Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:186:24:
    |
186 |   function addConsumer(uint64 _subId, address _consumer) external pure override {
    |                        ^^^^^^^^^^^^^


Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:186:39:
    |
186 |   function addConsumer(uint64 _subId, address _consumer) external pure override {
    |                                       ^^^^^^^^^^^^^^^^^


Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:190:27:
    |
190 |   function removeConsumer(uint64 _subId, address _consumer) external pure override {
    |                           ^^^^^^^^^^^^^


Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:190:42:
    |
190 |   function removeConsumer(uint64 _subId, address _consumer) external pure override {
    |                                          ^^^^^^^^^^^^^^^^^


Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:194:45:
    |
194 |   function requestSubscriptionOwnerTransfer(uint64 _subId, address _newOwner) external pure override {        
    |                                             ^^^^^^^^^^^^^


Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:194:60:
    |
194 |   function requestSubscriptionOwnerTransfer(uint64 _subId, address _newOwner) external pure override {        
    |                                                            ^^^^^^^^^^^^^^^^^


Warning: Unused function parameter. Remove or comment out the variable name to silence this warning.
   --> @chainlink/contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol:198:44:
    |
198 |   function acceptSubscriptionOwnerTransfer(uint64 _subId) external pure override {
    |                                            ^^^^^^^^^^^^^

[Utilitycoder]

Check your mock deployment file for "from.toLowerCase". That seems like a mistake. You should check and fix that line.

[Tosinkoa]

Here it is, there is no toLowerCase there.

const { developmentChains } = require("../helper-hardhat-config")
const { network } = require("hardhat")

const BASE_FEE = ethers.utils.parseEther("0.25")
const GAS_PRICE_LINK = 1e9

module.exports = async ({ getNamedAccounts, deployments }) => {
    const { deploy, log } = deployments
    const deployer = await getNamedAccounts()
    const args = [BASE_FEE, GAS_PRICE_LINK]

    if (developmentChains.includes(network.name)) {
        log("Local network detected! Deploying mocks...")
        await deploy("VRFCoordinatorV2Mock", {
            from: deployer,
            log: true,
            args: args
        })
    }
    log("Mocks Deployed!")
    log("--------------------------------------")

}

module.exports.tags = ["all", "mocks"]```

[Tosinkoa]

Could it be the from: deloyer that is throwing the error? And also I only used toLowerCase() method in my verify.js file.

[Tosinkoa]

Here is the full error I was getting

Error: ERROR processing C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\deploy\00-deploy-mocks.js:
TypeError: from.toLowerCase is not a function
    at getFrom (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat-deploy\src\helpers.ts:1650:34)
    at _deploy (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat-deploy\src\helpers.ts:537:9)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at _deployOne (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat-deploy\src\helpers.ts:934:16)
    at Object.module.exports [as func] (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\deploy\00-deploy-mocks.js:14:9)
    at DeploymentsManager.executeDeployScripts (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat-deploy\src\DeploymentsManager.ts:1220:22)
    at DeploymentsManager.runDeploy (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat-deploy\src\DeploymentsManager.ts:1053:5)
    at SimpleTaskDefinition.action (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat-deploy\src\index.ts:409:5)
    at Environment._runTaskDefinition (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat\src\internal\core\runtime-environment.ts:219:14)
    at Environment.run (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\raffle-smart-contract\node_modules\hardhat\src\internal\core\runtime-environment.ts:131:14)
    at DeploymentsManager.executeDeployScripts (C:\Users\NEW\Desktop\Blockchain-dev-practice\smart-contract-lottery\r

[Tosinkoa]

The error was from the node module. And am not sure if I did something wrong in my code to make this error occur. I followed @PatrickAlphaC code thrice, but the error still persist @Utilitycoder

[02b2]

in your verify.js file make sure you have typed this line correctly.... line 14 verify.js //if (e.message.toLowerCase().includes("already verified"))

it looks like you have not defined toLowerCase correclty by forgetting the (), so your contracts deploy up until the verify stage then it throws an error, to test this comment out the verify contract functions so they do not run and retry deployment, this should work as the only place toLowerCase() is used is in the verify.js file.....also make sure you have the correct dependencies at the top of all of your .sol files :) :) :) i hope this helps

[Tosinkoa]

I've done this, when I comment it out, it still throws the error. When I trace where the error was coming from, it was from node module. There is this helper.ts file causing the error

[Tosinkoa]

The main cause for the from.toLowerCase was caused by not putting the deployer in a curly braces.
Like this:
const { deployer } = await getNamedAccounts()

Thanks you so much for helping me out. @Utilitycoder @02b2