Repository: bigtop
Updated Branches:
  refs/heads/master e0a288d55 -> 208d153ed


BIGTOP-2243. :itest-common:test is broken without JAVA_HOME being set


Project: http://git-wip-us.apache.org/repos/asf/bigtop/repo
Commit: http://git-wip-us.apache.org/repos/asf/bigtop/commit/894fb8bc
Tree: http://git-wip-us.apache.org/repos/asf/bigtop/tree/894fb8bc
Diff: http://git-wip-us.apache.org/repos/asf/bigtop/diff/894fb8bc

Branch: refs/heads/master
Commit: 894fb8bcf03a4f39b74c372db1b161de8b110a55
Parents: e0a288d
Author: Konstantin Boudnik <[email protected]>
Authored: Thu Jan 14 18:03:40 2016 -0800
Committer: Konstantin Boudnik <[email protected]>
Committed: Thu Jan 14 18:03:40 2016 -0800

----------------------------------------------------------------------
 .../test/groovy/org/apache/bigtop/itest/JarContentTest.groovy    | 2 +-
 build.gradle                                                     | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/bigtop/blob/894fb8bc/bigtop-test-framework/src/test/groovy/org/apache/bigtop/itest/JarContentTest.groovy
----------------------------------------------------------------------
diff --git 
a/bigtop-test-framework/src/test/groovy/org/apache/bigtop/itest/JarContentTest.groovy
 
b/bigtop-test-framework/src/test/groovy/org/apache/bigtop/itest/JarContentTest.groovy
index b583d8a..9543b46 100644
--- 
a/bigtop-test-framework/src/test/groovy/org/apache/bigtop/itest/JarContentTest.groovy
+++ 
b/bigtop-test-framework/src/test/groovy/org/apache/bigtop/itest/JarContentTest.groovy
@@ -45,7 +45,7 @@ class JarContentTest {
 
   @Test
   void testUnpackJarContainer() {
-    def destination = 'target/local.unpack.dir';
+    def destination = System.properties['buildDir'] ?: 
'target/local.unpack.dir';
     JarContent.unpackJarContainer('java.lang.String', destination, 'visitor');
     // expect to find a number of sun/reflect/generics/visitor
     // under destination folder

http://git-wip-us.apache.org/repos/asf/bigtop/blob/894fb8bc/build.gradle
----------------------------------------------------------------------
diff --git a/build.gradle b/build.gradle
index e7cd663..5064cb5 100644
--- a/build.gradle
+++ b/build.gradle
@@ -167,6 +167,10 @@ project(':itest-common') {
 
   artifacts.archives packageITest
 
+  test.doFirst {
+    if (System.getenv('JAVA_HOME') == null)
+      throw new GradleException("JAVA_HOME is required to test 
${project.name}")
+  }
   test {
     exclude '**/Dummy*'
   }

Reply via email to