You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by ah...@apache.org on 2016/12/27 08:52:57 UTC

zeppelin git commit: [HOTFIX] [ZEPPELIN-1863] Travis Selenium CI fail

Repository: zeppelin
Updated Branches:
  refs/heads/master 71f5cc000 -> 93c2dae9e


[HOTFIX] [ZEPPELIN-1863] Travis Selenium CI fail

### What is this PR for?
Currently, the Selenium CI test of Travis regarding every PR is failing.
I think because it is after merging #1777 PR.

Here is error log.
```
Caused by: org.openqa.selenium.NoSuchElementException: Unable to locate element: {"method":"xpath","selector":"(//div[ng-controller=\"ParagraphCtrl\"])[2]//div[class=\"control ng-scope\"]//ul[class=\"dropdown-menu\"]/li[1]"}

Results :
Tests in error:
  ZeppelinIT.testAngularRunParagraph:291 � NoSuchElement Unable to locate elemen...
```

### What type of PR is it?
[Bug Fix | Hot Fix]

### Todos
* [x] fix code regarding this problem

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1863

### How should this be tested?
- None

### Screenshots (if appropriate)
- None

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

Author: soralee <so...@nflabs.com>

Closes #1806 from soralee/ZEPPELIN-1863 and squashes the following commits:

b7a8c07 [soralee] travis CI fail for ZeppelinIT.testAngularRunParagraph:291


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

Branch: refs/heads/master
Commit: 93c2dae9e2242168f10aa0daa80d91c06962fa6e
Parents: 71f5cc0
Author: soralee <so...@nflabs.com>
Authored: Tue Dec 27 16:42:20 2016 +0900
Committer: ahyoungryu <ah...@apache.org>
Committed: Tue Dec 27 17:52:50 2016 +0900

----------------------------------------------------------------------
 .../src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/93c2dae9/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
----------------------------------------------------------------------
diff --git a/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java b/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
index 9f3611c..07db2da 100644
--- a/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
+++ b/zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
@@ -289,7 +289,7 @@ public class ZeppelinIT extends AbstractZeppelinIT {
 
       // Get 2nd paragraph id
       final String secondParagraphId = driver.findElement(By.xpath(getParagraphXPath(2)
-              + "//div[@class=\"control ng-scope\"]//ul[@class=\"dropdown-menu\"]/li[1]"))
+              + "//div[@class=\"control ng-scope\"]//ul[@class=\"dropdown-menu dropdown-menu-right\"]/li[1]"))
               .getAttribute("textContent");
 
       assertTrue("Cannot find paragraph id for the 2nd paragraph", isNotBlank(secondParagraphId));