PR comments

globus · Sep 4, 2024 · 59889bd · 59889bd
1 parent bc56f4b
commit 59889bd
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/compute_endpoint/tests/unit/test_engines.py b/compute_endpoint/tests/unit/test_engines.py
@@ -373,13 +373,15 @@ def test_gcmpiengine_default_executor(randomstring):
     assert k["encrypted"] is True, "Expect encrypted by default"
 
 
-def test_gcmpiengine_accepts_resource_specification(task_uuid):
+def test_gcmpiengine_accepts_resource_specification(task_uuid, randomstring):
+    spec = {"some": "conf", "sentinel": randomstring()}
     with mock.patch.object(GlobusMPIEngine, "_ExecutorClass") as mock_ex:
         mock_ex.__name__ = "ClassName"
         mock_ex.return_value = mock.Mock(launch_cmd="")
         engine = GlobusMPIEngine()
-        engine.submit(
-            str(task_uuid), b"some task", resource_specification={"some": "conf"}
-        )
+        engine.submit(str(task_uuid), b"some task", resource_specification=spec)
 
     assert engine.executor.submit.called, "Verify test: correct internal method invoked"
+
+    a, _k = engine.executor.submit.call_args
+    assert spec in a