Skip to content

test: events & TickSpacingSetter#922

Merged
adhusson merged 2 commits into
fix-ordering-argsfrom
fix-ordering-args-event-tests
May 26, 2026
Merged

test: events & TickSpacingSetter#922
adhusson merged 2 commits into
fix-ordering-argsfrom
fix-ordering-args-event-tests

Conversation

@adhusson
Copy link
Copy Markdown
Contributor

  • add missing events test
  • fix others
  • add tick spacing setter success test

@adhusson adhusson changed the base branch from main to fix-ordering-args May 26, 2026 10:11
@adhusson adhusson requested review from MathisGD, QGarchery and peyha and removed request for MathisGD May 26, 2026 10:11
Copy link
Copy Markdown
Contributor

@peyha peyha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Liquidate event is not tested with healthyPath=true
Take event is only tested once in ContinuousFeeTest, would be worse adding a simple test in TakeTest as well

@adhusson adhusson merged commit c9427b6 into fix-ordering-args May 26, 2026
40 checks passed
@adhusson adhusson deleted the fix-ordering-args-event-tests branch May 26, 2026 14:26
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