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.

Sign up for GitHub

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

Jump to bottom

Merge MotorController Tests #115

Closed
CoolSpy3 opened this issue Jul 15, 2024 · 0 comments · Fixed by #123
Closed

Merge MotorController Tests #115

CoolSpy3 opened this issue Jul 15, 2024 · 0 comments · Fixed by #123
Assignees
@brettle
Labels
enhancement New feature or request

Comments

@CoolSpy3
Copy link
Member

Currently the MotorController tests are all run simultaneously, but their results are reported individually. This leads to unnecessary runs of the test world. It would be better if the tests were merged, so that the world only runs once and reports the results from all the tests.

Migrated from this PR comment.
@CoolSpy3 CoolSpy3 added the enhancement New feature or request label Jul 15, 2024
@CoolSpy3 CoolSpy3 mentioned this issue Jul 15, 2024
Merged
@brettle brettle self-assigned this Jul 18, 2024
brettle added a commit that referenced this issue Jul 18, 2024
@brettle
Fix Merge MotorController Tests #115
092917b
@brettle brettle mentioned this issue Jul 18, 2024
Merged
brettle added a commit that referenced this issue Jul 19, 2024
@brettle
Merge pull request #123 from DeepBlueRobotics/merge-MotorControllerTests
5be9c56 
Fix Merge `MotorController` Tests #115
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brettle brettle

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Merge MotorController Tests #115 DeepBlueRobotics/DeepBlueSim
2 participants
@brettle @CoolSpy3