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 );
     }
 }


Reply via email to