Flag to run tests with static ids

Author: FilipeNV1

tests/conftest.py

@@ -0,0 +1,31 @@
+"""Pytest configuration for test suite."""
+import uuid
+import pytest
+
+
+def pytest_addoption(parser):
+    """Add custom command-line options to pytest."""
+    parser.addoption(
+        "--fixed-ids",
+        action="store_true",
+        default=False,
+        help="Use fixed car IDs instead of random UUIDs (useful for frontend testing)",
+    )
+
+
+@pytest.fixture(scope="session")
+def use_fixed_ids(request):
+    """Fixture that returns whether to use fixed car IDs."""
+    return request.config.getoption("--fixed-ids")
+
+
+@pytest.fixture
+def get_car_id(use_fixed_ids):
+    """Fixture that returns a function to generate car IDs."""
+    def _get_car_id(base_name: str) -> str:
+        """Generate car ID: random UUID by default, or fixed name with --fixed-ids flag."""
+        if use_fixed_ids:
+            return base_name
+        return f"{base_name}-{str(uuid.uuid4())[:8]}"
+    
+    return _get_car_id

tests/test_accident.py

@@ -91,11 +91,10 @@ def collect_alerts(alert_queue: queue.Queue, car_ids: list[str], timeout: float
     return alerts
 
 
-def test_accident_directional_notification():
-    random_id = str(uuid.uuid4())[:8]
-    accident_car = f"accident-car-{random_id}"
-    car_behind = f"car-behind-{random_id}"
-    car_ahead = f"car-ahead-{random_id}"
+def test_accident_directional_notification(get_car_id):
+    accident_car = get_car_id("accident-car")
+    car_behind = get_car_id("car-behind")
+    car_ahead = get_car_id("car-ahead")
 
     ensure_car_exists(accident_car)
     ensure_car_exists(car_behind)
@@ -185,6 +184,17 @@ def other_cars_thread():
         time.sleep(2)
         alerts = collect_alerts(alert_queue, [car_behind, car_ahead], timeout=ALERT_TIMEOUT)
 
+    # Remove car device files to avoid interference with next test
+    accident_car_file = SIM_DIR / "devices" / f"{accident_car}.json"
+    car_behind_file = SIM_DIR / "devices" / f"{car_behind}.json"
+    car_ahead_file = SIM_DIR / "devices" / f"{car_ahead}.json"
+    if accident_car_file.exists():
+        accident_car_file.unlink()
+    if car_behind_file.exists():
+        car_behind_file.unlink()
+    if car_ahead_file.exists():
+        car_ahead_file.unlink()
+
     # Assertions
     assert len(alerts[car_behind]) > 0, (
         f"Car BEHIND should receive accident alerts. "
@@ -197,17 +207,6 @@ def other_cars_thread():
         f"Got {len(alerts[car_ahead])} alerts."
     )
 
-    # Remove car device files to avoid interference with next test
-    accident_car_file = SIM_DIR / "devices" / f"{accident_car}.json"
-    car_behind_file = SIM_DIR / "devices" / f"{car_behind}.json"
-    car_ahead_file = SIM_DIR / "devices" / f"{car_ahead}.json"
-    if accident_car_file.exists():
-        accident_car_file.unlink()
-    if car_behind_file.exists():
-        car_behind_file.unlink()
-    if car_ahead_file.exists():
-        car_ahead_file.unlink()
-
 
 if __name__ == "__main__":
     test_accident_directional_notification()

tests/test_curved_route.py

@@ -38,10 +38,9 @@ def send_position(car_name: str, lat: float, lon: float) -> None:
     ], check=True)
 
 
-def test_curved_route():
+def test_curved_route(get_car_id):
     """Test vehicle navigation on a curved route with complex trajectory."""
-    random_id = str(uuid.uuid4())[:8]
-    car = f"curved-route-car-{random_id}"
+    car = get_car_id("curved-route-car")
     ensure_car_exists(car)
 
     # subscribe to position updates topic

tests/test_emergency_vehicle.py

@@ -34,10 +34,9 @@ def send_position(car_name: str, lat: float, lon: float) -> None:
     )
 
 
-def test_emergency_vehicle():
-    random_id = str(uuid.uuid4())[:8]
-    car_regular = f"ev-test-regular-{random_id}"      # regular car (victim)
-    car_emergency = f"ev-test-emergency-{random_id}"  # emergency vehicle
+def test_emergency_vehicle(get_car_id):
+    car_regular = get_car_id("ev-test-regular")      # regular car (victim)
+    car_emergency = get_car_id("ev-test-emergency")  # emergency vehicle
 
     ensure_car_exists(car_regular, emergency=False)
     ensure_car_exists(car_emergency, emergency=True)

tests/test_highway_entry.py

@@ -26,12 +26,11 @@ def on_highway_entry_alert(client, userdata, msg):
     ALERTS.append(json.loads(msg.payload.decode()))
 
 
-def test_highway_entry_unsafe():
+def test_highway_entry_unsafe(get_car_id):
     ALERTS.clear()  # Clear alerts at the start
 
-    random_id = str(uuid.uuid4())[:8]
-    highway_car = f"highway-car-{random_id}"
-    entering_car = f"entering-car-{random_id}"
+    highway_car = get_car_id("highway-car")
+    entering_car = get_car_id("entering-car")
     ensure_car_exists(highway_car)
     ensure_car_exists(entering_car)
 
@@ -103,12 +102,11 @@ def test_highway_entry_unsafe():
     assert len(unsafe_alerts) > 0, f"Expected unsafe alert but got: {ALERTS}"
 
 
-def test_highway_entry_safe():
+def test_highway_entry_safe(get_car_id):
     ALERTS.clear()
 
-    random_id = str(uuid.uuid4())[:8]
-    highway_car = f"highway-car-{random_id}"
-    entering_car = f"entering-car-{random_id}"
+    highway_car = get_car_id("highway-car-2")
+    entering_car = get_car_id("entering-car-2")
 
     ensure_car_exists(highway_car)
     ensure_car_exists(entering_car)

tests/test_overtaking.py

@@ -25,10 +25,9 @@ def send_position(car_name: str, lat: float, lon: float) -> None:
         car_name, str(lat), str(lon)
     ], check=True)
 
-def test_overtaking():
-    random_id = str(uuid.uuid4())[:8]
-    car_slow = f"overtaking-car-front-{random_id}" # victim
-    car_fast = f"overtaking-car-behind-{random_id}" # overtaker
+def test_overtaking(get_car_id):
+    car_slow = get_car_id("overtaking-car-front") # victim
+    car_fast = get_car_id("overtaking-car-behind") # overtaker
 
     ensure_car_exists(car_slow)
     ensure_car_exists(car_fast)

tests/test_speeding.py

@@ -25,9 +25,8 @@ def on_speed_alert(client, userdata, msg):
     ALERTS.append(json.loads(msg.payload.decode()))
 
 
-def test_speeding():
-    random_id = str(uuid.uuid4())[:8]
-    car = f"speed-car-{random_id}"
+def test_speeding(get_car_id):
+    car = get_car_id("speed-car")
     ensure_car_exists(car)
 
     # subscribe to speed alerts