[zeppelin] branch master updated: [ZEPPELIN-4339] Flaky test org.apache.zeppelin.AbstractZeppelinIT#testAngularDisplay

[mo...@apache.org]

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

moon 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 4fe2e20  [ZEPPELIN-4339] Flaky test org.apache.zeppelin.AbstractZeppelinIT#testAngularDisplay
4fe2e20 is described below

commit 4fe2e20caa6e37e353d51d46aea41320e07be76e
Author: Lee moon soo <mo...@apache.org>
AuthorDate: Thu Sep 19 15:54:55 2019 -0700

    [ZEPPELIN-4339] Flaky test org.apache.zeppelin.AbstractZeppelinIT#testAngularDisplay
    
    ### What is this PR for?
    This PR try to fix flaky test described in https://issues.apache.org/jira/browse/ZEPPELIN-4339
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-4339
    
    ### Questions:
    * Does the licenses files need update? no
    * Is there breaking changes for older versions? no
    * Does this needs documentation? no
    
    Author: Lee moon soo <mo...@apache.org>
    
    Closes #3457 from Leemoonsoo/ZEPPELIN-4339 and squashes the following commits:
    
    41d5c88e8 [Lee moon soo] wait for RUN button to be rendered before click
---
 .../src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java         | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/zeppelin-integration/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java b/zeppelin-integration/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
index b7cf80d..3a8bf58 100644
--- a/zeppelin-integration/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
+++ b/zeppelin-integration/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
@@ -55,7 +55,9 @@ abstract public class AbstractZeppelinIT {
   }
 
   protected void runParagraph(int paragraphNo) {
-    driver.findElement(By.xpath(getParagraphXPath(paragraphNo) + "//span[@class='icon-control-play']")).click();
+    By by = By.xpath(getParagraphXPath(paragraphNo) + "//span[@class='icon-control-play']");
+    pollingWait(by, 5);
+    driver.findElement(by).click();
   }