fix broken tests

Author: dididy

zeppelin-web-angular/e2e/models/base-page.ts

@@ -23,6 +23,5 @@ export class BasePage {
 
   async waitForPageLoad(): Promise<void> {
     await this.page.waitForLoadState('domcontentloaded', { timeout: 15000 });
-    await this.page.waitForLoadState('networkidle', { timeout: 30000 });
   }
 }

zeppelin-web-angular/e2e/tests/notebook/keyboard/notebook-keyboard-shortcuts.spec.ts

@@ -208,7 +208,8 @@ test.describe.serial('Comprehensive Keyboard Shortcuts (ShortcutsMap)', () => {
       await keyboardPage.setCodeEditorContent('line1\nline2\nline3');
 
       // Position cursor at beginning of first line
-      await keyboardPage.pressKey('Control+Home');
+      await keyboardPage.pressKey('Control+A');
+      await keyboardPage.pressKey('ArrowLeft');
 
       // When: User presses Control+N (should move cursor down one line)
       await keyboardPage.pressMoveCursorDown();

zeppelin-web-angular/e2e/utils.ts

@@ -224,14 +224,6 @@ export async function waitForZeppelinReady(page: Page): Promise<void> {
       return;
     }
 
-    // Additional check: ensure we're not stuck on login page
-    await page
-      .waitForFunction(() => !window.location.href.includes('#/login'), { timeout: 10000 })
-      .catch(() => {
-        // If still on login page, this is expected - login will handle redirect
-        console.log('Still on login page - this is normal if authentication is required');
-      });
-
     // Wait for Angular and Zeppelin to be ready with more robust checks
     await page.waitForFunction(
       () => {