[TESTING] Tests needed for ERC721

[mudgen]

Testing Issue

Tests need to be written for the following:

• ERC721Facet
• LibERC721
• ERC721Enumerable
• LibERC721Enumerable
• ERC721BurnFacet

Some tests currently exist for the ERC721EnumerableBurn facet.

Help us improve our test coverage and quality:

Helpful Information

Have a question? Please check our contribution guide first - your answer might already be there!

Want to discuss something? For general questions, ideas, or brainstorming, please browse our discussions or start a new one.

You can also join our Discord to discuss the testing issue.

[akronim26]

Could I take this up?

[mudgen]

@akronim26 Yes, it yours. I really appreciate your help.

[ogazboiz]

@mudgen can i contribute to this

[mudgen]

@ogazboiz See if @akronim26 can use your help.

[akronim26]

@ogazboiz See if @akronim26 can use your help.

Yeah, @ogazboiz can write tests for the ERC721Enumerable and LibERC721Enumerable and I can write for other 3.

[ogazboiz]

@akronim26 alright

[mudgen]

@akronim26 @ogazboiz Sounds good. It would be good to have consistency with these tests, so perhaps you two can review each others tests and agree about how things are done. I am glad we have a team.

[akronim26]

I have created PR #218 for 3 of the files. @mudgen and @ogazboiz, whenever you get a moment, could you kindly review it?

[ogazboiz]

I have created a PR #218 for 3 of the files. @mudgen and @ogazboiz, whenever you get a moment, could you kindly review it?

so i am doing the other testing

[ogazboiz]

I have created PR #221 for ERC721Enumerable and LibERC721Enumerable of the files. @mudgen, @akronim26 and @maxnorm , whenever you get a moment, could you kindly review it?