Lesson-14 basicNft test failed.

[TimeKeyRoll]

Hi! tests are getting failed.
New repo : https://github.com/TimeKeyRoll/hardhat-nft1-fcc.git

[TimeKeyRoll]

@othaime-en yes I changed the name what you mentioned in for previous repo. but nothing changed.

@krakxn Here is all clean and new.

I have made this new repo. Since that one kept giving me artifacts error. there is this error with new repo.

[krakxn]

I suppose it is fixed by now; good luck! :)

The new error can be solved by incorporating the changes in the initial PR (the first one, package.json); looking at the answers, the same fixes were covered here.

How you can avoid such errors in the future: follow a strict naming convention, I think I have mentioned this to you before. If everything is uniform everywhere, it is always very easy to debug the code. Of course, if it not of habit to you yet, just keep it in your mind whenever naming variables, functions, parameters and anything which requires "naming" to be correct and would lead to an error otherwise.

[TimeKeyRoll]

@krakxn thankyou very much. I did not even know from you PR what was the error.
The truth is I am more interested in knowing ERROR and fixing it by myself.
Probably it is not you but me who just wants to understand before I make any change to my repo.

I don't want a quick solution but an answer to the problem which could be fixed and Don't want that problem to trouble me in future. lol
The idea is to to learn/understand than easy fixing.
But thanks man.You really took time to answer.

[krakxn]

@krakxn thankyou very much. I did not even know from you PR what was the error. The truth is I am more interested in knowing ERROR and fixing it by myself. Probably it is not you but me who just wants to understand before I make any change to my repo.

I don't want a quick solution but an answer to the problem which could be fixed and Don't want that problem to trouble me in future. lol The idea is to to learn/understand than easy fixing. But thanks man.You really took time to answer.

Yes, I understand that, and that indeed is the mentality to have.

Again, good luck!

[othaime-en]

Hey @TimeKeyRoll I think I've found something. Your deploy scripts has these tags module.exports.tags = ["all", "basicnft", "main"] while i'm seeing in your test script, you are trying to get deployments on line 17 with await deployments.fixtures(["mocks", "BasicNft"]) this is incorrect because your deploy script doesn't have a BasicNft tag inside it. So on line 17 change BasicNft to basicnft and tell me if it fixes your issue.

[TimeKeyRoll]

["all", "basicnft", "main"] changes this to ["all", "Basicnft", "main"]. since contract name is BasicNft.so everywhere it is in caps.
And nothing changed. lol
This Error won't go @othaime-en

[othaime-en]

The tag names don't have to be the same as the contract name. Just do the simple correction I've told you (only change line 17 in your test script) and run npx hardhat deploy again. If the error still persists, tell me. I'll have to run your code locally.

[TimeKeyRoll]

Hey! You did find the right error but i was missin n i mean it should have N but wow. its is passing now.

[othaime-en]

That's exactly what I was telling you. To change basicNft to basicnft in your test script. Glad you realized it though.

[TimeKeyRoll]

OMG i just did realise later you were talking about N thing. I was passin looking at the same message again and again lol
Next time I will be more aware of these kinda things. Thankyou @othaime-en it took me 2 days to fix and a whole lot people to help haha

[alymurtazamemon]

@TimeKeyRoll What ever the name of basicnft tag is here in you basic nft deploy script

module.exports.tags = ["all", "basicnft", "main"];

it should be similar here

await deployments.fixture(["basicnft"]);

[othaime-en]

@alymurtazamemon You saw it too :)

[TimeKeyRoll]

The error was n it should be have N :/
Feel really terrible now.
But glad you noticed @alymurtazamemon