Author: rvs
Date: Sun Jun 24 20:59:03 2012
New Revision: 1353330
URL: http://svn.apache.org/viewvc?rev=1353330&view=rev
Log:
BIGTOP-649. PackageTestCommon.checkRemoval() is broken (Andrew Bayer via rvs)
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/TestPackagesBasics.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=1353330&r1=1353329&r2=1353330&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
Sun Jun 24 20:59:03 2012
@@ -480,15 +480,6 @@ class PackageTestCommon {
}
}
- 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",
- pm.isInstalled(pkg), equalTo(false));
-
- checkPackageFilesGotRemoved(getMap(golden.content));
- }
-
static public boolean isUpgrade() {
return System.getProperty("cdh.prev.repo.version", "") != "";
}
Modified:
incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesBasics.groovy
URL:
http://svn.apache.org/viewvc/incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesBasics.groovy?rev=1353330&r1=1353329&r2=1353330&view=diff
==============================================================================
---
incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesBasics.groovy
(original)
+++
incubator/bigtop/trunk/bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesBasics.groovy
Sun Jun 24 20:59:03 2012
@@ -257,4 +257,14 @@ class TestPackagesBasics extends Package
return res;
}
+
+ 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",
+ pm.isInstalled(pkg), equalTo(false));
+
+ checkPackageFilesGotRemoved(getMap(golden.content));
+ }
+
}
