Skip to content

Runtime selection of field vs simulated gameplay tests#3744

Draft
nycrat wants to merge 8 commits into
masterfrom
avah/combine_simulated_and_field_tests
Draft

Runtime selection of field vs simulated gameplay tests#3744
nycrat wants to merge 8 commits into
masterfrom
avah/combine_simulated_and_field_tests

Conversation

@nycrat
Copy link
Copy Markdown
Member

@nycrat nycrat commented May 27, 2026

Description

This PR combines simulated and field gameplay test runners under the common TbotsTestRunner interface. This allows all tests to be run as either field tests or simulated tests with just a different flag passed into the bazel test command.

This PR contains many refactoring changes to our existing python testing framework. These changes were important to improving code quality while implementing the changes, and they will make it easier to implement #3369 in the future.

Testing Done

Resolved Issues

resolves #2908

Length Justification and Key Files to Review

A lot of just renaming and combining the simulated_tests and field_tests under the gameplay_tests package. None of the logic is changed for validations and existing tests.

Review Checklist

It is the reviewers responsibility to also make sure every item here has been covered

  • Function & Class comments: All function definitions (usually in the .h file) should have a javadoc style comment at the start of them. For examples, see the functions defined in thunderbots/software/geom. Similarly, all classes should have an associated Javadoc comment explaining the purpose of the class.
  • Remove all commented out code
  • Remove extra print statements: for example, those just used for testing
  • Resolve all TODO's: All TODO (or similar) statements should either be completed or associated with a github issue

@nycrat nycrat force-pushed the avah/combine_simulated_and_field_tests branch from 6565caa to 4c20588 Compare May 27, 2026 20:31
@nycrat nycrat force-pushed the avah/combine_simulated_and_field_tests branch from 4c20588 to f7b544d Compare May 28, 2026 03:14
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.

Support running a test in both in simulated mode and field mode

1 participant