GitHub user eranwitkon opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/254
ZEPPELIN-241- Add tests for REST API
Can someone help in this one?
```
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 9.18 sec
<<< FAILURE! - in org.apache.zeppelin.rest.ZeppelinRestApiTest
org.apache.zeppelin.rest.ZeppelinRestApiTest Time elapsed: 9.176 sec <<<
ERROR!
java.lang.NullPointerException: null
at
org.apache.zeppelin.rest.AbstractTestRestApi.startUp(AbstractTestRestApi.java:134)
at
org.apache.zeppelin.rest.ZeppelinRestApiTest.init(ZeppelinRestApiTest.java:64)
10:08:12,486 INFO org.apache.zeppelin.server.ZeppelinServer:116 - Shutting
down Zeppelin Server ...
10:08:12,487 INFO org.apache.zeppelin.server.ZeppelinServer:122 - Bye
Results :
Tests in error:
ZeppelinRestApiTest.init:64->AbstractTestRestApi.startUp:134 »
NullPointer
```
Not sure what is the problem but I will be happy to understand if these
test work locally for someone else.
I don't have CI environment so line AbstractTestRestApi.startUp:134 should
not be reached at all...
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/eranwitkon/incubator-zeppelin 241
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/254.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #254
----
commit f8638715b1e5f93771b11d7f149800196ca93f77
Author: eranwitkon <[email protected]>
Date: 2015-08-19T07:51:44Z
Add argument check for option and properties to protect from NPE
(cherry picked from commit 39ac2c1)
commit da0717f398828084dc9a99f9b398ec6d2217713a
Author: eranwitkon <[email protected]>
Date: 2015-08-21T20:20:36Z
Fix getRegisteredInterpreterList,
avoid adding null to the list when class name from config is not registered
(only interpreters available on the disk (has interpreter folder) are
registered.
findRegisteredInterpreterByClassName return null when class name not found.
(cherry picked from commit 6724e54)
commit 1aa806f9b3b2ac2e99db68797652e63a77238838
Author: eranwitkon <[email protected]>
Date: 2015-08-19T07:51:44Z
Fix CI problem
commit 6075a6670c6cc31174423a87814bf06ac2649f32
Author: eranwitkon <[email protected]>
Date: 2015-08-21T21:52:40Z
fix CI problem
commit b463100e85efc769a509467b55c46c1559d98e17
Author: eranwitkon <[email protected]>
Date: 2015-08-22T07:18:25Z
Remove unit test
commit 99e557301849f6a67dca673708a7fb4fd0239fce
Author: eranwitkon <[email protected]>
Date: 2015-08-19T07:51:44Z
Add test methods for REST API
commit 28f55b3d14a5cfcfce4542fcac9eec673cd0d0ce
Author: eranwitkon <[email protected]>
Date: 2015-08-22T07:45:53Z
Add Origin header and fix some unit test
commit 1365a12b5fd8d0c80fc2e6ba533942120808b567
Author: eranwitkon <[email protected]>
Date: 2015-08-25T06:50:33Z
Merge remote-tracking branch 'upstream/master' into 241
Conflicts:
zeppelin-server/src/test/java/org/apache/zeppelin/rest/AbstractTestRestApi.java
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
