Simplify E2E tests - remove excessive timeouts

- Remove explicit setTimeout calls (default 30s is sufficient)
- Replace waitForTimeout(5000/6000) with proper waitForAppLoad()
- Wait for inner iframe visibility instead of fixed delays
- Keep only 500ms stabilization for screenshot animations

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: ochafik

tests/e2e/servers.spec.ts

@@ -20,9 +20,18 @@ const SERVERS = [
   { key: "threejs", index: 7, name: "Three.js Server" },
 ];
 
+/**
+ * Wait for the MCP App to load inside nested iframes.
+ * Structure: page > iframe (sandbox) > iframe (app)
+ */
+async function waitForAppLoad(page: import("@playwright/test").Page) {
+  // Wait for outer iframe (sandbox)
+  const outerFrame = page.frameLocator("iframe").first();
+  // Wait for inner iframe (app) to be visible
+  await expect(outerFrame.locator("iframe")).toBeVisible();
+}
+
 test.describe("Host UI", () => {
-  // Increase timeout for iframe-heavy tests
-  test.setTimeout(90000);
   test("initial state shows controls", async ({ page }) => {
     await page.goto("/");
     await expect(page.locator("label:has-text('Server')")).toBeVisible();
@@ -32,57 +41,41 @@ test.describe("Host UI", () => {
 
   test("screenshot of initial state", async ({ page }) => {
     await page.goto("/");
-    await page.waitForTimeout(1000);
+    await expect(page.locator('button:has-text("Call Tool")')).toBeVisible();
     await expect(page).toHaveScreenshot("host-initial.png");
   });
 });
 
 // Generate tests for each server
 for (const server of SERVERS) {
   test.describe(`${server.name}`, () => {
-    test.setTimeout(90000);
-
     test(`loads app UI`, async ({ page }) => {
       await page.goto("/");
 
-      // Select server
-      const serverSelect = page.locator("select").first();
-      await serverSelect.selectOption({ index: server.index });
-
-      // Click Call Tool
+      // Select server and call tool
+      await page.locator("select").first().selectOption({ index: server.index });
       await page.click('button:has-text("Call Tool")');
 
-      // Wait for outer iframe
-      await page.waitForSelector("iframe", { timeout: 10000 });
-
-      // Wait for content to load (generous timeout for nested iframes)
-      await page.waitForTimeout(5000);
-
-      // Verify iframe structure exists
-      const outerFrame = page.frameLocator("iframe").first();
-      await expect(outerFrame.locator("iframe")).toBeVisible({
-        timeout: 10000,
-      });
+      // Wait for app to load in nested iframes
+      await waitForAppLoad(page);
     });
 
     test(`screenshot matches golden`, async ({ page }) => {
       await page.goto("/");
 
-      // Select server
-      const serverSelect = page.locator("select").first();
-      await serverSelect.selectOption({ index: server.index });
-
-      // Click Call Tool
+      // Select server and call tool
+      await page.locator("select").first().selectOption({ index: server.index });
       await page.click('button:has-text("Call Tool")');
 
-      // Wait for app to fully load
-      await page.waitForSelector("iframe", { timeout: 10000 });
-      await page.waitForTimeout(6000); // Extra time for nested iframe content
+      // Wait for app to load
+      await waitForAppLoad(page);
+
+      // Brief stabilization for animations/rendering
+      await page.waitForTimeout(500);
 
       // Take screenshot
       await expect(page).toHaveScreenshot(`${server.key}.png`, {
         maxDiffPixelRatio: 0.1, // 10% tolerance for dynamic content
-        timeout: 10000,
       });
     });
   });