lint

spiffworkflow-backend/src/spiffworkflow_backend/routes/process_models_controller.py

@@ -1,7 +1,5 @@
 """APIs for dealing with process groups, process models, and process instances."""
 import json
-from spiffworkflow_backend.services.file_system_service import FileSystemService
-from spiffworkflow_backend.services.process_model_test_runner_service import ProcessModelTestRunner
 import os
 import re
 from hashlib import sha256
@@ -32,6 +30,7 @@ from spiffworkflow_backend.routes.process_api_blueprint import _get_process_mode
 from spiffworkflow_backend.routes.process_api_blueprint import (
     _un_modify_modified_process_model_id,
 )
+from spiffworkflow_backend.services.file_system_service import FileSystemService
 from spiffworkflow_backend.services.git_service import GitCommandError
 from spiffworkflow_backend.services.git_service import GitService
 from spiffworkflow_backend.services.git_service import MissingGitConfigsError
@@ -45,6 +44,7 @@ from spiffworkflow_backend.services.process_model_service import ProcessModelSer
 from spiffworkflow_backend.services.process_model_service import (
     ProcessModelWithInstancesNotDeletableError,
 )
+from spiffworkflow_backend.services.process_model_test_runner_service import ProcessModelTestRunner
 from spiffworkflow_backend.services.spec_file_service import (
     ProcessModelFileInvalidError,
 )

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/exclusive-gateway-based-on-user-task/choose-your-branch-schema.json

@@ -8,4 +8,4 @@
     }
   },
   "required": []
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/exclusive-gateway-based-on-user-task/choose-your-branch-uischema.json

@@ -2,4 +2,4 @@
   "ui:order": [
     "branch"
   ]
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/exclusive-gateway-based-on-user-task/process_model.json

@@ -8,4 +8,4 @@
     "metadata_extraction_paths": null,
     "primary_file_name": "exclusive_gateway_based_on_user_task.bpmn",
     "primary_process_id": "exclusive_gateway_based_on_user_task_process"
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/loopback-to-user-task/process_model.json

@@ -8,4 +8,4 @@
     "metadata_extraction_paths": null,
     "primary_file_name": "loopback_to_user_task.bpmn",
     "primary_process_id": "loopback_to_user_task_process"
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/loopback-to-user-task/user-input-schema.json

@@ -8,4 +8,4 @@
     }
   },
   "required": []
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/loopback-to-user-task/user-input-uischema.json

@@ -2,4 +2,4 @@
   "ui:order": [
     "user_input_variable"
   ]
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/loopback/process_model.json

@@ -8,4 +8,4 @@
     "metadata_extraction_paths": null,
     "primary_file_name": "loopback.bpmn",
     "primary_process_id": "loopback_process"
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/multiple-test-files/process_model.json

@@ -8,4 +8,4 @@
     "metadata_extraction_paths": null,
     "primary_file_name": "a.bpmn",
     "primary_process_id": "ProcessA"
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/multiple-test-files/test_a.json

@@ -2,4 +2,4 @@
   "test_case_1": {
     "expected_output_json": { "a": 1 }
   }
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/multiple-test-files/test_b.json

@@ -5,4 +5,4 @@
   "test_case_2": {
     "expected_output_json": { "b": 1 }
   }
-}
+}

spiffworkflow-backend/tests/data/bpmn_unit_test_process_models/expected-to-pass/process_group.json

@@ -6,4 +6,4 @@
     "parent_groups": null,
     "process_groups": [],
     "process_models": []
-}
+}

spiffworkflow-backend/tests/spiffworkflow_backend/unit/test_process_model_test_runner.py

@@ -62,13 +62,21 @@ class TestProcessModelTestRunner(BaseTest):
         process_model_test_runner = self._run_model_tests(bpmn_process_directory_name="multiple-test-files")
         assert len(process_model_test_runner.test_case_results) == 3
 
-        process_model_test_runner = self._run_model_tests(bpmn_process_directory_name="multiple-test-files", test_case_file='test_a.json')
+        process_model_test_runner = self._run_model_tests(
+            bpmn_process_directory_name="multiple-test-files", test_case_file="test_a.json"
+        )
         assert len(process_model_test_runner.test_case_results) == 1
 
-        process_model_test_runner = self._run_model_tests(bpmn_process_directory_name="multiple-test-files", test_case_file='test_b.json')
+        process_model_test_runner = self._run_model_tests(
+            bpmn_process_directory_name="multiple-test-files", test_case_file="test_b.json"
+        )
         assert len(process_model_test_runner.test_case_results) == 2
 
-        process_model_test_runner = self._run_model_tests(bpmn_process_directory_name="multiple-test-files", test_case_file='test_b.json', test_case_identifier='test_case_2')
+        process_model_test_runner = self._run_model_tests(
+            bpmn_process_directory_name="multiple-test-files",
+            test_case_file="test_b.json",
+            test_case_identifier="test_case_2",
+        )
         assert len(process_model_test_runner.test_case_results) == 1
 
     def test_can_test_process_model_call_activity(
@@ -99,8 +107,11 @@ class TestProcessModelTestRunner(BaseTest):
         assert len(process_model_test_runner.test_case_results) == 1
 
     def _run_model_tests(
-        self, bpmn_process_directory_name: Optional[str] = None, parent_directory: str = "expected-to-pass",
+        self,
-        test_case_file: Optional[str] = None, test_case_identifier: Optional[str] = None,
+        bpmn_process_directory_name: Optional[str] = None,
+        parent_directory: str = "expected-to-pass",
+        test_case_file: Optional[str] = None,
+        test_case_identifier: Optional[str] = None,
     ) -> ProcessModelTestRunner:
         base_process_model_dir_path_segments = [FileSystemService.root_path(), parent_directory]
         path_segments = base_process_model_dir_path_segments