Lesson 14: BasicNft Unit test

[ManuWeb3]

Hi all,

Just saw the below Unit test-script snippet at this course's repo and saw getContract() without deployer argument.

Usually, we have been taught to input deployer as the second arg. in getContract("BasicNft", deployer) to establish a connection of deployer with the returned / deployed instance of BasicNft.sol.

But, this still works.

So, does it mean that deployer is optional and the connection gets established by itself by default??

beforeEach(async () => {
              accounts = await ethers.getSigners()
              deployer = accounts[0]
              await deployments.fixture(["basicnft"])
              basicNft = await ethers.getContract("BasicNft")
          })

[RoboCrypter]

@ManuWeb3 : Yes both instances will work just fine, If you are only using the deployer,
but the good practice what I do is to connect the deployer or whatever user is when getting contract.

Sometimes you have more than one Users so you have to deploy with by connecting them to the contract.
This usually we have to do when we are testing our contract.
For Example:
basicNft = await ethers.getContract("BasicNft", deployer)
basicNft = await ethers.getContract("BasicNft", user)

[ManuWeb3]

Thanks @abossofmyself for the quick reply.
Even I prefer to follow the same good practice to mention deployer all the time.