Fixed the audit log playwright failures (#25990)

* Fixed the audit log playwright failures

* minor fix

* minor fix

* minor fix

* addressed gitar comment

* addressed gitar comment

Author: Rohit0301

openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts

@@ -11,16 +11,17 @@
  *  limitations under the License.
  */
 import { APIRequestContext, expect, Page, test } from '@playwright/test';
+import { PLAYWRIGHT_BASIC_TEST_TAG_OBJ } from '../../constant/config';
 import { GlobalSettingOptions } from '../../constant/settings';
 import { getApiContext, redirectToHomePage } from '../../utils/common';
 import { settingClick } from '../../utils/sidebar';
-import { PLAYWRIGHT_BASIC_TEST_TAG_OBJ } from '../../constant/config';
 
 const navigateToAuditLogsPage = async (page: Page) => {
-  const logRequest=page.waitForResponse('/api/v1/audit/logs?*');
+  const logRequest = page.waitForResponse('/api/v1/audit/logs?*');
   await settingClick(page, GlobalSettingOptions.AUDIT_LOGS);
   await logRequest;
-  await page.waitForSelector('[data-testid="loader"]', { state: 'detached' });
+  await page.waitForSelector('.ant-skeleton', { state: 'detached' });
+  await page.getByTestId('audit-log-list').waitFor({ state: 'visible' });
 };
 
 test.use({ storageState: 'playwright/.auth/admin.json' });
@@ -118,6 +119,9 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
 
       // Verify Time filter is active
       const timeFilterTag = page.getByTestId('filter-chip-time');
+      await page.waitForSelector('.ant-skeleton', {
+        state: 'detached',
+      });
       await expect(timeFilterTag).toBeVisible();
     });
 
@@ -289,7 +293,13 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
       await userFilter.click();
 
       const searchInput = page.getByTestId('search-input');
+      const userSearchResponse = page.waitForResponse(
+        (response) =>
+          response.url().includes('/api/v1/search/query') &&
+          response.url().includes('index=user_search_index')
+      );
       await searchInput.fill('admin');
+      await userSearchResponse;
 
       const adminOption = page
         .locator('.ant-dropdown-menu')
@@ -330,8 +340,6 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
         const searchInput = page.getByTestId('search-input');
         await searchInput.fill(entityType);
 
-        await page.waitForTimeout(500);
-
         const entityOption = page.locator('.ant-dropdown-menu-item').first();
         await expect(entityOption).toBeVisible();
 
@@ -404,6 +412,9 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
       await searchInput.press('Enter');
       const response = await auditLogResponse;
       expect(response.status()).toBe(200);
+      await page.waitForSelector('.ant-skeleton', {
+        state: 'detached',
+      });
     });
 
     await test.step('Verify Clear button appears after search', async () => {
@@ -412,8 +423,15 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
     });
 
     await test.step('Clear search', async () => {
+      const auditLogResponse = page.waitForResponse((response) =>
+        response.url().includes('/api/v1/audit')
+      );
       const clearButton = page.getByTestId('clear-filters');
       await clearButton.click();
+      await auditLogResponse;
+      await page.waitForSelector('.ant-skeleton', {
+        state: 'detached',
+      });
 
       const searchInput = page.getByPlaceholder('Search audit logs');
       await expect(searchInput).toHaveValue('');
@@ -434,14 +452,24 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
       await searchInput.press('Enter');
       const response = await auditLogResponse;
       expect(response.status()).toBe(200);
+      await page.waitForSelector('.ant-skeleton', {
+        state: 'detached',
+      });
 
       // Clear search
       const clearButton = page.getByTestId('clear-filters');
       if (await clearButton.isVisible()) {
+        const auditLogResponseClear = page.waitForResponse((response) =>
+          response.url().includes('/api/v1/audit')
+        );
         await clearButton.click();
+        await auditLogResponseClear;
       }
 
       // Search with uppercase term - should return similar results
+      await page.waitForSelector('.ant-skeleton', {
+        state: 'detached',
+      });
       await searchInput.fill('ADMIN');
 
       const auditLogResponse2 = page.waitForResponse(
@@ -453,6 +481,9 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
       await searchInput.press('Enter');
       const response2 = await auditLogResponse2;
       expect(response2.status()).toBe(200);
+      await page.waitForSelector('.ant-skeleton', {
+        state: 'detached',
+      });
     });
   });
 
@@ -678,6 +709,9 @@ test.describe(
           await searchInput.press('Enter');
           const response = await auditLogResponse;
           expect(response.status()).toBe(200);
+          await page.waitForSelector('.ant-skeleton', {
+            state: 'detached',
+          });
           const responseData = await response.json();
 
           // Verify response has expected structure
@@ -788,11 +822,15 @@ test.describe(
       await test.step('Enter a search term', async () => {
         const searchInput = page.getByPlaceholder('Search audit logs');
         await searchInput.fill('admin');
-        await searchInput.press('Enter');
 
-        await page.waitForResponse((response) =>
+        const auditResponse = page.waitForResponse((response) =>
           response.url().includes('/api/v1/audit')
         );
+        await searchInput.press('Enter');
+        await auditResponse;
+        await page.waitForSelector('.ant-skeleton', {
+          state: 'detached',
+        });
       });
 
       await test.step('Open Export modal', async () => {
@@ -1601,6 +1639,9 @@ test.describe(
             await searchInput.press('Enter');
             const response = await searchResponse;
             expect(response.status()).toBe(200);
+            await page.waitForSelector('.ant-skeleton', {
+              state: 'detached',
+            });
             const responseData = await response.json();
 
             // Should find at least one entry