fix(ApiDataFactory): Improve cleanup reliability in _after() method

- Add error handling to catch and log deletion failures without breaking Promise.all
- Explicitly clear created items array after successful cleanup
- Check for undefined deletePromise before adding to promises array

This should help prevent test failures where data from previous tests
accumulates, especially in CI environments where timing may differ.

The cleanup now:
1. Catches individual deletion errors to prevent Promise.all rejection
2. Logs deletion errors for debugging
3. Clears the created items tracker after all deletions complete
4. Handles cases where _requestDelete returns undefined

Author: kobenguyent

lib/helper/ApiDataFactory.js

@@ -241,10 +241,22 @@ class ApiDataFactory extends Helper {
       if (!createdItems.length) continue
       this.debug(`Deleting ${createdItems.length} ${factoryName}(s)`)
       for (const id in createdItems) {
-        promises.push(this._requestDelete(factoryName, createdItems[id]))
+        const deletePromise = this._requestDelete(factoryName, createdItems[id])
+        if (deletePromise) {
+          promises.push(deletePromise.catch(err => {
+            this.debugSection('Delete Error', `Failed to delete ${factoryName} with id ${createdItems[id]}: ${err.message}`)
+            // Don't reject Promise.all, just log the error
+            return Promise.resolve()
+          }))
+        }
       }
     }
-    return Promise.all(promises)
+    return Promise.all(promises).then(() => {
+      // Clear the created items after successful cleanup
+      for (const factoryName in this.created) {
+        this.created[factoryName] = []
+      }
+    })
   }
 
   /**