fix(ApiDataFactory): Ensure uppercase DELETE method and add better logging

Changes:
1. Convert HTTP method to uppercase (DELETE not 'delete') for strict servers
2. Add detailed logging for delete operations (request + response)
3. Add error handling and logging for failed deletions
4. Check index exists before splicing from created array
5. Make afterEach async and add 200ms delay for Docker file sync

The lowercase 'delete' method came from using the config object key directly.
Some servers/proxies may reject lowercase HTTP methods.

Added comprehensive logging to help diagnose deletion issues in CI:
- Logs DELETE request details before sending
- Logs successful deletion with status code
- Logs and re-throws deletion errors

The additional 200ms delay in afterEach (on top of the 100ms in _after)
gives Docker environments extra time for file system synchronization.

Author: kobenguyent

lib/helper/ApiDataFactory.js

@@ -401,7 +401,7 @@ Current file error: ${err.message}`)
       const url = this.factories[factory].delete[method].replace('{id}', id)
 
       request = {
-        method,
+        method: method.toUpperCase(), // Ensure HTTP method is uppercase
         url,
       }
     }
@@ -412,10 +412,18 @@ Current file error: ${err.message}`)
       return this.debugSection('Please configure the delete request in your ApiDataFactory helper', "delete: () => ({ method: 'DELETE',  url: '/api/users' })")
     }
 
-    return this.restHelper._executeRequest(request).then(() => {
+    this.debugSection('Deleting', `${request.method} ${request.baseURL}${request.url} (ID: ${id})`)
+
+    return this.restHelper._executeRequest(request).then((resp) => {
       const idx = this.created[factory].indexOf(id)
-      this.debugSection('Deleted Id', `Id: ${id}`)
-      this.created[factory].splice(idx, 1)
+      this.debugSection('Deleted Successfully', `${factory} ID: ${id}, Status: ${resp.status || 'OK'}`)
+      if (idx !== -1) {
+        this.created[factory].splice(idx, 1)
+      }
+      return resp
+    }).catch(err => {
+      this.debugSection('Delete Failed', `${factory} ID: ${id}, Error: ${err.message}`)
+      throw err
     })
   }
 }

test/rest/ApiDataFactory_test.js

@@ -52,7 +52,11 @@ describe('ApiDataFactory', function () {
     setTimeout(done, 5000)
   })
 
-  afterEach(() => I._after())
+  afterEach(async () => {
+    await I._after()
+    // Add extra delay to ensure cleanup propagates in Docker environments
+    await new Promise(resolve => setTimeout(resolve, 200))
+  })
 
   describe('create and cleanup records', function () {
     this.retries(2)