fix broken test

Author: dididy

zeppelin-web-angular/e2e/models/notebook-paragraph-page.ts

@@ -71,10 +71,6 @@ export class NotebookParagraphPage extends BasePage {
     return await this.resultDisplay.isVisible();
   }
 
-  async isCodeEditorVisible(): Promise<boolean> {
-    return await this.codeEditor.isVisible();
-  }
-
   async getFooterText(): Promise<string> {
     return (await this.footerInfo.textContent()) || '';
   }

zeppelin-web-angular/e2e/models/notebook-paragraph-page.util.ts

@@ -59,9 +59,7 @@ export class NotebookParagraphUtil {
   }
 
   async verifyCodeEditorVisibility(): Promise<void> {
-    const isVisible = await this.paragraphPage.isCodeEditorVisible();
-    expect(isVisible).toBe(true);
-    await expect(this.paragraphPage.codeEditor).toBeVisible();
+    await expect(this.paragraphPage.codeEditor).toBeVisible({ timeout: 10000 });
   }
 
   async verifyResultDisplaySystem(): Promise<void> {

zeppelin-web-angular/e2e/tests/theme/dark-mode.spec.ts

@@ -81,7 +81,7 @@ test.describe('Dark Mode Theme Switching', () => {
 
     await test.step('GIVEN: No localStorage, System preference is Light', async () => {
       await page.emulateMedia({ colorScheme: 'light' });
-      await page.goto('/#/');
+      await page.goto('/');
       await waitForZeppelinReady(page);
       // When no explicit theme is set, it defaults to 'system' mode
       // Even in system mode with light preference, the icon should be robot
@@ -92,23 +92,23 @@ test.describe('Dark Mode Theme Switching', () => {
 
     await test.step('GIVEN: No localStorage, System preference is Dark (initial system state)', async () => {
       await themePage.setThemeInLocalStorage('system');
-      await page.goto('/#/');
+      await page.goto('/');
       await waitForZeppelinReady(page);
       await themePage.assertSystemTheme(); // Robot icon for system theme
     });
 
     await test.step("GIVEN: localStorage is 'dark', System preference is Light", async () => {
       await themePage.setThemeInLocalStorage('dark');
       await page.emulateMedia({ colorScheme: 'light' });
-      await page.goto('/#/');
+      await page.goto('/');
       await waitForZeppelinReady(page);
       await themePage.assertDarkTheme(); // localStorage should override system
     });
 
     await test.step("GIVEN: localStorage is 'system', THEN: Emulate system preference change to Light", async () => {
       await themePage.setThemeInLocalStorage('system');
       await page.emulateMedia({ colorScheme: 'light' });
-      await page.goto('/#/');
+      await page.goto('/');
       await waitForZeppelinReady(page);
       await expect(themePage.rootElement).toHaveClass(/light/);
       await expect(themePage.rootElement).toHaveAttribute('data-theme', 'light');
@@ -118,7 +118,7 @@ test.describe('Dark Mode Theme Switching', () => {
     await test.step("GIVEN: localStorage is 'system', THEN: Emulate system preference change to Dark", async () => {
       await themePage.setThemeInLocalStorage('system');
       await page.emulateMedia({ colorScheme: 'dark' });
-      await page.goto('/#/');
+      await page.goto('/');
       await waitForZeppelinReady(page);
       await expect(themePage.rootElement).toHaveClass(/dark/);
       await expect(themePage.rootElement).toHaveAttribute('data-theme', 'dark');

zeppelin-web-angular/e2e/utils.ts

@@ -374,7 +374,7 @@ const navigateViaHomePageFallback = async (page: Page, baseNotebookName: string)
 };
 
 const extractFirstParagraphId = async (page: Page): Promise<string> => {
-  await page.locator('zeppelin-notebook-paragraph').first().waitFor({ state: 'visible', timeout: 10000 });
+  await page.locator('zeppelin-notebook-paragraph').first().waitFor({ state: 'visible', timeout: 20000 });
 
   const paragraphContainer = page.locator('zeppelin-notebook-paragraph').first();
   const dropdownTrigger = paragraphContainer.locator('a[nz-dropdown]');