You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by vs...@apache.org on 2008/09/02 00:05:12 UTC

svn commit: r691075 - in /maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src: main/java/org/apache/maven/scm/provider/starteam/command/changelog/ test/java/org/apache/maven/scm/provider/starteam/command/changelog/

Author: vsiveton
Date: Mon Sep  1 15:05:11 2008
New Revision: 691075

URL: http://svn.apache.org/viewvc?rev=691075&view=rev
Log:
o fixed the uncaught error in test 
java.text.ParseException: Unparseable date: "11/09/02 15:08:20 GMT+02:00" on French system

Modified:
    maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/main/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumer.java
    maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/test/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumerTest.java

Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/main/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumer.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/main/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumer.java?rev=691075&r1=691074&r2=691075&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/main/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumer.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/main/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumer.java Mon Sep  1 15:05:11 2008
@@ -30,6 +30,7 @@
 import java.util.ArrayList;
 import java.util.Date;
 import java.util.List;
+import java.util.Locale;
 
 /**
  * @author <a href="mailto:evenisse@apache.org">Emmanuel Venisse</a>
@@ -38,7 +39,7 @@
 public class StarteamChangeLogConsumer
     extends AbstractConsumer
 {
-    private SimpleDateFormat localFormat = new SimpleDateFormat();
+    private SimpleDateFormat localFormat = new SimpleDateFormat( "", Locale.getDefault() );
 
     private List entries = new ArrayList();
 

Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/test/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumerTest.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/test/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumerTest.java?rev=691075&r1=691074&r2=691075&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/test/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumerTest.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-starteam/src/test/java/org/apache/maven/scm/provider/starteam/command/changelog/StarteamChangeLogConsumerTest.java Mon Sep  1 15:05:11 2008
@@ -104,11 +104,29 @@
     {
         List entries = parseTestFile();
 
-        //ensure the filename in the first ChangeSet has correct relative path
+        // ensure the filename in the first ChangeSet has correct relative path
         ChangeSet entry = (ChangeSet) entries.get( 1 );
 
         assertTrue( entry.containsFilename( "./maven/src/File2.java", null ) );
     }
 
+    public void testLocales()
+        throws Exception
+    {
+        Locale currentLocale = Locale.getDefault();
 
+        Locale[] availableLocales = Locale.getAvailableLocales();
+        try
+        {
+            for ( int i = 0; i < availableLocales.length; i++ )
+            {
+                Locale.setDefault( availableLocales[i] );
+                parseTestFile();
+            }
+        }
+        finally
+        {
+            Locale.setDefault( currentLocale );
+        }
+    }
 }