Skip to content

Conversation

@siddu0523
Copy link
Contributor

  • Reject sqrtPrice = 0 (must be greater than zero)
  • Reject sqrtPrice outside valid tick range [MIN_SQRT_RATIO, MAX_SQRT_RATIO]
  • MIN_SQRT_RATIO = 5.42e-20 (tick -887272)
  • MAX_SQRT_RATIO = 1.84e+19 (tick +887272)

Added unit tests for:

  • Zero value rejection
  • Below minimum rejection
  • Above maximum rejection
  • Boundary acceptance tests

- Reject sqrtPrice = 0 (must be greater than zero)
- Reject sqrtPrice outside valid tick range [MIN_SQRT_RATIO, MAX_SQRT_RATIO]
- MIN_SQRT_RATIO = 5.42e-20 (tick -887272)
- MAX_SQRT_RATIO = 1.84e+19 (tick +887272)

Added unit tests for:
- Zero value rejection
- Below minimum rejection
- Above maximum rejection
- Boundary acceptance tests
@siddu0523 siddu0523 requested a review from Jehosephat January 13, 2026 16:30
@siddu0523 siddu0523 self-assigned this Jan 13, 2026
@siddu0523 siddu0523 merged commit 785cc2d into main Jan 14, 2026
1 check passed
@siddu0523 siddu0523 deleted the feature/comprehensive-test-coverage branch January 14, 2026 16:36
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.

3 participants