(zeppelin) branch master updated: [ZEPPELIN-6713] Stabilize editor blur E2E assertion

[yo...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

voidmatcha pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/master by this push:
     new 156cdd1e1b [ZEPPELIN-6713] Stabilize editor blur E2E assertion
156cdd1e1b is described below

commit 156cdd1e1b29eb58aae5020d31d2042c6ea3c491
Author: YONGJAE LEE (이용재) <de...@gmail.com>
AuthorDate: Wed Oct 7 21:32:00 2026 +0900

    [ZEPPELIN-6713] Stabilize editor blur E2E assertion
    
    ### What is this PR for?
    
    NB-PARITY-003 can fail intermittently when Monaco's suggestion widget is open because the first Escape dismisses the widget instead of removing editor focus.
    
    This PR makes the assertion deterministic by dismissing the suggestion widget first, waiting for it to close, and then pressing Escape again to blur the editor.
    
    ### What type of PR is it?
    
    Bug Fix
    
    ### What is the Jira issue?
    
    ZEPPELIN-6713
    
    ### How should this be tested?
    
    ```bash
    CI=true PLAYWRIGHT_BASE_URL=http://127.0.0.1:8081 npm exec -- playwright test \
      e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts \
      --grep <at>NB-PARITY-003 \
      --project=chromium \
      --project=firefox \
      --project=webkit \
      --repeat-each=100 \
      --retries=0 \
      --workers=3 \
      --no-deps
    ```
    
    The test passed 100 times on each browser without retries:
    
    * Chromium: 100/100
    * Firefox: 100/100
    * WebKit: 100/100
    * Total: 300/300
    
    ### Questions
    
    * Does the license file need updating? No
    * Is there breaking changes for older versions? No
    * Does this need documentation? No
    
    
    Closes #5549 from voidmatcha/ZEPPELIN-6713-deterministic-blur.
    
    Signed-off-by: YONGJAE LEE <de...@gmail.com>
---
 .../e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts      | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/zeppelin-web-angular/e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts b/zeppelin-web-angular/e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts
index 21bd0c6c8c..35a93f3d31 100644
--- a/zeppelin-web-angular/e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts
+++ b/zeppelin-web-angular/e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts
@@ -75,7 +75,9 @@ test.describe('Notebook Paragraph Functionality', () => {
         await expect(paragraphPage.codeEditorHost).toHaveClass(/\bfocused\b/);
         await expect(paragraphPage.codeEditorHost).toHaveClass(/\bdirty\b/);
 
-        await page.keyboard.press('Escape');
+        await keyboard.pressEscape();
+        await expect(keyboard.autocompletePopup).toBeHidden();
+        await keyboard.pressEscape();
         await expect(paragraphPage.codeEditorHost).not.toHaveClass(/\bfocused\b/);
       });
     }