LESSON 12: Error: invalid address or ENS name

[DorianDaSilva]

ERROR

   1) ManualToken Unit Test
       transfers
         Should transfer tokens succesfully to desired address:
     Error: invalid address or ENS name (argument="name", value=undefined, code=INVALID_ARGUMENT, version=contracts/5.6.2)
      at Logger.makeError (node_modules/@ethersproject/logger/src.ts/index.ts:261:28)
      at Logger.throwError (node_modules/@ethersproject/logger/src.ts/index.ts:273:20)
      at Logger.throwArgumentError (node_modules/@ethersproject/logger/src.ts/index.ts:277:21)
      at /home/ionmind/hh-fcc/hardhat-erc20-fcc/node_modules/@ethersproject/contracts/src.ts/index.ts:123:16
      at step (node_modules/@ethersproject/contracts/lib/index.js:48:23)
      at Object.next (node_modules/@ethersproject/contracts/lib/index.js:29:53)
      at fulfilled (node_modules/@ethersproject/contracts/lib/index.js:20:58)

  2) ManualToken Unit Test
       transfers
         emits a transfer event, when transfer happens:
     Error: invalid address or ENS name (argument="name", value=undefined, code=INVALID_ARGUMENT, version=contracts/5.6.2)
      at Logger.makeError (node_modules/@ethersproject/logger/src.ts/index.ts:261:28)
      at Logger.throwError (node_modules/@ethersproject/logger/src.ts/index.ts:273:20)
      at Logger.throwArgumentError (node_modules/@ethersproject/logger/src.ts/index.ts:277:21)
      at /home/ionmind/hh-fcc/hardhat-erc20-fcc/node_modules/@ethersproject/contracts/src.ts/index.ts:123:16
      at step (node_modules/@ethersproject/contracts/lib/index.js:48:23)
      at Object.next (node_modules/@ethersproject/contracts/lib/index.js:29:53)
      at fulfilled (node_modules/@ethersproject/contracts/lib/index.js:20:58)

  3) ManualToken Unit Test
       allowances
         It should approve other address to spend token:
     Error: invalid address or ENS name (argument="name", value=undefined, code=INVALID_ARGUMENT, version=contracts/5.6.2)
      at Logger.makeError (node_modules/@ethersproject/logger/src.ts/index.ts:261:28)
      at Logger.throwError (node_modules/@ethersproject/logger/src.ts/index.ts:273:20)
      at Logger.throwArgumentError (node_modules/@ethersproject/logger/src.ts/index.ts:277:21)
      at /home/ionmind/hh-fcc/hardhat-erc20-fcc/node_modules/@ethersproject/contracts/src.ts/index.ts:123:16
      at step (node_modules/@ethersproject/contracts/lib/index.js:48:23)
      at Object.next (node_modules/@ethersproject/contracts/lib/index.js:29:53)
      at fulfilled (node_modules/@ethersproject/contracts/lib/index.js:20:58)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)

UNIT TEST

const { messagePrefix } = require("@ethersproject/hash")
const { assert, expect } = require("chai")
const { getNamedAccounts, deployments, ethers } = require("hardhat")
const {
    INITIAL_SUPPLY,
    TOKEN_NAME,
    TOKEN_SYMBOL,
} = require("../helper-hardhat-config")

describe("ManualToken Unit Test", function () {
    const multiplier = 10 ** 18 //Set proper decimals (18)
    let token, deployer, user

    beforeEach(async function () {
        const accounts = await getNamedAccounts()
        deployer = accounts.deployer
        user = accounts.user
        await deployments.fixture("all")
        token = await ethers.getContract("ManualToken", deployer)
    })

    it("Token deployed!!!", async function () {
        assert(token.address)
    })

    describe("constructor", async function () {
        it("Has correct initial supply of our token", async function () {
            const totalSupply = await token.totalSupply()
            assert.equal(totalSupply.toString(), INITIAL_SUPPLY)
        })

        it("initilize token with the proper name & symbol", async function () {
            const name = (await token.name()).toString()
            assert.equal(name, "Tokenator")
            const symbol = (await token.symbol()).toString()
            assert.equal = (symbol, "TOK")
        })
    })

    describe("Minting token", async function () {
        it("user cannot mint", async function () {
            try {
                await token.Mint(deployer, 100)
                assert(false)
            } catch (e) {
                assert(e)
            }
        })
    })

    describe("transfers", function() {
        it("Should transfer tokens succesfully to desired address", async function() {
            const transferTokens = ethers.utils.parseEther("10")
            await token.transfer(user, transferTokens)
            expect(await token.balanceOf(user)).to.equal(transferTokens)
        })
        it("emits a transfer event, when transfer happens", async function() {
            await expect(
                await token.transfer(user, (10 * multiplier).toString())
            ).to.emit(token, "Transfer")
        })
    })

    describe("allowances", function() {
        const amount = (20 * multiplier).toString()
        beforeEach(async function() {
            playerToken = await ethers.getContract("ManualToken", user)
        })

        it("It should approve other address to spend token", async function() {
            tokenToSpend = ethers.utils.parseEther("5")
            await token.approve(user, tokenToSpend)
            const token1 = await ethers.getContract("ManualToken")
            await token1.transferFrom(deployer, user, tokenToSpend)
            expect(await token1.balanceOf(user)).to.equal(tokenToSpend)
        })
    })
})

