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

Split exchanges orchestrator tests in smaller files for better clarity #319

Merged
merged 1 commit into from
Sep 18, 2022

Conversation

sjanel
Copy link
Owner

@sjanel sjanel commented Sep 18, 2022

exchangesorchestrator_test.cpp had more than 1000 lines. Split all tests into 3 (for now):

  • private tests (all except trades)
  • public tests
  • trade tests

@sjanel sjanel added the refactoring Code cleaning and refactoring label Sep 18, 2022
@sjanel sjanel force-pushed the chore/split-exchanges-orchestrator-tests branch from b87cdd4 to 7832524 Compare September 18, 2022 16:35
@sjanel sjanel merged commit 67ae512 into main Sep 18, 2022
@sjanel sjanel deleted the chore/split-exchanges-orchestrator-tests branch September 18, 2022 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Code cleaning and refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant