Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

component(dex): state-machine tests for execution #2145

Merged
merged 13 commits into from
Mar 16, 2023
Merged

component(dex): state-machine tests for execution #2145

merged 13 commits into from
Mar 16, 2023

Conversation

erwanor
Copy link
Member

@erwanor erwanor commented Mar 10, 2023

This adds tests to the DEX execution implementation, in particular, it tests against a simple order book that:

  • filling an entire order in one go works as expected
  • round-tripping against a position works as expected
  • partial fills work as expected when a scaling factor is used

@erwanor erwanor temporarily deployed to smoke-test March 10, 2023 21:15 — with GitHub Actions Inactive
@erwanor erwanor temporarily deployed to smoke-test March 13, 2023 13:29 — with GitHub Actions Inactive
@erwanor erwanor temporarily deployed to smoke-test March 14, 2023 15:56 — with GitHub Actions Inactive
@erwanor erwanor marked this pull request as ready for review March 14, 2023 19:50
@erwanor erwanor requested a review from zbuc March 14, 2023 19:50
@erwanor erwanor temporarily deployed to smoke-test March 14, 2023 19:53 — with GitHub Actions Inactive
@erwanor erwanor temporarily deployed to smoke-test March 16, 2023 13:50 — with GitHub Actions Inactive
@erwanor erwanor merged commit d4b0b5d into main Mar 16, 2023
@erwanor erwanor deleted the s-order-book branch March 16, 2023 14:53
@hdevalence hdevalence mentioned this pull request Mar 20, 2023
65 tasks
@cratelyn cratelyn added the A-dex Area: Relates to the dex label Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dex Area: Relates to the dex
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants