transfer jobs as tuple, not set

bihealth · Dec 8, 2023 · 2fa772c · 2fa772c
1 parent 70908b7
commit 2fa772c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/cubi_tk/sodar/ingest.py b/cubi_tk/sodar/ingest.py
@@ -259,7 +259,7 @@ def build_file_list(self) -> typing.List[typing.Dict[Path, Path]]:
                     output_paths.append({"spath": src, "ipath": Path(src.name)})
         return output_paths
 
-    def build_jobs(self, source_paths: typing.Iterable[Path]) -> typing.Set[TransferJob]:
+    def build_jobs(self, source_paths: typing.Iterable[Path]) -> typing.Tuple[TransferJob]:
         """Build file transfer jobs."""
 
         transfer_jobs = []
@@ -289,7 +289,7 @@ def build_jobs(self, source_paths: typing.Iterable[Path]) -> typing.Set[Transfer
                 )
             )
 
-        return set(transfer_jobs)
+        return tuple(transfer_jobs)
 
 
 def setup_argparse(parser: argparse.ArgumentParser) -> None:

diff --git a/tests/test_sodar_ingest.py b/tests/test_sodar_ingest.py
@@ -177,6 +177,9 @@ class DummyColl(object):
         lz_uuid,
     ]
 
+    # to make it sortable
+    mockjob.return_value.path_local = 1
+
     # Test env file missing
     with pytest.raises(SystemExit):
         main(argv)