From 0d93857601374961a0d8e02c6843cb2ae8d91cf3 Mon Sep 17 00:00:00 2001
From: Adrian Cole <adrian@jclouds.org>
Date: Fri, 13 Apr 2012 13:34:21 -0600
Subject: [PATCH] adjusted test glitches

---
 .../compute/StubComputeServiceIntegrationTest.java     |  8 +-------
 .../compute/internal/BaseComputeServiceLiveTest.java   | 10 ++++++----
 .../org/jclouds/rest/internal/BaseContextLiveTest.java |  2 ++
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java b/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
index 8b42c1e2eb..340817a75f 100644
--- a/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
+++ b/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
@@ -32,7 +32,6 @@ import java.util.Set;
 import java.util.concurrent.ConcurrentMap;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
-import java.util.concurrent.TimeoutException;
 
 import org.easymock.IArgumentMatcher;
 import org.jclouds.compute.domain.ExecResponse;
@@ -370,7 +369,7 @@ public class StubComputeServiceIntegrationTest extends
             client.connect();
 
             expect(client.exec("echo hello")).andReturn(new ExecResponse("hello", "", 0));
-            expect(client.exec("java -version")).andReturn(new ExecResponse("", "1.7", 0));
+            expect(client.exec("java -version")).andReturn(new ExecResponse("", "OpenJDK", 0));
 
             client.disconnect();
          }
@@ -530,9 +529,4 @@ public class StubComputeServiceIntegrationTest extends
       super.testDestroyNodes();
    }
 
-   @Override
-   protected void cleanup() throws InterruptedException, ExecutionException, TimeoutException {
-      super.cleanup();
-   }
-
 }
\ No newline at end of file
diff --git a/compute/src/test/java/org/jclouds/compute/internal/BaseComputeServiceLiveTest.java b/compute/src/test/java/org/jclouds/compute/internal/BaseComputeServiceLiveTest.java
index 6aa7841043..bbe9921f50 100644
--- a/compute/src/test/java/org/jclouds/compute/internal/BaseComputeServiceLiveTest.java
+++ b/compute/src/test/java/org/jclouds/compute/internal/BaseComputeServiceLiveTest.java
@@ -98,7 +98,7 @@ import org.jclouds.scriptbuilder.statements.login.AdminAccess;
 import org.jclouds.ssh.SshClient;
 import org.jclouds.ssh.SshException;
 import org.jclouds.util.Strings2;
-import org.testng.annotations.AfterTest;
+import org.testng.annotations.AfterClass;
 import org.testng.annotations.BeforeGroups;
 import org.testng.annotations.Test;
 
@@ -878,7 +878,7 @@ public abstract class BaseComputeServiceLiveTest<S, A, C extends ComputeServiceC
          ExecResponse hello = ssh.exec("echo hello");
          assertEquals(hello.getOutput().trim(), "hello");
          ExecResponse exec = ssh.exec("java -version");
-         assert exec.getError().indexOf("OpenJDK") != -1 || exec.getOutput().indexOf("OpenJDK") != -1 : exec
+         assert exec.getError().indexOf("OpenJDK") != -1 || exec.getOutput().indexOf("1.7") != -1 : exec
                + "\n"
                + ssh.exec("cat /tmp/" + taskName + "/" + taskName + ".sh /tmp/" + taskName + "/stdout.log /tmp/"
                      + taskName + "/stderr.log");
@@ -888,11 +888,13 @@ public abstract class BaseComputeServiceLiveTest<S, A, C extends ComputeServiceC
       }
    }
 
-   @AfterTest
-   protected void cleanup() throws InterruptedException, ExecutionException, TimeoutException {
+   @AfterClass(groups = { "integration", "live" })
+   @Override
+   protected void tearDownContext() {
       if (nodes != null) {
          testDestroyNodes();
       }
+      super.tearDownContext();
    }
 
    @Override
diff --git a/core/src/test/java/org/jclouds/rest/internal/BaseContextLiveTest.java b/core/src/test/java/org/jclouds/rest/internal/BaseContextLiveTest.java
index 538eee99e4..edb136f225 100644
--- a/core/src/test/java/org/jclouds/rest/internal/BaseContextLiveTest.java
+++ b/core/src/test/java/org/jclouds/rest/internal/BaseContextLiveTest.java
@@ -96,6 +96,7 @@ public abstract class BaseContextLiveTest<C extends Closeable> {
    /**
     * @see org.jclouds.providers.Providers#withId
     */
+   @SuppressWarnings("unchecked")
    protected ProviderMetadata<?, ?, C, ?> createProviderMetadata() {
       try {
          return (ProviderMetadata<?, ?, C, ?>) Providers.withId(provider);
@@ -107,6 +108,7 @@ public abstract class BaseContextLiveTest<C extends Closeable> {
    /**
     * @see org.jclouds.apis.Apis#withId
     */
+   @SuppressWarnings("unchecked")
    protected ApiMetadata<?, ?, C, ?> createApiMetadata() {
       try {
          return (ApiMetadata<?, ?, C, ?>) Apis.withId(provider);