[MINOR] Verify appId is not null in LivyInterpreterIT

zjffdu · zjffdu · commit 6fc3057d5296 · 2018-06-07T15:23:39.000+08:00
### What is this PR for? Someone complain that they could not get appId, this PR just try to verify appId returned by livy rest api is not null. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * No jira created ### How should this be tested? * CI pass ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Jeff Zhang <zjffdu@apache.org> Closes #2999 from zjffdu/minor_livy and squashes the following commits: eae0cf5 [Jeff Zhang] [MINOR] Verify appId is null in LivyInterpreterIT
diff --git a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
@@ -495,7 +495,8 @@ public void testSparkInterpreterWithDisplayAppInfo_StringWithoutTruncation()
       InterpreterResult result = sparkInterpreter.interpret("sc.version", context);
       assertEquals(InterpreterResult.Code.SUCCESS, result.code());
       assertEquals(2, result.message().size());
-      assertTrue(result.message().get(1).getData().contains("Spark Application Id"));
+      // check yarn appId and ensure it is not null
+      assertTrue(result.message().get(1).getData().contains("Spark Application Id: application_"));
 
       // html output
       String htmlCode = "println(\"%html <h1> hello </h1>\")";
