Unified all py tests (UBC-Thunderbots#3513)

* Unified all py tests

* [pre-commit.ci lite] apply automatic fixes

* Updated missed files

* [pre-commit.ci lite] apply automatic fixes

---------

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

Author: RockyCao23

src/software/ai/hl/stp/play/crease_defense/crease_defense_play_test.py

@@ -1,12 +1,11 @@
-import sys
-
-import pytest
-
 import software.python_bindings as tbots_cpp
 from proto.play_pb2 import Play, PlayName
 from proto.import_all_protos import *
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 def test_crease_defense_play(simulated_test_runner):
@@ -79,5 +78,4 @@ def test_crease_defense_play(simulated_test_runner):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/defense/defense_play_test.py

@@ -1,5 +1,3 @@
-import sys
-
 import pytest
 
 import software.python_bindings as tbots_cpp
@@ -10,6 +8,9 @@
 )
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 @pytest.mark.parametrize(
@@ -170,5 +171,4 @@ def setup(*args):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/enemy_free_kick/enemy_free_kick_play_test.py

@@ -1,5 +1,3 @@
-import sys
-
 import pytest
 
 import software.python_bindings as tbots_cpp
@@ -18,6 +16,9 @@
 )
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 @pytest.mark.parametrize(
@@ -164,5 +165,4 @@ def setup(*args):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/example/example_play_test.py

@@ -1,7 +1,3 @@
-import sys
-
-import pytest
-
 import software.python_bindings as tbots_cpp
 from software.simulated_tests.robot_enters_region import (
     NumberOfRobotsEventuallyExitsRegion,
@@ -11,6 +7,9 @@
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
 from proto.play_pb2 import Play, PlayName
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 def test_example_play(simulated_test_runner):
@@ -96,5 +95,4 @@ def setup(*args):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/free_kick/free_kick_play_test.py

@@ -1,5 +1,3 @@
-import sys
-
 import pytest
 
 import software.python_bindings as tbots_cpp
@@ -9,6 +7,9 @@
 from software.simulated_tests.robot_enters_region import RobotEventuallyEntersRegion
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 def free_kick_play_setup(
@@ -243,5 +244,4 @@ def test_free_kick_play_both(simulated_test_runner, ball_initial_pos):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/halt_play/halt_play_test.py

@@ -1,11 +1,10 @@
-import sys
-
-import pytest
-
 import software.python_bindings as tbots_cpp
 from software.simulated_tests.robot_speed_threshold import *
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 # TODO issue  #2599 - Remove Duration parameter from test
@@ -81,5 +80,4 @@ def setup(*args):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/kickoff_play_test.py

@@ -1,5 +1,3 @@
-import sys
-
 import pytest
 
 import software.python_bindings as tbots_cpp
@@ -10,6 +8,9 @@
 from proto.import_all_protos import *
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 from software.simulated_tests.or_validation import OrValidation
 
 
@@ -144,5 +145,4 @@ def test_kickoff_play(simulated_test_runner, is_friendly_test):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/ai/hl/stp/play/offense/offense_play_test.py

@@ -1,14 +1,13 @@
-import sys
-
-import pytest
-
 import software.python_bindings as tbots_cpp
 from proto.play_pb2 import Play, PlayName
 from software.simulated_tests.friendly_team_scored import *
 from software.simulated_tests.ball_enters_region import *
 from software.simulated_tests.friendly_has_ball_possession import *
 from proto.message_translation.tbots_protobuf import create_world_state
 from proto.ssl_gc_common_pb2 import Team
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 def test_offense_play(simulated_test_runner):
@@ -99,5 +98,4 @@ def setup(start_point):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/field_tests/passing_field_test.py

@@ -1,12 +1,12 @@
-import pytest
-
 import software.python_bindings as tbots_cpp
-import sys
 from proto.import_all_protos import *
 from software.field_tests.field_test_fixture import *
 from software.simulated_tests.friendly_receives_ball_slow import (
     FriendlyAlwaysReceivesBallSlow,
 )
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 
 def test_passing(field_test_runner):
@@ -105,5 +105,4 @@ def test_passing(field_test_runner):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)

src/software/field_tests/pivot_kick_field_test.py

@@ -1,12 +1,12 @@
-import pytest
-
-import sys
 import math
 from proto.import_all_protos import *
 from software.field_tests.field_test_fixture import *
 
 from software.simulated_tests.simulated_test_fixture import *
 from software.logger.logger import create_logger
+from software.simulated_tests.simulated_test_fixture import (
+    pytest_main,
+)
 
 logger = create_logger(__name__)
 
@@ -45,5 +45,4 @@ def test_pivot_kick(field_test_runner):
 
 
 if __name__ == "__main__":
-    # Run the test, -s disables all capturing at -vv increases verbosity
-    sys.exit(pytest.main([__file__, "-svv"]))
+    pytest_main(__file__)