You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ga...@apache.org on 2008/06/11 21:56:48 UTC

svn commit: r666811 - /geronimo/server/trunk/testsupport/testsupport-commands/src/main/java/org/apache/geronimo/testsupport/commands/CommandTestSupport.java

Author: gawor
Date: Wed Jun 11 12:56:47 2008
New Revision: 666811

URL: http://svn.apache.org/viewvc?rev=666811&view=rev
Log:
close jmx connection

Modified:
    geronimo/server/trunk/testsupport/testsupport-commands/src/main/java/org/apache/geronimo/testsupport/commands/CommandTestSupport.java

Modified: geronimo/server/trunk/testsupport/testsupport-commands/src/main/java/org/apache/geronimo/testsupport/commands/CommandTestSupport.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/testsupport/testsupport-commands/src/main/java/org/apache/geronimo/testsupport/commands/CommandTestSupport.java?rev=666811&r1=666810&r2=666811&view=diff
==============================================================================
--- geronimo/server/trunk/testsupport/testsupport-commands/src/main/java/org/apache/geronimo/testsupport/commands/CommandTestSupport.java (original)
+++ geronimo/server/trunk/testsupport/testsupport-commands/src/main/java/org/apache/geronimo/testsupport/commands/CommandTestSupport.java Wed Jun 11 12:56:47 2008
@@ -51,9 +51,14 @@
         } catch (Exception e) {
             throw new RuntimeException("Unable to setup ServerProxy", e);
         }
+        
         String home = server.getGeronimoHome();
-        if (server.getLastError() != null) {
-            throw new RuntimeException("Failed to get Geronimo home", server.getLastError());
+        Throwable exception = server.getLastError();
+        
+        server.closeConnection();
+        
+        if (exception != null) {
+            throw new RuntimeException("Failed to get Geronimo home", exception);
         } else {
             return home;
         }