From bf34395a5703968611b7e913697ff202e13bcffa Mon Sep 17 00:00:00 2001
From: NandanAnantharamu <67676905+NandanAnantharamu@users.noreply.github.com>
Date: Fri, 7 Jun 2024 18:56:02 +0530
Subject: [PATCH] test: flay table header validation (#34080)

Xpath used to read the header text is flaky.

Solution:
Replacing xpath with css locator to make it stable

EE PR: https://github.com/appsmithorg/appsmith-ee/pull/4382
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Tests**
- Updated spec names and file paths for tests related to Community
Issues and TableV2 widget.

- **Refactor**
- Simplified the selector for table headers to improve readability and
maintainability.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
---
 app/client/cypress/support/Pages/Table.ts | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/app/client/cypress/support/Pages/Table.ts b/app/client/cypress/support/Pages/Table.ts
index 7847e109e1..bac5097877 100644
--- a/app/client/cypress/support/Pages/Table.ts
+++ b/app/client/cypress/support/Pages/Table.ts
@@ -37,9 +37,7 @@ export class Table {
   private assertHelper = ObjectsRegistry.AssertHelper;
 
   private _tableWrap = "//div[contains(@class,'tableWrap')]";
-  private _tableHeader =
-    this._tableWrap +
-    "//div[contains(@class,'thead')]//div[contains(@class,'tr')][1]";
+  private _tableHeader = ".thead div[role=columnheader]";
   private _columnHeader = (columnName: string) =>
     this._tableWrap +
     "//div[contains(@class,'thead')]//div[contains(@class,'tr')][1]//div[@role='columnheader']//div[contains(text(),'" +
@@ -258,7 +256,7 @@ export class Table {
   }
 
   public AssertTableHeaderOrder(expectedOrder: string) {
-    cy.xpath(this._tableHeader)
+    cy.get(this._tableHeader)
       .invoke("text")
       .then((x) => {
         expect(x).to.eq(expectedOrder);