fix(gen-ai): Restore original timeout pattern for MCP tab visibility check

toklumpp · claude · toklumpp · commit e31fdc86f6b7 · 2026-04-23T10:12:42.000+01:00
Restore the original chained assertion pattern instead of the simplified
version. The chained approach ensures the element exists in the DOM before
checking visibility, which is more reliable when waiting for components
that may remount after modal interactions.

The timeout applies to the existence check, and visibility is verified
immediately after with Cypress built-in retry logic.

Co-Authored-By: Claude Sonnet 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/packages/gen-ai/frontend/src/__tests__/cypress/cypress/pages/playgroundPage/mcpTab.ts b/packages/gen-ai/frontend/src/__tests__/cypress/cypress/pages/playgroundPage/mcpTab.ts
@@ -43,7 +43,7 @@ class MCPTab {
   }
 
   verifyMCPTabVisible(): void {
-    this.findMCPServersTable().should('be.visible', { timeout: 30000 });
+    this.findMCPServersTable().should('exist', { timeout: 30000 }).and('be.visible');
   }
 
   private findCheckedCheckboxes(): Cypress.Chainable<JQuery<HTMLElement>> {

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ class MCPTab {`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`verifyMCPTabVisible(): void {`
`46`		`- this.findMCPServersTable().should('be.visible', { timeout: 30000 });`
	`46`	`+ this.findMCPServersTable().should('exist', { timeout: 30000 }).and('be.visible');`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`private findCheckedCheckboxes(): Cypress.Chainable<JQuery<HTMLElement>> {`