Use @DefaultTimeZone in test instead of setting a global variable

Author: garydgregory

pom.xml

@@ -52,6 +52,11 @@
       <artifactId>junit-jupiter-params</artifactId>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.junit-pioneer</groupId>
+      <artifactId>junit-pioneer</artifactId>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.easymock</groupId>
       <artifactId>easymock</artifactId>

src/test/java/org/apache/commons/collections4/FactoryUtilsTest.java

@@ -27,11 +27,11 @@
 import java.io.IOException;
 import java.io.Serializable;
 import java.util.Date;
-import java.util.TimeZone;
 
 import org.apache.commons.collections4.functors.ConstantFactory;
 import org.apache.commons.collections4.functors.ExceptionFactory;
 import org.junit.jupiter.api.Test;
+import org.junitpioneer.jupiter.DefaultTimeZone;
 
 /**
  * Tests the org.apache.commons.collections.FactoryUtils class.
@@ -129,8 +129,8 @@ void testExceptionFactory() {
     }
 
     @Test
+    @DefaultTimeZone("GMT")
     void testInstantiateFactoryComplex() {
-        TimeZone.setDefault(TimeZone.getTimeZone("GMT"));
         // 2nd Jan 1970
         final Factory<Date> factory = FactoryUtils.instantiateFactory(Date.class,
             new Class[] {Integer.TYPE, Integer.TYPE, Integer.TYPE},