Skip to content

test: add zero duration pool coverage (#496)#529

Merged
Yunusabdul38 merged 1 commit intoWeb3Novalabs:mainfrom
edehvictor:codex/test-zero-duration-pool-496
Mar 28, 2026
Merged

test: add zero duration pool coverage (#496)#529
Yunusabdul38 merged 1 commit intoWeb3Novalabs:mainfrom
edehvictor:codex/test-zero-duration-pool-496

Conversation

@edehvictor
Copy link
Copy Markdown
Contributor

Closes #496

Changes

  • add a unit test in test.rs covering pool creation with zero duration
  • verify that create_pool fails when end_time == current ledger timestamp
  • keep the new test alongside the existing pool-duration boundary tests for clarity

Testing

  • cargo fmt
  • cargo test test_create_pool_rejects_zero_duration -- --nocapture (blocked locally: missing MSVC linker link.exe on this machine)

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 28, 2026

@edehvictor is attempting to deploy a commit to the shola's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Mar 28, 2026

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

@Yunusabdul38 Yunusabdul38 merged commit 9de53f2 into Web3Novalabs:main Mar 28, 2026
1 of 2 checks 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 Test for Zero Duration Pool

2 participants