fix(gen-ai): Fix timeout handling in MCP tab visibility check

toklumpp · claude · toklumpp · commit 0a161bcebdd7 · 2026-04-22T09:56:12.000+01:00
Simplify verifyMCPTabVisible() to use a single should() assertion
with timeout parameter instead of chaining exist and be.visible.
This ensures the 30-second timeout is properly applied to the
visibility check, improving reliability when waiting for the MCP
servers table after modal interactions and tab remounts.

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('exist', { timeout: 30000 }).and('be.visible');
+    this.findMCPServersTable().should('be.visible', { timeout: 30000 });
   }
 
   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('exist', { timeout: 30000 }).and('be.visible');`
	`46`	`+ this.findMCPServersTable().should('be.visible', { timeout: 30000 });`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`private findCheckedCheckboxes(): Cypress.Chainable<JQuery<HTMLElement>> {`