Skip to content

test: cover farming pool lock credit edges#57

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

test: cover farming pool lock credit edges#57
jamilahmadzai wants to merge 1 commit into
SmartDropLabs:mainfrom
jamilahmadzai:test/farming-pool-lock-coverage-21

Conversation

@jamilahmadzai

Copy link
Copy Markdown
Contributor

Closes #21

Bounty: GrantFox OSS campaign for issue #21

Summary

  • add exact balance assertions for lock and full/partial unlock paths
  • assert the locked and unlocked farming-pool event payloads, filtering out SAC transfer events
  • cover credit continuity across multiple lock windows so later deposits only accrue for later ledgers
  • align the partial unlock test with the 500 locked / 200 unlocked / 300 remaining case from the issue

Validation

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

Notes

  • test-only change; no contract runtime behavior changed
  • Soroban still emits the existing Events::publish deprecation warnings from contract code

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