Initial scheduler test fixes

Signed-off-by: Samuel Monson <[email protected]>

Author: sjmonson

tests/unit/scheduler/test_constraints.py

@@ -286,11 +286,11 @@ def test_create_constraint_raises(self, valid_instances):
     def test_call_raises(self, valid_instances):
         """Test that calling constraint raises RuntimeError."""
         instance, _ = valid_instances
-        state = SchedulerState(node_id="test_node", num_processes=1, start_time=0.0)
+        state = SchedulerState(node_id=0, num_processes=1, start_time=0.0)
         request = ScheduledRequestInfo(
             request_id="test_request",
             status="pending",
-            scheduler_node_id="test_node",
+            scheduler_node_id=0,
             scheduler_process_id=1,
             scheduler_start_time=0.0,
         )

tests/unit/scheduler/test_environment.py

@@ -246,7 +246,7 @@ async def test_sync_run_start(self, valid_instances, mock_time, delay, expected)
 
         with (
             patch("time.time", return_value=mock_time),
-            patch("guidellm.scheduler.environment.settings") as mock_settings,
+            patch("guidellm.scheduler.environments.settings") as mock_settings,
         ):
             mock_settings.scheduler_start_delay_non_distributed = delay
             start_time = await instance.sync_run_start()

tests/unit/scheduler/test_objects.py

@@ -110,7 +110,7 @@ def test_generic_type_parameters(self):
 
         if hasattr(generic_base, "__args__"):
             type_params = generic_base.__args__
-            assert len(type_params) == 3, "Should have 3 type parameters"
+            assert len(type_params) == 2, "Should have 2 type parameters"
             param_names = [param.__name__ for param in type_params]
             expected_names = ["RequestT", "ResponseT"]
             assert param_names == expected_names
@@ -119,7 +119,7 @@ def test_generic_type_parameters(self):
     def test_implementation_construction(self):
         """Test that a complete concrete implementation can be instantiated."""
 
-        class ConcreteBackend(BackendInterface[str, MeasuredRequestTimings, str]):
+        class ConcreteBackend(BackendInterface[str, str]):
             @property
             def processes_limit(self) -> int | None:
                 return 4
@@ -162,7 +162,7 @@ async def resolve(
     async def test_implementation_async_methods(self):  # noqa: C901
         """Test that async methods work correctly in concrete implementation."""
 
-        class AsyncBackend(BackendInterface[dict, MeasuredRequestTimings, dict]):
+        class AsyncBackend(BackendInterface[dict, dict]):
             def __init__(self):
                 self.startup_called = False
                 self.validate_called = False
@@ -434,7 +434,6 @@ def valid_instances(self, request):
     @pytest.mark.smoke
     def test_class_signatures(self):
         """Test MeasuredRequestTimings inheritance and type relationships."""
-        assert issubclass(MeasuredRequestTimings, StandardBaseModel)
         assert hasattr(MeasuredRequestTimings, "model_dump")
         assert hasattr(MeasuredRequestTimings, "model_validate")