Skip to content

test: cover farming pool boost credit invariants - #56

Open
jamilahmadzai wants to merge 1 commit into
SmartDropLabs:mainfrom
jamilahmadzai:test/farming-pool-boost-coverage-22
Open

test: cover farming pool boost credit invariants#56
jamilahmadzai wants to merge 1 commit into
SmartDropLabs:mainfrom
jamilahmadzai:test/farming-pool-boost-coverage-22

Conversation

@jamilahmadzai

Copy link
Copy Markdown
Contributor

Closes #22

Bounty: GrantFox OSS campaign for issue #22

Summary

  • add exact boost arithmetic coverage for full allocation at 3x and 1x multipliers
  • assert boost and multiplier event payloads, admin authorization, and non-retroactive checkpointing
  • cover monotonic get_credits, boosted-vs-unboosted accrual, stake/lock credit separation, and stake/unstake balance round trip

Validation

  • cargo fmt --all -- --check
  • cargo build
  • cargo test -p farming-pool boost -- --nocapture
  • cargo test -p farming-pool -- --nocapture
  • cargo test --workspace -- --nocapture
  • git diff --check

Security notes

  • test-only change; no contract runtime behavior changed
  • verifies boost and multiplier updates checkpoint old credit windows instead of retroactively recalculating banked credits

@jamilahmadzai
jamilahmadzai force-pushed the test/farming-pool-boost-coverage-22 branch from d7e507b to 44c29bc Compare August 28, 2026 19:28
@netlify

netlify Bot commented Aug 28, 2026

Copy link
Copy Markdown

Deploy Preview for sdcontracts ready!

Name Link
🔨 Latest commit 44c29bc
🔍 Latest deploy log https://app.netlify.com/projects/sdcontracts/deploys/6a91e1724344450008c76306
😎 Deploy Preview https://deploy-preview-56--sdcontracts.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant