do not filter files on process model show and added sorting to files w/ burnettk

src/spiffworkflow_backend/models/file.py

@@ -88,10 +88,12 @@ CONTENT_TYPES = {
 }
 
 
-@dataclass
+@dataclass(order=True)
 class File:
     """File."""
 
+    sort_index: str = field(init=False)
+
     content_type: str
     name: str
     type: str
@@ -107,6 +109,10 @@ class File:
     process_group_id: Optional[str] = None
     archived: bool = False
 
+    def __post_init__(self):
+        """__post_init__."""
+        self.sort_index = f"{self.type}:{self.name}"
+
     @classmethod
     def from_file_system(
         cls, file_name, file_type, content_type, last_modified, file_size

src/spiffworkflow_backend/routes/process_api_blueprint.py

@@ -115,7 +115,7 @@ def process_model_show(process_group_id, process_model_id):
             )
         )
 
-    files = SpecFileService.get_files(process_model, extension_filter="bpmn")
+    files = sorted(SpecFileService.get_files(process_model))
     process_model.files = files
     process_model_json = ProcessModelInfoSchema().dump(process_model)
     return process_model_json

src/spiffworkflow_backend/scripts/script.py

@@ -49,7 +49,6 @@ class Script:
         We may be able to remove the task for each of these calls if we are not using it other than potentially
         updating the task data.
         """
-
         def make_closure(subclass, task, workflow_id):
             """Yes - this is black magic.
 
@@ -85,7 +84,6 @@ class Script:
         We may be able to remove the task for each of these calls if we are not using it other than potentially
         updating the task data.
         """
-
         def make_closure_validate(subclass, task, workflow_id):
             """Make_closure_validate."""
             instance = subclass()

tests/spiffworkflow_backend/unit/test_file.py

@@ -0,0 +1,19 @@
+"""test_file."""
+
+from spiffworkflow_backend.models.file import File
+
+
+def test_files_can_be_sorted() -> None:
+    """Test_files_can_be_sorted."""
+    europe = create_test_file(type="bpmn", name="europe")
+    asia = create_test_file(type="bpmn", name="asia")
+    africa = create_test_file(type="dmn", name="africa")
+    oceania = create_test_file(type="dmn", name="oceania")
+
+    mylist = [europe, oceania, asia, africa]
+    assert sorted(mylist) == [asia, europe, africa, oceania]
+
+
+def create_test_file(type: str, name: str):
+    """Create_test_file."""
+    return File(type=type, name=name, content_type=type, document={}, last_modified="Tuesday", size="1")