This is the mail archive of the
mauve-patches@sourceware.org
mailing list for the Mauve project.
FYI: Test Fix
- From: Joshua Sumali <jsumali at redhat dot com>
- To: mauve-patches at sources dot redhat dot com
- Date: Fri, 13 Jul 2007 16:22:17 -0400
- Subject: FYI: Test Fix
This patch fixed a couple of tests in gnu/testlet/java/text/SimpleDateFormat
It just needed a simple time zone change to match the locale. Now passes
on openJDK.
2007-07-13 Joshua Sumali <jsumali@redhat.com>
* gnu/testlet/java/text/SimpleDateFormat/getDateFormatSymbols.java
(test): Fixed incorrect test.
* gnu/testlet/java/text/SimpleDateFormat/setDateFormatSymbols.java
(test): Fixed incorrect test.
#P mauve
Index: gnu/testlet/java/text/SimpleDateFormat/getDateFormatSymbols.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/java/text/SimpleDateFormat/getDateFormatSymbols.java,v
retrieving revision 1.1
diff -u -r1.1 getDateFormatSymbols.java
--- gnu/testlet/java/text/SimpleDateFormat/getDateFormatSymbols.java 11 Nov 2004 09:37:20 -0000 1.1
+++ gnu/testlet/java/text/SimpleDateFormat/getDateFormatSymbols.java 13 Jul 2007 20:16:01 -0000
@@ -28,6 +28,7 @@
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.Locale;
+import java.util.TimeZone;
/**
* Some checks for the getDateFormatSymbols() method in the SimpleDateFormat
@@ -47,6 +48,7 @@
// symbols, so updating them should not affect the results of the
// date formatter...
SimpleDateFormat sdf = new SimpleDateFormat("E", Locale.UK);
+ sdf.setTimeZone(TimeZone.getTimeZone("UTC"));
Date jan1_2005 = new Date(1104537600000L);
harness.check(sdf.format(jan1_2005), "Sat");
DateFormatSymbols s = sdf.getDateFormatSymbols();
#P mauve
Index: gnu/testlet/java/text/SimpleDateFormat/setDateFormatSymbols.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/java/text/SimpleDateFormat/setDateFormatSymbols.java,v
retrieving revision 1.1
diff -u -r1.1 setDateFormatSymbols.java
--- gnu/testlet/java/text/SimpleDateFormat/setDateFormatSymbols.java 11 Nov 2004 09:37:20 -0000 1.1
+++ gnu/testlet/java/text/SimpleDateFormat/setDateFormatSymbols.java 13 Jul 2007 20:14:26 -0000
@@ -28,6 +28,7 @@
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.Locale;
+import java.util.TimeZone;
/**
* Some checks for the setDateFormatSymbols() method in the SimpleDateFormat
@@ -45,6 +46,7 @@
{
// check that changing the short weekdays does work...
SimpleDateFormat sdf = new SimpleDateFormat("E", Locale.UK);
+ sdf.setTimeZone(TimeZone.getTimeZone("UTC"));
Date jan1_2005 = new Date(1104537600000L);
harness.check(sdf.format(jan1_2005), "Sat");
DateFormatSymbols s = sdf.getDateFormatSymbols();