Thanks, @lin121291 for implementing review comments. I suggest resolving comments after you are done. I will review this soon

Pagination needs to follow the established pattern from PR #59400:

The current implementation differs from our merged pagination tests. Please reference:

• File: dags-list.spec.ts
• PR: Feat e2e test for Dags pagination #59400 (Feat e2e test for Dags pagination)

1. Use [data-testid="next/prev"] buttons instead of page numbers:
   // Current (incorrect)
   const page2Button = this.page.getByRole("button", { name: /page 2|^2$/ });

// Reference pattern (from DagsPage.ts)
this.paginationNextButton = page.locator('[data-testid="next"]');
this.paginationPrevButton = page.locator('[data-testid="prev"]');2. Remove if (hasPage2) conditional - beforeAll ensures data exists, pagination should work.

3. Verify data actually changed using expect.poll():
   // Reference from dags-list.spec.ts
   const initialData = await dagsPage.getDagNames();
   await dagsPage.clickNextPage();
   const dataAfterNext = await dagsPage.getDagNames();
   expect(dataAfterNext).not.toEqual(initialData); // Verifies data changed!See: Feat e2e test for Dags pagination #59400

@vatsrahul1001 Fixed! Now following the DagsPage pattern from PR #59400:

• Using data-testid="next/prev" selectors
• Removed conditional pagination checks
• Verifying data changes with `expect.poll()

Thanks !!

@vatsrahul1001 Fixed! Now following the DagsPage pattern from PR #59400:

• Using data-testid="next/prev" selectors
• Removed conditional pagination checks
• Verifying data changes with `expect.poll()

Thanks !!

Nice!