===================================================================================

I have tried all sorts of modifications and permutations since yesterday but test keeps failing.

It seems to be related with the require(_to != address(0x0) but I am not experienced enough to figure out how to fix it and solutions online are unrelated.

[sv250349]

Should this line await deployments.fixture("all") not be await deployments.fixture(["all"]) also double check that you have await where it is needed.

[DorianDaSilva]

@sv250349 Thanks for the input...tried all of the above however still getting the same error.

[adityabhattad2021]

Try logging address of the user before calling the transfer function.

[DorianDaSilva]

@adityabhattad2021
Step 1: modified user to user1 - Still getting an error here
Step 2: I added the 18 decimals to the INITIAL_SUPPLY in the helper-hardhat-config.js

This fixed the error for the transfer tests, now I need to fix the allowance tests

[RoboCrypter]

@DorianDaSilva : Yup exactly what I told you to do in your other disscussion #1984!

[adityabhattad2021]

Here I think the problem is

describe("allowances", function() {
        const amount = (20 * multiplier).toString()
        beforeEach(async function() {
            playerToken = await ethers.getContract("ManualToken", user)
        })

        it("It should approve other address to spend token", async function() {
            tokenToSpend = ethers.utils.parseEther("5")
            await token.approve(user, tokenToSpend)
            const token1 = await ethers.getContract("ManualToken")
            await token1.transferFrom(deployer, user, tokenToSpend)
            expect(await token1.balanceOf(user)).to.equal(tokenToSpend)
        })
    })

with these lines

tokenToSpend = ethers.utils.parseEther("5")
await token.approve(user, tokenToSpend)
const token1 = await ethers.getContract("ManualToken")
await token1.transferFrom(deployer, user, tokenToSpend)

as token1 that you defined is connected to deployer and not the user .
Try this:

describe("allowances", function() {
        it("It should approve other address to spend token", async function() {
            tokenToSpend = ethers.utils.parseEther("5")
            await token.approve(user1, tokenToSpend)
            const userConnectedContract=await token.connect(user1)
            await userConnectedContract.transferFrom(deployer, user1, tokenToSpend)
            assert.equal(await userConnectedContract.balanceOf(user1),tokenToSpend)
        })
    })

[DorianDaSilva]

@adityabhattad2021 Thank you for sharing this solution, it really helped me fix the test.

However, running the lines below as is returned Error: VoidSigner cannot sign transactions

=================================================================================

Your code

describe("allowances", function() {
        it("It should approve other address to spend token", async function() {
            tokenToSpend = ethers.utils.parseEther("5")
            await token.approve(user1, tokenToSpend)
            const userConnectedContract = await token.connect(user1)
            await userConnectedContract.transferFrom(deployer, user1, tokenToSpend)
            assert.equal(await userConnectedContract.balanceOf(user1),tokenToSpend)
        })
    })

After a little digging around I managed to make it work as seen below

====================================================================================

Working code

        it("It should approve other address to spend token", async function() {
            tokenTransfer = ethers.utils.parseEther("5")
            await token.approve(user1, tokenTransfer)
            const userConnectedContract = await ethers.getContract("ManualToken", user1)
            await token.connect(user1)
            await userConnectedContract.transferFrom(deployer, user1, tokenTransfer)
            expect(await userConnectedContract.balanceOf(user1)).to.equal(tokenTransfer)
        })

To do so I replaced

const userConnectedContract = await token.connect(user1)

with

const userConnectedContract = await ethers.getContract("ManualToken", user1)
await token.connect(user1)

==================================================================================

It seems userConnectedContract needed to get contract first to then .connect() contract to user1

Either way, it wouldn't have happened without your input...Thank you so much for the help!

[adityabhattad2021]

Very happy that I could help!

[DorianDaSilva]

you're the man...thanks again! :)

[dealeros]

The problem is that we need SignerWithAddress type to use <contract>.connect()
One of the solutions can be instead:

let deplyer, user
const accounts = await getNamedAccounts()
deployer = accounts.deployer
user = accounts.user

do like that:

let deplyer, user
;[deployer, user] = await ethers.getSigners()

with that you can use just: const userConnectedContract = await token.connect(user)

and when you need an address as string i.e. deployer.address

[krakxn]

Is this solved? If so, please close it for others to refer to. Thank you! :)