You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by mi...@apache.org on 2016/11/22 14:04:32 UTC

zeppelin git commit: [ZEPPELIN-1623] Fix flaky test - testEditOnDoubleClick

Repository: zeppelin
Updated Branches:
  refs/heads/master 0bcbfb922 -> 4dd36bfa3


[ZEPPELIN-1623] Fix flaky test - testEditOnDoubleClick

### What is this PR for?
#1657 is merged, but CI failure still exists on master. Check [this link](https://issues.apache.org/jira/browse/ZEPPELIN-1623) to see the test failure.
I will trigger this PR 5 times and attach CI link on comments if all of them pass to be sure that the issue is gone.

### What type of PR is it?
Hot Fix

### What is the Jira issue?
[ZEPPELIN-1623](https://issues.apache.org/jira/browse/ZEPPELIN-1623)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Mina Lee <mi...@apache.org>

Closes #1667 from minahlee/ZEPPELIN-1623 and squashes the following commits:

ba750a4 [Mina Lee] Fix testEditOnDoubleClick flaky test


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/4dd36bfa
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/4dd36bfa
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/4dd36bfa

Branch: refs/heads/master
Commit: 4dd36bfa312285a13347fb15beadcb0baa30ed23
Parents: 0bcbfb9
Author: Mina Lee <mi...@apache.org>
Authored: Mon Nov 21 22:48:04 2016 +0100
Committer: Mina Lee <mi...@apache.org>
Committed: Tue Nov 22 15:04:24 2016 +0100

----------------------------------------------------------------------
 .../org/apache/zeppelin/integration/ParagraphActionsIT.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/4dd36bfa/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
----------------------------------------------------------------------
diff --git a/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java b/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
index 6cf46ec..79e1d79 100644
--- a/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
+++ b/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
@@ -439,9 +439,10 @@ public class ParagraphActionsIT extends AbstractZeppelinIT {
       Actions action = new Actions(driver);
 
       waitForParagraph(1, "READY");
-      pollingWait(By.xpath(getParagraphXPath(1) + "//textarea"), MAX_PARAGRAPH_TIMEOUT_SEC);
-      driver.findElement(By.xpath(getParagraphXPath(1) + "//textarea")).sendKeys(Keys.SHIFT + "5");
-      driver.findElement(By.xpath(getParagraphXPath(1) + "//textarea")).sendKeys("md" + Keys.ENTER);
+
+      setTextOfParagraph(1, "%md");
+      driver.findElement(By.xpath(getParagraphXPath(1) + "//textarea")).sendKeys(Keys.ARROW_RIGHT);
+      driver.findElement(By.xpath(getParagraphXPath(1) + "//textarea")).sendKeys(Keys.ENTER);
       driver.findElement(By.xpath(getParagraphXPath(1) + "//textarea")).sendKeys(Keys.SHIFT + "3");
       driver.findElement(By.xpath(getParagraphXPath(1) + "//textarea")).sendKeys(" abc");