directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From seelm...@apache.org
Subject svn commit: r1770062 - in /directory/shared/trunk/util: ./ src/main/java/org/apache/directory/api/util/exception/ src/test/java/org/apache/directory/api/util/exception/
Date Wed, 16 Nov 2016 21:49:53 GMT
Author: seelmann
Date: Wed Nov 16 21:49:52 2016
New Revision: 1770062

URL: http://svn.apache.org/viewvc?rev=1770062&view=rev
Log:
Print stacktrace to dedicated output

Added:
    directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/
    directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/MultiExceptionTest.java
    directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/RuntimeMultiExceptionTest.java
Modified:
    directory/shared/trunk/util/pom.xml
    directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/MultiException.java
    directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/RuntimeMultiException.java

Modified: directory/shared/trunk/util/pom.xml
URL: http://svn.apache.org/viewvc/directory/shared/trunk/util/pom.xml?rev=1770062&r1=1770061&r2=1770062&view=diff
==============================================================================
--- directory/shared/trunk/util/pom.xml (original)
+++ directory/shared/trunk/util/pom.xml Wed Nov 16 21:49:52 2016
@@ -80,6 +80,16 @@
           </instructions>
         </configuration>
       </plugin>
+      <plugin>
+        <groupId>de.thetaphi</groupId>
+        <artifactId>forbiddenapis</artifactId>
+        <configuration>
+          <excludes>
+            <exclude>**/MultiExceptionTest.class</exclude>
+            <exclude>**/RuntimeMultiExceptionTest.class</exclude>
+          </excludes>
+        </configuration>
+      </plugin>
     </plugins>
   </build>
 </project>

Modified: directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/MultiException.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/MultiException.java?rev=1770062&r1=1770061&r2=1770062&view=diff
==============================================================================
--- directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/MultiException.java
(original)
+++ directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/MultiException.java
Wed Nov 16 21:49:52 2016
@@ -139,7 +139,7 @@ public class MultiException extends Exce
                 out.println( "\n\t<<========= Next Nested Exception" + " ========>>\n"
);
             }
 
-            throwable.printStackTrace();
+            throwable.printStackTrace( out );
         }
 
         out.println( "\n\t<<========= Last Nested Exception" + " ========>>\n"
);
@@ -171,7 +171,7 @@ public class MultiException extends Exce
                 out.println( "\n\t<<========= Next Nested Exception" + " ========>>\n"
);
             }
 
-            throwable.printStackTrace();
+            throwable.printStackTrace( out );
         }
 
         out.println( "\n\t<<========= Last Nested Exception" + " ========>>\n"
);

Modified: directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/RuntimeMultiException.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/RuntimeMultiException.java?rev=1770062&r1=1770061&r2=1770062&view=diff
==============================================================================
--- directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/RuntimeMultiException.java
(original)
+++ directory/shared/trunk/util/src/main/java/org/apache/directory/api/util/exception/RuntimeMultiException.java
Wed Nov 16 21:49:52 2016
@@ -149,7 +149,7 @@ public class RuntimeMultiException exten
                 out.println( "\n\t<<========= Next Nested Exception" + " ========>>\n"
);
             }
 
-            throwable.printStackTrace();
+            throwable.printStackTrace( out );
         }
 
         out.println( "\n\t<<========= Last Nested Exception" + " ========>>\n"
);
@@ -181,7 +181,7 @@ public class RuntimeMultiException exten
                 out.println( "\n\t<<========= Next Nested Exception" + " ========>>\n"
);
             }
 
-            throwable.printStackTrace();
+            throwable.printStackTrace( out );
         }
 
         out.println( "\n\t<<========= Last Nested Exception" + " ========>>\n"
);

