Updated Branches:
  refs/heads/master 899e7bf0e -> 2b35a9255

BIGTOP-901. Oozie smoke tests mightn't be able to obtain the jobtracker or 
namenode hostnames, using old config. names

            The original patch missed a couple of points that weren't apparent
            until real cluster test run.


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

Branch: refs/heads/master
Commit: 2b35a925556c97a7c78eaf355d8a7979a253738d
Parents: 899e7bf
Author: Konstantin Boudnik <[email protected]>
Authored: Tue Apr 2 16:16:24 2013 -0700
Committer: Konstantin Boudnik <[email protected]>
Committed: Tue Apr 2 16:16:24 2013 -0700

----------------------------------------------------------------------
 .../bigtop/itest/ooziesmoke/TestOozieSmoke.groovy  |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/bigtop/blob/2b35a925/bigtop-tests/test-artifacts/oozie/src/main/groovy/org/apache/bigtop/itest/ooziesmoke/TestOozieSmoke.groovy
----------------------------------------------------------------------
diff --git 
a/bigtop-tests/test-artifacts/oozie/src/main/groovy/org/apache/bigtop/itest/ooziesmoke/TestOozieSmoke.groovy
 
b/bigtop-tests/test-artifacts/oozie/src/main/groovy/org/apache/bigtop/itest/ooziesmoke/TestOozieSmoke.groovy
index 56e34d4..23a00cb 100644
--- 
a/bigtop-tests/test-artifacts/oozie/src/main/groovy/org/apache/bigtop/itest/ooziesmoke/TestOozieSmoke.groovy
+++ 
b/bigtop-tests/test-artifacts/oozie/src/main/groovy/org/apache/bigtop/itest/ooziesmoke/TestOozieSmoke.groovy
@@ -19,7 +19,7 @@ package org.apache.bigtop.itest.ooziesmoke
 
 import org.junit.Test
 
-import static junit.framework.Assert.assertNull
+import static junit.framework.Assert.assertNotNull
 import static org.junit.Assert.assertTrue
 import org.apache.bigtop.itest.shell.Shell
 import org.junit.AfterClass
@@ -43,12 +43,12 @@ class TestOozieSmoke {
     conf.addResource('yarn-site.xml');
 
     oozie_url = System.getProperty("org.apache.bigtop.itest.oozie_url", 
"http://localhost:11000/oozie";);
-    resourcemanager = ${conf.get("yarn.resourcemanager.address")}
+    resourcemanager = conf.get("yarn.resourcemanager.address")
     resourcemanager = 
System.getProperty("org.apache.bigtop.itest.resourcemanager", resourcemanager);
-    namenode = ${conf.get('fs.defaultFS')} ? ${conf.get('fs.defaultFS')} : 
${conf.get('fs.default.name')}
+    namenode = conf.get('fs.defaultFS') ? conf.get('fs.defaultFS') : 
conf.get('fs.default.name')
     namenode = System.getProperty("org.apache.bigtop.itest.namenode", 
namenode);
-    assertNull("resourcemanager hostname isn't set", resourcemanager)
-    assertNull("namenode hostname isn't set", namenode)
+    assertNotNull("resourcemanager hostname isn't set", resourcemanager)
+    assertNotNull("namenode hostname isn't set", namenode)
 
     oozie_tar_home = 
System.getProperty("org.apache.bigtop.itest.oozie_tar_home",
                                         (new 
File("/usr/share/doc/packages/oozie/")).exists() ?

Reply via email to