Remove some but not all System.out.println calls.
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/exec/trunk@1067181 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java b/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
index c648e9e..d487a40 100644
--- a/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
+++ b/src/test/java/org/apache/commons/exec/DefaultExecutorTest.java
@@ -72,8 +72,6 @@
protected void setUp() throws Exception {
- System.out.println(">>> Executing " + getName() + " ...");
-
// delete the marker file
this.foreverOutputFile.getParentFile().mkdirs();
if(this.foreverOutputFile.exists()) {
@@ -487,9 +485,8 @@
int exitValue = executor.execute(cl);
fis.close();
String result = baos.toString().trim();
- System.out.println(result);
- assertTrue(result.indexOf("Finished reading from stdin") > 0);
- assertFalse(exec.isFailure(exitValue));
+ assertTrue(result, result.indexOf("Finished reading from stdin") > 0);
+ assertFalse("exitValue=" + exitValue, exec.isFailure(exitValue));
}
else if(OS.isFamilyWindows()) {
System.err.println("The code samples to do that in windows look like a joke ... :-( .., no way I'm doing that");
@@ -739,10 +736,6 @@
int exitValue = exec.execute(cmdl);
String result = baos.toString().trim();
- System.out.println("=== Expected ===");
- System.out.println(expected);
- System.out.println("=== Result ===");
- System.out.println(result);
assertFalse(exec.isFailure(exitValue));
assertEquals(expected, result);
}
@@ -797,10 +790,6 @@
int exitValue = exec.execute(cmdl);
String result = baos.toString().trim();
- System.out.println("=== Expected ===");
- System.out.println(expected);
- System.out.println("=== Result ===");
- System.out.println(result);
assertFalse(exec.isFailure(exitValue));
if(OS.isFamilyUnix()) {
diff --git a/src/test/java/org/apache/commons/exec/LogOutputStreamTest.java b/src/test/java/org/apache/commons/exec/LogOutputStreamTest.java
index caebb34..b5dfc91 100644
--- a/src/test/java/org/apache/commons/exec/LogOutputStreamTest.java
+++ b/src/test/java/org/apache/commons/exec/LogOutputStreamTest.java
@@ -41,7 +41,6 @@
protected void setUp() throws Exception {
- System.out.println(">>> Executing " + getName() + " ...");
this.systemOut = new SystemLogOutputStream(1);
this.exec.setStreamHandler(new PumpStreamHandler(systemOut, systemOut));
}
diff --git a/src/test/java/org/apache/commons/exec/StandAloneTest.java b/src/test/java/org/apache/commons/exec/StandAloneTest.java
index a540a2c..2677441 100644
--- a/src/test/java/org/apache/commons/exec/StandAloneTest.java
+++ b/src/test/java/org/apache/commons/exec/StandAloneTest.java
@@ -36,7 +36,6 @@
public void testMe() throws Exception {
if(OS.isFamilyUnix()) {
File testScript = TestUtil.resolveScriptForOS("./src/test/scripts/standalone");
- System.out.println("Executing the following test script : " + testScript.getAbsolutePath());
Executor exec = new DefaultExecutor();
exec.setStreamHandler(new PumpStreamHandler());
CommandLine cl = new CommandLine(testScript);
diff --git a/src/test/java/org/apache/commons/exec/environment/EnvironmentUtilTest.java b/src/test/java/org/apache/commons/exec/environment/EnvironmentUtilTest.java
index 4607e17..e7bf68b 100644
--- a/src/test/java/org/apache/commons/exec/environment/EnvironmentUtilTest.java
+++ b/src/test/java/org/apache/commons/exec/environment/EnvironmentUtilTest.java
@@ -64,7 +64,7 @@
for(int i=0; i<envArgs.length; i++) {
assertNotNull("Entry "+i+" should not be null",envArgs[i]);
assertTrue("Entry "+i+" should not be empty",envArgs[i].length() > 0);
- System.out.println(envArgs[i]);
+ // System.out.println(envArgs[i]);
}
}