Skip to content

Commit 7d9a34c

Browse files
dididyjongyoul
authored andcommitted
fix broken tests
1 parent 882d4aa commit 7d9a34c

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

zeppelin-web-angular/e2e/models/folder-rename-page.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,10 @@ export class FolderRenamePage extends BasePage {
8181
}
8282

8383
async clearNewName(): Promise<void> {
84-
await this.renameInput.clear();
85-
await expect(this.renameInput).toHaveValue('');
84+
await this.renameInput.click();
85+
await this.renameInput.press('Control+a');
86+
await this.renameInput.press('Backspace');
87+
await expect(this.renameInput).toHaveValue('', { timeout: 5000 });
8688
}
8789

8890
async clickConfirm(): Promise<void> {

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1030,8 +1030,7 @@ test.describe.serial('Comprehensive Keyboard Shortcuts (ShortcutsMap)', () => {
10301030
// Rapid Shift+Enter operations
10311031
for (let i = 0; i < 3; i++) {
10321032
await keyboardPage.pressRunParagraph();
1033-
// JUSTIFIED: single-paragraph test notebook; first() is deterministic
1034-
await expect(keyboardPage.paragraphResult.first()).toBeVisible({ timeout: 15000 });
1033+
await keyboardPage.waitForParagraphExecution(0);
10351034
await keyboardPage.page.waitForTimeout(500); // JUSTIFIED: brief gap between rapid sequential runs to prevent WebSocket message overlap
10361035
}
10371036

0 commit comments

Comments
 (0)