bigtop-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r..@apache.org
Subject svn commit: r1166271 - in /incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke: PackageTestCommon.groovy TestPackagesSingleNode.groovy
Date Wed, 07 Sep 2011 17:15:03 GMT
Author: rvs
Date: Wed Sep  7 17:15:02 2011
New Revision: 1166271

URL: http://svn.apache.org/viewvc?rev=1166271&view=rev
Log:
BIGTOP-83. Advanced checks in checkRemoval need to be moved out

Modified:
    incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/PackageTestCommon.groovy
    incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesSingleNode.groovy

Modified: incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/PackageTestCommon.groovy
URL: http://svn.apache.org/viewvc/incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/PackageTestCommon.groovy?rev=1166271&r1=1166270&r2=1166271&view=diff
==============================================================================
--- incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/PackageTestCommon.groovy
(original)
+++ incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/PackageTestCommon.groovy
Wed Sep  7 17:15:02 2011
@@ -412,17 +412,16 @@ class PackageTestCommon {
     }
   }
 
-  public void checkRemoval() {
-    checkComplimentary32bitInstall();
-
+  public void checkDaemonStart() {
     // We need to start service for one more time to make sure that package
     // removal would succeed even when services are still running
     pkg.getServices().each { name, svc ->
       checkThat("can not start service $name",
                 svc.start(), equalTo(0));
     }
-    sleep(3001); // TODO FIXME: CDH-2816 should address the timing of daemons start.
+  }
 
+  public void checkRemoval() {
     checkThat("package $name failed to be removed",
               pkg.remove(), equalTo(0));
     checkThat("package $name is NOT expected to remain installed after removal",

Modified: incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesSingleNode.groovy
URL: http://svn.apache.org/viewvc/incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesSingleNode.groovy?rev=1166271&r1=1166270&r2=1166271&view=diff
==============================================================================
--- incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesSingleNode.groovy
(original)
+++ incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesSingleNode.groovy
Wed Sep  7 17:15:02 2011
@@ -57,4 +57,13 @@ class TestPackagesSingleNode extends Tes
   void testPackageServices() {
     checkServices(getMap(golden.services));
   }
-}
+
+  @RunStage(level=1)
+  @Test
+  void testPackageRemove() {
+    checkComplimentary32bitInstall();
+    checkDaemonStart();
+    sleep(3001); // TODO FIXME: CDH-2816 should address the timing of daemons start.
+    checkRemoval();
+  }
+}
\ No newline at end of file



Mime
View raw message