Fix APIActions header validation and add robust API integration tests

kavitaj11 · kavitaj11 · commit 498b45a2ca59 · 2026-02-09T21:15:47.000-06:00
diff --git a/apiresponse/advice_error_2026-02-09T05-47-27-313Z.json b/apiresponse/advice_error_2026-02-09T05-47-27-313Z.json
diff --git a/apiresponse/advice_error_2026-02-09T05-47-27-324Z.json b/apiresponse/advice_error_2026-02-09T05-47-27-324Z.json
diff --git a/apiresponse/advice_error_2026-02-09T05-47-27-363Z.json b/apiresponse/advice_error_2026-02-09T05-47-27-363Z.json
diff --git a/apiresponse/advice_error_2026-02-09T05-47-27-580Z.json b/apiresponse/advice_error_2026-02-09T05-47-27-580Z.json
diff --git a/apiresponse/advice_error_2026-02-09T05-47-30-911Z.json b/apiresponse/advice_error_2026-02-09T05-47-30-911Z.json
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-09-04-021Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-09-04-021Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-09-04-082Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-09-04-082Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-09-04-159Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-09-04-159Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-09-07-753Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-09-07-753Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-09-08-070Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-09-08-070Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-10-35-795Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-10-35-795Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-10-35-796Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-10-35-796Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-10-35-816Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-10-35-816Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-10-39-274Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-10-39-274Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-10-39-306Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-10-39-306Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-13-01-393Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-13-01-393Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-13-01-444Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-13-01-444Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-13-01-512Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-13-01-512Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-13-04-489Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-13-04-489Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_get_post_2026-02-10T03-13-04-659Z.json b/apiresponse/apiactions_get_post_2026-02-10T03-13-04-659Z.json
@@ -0,0 +1,6 @@
+{
+  "userId": 1,
+  "id": 1,
+  "title": "sunt aut facere repellat provident occaecati excepturi optio reprehenderit",
+  "body": "quia et suscipit\nsuscipit recusandae consequuntur expedita et cum\nreprehenderit molestiae ut ut quas totam\nnostrum rerum est autem sunt rem eveniet architecto"
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-09-04-066Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-09-04-066Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-09-04-184Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-09-04-184Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-09-04-188Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-09-04-188Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-09-07-674Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-09-07-674Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-09-07-789Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-09-07-789Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-10-35-844Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-10-35-844Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-10-35-886Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-10-35-886Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-10-35-888Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-10-35-888Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-10-39-328Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-10-39-328Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-10-39-382Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-10-39-382Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-13-01-401Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-13-01-401Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-13-01-474Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-13-01-474Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-13-01-659Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-13-01-659Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-13-04-587Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-13-04-587Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/apiresponse/apiactions_post_post_2026-02-10T03-13-04-719Z.json b/apiresponse/apiactions_post_post_2026-02-10T03-13-04-719Z.json
@@ -0,0 +1,6 @@
+{
+  "title": "foo",
+  "body": "bar",
+  "userId": 1,
+  "id": 101
+}
diff --git a/tests/api/APIActionsIntegration.spec.js b/tests/api/APIActionsIntegration.spec.js
@@ -0,0 +1,38 @@
+const { test, expect } = require('@playwright/test');
+const APIActions = require('../../utils/APIActions');
+const { saveApiResponse } = require('../../utils/saveApiResponse');
+
+const JSONPLACEHOLDER_API_URL = 'https://jsonplaceholder.typicode.com';
+const apiActions = new APIActions();
+
+test.describe('API Actions Utility Integration', () => {
+  test('should GET a post and verify status and body fields', async ({ request }) => {
+    const response = await request.get(`${JSONPLACEHOLDER_API_URL}/posts/1`);
+    await apiActions.verifyStatusCode(response);
+    const body = await response.json();
+    saveApiResponse('apiactions_get_post', body);
+    await apiActions.verifyResponseBody('userId|id|title|body', body, 'JSON Body');
+  });
+
+  test('should POST a new post and verify status and response', async ({ request }) => {
+    const payload = { title: 'foo', body: 'bar', userId: 1 };
+    const response = await request.post(`${JSONPLACEHOLDER_API_URL}/posts`, {
+      data: payload,
+      headers: { 'Accept': 'application/json' }
+    });
+    await apiActions.verifyStatusCode(response);
+    const body = await response.json();
+    saveApiResponse('apiactions_post_post', body);
+    await apiActions.verifyResponseBody('id|title|body|userId', body, 'JSON Body');
+  });
+
+  test('should print and verify response headers for GET', async ({ request }) => {
+    const response = await request.get(`${JSONPLACEHOLDER_API_URL}/posts/1`);
+    await apiActions.verifyStatusCode(response);
+    const headers = response.headersArray();
+    // Print all response headers for debugging
+    console.log('Actual response headers:', headers.map(h => h.name));
+    // Only check for 'content-type' which should always be present
+    await apiActions.verifyResponseHeader(['content-type'], headers, 'Headers');
+  });
+});
diff --git a/utils/APIActions.js b/utils/APIActions.js
@@ -24,10 +24,12 @@ class APIActions {
     async verifyResponseHeader(expectedResponseHeaderParams, responsePart, responseType) {
         let status = true;
         let fieldNames = 'Parameter';
-        for (let responseKey of responsePart) {
-            if (!(expectedResponseHeaderParams.includes(responseKey.name.trim()))) {
+        // Support both arrays of header objects ({name: ...}) and arrays of header name strings
+        const headerNames = responsePart.map(h => typeof h === 'string' ? h.toLowerCase() : (h.name ? h.name.toLowerCase() : ''));
+        for (let expectedHeader of expectedResponseHeaderParams) {
+            if (!headerNames.includes(expectedHeader.toLowerCase())) {
                 status = false;
-                fieldNames = fieldNames + ' ,' + responseKey.name;
+                fieldNames = fieldNames + ', ' + expectedHeader;
                 break;
             }
         }
