(zeppelin) branch master updated: [ZEPPELIN-6088] Fix some flaky tests in zeppelin-integration (#4826)

[chengpan]

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

chengpan 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 7ae89809da [ZEPPELIN-6088] Fix some flaky tests in 
zeppelin-integration (#4826)
7ae89809da is described below

commit 7ae89809da9314b2c12b716ff6c1d140c79f7041
Author: ChanHo Lee <chanho0...@gmail.com>
AuthorDate: Sat Sep 14 11:51:26 2024 +0900

    [ZEPPELIN-6088] Fix some flaky tests in zeppelin-integration (#4826)
    
    * Fix flaky SparkParagraphIT.testCancelPyspark
    
    * Fix flaky PersonalizeActionsIT.testGraphAction
---
 .../test/java/org/apache/zeppelin/integration/PersonalizeActionsIT.java | 2 ++
 .../src/test/java/org/apache/zeppelin/integration/SparkParagraphIT.java | 1 +
 2 files changed, 3 insertions(+)

diff --git 
a/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/PersonalizeActionsIT.java
 
b/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/PersonalizeActionsIT.java
index 6b66197963..bc4e0277c0 100644
--- 
a/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/PersonalizeActionsIT.java
+++ 
b/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/PersonalizeActionsIT.java
@@ -19,6 +19,7 @@ package org.apache.zeppelin.integration;
 import org.apache.zeppelin.AbstractZeppelinIT;
 import org.apache.zeppelin.MiniZeppelinServer;
 import org.apache.zeppelin.WebDriverManager;
+import org.apache.zeppelin.ZeppelinITUtils;
 import org.apache.zeppelin.test.DownloadUtils;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.AfterEach;
@@ -239,6 +240,7 @@ public class PersonalizeActionsIT extends 
AbstractZeppelinIT {
 
       pollingWait(By.xpath(getParagraphXPath(1) +
           "//button[contains(@uib-tooltip, 'Table')]"), 
MAX_BROWSER_TIMEOUT_SEC).click();
+      ZeppelinITUtils.sleep(1000, false);
       assertEquals("fa fa-table", 
manager.getWebDriver().findElement(By.xpath(getParagraphXPath(1)
           + "//button[contains(@class," +
           "'btn btn-default btn-sm ng-binding ng-scope 
active')]//i")).getAttribute("class"));
diff --git 
a/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/SparkParagraphIT.java
 
b/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/SparkParagraphIT.java
index a89a4c4db0..a81ab893dd 100644
--- 
a/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/SparkParagraphIT.java
+++ 
b/zeppelin-integration/src/test/java/org/apache/zeppelin/integration/SparkParagraphIT.java
@@ -188,6 +188,7 @@ class SparkParagraphIT extends AbstractZeppelinIT {
       ZeppelinITUtils.sleep(2000, false);
       cancelParagraph(1);
       waitForParagraph(1, "ABORT");
+      ZeppelinITUtils.sleep(1000, false);
 
       assertEquals("ABORT", getParagraphStatus(1),
           "First paragraph status is " + getParagraphStatus(1));