fix: scope toast-container locators to main content area in e2e tests

SpaceSwitcher in the sidebar also renders a ToastContainer, causing
Playwright strict mode violations when getByTestId('toast-container')
resolves to 2 elements. Scoped all toast locators to
page.getByRole('main') to target only the main content area.

Author: Mohammod Al Amin Ashik

tests/e2e/pages/BasePage.ts

@@ -73,7 +73,7 @@ export abstract class BasePage {
    * Get text content of the first visible toast
    */
   async getToastText(): Promise<string | null> {
-    const toast = this.page.getByTestId('toast-container').locator('[role="alert"]').first();
+    const toast = this.page.getByRole('main').getByTestId('toast-container').locator('[role="alert"]').first();
     return toast.textContent();
   }
 
@@ -85,9 +85,9 @@ export abstract class BasePage {
   }
 
   /**
-   * Assert that a toast container is present in the DOM
+   * Assert that a toast container is present in the main content area
    */
   get toastContainer(): Locator {
-    return this.page.getByTestId('toast-container');
+    return this.page.getByRole('main').getByTestId('toast-container');
   }
 }

tests/e2e/pages/SettingsPage.ts

@@ -27,7 +27,7 @@ export class SettingsPage extends BasePage {
     this.autoLaunchSwitch = page.getByTestId('auto-launch-switch');
     this.startMinimizedSwitch = page.getByTestId('start-minimized-switch');
     this.closeToTraySwitch = page.getByTestId('close-to-tray-switch');
-    this.toastContainer = page.getByTestId('toast-container');
+    this.toastContainer = page.getByRole('main').getByTestId('toast-container');
   }
 
   async selectTheme(theme: 'light' | 'dark' | 'system') {
@@ -60,7 +60,7 @@ export class SettingsPage extends BasePage {
   }
 
   async getToastText() {
-    const toast = this.page.getByTestId('toast-container').locator('[role="alert"]').first();
+    const toast = this.page.getByRole('main').getByTestId('toast-container').locator('[role="alert"]').first();
     return toast.textContent();
   }
 

tests/e2e/specs/featuresets.spec.ts

@@ -80,7 +80,7 @@ test.describe('Feature Set Toast Container', () => {
     await page.locator('nav button:has-text("FeatureSets")').click({ force: true });
     await expect(page.getByRole('heading', { name: 'Feature Sets' }).first()).toBeVisible();
 
-    await expect(page.getByTestId('toast-container')).toBeAttached();
+    await expect(page.getByRole('main').getByTestId('toast-container')).toBeAttached();
   });
 });
 
@@ -176,8 +176,8 @@ test.describe('Feature Set Panel Save Toast', () => {
       await page.getByRole('button', { name: /Save Changes/i }).click();
 
       // Wait for success toast
-      await expect(page.getByTestId('toast-success')).toBeVisible({ timeout: 5000 });
-      const toastText = await page.getByTestId('toast-container').locator('[role="alert"]').first().textContent();
+      await expect(page.getByTestId('toast-success').first()).toBeVisible({ timeout: 5000 });
+      const toastText = await page.getByRole('main').getByTestId('toast-container').locator('[role="alert"]').first().textContent();
       expect(toastText).toContain('Changes saved');
     }
   });
@@ -201,8 +201,8 @@ test.describe('Feature Set Panel Save Toast', () => {
       if (await saveButton.isVisible()) {
         await saveButton.click();
 
-        await expect(page.getByTestId('toast-error')).toBeVisible({ timeout: 5000 });
-        const toastText = await page.getByTestId('toast-container').locator('[role="alert"]').first().textContent();
+        await expect(page.getByTestId('toast-error').first()).toBeVisible({ timeout: 5000 });
+        const toastText = await page.getByRole('main').getByTestId('toast-container').locator('[role="alert"]').first().textContent();
         expect(toastText).toContain('Failed to save');
       }
     }

tests/e2e/specs/servers.spec.ts

@@ -126,8 +126,8 @@ test.describe('Server Toast Notifications', () => {
       if (await clearBtn.isVisible()) {
         await clearBtn.click();
 
-        await expect(page.getByTestId('toast-success')).toBeVisible({ timeout: 5000 });
-        const toastText = await page.getByTestId('toast-container').locator('[role="alert"]').first().textContent();
+        await expect(page.getByTestId('toast-success').first()).toBeVisible({ timeout: 5000 });
+        const toastText = await page.getByRole('main').getByTestId('toast-container').locator('[role="alert"]').first().textContent();
         expect(toastText).toContain('Logs cleared');
       }
     }
@@ -149,8 +149,8 @@ test.describe('Server Toast Notifications', () => {
       if (await copyBtn.isVisible()) {
         await copyBtn.click();
 
-        await expect(page.getByTestId('toast-success')).toBeVisible({ timeout: 5000 });
-        const toastText = await page.getByTestId('toast-container').locator('[role="alert"]').first().textContent();
+        await expect(page.getByTestId('toast-success').first()).toBeVisible({ timeout: 5000 });
+        const toastText = await page.getByRole('main').getByTestId('toast-container').locator('[role="alert"]').first().textContent();
         expect(toastText).toContain('Path copied');
       }
     }

tests/e2e/specs/settings.spec.ts

@@ -303,8 +303,8 @@ test.describe('Settings', () => {
 
       await page.locator('nav button:has-text("Settings")').click();
 
-      // Toast container should exist (even if empty)
-      const toastContainer = page.getByTestId('toast-container');
+      // Toast container should exist in main content (even if empty)
+      const toastContainer = page.getByRole('main').getByTestId('toast-container');
       await expect(toastContainer).toBeAttached();
     });