search

PatrickAlphaC / hardhat-nft-fcc

102 stars 140 forks source link

Change from .tokenUri("0") to .getDogTokenUris("0") #5

Closed Gnome101 closed 2 years ago

Gnome101 commented 2 years ago

I noticed that in the fulfill random words test, the portion of the test that grabs the token URI, doesn't grab the Dog Token Uris with the correct method. I also changed it from inputting a number to a string to make it more like the rest of the code.

PatrickAlphaC commented 2 years ago

So a bit confusing, but in this test we actually are looking for the tokenURIs, not the dog tokenURIs (although, we probably should have a test for both).

Remember that the tokenURI has a imageURI inside it pointing to what the dog looks like. I do like the improvement that we use a string instead of a number! I'll include that change, thanks!