You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by nd...@apache.org on 2006/07/16 00:38:18 UTC

svn commit: r422308 - in /incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging: FileHandlerTest.java SocketHandlerTest.java

Author: ndbeyer
Date: Sat Jul 15 15:38:18 2006
New Revision: 422308

URL: http://svn.apache.org/viewvc?rev=422308&view=rev
Log:
Apply patch for HARMONY-867: [classlib[util] unit test org.apache.harmony.logging.tests.java.util.logging.FileHandlerTest is incorrect

Modified:
    incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/FileHandlerTest.java
    incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/SocketHandlerTest.java

Modified: incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/FileHandlerTest.java
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/FileHandlerTest.java?rev=422308&r1=422307&r2=422308&view=diff
==============================================================================
--- incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/FileHandlerTest.java (original)
+++ incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/FileHandlerTest.java Sat Jul 15 15:38:18 2006
@@ -503,8 +503,8 @@
 		}
 		try {
 			handler.publish(null);
-			fail("should throw null exception");
 		} catch (NullPointerException e) {
+			fail("should not throw NPE");
 		}
 		try {
 			FileHandler h = new FileHandler(null, false);

Modified: incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/SocketHandlerTest.java
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/SocketHandlerTest.java?rev=422308&r1=422307&r2=422308&view=diff
==============================================================================
--- incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/SocketHandlerTest.java (original)
+++ incubator/harmony/enhanced/classlib/trunk/modules/logging/src/test/java/org/apache/harmony/logging/tests/java/util/logging/SocketHandlerTest.java Sat Jul 15 15:38:18 2006
@@ -594,8 +594,6 @@
 		h.setLevel(Level.INFO);
 		try {
 			h.publish(null);
-			fail("Should throw NullPointerException!");
-		} catch (NullPointerException e) {
 		} finally {
 			h.close();
 			// ensure the thread exits and the port becomes available again