Mailing-List: contact derby-commits-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Derby Development" <derby-dev@db.apache.org>
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1409113 -
 /db/derby/code/trunk/java/testing/org/apache/derbyTesting/junit/NetworkServerTestSetup.java
Date: Wed, 14 Nov 2012 08:22:43 -0000
To: derby-commits@db.apache.org
From: kahatlen@apache.org
Message-Id: <20121114082243.4A4D92388AB9@eris.apache.org>

Author: kahatlen
Date: Wed Nov 14 08:22:42 2012
New Revision: 1409113

URL: http://svn.apache.org/viewvc?rev=1409113&view=rev
Log:
DERBY-5992: Use BaseTestCase.execJavaCmd() to start process from NetworkServerTestSetup

Modified:
    db/derby/code/trunk/java/testing/org/apache/derbyTesting/junit/NetworkServerTestSetup.java

Modified: db/derby/code/trunk/java/testing/org/apache/derbyTesting/junit/NetworkServerTestSetup.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/testing/org/apache/derbyTesting/junit/NetworkServerTestSetup.java?rev=1409113&r1=1409112&r2=1409113&view=diff
==============================================================================
--- db/derby/code/trunk/java/testing/org/apache/derbyTesting/junit/NetworkServerTestSetup.java (original)
+++ db/derby/code/trunk/java/testing/org/apache/derbyTesting/junit/NetworkServerTestSetup.java Wed Nov 14 08:22:42 2012
@@ -33,6 +33,7 @@ import java.security.PrivilegedActionExc
 import java.security.PrivilegedExceptionAction;
 import java.sql.SQLException;
 import java.util.ArrayList;
+import java.util.Arrays;
 import junit.framework.Test;
 import org.apache.derby.drda.NetworkServerControl;
 
@@ -301,8 +302,6 @@ final public class NetworkServerTestSetu
     
     private void startWithCommand() throws Exception
     {
-        final TestConfiguration config = TestConfiguration.getCurrent();
-        
         // start the server through the command line
         // arguments using a new thread to do so.
         new Thread(
@@ -320,14 +319,8 @@ final public class NetworkServerTestSetu
     private SpawnedProcess startSeparateProcess() throws Exception
     {
         ArrayList       al = new ArrayList();
-        String              classpath = BaseTestCase.getSystemProperty( "java.class.path" );
         boolean         skipHostName = false;
 
-        al.add( BaseTestCase.getJavaExecutableName() );
-        al.add( "-Demma.verbosity.level=silent" );
-        al.add( "-classpath" );
-        al.add( classpath );
-        
         // Loading from classes need to work-around the limitation of the
         // default policy file doesn't work with classes.  Similarly, if we are
         // running with Emma we don't run with the security manager, as the
@@ -377,48 +370,13 @@ final public class NetworkServerTestSetu
             if ( HOST_OPTION.equals( startupArgs[ i ] ) ) { skipHostName = true; }
         }
 
-        String[]    defaultArgs = getDefaultStartupArgs( skipHostName );
-
-        count = defaultArgs.length;
-        for ( int i = 0; i < count; i++ )
-        {
-            al.add( defaultArgs[ i ] );
-        }
-
-        count = startupArgs.length;
-        for ( int i = 0; i < count; i++ )
-        {
-            al.add( startupArgs[ i ] );
-        }
+        al.addAll(Arrays.asList(getDefaultStartupArgs(skipHostName)));
+        al.addAll(Arrays.asList(startupArgs));
 
         final   String[]  command = new String[ al.size() ];
         al.toArray(command);
 
-        String startcommand ="";
-        for (int i = 0 ; i < command.length ; i++) {
-        	startcommand += command[i] + " ";
-        }
-
-        BaseTestCase.println("XXX server startup command = " +
-	startcommand + "\n");
-
-        Process serverProcess;
-        
-        try {
-            serverProcess = (Process)
-                AccessController.doPrivileged
-                (
-                 new PrivilegedExceptionAction()
-                 {
-                     public Object run() throws IOException
-                     {
-                         return Runtime.getRuntime().exec(command);
-                     }
-                 }
-                );
-        } catch (PrivilegedActionException e) {
-            throw e.getException();
-        }
+        Process serverProcess = BaseTestCase.execJavaCmd(command);
 
         return new SpawnedProcess(serverProcess, "SpawnedNetworkServer");
     }