Skip to content

test: add unit tests for issues #487 #489 #491 #492#498

Merged
Chucks1093 merged 7 commits into
accesslayerorg:mainfrom
Godsmiracle001:feat/tests-issues-487-489-491-492
Jun 27, 2026
Merged

test: add unit tests for issues #487 #489 #491 #492#498
Chucks1093 merged 7 commits into
accesslayerorg:mainfrom
Godsmiracle001:feat/tests-issues-487-489-491-492

Conversation

@Godsmiracle001

Copy link
Copy Markdown
Contributor

#487 - distribute_dividend reverts when creator has zero total supply

#489 - multi-holder dividend distribution majority-holder proportional split

#491 - supply cap regression: buy that partially exceeds cap must revert

#492 - assert_supply_equals_holder_sum invariant helper (used in 6 places)

Summary

Testing

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace

Checklist

  • Linked issue or backlog item
  • Added or updated creator-keys unit/integration tests for every changed contract behavior, including failure paths for new or reachable ContractError variants
  • Ran cargo fmt --all -- --check, cargo clippy --workspace --all-targets -- -D warnings, and cargo test --workspace, or explained exactly why a command was not run
  • Reviewed persistent storage changes against docs/storage-key-invariants.md; any storage layout change includes a migration/backward-compatibility note
  • Confirmed event names, topic order, payload field order, and field meanings remain compatible with docs/contract-event-conventions.md, or documented the breaking change and versioning plan
  • Updated docs for any changed public contract interface, read-only method, event schema, storage behavior, fee logic, or deployment workflow
  • Scope stays limited to one contract concern and does not include unrelated formatting, lockfile, generated artifact, or dependency changes

close #487
close #489
close #491
close #492

…ccesslayerorg#491 accesslayerorg#492

accesslayerorg#487 - distribute_dividend reverts when creator has zero total supply

accesslayerorg#489 - multi-holder dividend distribution majority-holder proportional split

accesslayerorg#491 - supply cap regression: buy that partially exceeds cap must revert

accesslayerorg#492 - assert_supply_equals_holder_sum invariant helper (used in 6 places)
@drips-wave

drips-wave Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Godsmiracle001 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 f1deec9 into accesslayerorg:main Jun 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment