You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2014/04/21 19:03:37 UTC

svn commit: r1588928 - /maven/shared/trunk/maven-shared-utils/src/test/java/org/apache/maven/shared/utils/CaseTest.java

Author: hboutemy
Date: Mon Apr 21 17:03:37 2014
New Revision: 1588928

URL: http://svn.apache.org/r1588928
Log:
make sure default locale is restored, even in case of failure

Modified:
    maven/shared/trunk/maven-shared-utils/src/test/java/org/apache/maven/shared/utils/CaseTest.java

Modified: maven/shared/trunk/maven-shared-utils/src/test/java/org/apache/maven/shared/utils/CaseTest.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-shared-utils/src/test/java/org/apache/maven/shared/utils/CaseTest.java?rev=1588928&r1=1588927&r2=1588928&view=diff
==============================================================================
--- maven/shared/trunk/maven-shared-utils/src/test/java/org/apache/maven/shared/utils/CaseTest.java (original)
+++ maven/shared/trunk/maven-shared-utils/src/test/java/org/apache/maven/shared/utils/CaseTest.java Mon Apr 21 17:03:37 2014
@@ -21,6 +21,7 @@ package org.apache.maven.shared.utils;
 
 import java.util.Locale;
 
+import org.junit.AfterClass;
 import org.junit.Assert;
 import org.junit.Test;
 
@@ -50,6 +51,14 @@ public class CaseTest
     /** turkish dotted I = İ */
     private final static char DOTTED_I = '\u0130';
 
+    private final Locale savedDefaultLocale = Locale.getDefault();
+
+    @AfterClass
+    public void restoreDefaultLocale()
+    {
+        Locale.setDefault( savedDefaultLocale );
+    }
+
     /**
      * test the known case of upper I which doesn't give commonly expected i in Turkish locale, but ı (dotless i).
      * @see <a href="http://mattryall.net/blog/2009/02/the-infamous-turkish-locale-bug">The infamous Turkish locale bug</a>
@@ -102,8 +111,6 @@ public class CaseTest
         final String lower = "abcdefghijklmnopqrstuvwxyz";
         final String upper = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
 
-        Locale savedDefaultLocale = Locale.getDefault();
-
         for ( Locale locale : Locale.getAvailableLocales() )
         {
             // check that toUpper() == toUpper(default locale) and toLower() = toLower(default locale)
@@ -136,7 +143,5 @@ public class CaseTest
             assertTrue( "'" + expectedToUpperCase + "'.equalsIgnoreCase('" + lower + "')",
                         expectedToUpperCase.equalsIgnoreCase( lower ) );
         }
-
-        Locale.setDefault( savedDefaultLocale );
     }
 }