feat: add tests for issues #471, #472, #475, #477

[privexlabs]

Summary

Resolves four test-coverage issues across the creator-keys contract.

• Add unit tests for protocol fee recipient update emitting correct event fields #477 Add field-level assertions for ProtocolFeeRecipientUpdated event (old_recipient, new_recipient)
• Add unit tests for get_buyback_quote returning zero for zero amount #475 Change get_buyback_quote to return 0 for amount=0 (view functions should not revert on zero input); add
  tests
• Add helper for building a multi-holder test fixture with varied balances in one call #471 Add setup_holders fixture helper to contract_test_env; add confirmation tests
• Add regression test for locked allocation moving bonding curve supply at registration #472 Add regression tests confirming locked allocation initialises bonding-curve supply at registration

Changes

• src/lib.rs — get_buyback_quote: return Ok(0) for amount == 0 instead of NotPositiveAmount; the actual
  buyback operation continues to reject zero amounts
• tests/buyback.rs — narrow test_buyback_zero_amount_reverts to only assert on the buyback operation (not the
  quote view)
• tests/contract_test_env/mod.rs — add setup_holders(env, client, creator, &[(Address, u32)]) -> u32
• tests/protocol_fee_recipient_updated_event.rs (new) — 3 tests for Add unit tests for protocol fee recipient update emitting correct event fields #477
• tests/get_buyback_quote_zero_amount.rs (new) — 3 tests for Add unit tests for get_buyback_quote returning zero for zero amount #475
• tests/setup_holders_helper.rs (new) — 2 tests for Add helper for building a multi-holder test fixture with varied balances in one call #471
• tests/locked_allocation_bonding_curve_supply.rs (new) — 3 tests for Add regression test for locked allocation moving bonding curve supply at registration #472

Test plan

• cargo test -p creator-keys passes with no failures
• test_protocol_fee_recipient_updated_event_old_recipient_field confirms old address
• test_protocol_fee_recipient_updated_event_new_recipient_field confirms new address
• test_get_buyback_quote_returns_zero_for_zero_amount confirms zero return
• test_setup_holders_produces_correct_balances confirms per-wallet balances
• test_locked_allocation_sets_total_supply_immediately confirms supply at registration
• test_buy_quote_reflects_locked_supply confirms price reflects locked supply

Closes #477
Closes #475
Closes #471
Closes #472

[drips-wave]

@privexlabs 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