Skip to content

test(creator-keys): add coverage for creator fee recipient update val… - #696

Merged
Chucks1093 merged 1 commit into
accesslayerorg:mainfrom
funds0033-cmyk:test/681-update-creator-fee-recipient-zero-address
Jul 30, 2026
Merged

test(creator-keys): add coverage for creator fee recipient update val…#696
Chucks1093 merged 1 commit into
accesslayerorg:mainfrom
funds0033-cmyk:test/681-update-creator-fee-recipient-zero-address

Conversation

@funds0033-cmyk

Copy link
Copy Markdown
Contributor

Adds focused unit tests for the update_creator_fee_recipient functionality to verify successful updates, authorization checks, and zero-address validation.

What was added
Added creator-keys/tests/update_creator_fee_recipient.rs.
Added test coverage for:
Valid fee recipient updates are persisted correctly.
Zero-address recipients are rejected with ContractError::ZeroAddress.
Unauthorized callers cannot update the fee recipient and no state changes occur.
A second valid update replaces the previous recipient.
The previous recipient is no longer stored after an update.
Notes
The issue description references an invalid_recipient error, but the contract currently returns ContractError::ZeroAddress. The tests were written against the contract's actual behavior.
This PR only adds test coverage and does not modify contract logic.
Validation
Local cargo test could not be executed because the local environment is missing the MSVC link.exe toolchain.
The tests are intended to be validated by the project's GitHub CI.
Closes #681

@drips-wave

drips-wave Bot commented Jul 30, 2026

Copy link
Copy Markdown

@funds0033-cmyk Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Chucks1093
Chucks1093 merged commit c7fa93d into accesslayerorg:main Jul 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit tests for the creator fee recipient update function rejecting a zero address as the new recipient

2 participants