Feat: Add missing tests for Erc1155 E2E Tests

[Dhanraj30]

---

Description

This pull request adds the missing receiver-related tests for the _burn and _burn_batch functions in the erc1155.rs file.

Changes

• Added tests for _burn and _burn_batch:
  • errors_when_receiver_reverts_with_reason
  • errors_when_receiver_reverts_without_reason
  • errors_when_receiver_panics
  • errors_when_invalid_receiver_contract

Issue

Resolves #482

Checklist

• Tests
• Documentation
• Changelog

---

[netlify]

✅ Deploy Preview for contracts-stylus canceled.

Name	Link
🔨 Latest commit	5000439
🔍 Latest deploy log	https://app.netlify.com/sites/contracts-stylus/deploys/6782427341a4ef0008000a98

[0xNeshi]

Hey @Dhanraj30, thanks for wanting to contribute!

I took a look at the changes, and it turns out we don't need to test burn and burn_batch for reverts in ERC1155Receiver. Here's why: both functions use _update_with_acceptance_check internally, which doesn't do anything with the ERC1155Receiver contract when to is Address::ZERO - which is always the case for burns.

Sorry you spent time working on this! 🙏

Would you be interested in tackling some other issues instead? We could really use help cleaning up the Rust code in our docs (#480). There's also a quick win available - just reordering our docs sections (#476).