Cleaned up the deleteSboms helper function.

Signed-off-by: Vilem Obratil <vobratil@redhat.com>

Author: vobratil

e2e/tests/api/features/performance-delete.ts

@@ -103,15 +103,20 @@ test.describe("Performance / Deletion", { tag: "@performance" }, () => {
   test.afterEach(async ({ axios }) => {
     logger.info("Cleaning up SBOMs after deletion performance tests.");
 
-    await deleteSboms(axios, sbomIds).then((success) => {
-      if (success) {
-        logger.info("All SBOMs were deleted successfully.");
-      } else {
-        logger.warn(
-          "One or more SBOMs could not be deleted. Check the logs and/or consider deleting the SBOMs manually.",
-        );
-      }
-    });
+    let responses = await deleteSboms(axios, sbomIds);
+
+    if (
+      responses.every(
+        (result) =>
+          result.status === "fulfilled" && result.value?.status === 200,
+      )
+    ) {
+      logger.info("All SBOMS have been deleted successfully.");
+    } else {
+      logger.warn(
+        "Some SBOM deletions were unsuccessful. Check the logs and/or consider deleting the SBOMs manually.",
+      );
+    }
 
     sbomIds = [];
   });

e2e/tests/api/helpers/delete.ts

@@ -15,16 +15,10 @@ export async function deleteSboms(axios: AxiosInstance, sbomIds: string[]) {
   }
 
   const deletionPromises = existingSbomIds.map((sbomId) =>
-    axios.delete(`/api/v2/sbom/${sbomId}`).catch((error) => {
-      logger.error(`Failed to delete SBOM with ID: ${sbomId}`, error);
-    }),
+    axios.delete(`/api/v2/sbom/${sbomId}`),
   );
 
-  const results = await Promise.allSettled(deletionPromises);
+  const responses = await Promise.allSettled(deletionPromises);
 
-  const allSuccessful = results.every(
-    (result) => result.status === "fulfilled" && result.value?.status === 200,
-  );
-
-  return allSuccessful;
+  return responses;
 }