[zeppelin] branch master updated: [ZEPPELIN-3984] Flaky test NotebookRestApiTest.testRunWithServerRestart

[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 df5933d  [ZEPPELIN-3984] Flaky test NotebookRestApiTest.testRunWithServerRestart
df5933d is described below

commit df5933d7f99f6d647f23b8e1ed9294a8b5cb563e
Author: Lee moon soo <mo...@apache.org>
AuthorDate: Thu Sep 19 15:32:43 2019 -0700

    [ZEPPELIN-3984] Flaky test NotebookRestApiTest.testRunWithServerRestart
    
    ### What is this PR for?
    This PR trying to fix flaky test described in https://issues.apache.org/jira/browse/ZEPPELIN-3984
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-3984
    
    ### 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 #3456 from Leemoonsoo/ZEPPELIN-3984 and squashes the following commits:
    
    592ad4abb [Lee moon soo] try not make NPE
---
 .../src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java b/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java
index 079fe5f..bc3bc04 100644
--- a/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java
+++ b/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java
@@ -422,7 +422,7 @@ public class NotebookRestApiTest extends AbstractTestRestApi {
       assertEquals(resp.get("status"), "OK");
       post2.releaseConnection();
 
-      assertEquals(p1.getReturn().toString(), Job.Status.FINISHED, p1.getStatus());
+      assertEquals(Job.Status.FINISHED, p1.getStatus());
       assertEquals(p2.getReturn().toString(), Job.Status.FINISHED, p2.getStatus());
       assertNotNull(p2.getReturn());
       assertEquals("abc\n", p2.getReturn().message().get(0).getData());