You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by mo...@apache.org on 2016/01/03 16:53:59 UTC

incubator-zeppelin git commit: Fix restapi testcase

Repository: incubator-zeppelin
Updated Branches:
  refs/heads/master f0383c699 -> fb75cb3d4


Fix restapi testcase

### What is this PR for?
To fix REST API testcase.

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

### Is there a relevant Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-536

### How should this be tested?
Run testcase.

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

Author: astroshim <hs...@nflabs.com>

Closes #572 from astroshim/fix-restapi-testcase and squashes the following commits:

27787a8 [astroshim] to pass build.
62e673b [astroshim] to rebuild
a702136 [astroshim] to fix build error.
a90b5c5 [astroshim] remove unused package
03f722a [astroshim] fix restapi testcase.


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

Branch: refs/heads/master
Commit: fb75cb3d4f3a49601a4fbd56e76e8fd1e0e4d2f5
Parents: f0383c6
Author: astroshim <hs...@nflabs.com>
Authored: Tue Dec 29 21:14:53 2015 +0900
Committer: Lee moon soo <mo...@apache.org>
Committed: Sun Jan 3 07:55:56 2016 -0800

----------------------------------------------------------------------
 .../java/org/apache/zeppelin/rest/ZeppelinRestApiTest.java    | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-zeppelin/blob/fb75cb3d/zeppelin-server/src/test/java/org/apache/zeppelin/rest/ZeppelinRestApiTest.java
----------------------------------------------------------------------
diff --git a/zeppelin-server/src/test/java/org/apache/zeppelin/rest/ZeppelinRestApiTest.java b/zeppelin-server/src/test/java/org/apache/zeppelin/rest/ZeppelinRestApiTest.java
index 0fde5bf..c7ac631 100644
--- a/zeppelin-server/src/test/java/org/apache/zeppelin/rest/ZeppelinRestApiTest.java
+++ b/zeppelin-server/src/test/java/org/apache/zeppelin/rest/ZeppelinRestApiTest.java
@@ -165,11 +165,14 @@ public class ZeppelinRestApiTest extends AbstractTestRestApi {
     }
     assertEquals("<p>markdown</p>\n", p.getResult().message());
 
+    
     // restart interpreter
     for (InterpreterSetting setting : note.getNoteReplLoader().getInterpreterSettings()) {
       if (setting.getName().equals("md")) {
-        // restart
-        ZeppelinServer.notebook.getInterpreterFactory().restart(setting.id());
+        // Call Restart Interpreter REST API
+        PutMethod put = httpPut("/interpreter/setting/restart/" + setting.id(), "");
+        assertThat("test interpreter restart:", put, isAllowed());
+        put.releaseConnection();
         break;
       }
     }