This is the mail archive of the mauve-patches@sourceware.org mailing list for the Mauve project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

FYI: Test Fix


This patch fixed an incorrect test in gnu/testlet/java/util/GregorianCalendar/internal.java

The test needed an extra else if conditional that it was missing. It now passes on openJDK.

2007-07-13 Joshua Sumali <jsumali@redhat.com>

   * gnu/testlet/java/util/GregorianCalendar/internal.java
   (test): Fixed incorrect test.

#P mauve
Index: gnu/testlet/java/util/GregorianCalendar/internal.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/java/util/GregorianCalendar/internal.java,v
retrieving revision 1.1
diff -u -r1.1 internal.java
--- gnu/testlet/java/util/GregorianCalendar/internal.java	18 Apr 2007 15:56:05 -0000	1.1
+++ gnu/testlet/java/util/GregorianCalendar/internal.java	13 Jul 2007 15:36:44 -0000
@@ -394,15 +394,17 @@
 		if (setOrder[i] == Calendar.WEEK_OF_YEAR)
 		  {
 		    // YEAR + DAY_OF_WEEK + WEEK_OF_YEAR
-		    // (the rest)
 		    expectTime = 1176361200000L;
 		  }
-		else
+		else if (setOrder[i] == Calendar.WEEK_OF_MONTH)
 		  {
 		    // YEAR + MONTH + WEEK_OF_MONTH + DAY_OF_WEEK
-		    // (some of them)
 		    expectTime = 1177570800000L;
 		  }
+		else {
+			// rest of them
+			expectTime = 1178175600000L;
+		}
 		break;
 	      }
 	  }


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]