fix modified job patch schema definition to refer to expected execute parameters

Author: fmigneault

tests/wps_restapi/test_jobs.py

@@ -1929,11 +1929,13 @@ def test_job_update_execution_parameters(self):
         """
         Test modification of the execution ``return`` and ``response`` options, going back-and-forth between approaches.
         """
+        test_inputs = {"message": "test"}
+        test_outputs = {"result": {"transmissionMode": ExecuteTransmissionMode.VALUE}}
         new_job = self.make_job(
             task_id=self.fully_qualified_test_name(), process=self.process_public.identifier, service=None,
             status=Status.CREATED, progress=0, access=Visibility.PUBLIC,
-            execute_mode=ExecuteMode.AUTO,
-            execute_response=ExecuteResponse.DOCUMENT,
+            execute_mode=ExecuteMode.AUTO, execute_response=ExecuteResponse.DOCUMENT,
+            inputs=test_inputs, outputs=test_outputs,
         )
 
         body = {}
@@ -1950,6 +1952,8 @@ def test_job_update_execution_parameters(self):
         assert resp.json["mode"] == ExecuteMode.AUTO
         assert resp.json["response"] == ExecuteResponse.RAW
         assert resp.json["headers"]["Prefer"] == f"return={ExecuteReturnPreference.REPRESENTATION}"
+        assert resp.json["inputs"] == test_inputs
+        assert resp.json["outputs"] == test_outputs
 
         body = {"response": ExecuteResponse.DOCUMENT}
         path = f"/jobs/{new_job.id}"
@@ -1962,6 +1966,8 @@ def test_job_update_execution_parameters(self):
         assert resp.json["mode"] == ExecuteMode.AUTO
         assert resp.json["response"] == ExecuteResponse.DOCUMENT
         assert resp.json["headers"]["Prefer"] == f"return={ExecuteReturnPreference.MINIMAL}"
+        assert resp.json["inputs"] == test_inputs
+        assert resp.json["outputs"] == test_outputs
 
         body = {}
         headers = {
@@ -1977,6 +1983,8 @@ def test_job_update_execution_parameters(self):
         assert resp.json["mode"] == ExecuteMode.AUTO
         assert resp.json["response"] == ExecuteResponse.RAW
         assert resp.json["headers"]["Prefer"] == f"return={ExecuteReturnPreference.REPRESENTATION}"
+        assert resp.json["inputs"] == test_inputs
+        assert resp.json["outputs"] == test_outputs
 
         body = {"response": ExecuteResponse.RAW}
         path = f"/jobs/{new_job.id}"
@@ -1989,6 +1997,8 @@ def test_job_update_execution_parameters(self):
         assert resp.json["mode"] == ExecuteMode.AUTO
         assert resp.json["response"] == ExecuteResponse.RAW
         assert resp.json["headers"]["Prefer"] == f"return={ExecuteReturnPreference.REPRESENTATION}"
+        assert resp.json["inputs"] == test_inputs
+        assert resp.json["outputs"] == test_outputs
 
     @pytest.mark.oap_part4
     def test_job_update_subscribers(self):

weaver/wps_restapi/swagger_definitions.py

@@ -6754,7 +6754,7 @@ class JobTitleNullable(OneOfKeywordSchema):
     ]
 
 
-class PatchJobBodySchema(Execute):
+class PatchJobBodySchema(ExecuteParameters):
     description = "Execution request contents to be updated."
     # 'missing=null' ensures that, if a field is provided with an "empty" definition (JSON null, no-field dict, etc.),
     # contents are passed down as is rather than dropping them (what 'missing=drop' would do due to DropableSchemaNode)