Added: directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/MultiExceptionTest.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/MultiExceptionTest.java?rev=1770062&view=auto
==============================================================================
--- directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/MultiExceptionTest.java
(added)
+++ directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/MultiExceptionTest.java
Wed Nov 16 21:49:52 2016
@@ -0,0 +1,84 @@
+package org.apache.directory.api.util.exception;
+
+
+import static org.hamcrest.CoreMatchers.containsString;
+import static org.hamcrest.CoreMatchers.equalTo;
+import static org.junit.Assert.assertThat;
+
+import java.io.ByteArrayOutputStream;
+import java.io.PrintStream;
+import java.io.PrintWriter;
+import java.io.StringWriter;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+
+public class MultiExceptionTest
+{
+
+    private PrintStream originalOut;
+    private PrintStream originalErr;
+
+
+    @Before
+    public void setUp()
+    {
+        originalOut = System.out;
+        originalErr = System.err;
+    }
+
+
+    @After
+    public void tearDown()
+    {
+        System.setOut( originalOut );
+        System.setErr( originalErr );
+    }
+
+
+    @Test
+    public void testPrintStacktracePrintWriterDoesNotWriteToSystemOutErr()
+    {
+        ByteArrayOutputStream systemOut = new ByteArrayOutputStream();
+        PrintStream systemPrintStream = new PrintStream( systemOut );
+        System.setOut( systemPrintStream );
+        System.setErr( systemPrintStream );
+
+        StringWriter customOut = new StringWriter();
+        PrintWriter customPrintWriter = new PrintWriter( customOut );
+        MultiException multiException = new MultiException( "multi" );
+        multiException.addThrowable( new Exception( "nested1" ) );
+        multiException.addThrowable( new Exception( "nested2" ) );
+        multiException.printStackTrace( customPrintWriter );
+
+        assertThat( customOut.toString(), containsString( "multi" ) );
+        assertThat( customOut.toString(), containsString( "nested1" ) );
+        assertThat( customOut.toString(), containsString( "nested2" ) );
+        assertThat( systemOut.size(), equalTo( 0 ) );
+    }
+
+
+    @Test
+    public void testPrintStacktraceToPrintStreamDoesNotWriteToSystemOutErr()
+    {
+        ByteArrayOutputStream systemOut = new ByteArrayOutputStream();
+        PrintStream systemPrintStream = new PrintStream( systemOut );
+        System.setOut( systemPrintStream );
+        System.setErr( systemPrintStream );
+
+        ByteArrayOutputStream customOut = new ByteArrayOutputStream();
+        PrintStream customPrintWriter = new PrintStream( customOut );
+        MultiException multiException = new MultiException( "multi" );
+        multiException.addThrowable( new Exception( "nested1" ) );
+        multiException.addThrowable( new Exception( "nested2" ) );
+        multiException.printStackTrace( customPrintWriter );
+
+        assertThat( customOut.toString(), containsString( "multi" ) );
+        assertThat( customOut.toString(), containsString( "nested1" ) );
+        assertThat( customOut.toString(), containsString( "nested2" ) );
+        assertThat( systemOut.size(), equalTo( 0 ) );
+    }
+
+}

Added: directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/RuntimeMultiExceptionTest.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/RuntimeMultiExceptionTest.java?rev=1770062&view=auto
==============================================================================
--- directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/RuntimeMultiExceptionTest.java
(added)
+++ directory/shared/trunk/util/src/test/java/org/apache/directory/api/util/exception/RuntimeMultiExceptionTest.java
Wed Nov 16 21:49:52 2016
@@ -0,0 +1,84 @@
+package org.apache.directory.api.util.exception;
+
+
+import static org.hamcrest.CoreMatchers.containsString;
+import static org.hamcrest.CoreMatchers.equalTo;
+import static org.junit.Assert.assertThat;
+
+import java.io.ByteArrayOutputStream;
+import java.io.PrintStream;
+import java.io.PrintWriter;
+import java.io.StringWriter;
+
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+
+public class RuntimeMultiExceptionTest
+{
+
+    private PrintStream originalOut;
+    private PrintStream originalErr;
+
+
+    @Before
+    public void setUp()
+    {
+        originalOut = System.out;
+        originalErr = System.err;
+    }
+
+
+    @After
+    public void tearDown()
+    {
+        System.setOut( originalOut );
+        System.setErr( originalErr );
+    }
+
+
+    @Test
+    public void testPrintStacktracePrintWriterDoesNotWriteToSystemOutErr()
+    {
+        ByteArrayOutputStream systemOut = new ByteArrayOutputStream();
+        PrintStream systemPrintStream = new PrintStream( systemOut );
+        System.setOut( systemPrintStream );
+        System.setErr( systemPrintStream );
+
+        StringWriter customOut = new StringWriter();
+        PrintWriter customPrintWriter = new PrintWriter( customOut );
+        RuntimeMultiException runtimeMultiException = new RuntimeMultiException( "multi"
);
+        runtimeMultiException.addThrowable( new Exception( "nested1" ) );
+        runtimeMultiException.addThrowable( new Exception( "nested2" ) );
+        runtimeMultiException.printStackTrace( customPrintWriter );
+
+        assertThat( customOut.toString(), containsString( "multi" ) );
+        assertThat( customOut.toString(), containsString( "nested1" ) );
+        assertThat( customOut.toString(), containsString( "nested2" ) );
+        assertThat( systemOut.size(), equalTo( 0 ) );
+    }
+
+
+    @Test
+    public void testPrintStacktraceToPrintStreamDoesNotWriteToSystemOutErr()
+    {
+        ByteArrayOutputStream systemOut = new ByteArrayOutputStream();
+        PrintStream systemPrintStream = new PrintStream( systemOut );
+        System.setOut( systemPrintStream );
+        System.setErr( systemPrintStream );
+
+        ByteArrayOutputStream customOut = new ByteArrayOutputStream();
+        PrintStream customPrintWriter = new PrintStream( customOut );
+        RuntimeMultiException runtimeMultiException = new RuntimeMultiException( "multi"
);
+        runtimeMultiException.addThrowable( new Exception( "nested1" ) );
+        runtimeMultiException.addThrowable( new Exception( "nested2" ) );
+        runtimeMultiException.printStackTrace( customPrintWriter );
+
+        assertThat( customOut.toString(), containsString( "multi" ) );
+        assertThat( customOut.toString(), containsString( "nested1" ) );
+        assertThat( customOut.toString(), containsString( "nested2" ) );
+        assertThat( systemOut.size(), equalTo( 0 ) );
+    }
+
+}



Mime
View raw message