diff --git a/socorro/external/fs/crashstorage.py b/socorro/external/fs/crashstorage.py index a1e9b6e87c..2f58ea2148 100644 --- a/socorro/external/fs/crashstorage.py +++ b/socorro/external/fs/crashstorage.py @@ -246,7 +246,7 @@ def catalog_crash(self, crash_id): self.get_processed_crash(crash_id) contents.append("fs_processed_crash") - return contents + return list(sorted(contents)) def delete_crash(self, crash_id): parent_dir = self._get_radixed_parent_directory(crash_id) diff --git a/socorro/external/gcs/crashstorage.py b/socorro/external/gcs/crashstorage.py index e5664448c5..b1f2c4694d 100644 --- a/socorro/external/gcs/crashstorage.py +++ b/socorro/external/gcs/crashstorage.py @@ -331,7 +331,7 @@ def catalog_crash(self, crash_id): f"could not parse dump_names: gs://{self.bucket}/{key}" ) - return contents + return list(sorted(contents)) def delete_crash(self, crash_id): # delete raw and processed crash diff --git a/socorro/tests/external/fs/test_fspermanentstorage.py b/socorro/tests/external/fs/test_fspermanentstorage.py index 0f1b3e4654..01c3c32e58 100644 --- a/socorro/tests/external/fs/test_fspermanentstorage.py +++ b/socorro/tests/external/fs/test_fspermanentstorage.py @@ -107,11 +107,11 @@ def test_catalog_crash(self, tmp_path): fs.save_raw_crash(raw_crash=raw_crash, dumps=dumps, crash_id=crash_id) fs.save_processed_crash(raw_crash=raw_crash, processed_crash=processed_crash) - assert fs.catalog_crash(crash_id=crash_id) == [ - "fs_raw_crash", - "fs_dump_upload_file_minidump", + assert list(sorted(fs.catalog_crash(crash_id=crash_id))) == [ "fs_dump_memory_report", + "fs_dump_upload_file_minidump", "fs_processed_crash", + "fs_raw_crash", ] def test_delete_crash(self, tmp_path): diff --git a/socorro/tests/external/gcs/test_crashstorage.py b/socorro/tests/external/gcs/test_crashstorage.py index af2b3c42bc..d038a12471 100644 --- a/socorro/tests/external/gcs/test_crashstorage.py +++ b/socorro/tests/external/gcs/test_crashstorage.py @@ -415,11 +415,11 @@ def test_catalog_crash(self, gcs_helper): ) assert crashstorage.catalog_crash(crash_id=crash_id) == [ - "gcs_raw_crash", - "gcs_processed_crash", - "gcs_dump_names", "gcs_dump_dump", "gcs_dump_memory_report", + "gcs_dump_names", + "gcs_processed_crash", + "gcs_raw_crash", ] def test_delete_crash(self, gcs_helper):