fix test setup timing issue.

Author: zhumin8

oauth2_http/java/com/google/auth/oauth2/LoggingConfigs.java

@@ -8,10 +8,12 @@ class LoggingConfigs {
 
   private static EnvironmentProvider environmentProvider = SystemEnvironmentProvider.getInstance();
   private static final Logger NO_OP_LOGGER = org.slf4j.helpers.NOPLogger.NOP_LOGGER;
-  private static final boolean LOGGING_ENABLED = isLoggingEnabled();
-  // expose this setter for testing purposes
+  private static boolean loggingEnabled = isLoggingEnabled();
+  // expose this setter only for testing purposes
   static void setEnvironmentProvider(EnvironmentProvider provider) {
     environmentProvider = provider;
+    // Recalculate LOGGING_ENABLED after setting the new provider
+    loggingEnabled = isLoggingEnabled();
   }
 
   private LoggingConfigs() {}
@@ -22,7 +24,7 @@ static Logger getLogger(Class<?> clazz) {
 
   // constructor with LoggerFactoryProvider to make testing easier
   static Logger getLogger(Class<?> clazz, LoggerFactoryProvider factoryProvider) {
-    if (LOGGING_ENABLED) {
+    if (loggingEnabled) {
       return factoryProvider.getLoggerFactory().getLogger(clazz.getName());
     } else {
       //  use SLF4j's NOP logger regardless of bindings

oauth2_http/javatests/com/google/auth/oauth2/LoggingConfigsTest.java

@@ -33,8 +33,6 @@ public class LoggingConfigsTest {
   @Before
   public void setup() {
     testEnvironmentProvider = new TestEnvironmentProvider();
-    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
-
     // need to setup a ConsoleAppender and attach to root logger because TestAppender
     // does not correctly capture MDC info
     LoggerContext loggerContext = (LoggerContext) LoggerFactory.getILoggerFactory();
@@ -66,6 +64,7 @@ public void tearDown() {
   @Test
   public void testGetLogger_loggingEnabled_slf4jBindingPresent() {
     testEnvironmentProvider.setEnv("GOOGLE_SDK_JAVA_LOGGING", "true");
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
     Logger logger = LoggingConfigs.getLogger(LoggingConfigsTest.class);
     assertTrue(logger instanceof org.slf4j.Logger);
     assertNotEquals(logger.getClass(), NOPLogger.class);
@@ -74,6 +73,7 @@ public void testGetLogger_loggingEnabled_slf4jBindingPresent() {
   @Test
   public void testGetLogger_loggingDisabled() {
     testEnvironmentProvider.setEnv("GOOGLE_SDK_JAVA_LOGGING", "false");
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
 
     Logger logger = LoggingConfigs.getLogger(LoggingConfigsTest.class);
     assertEquals(NOPLogger.class, logger.getClass());
@@ -82,6 +82,7 @@ public void testGetLogger_loggingDisabled() {
   @Test
   public void testGetLogger_loggingEnabled_noBinding() {
     testEnvironmentProvider.setEnv("GOOGLE_SDK_JAVA_LOGGING", "true");
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
     // Create a mock LoggerFactoryProvider
     LoggerFactoryProvider mockLoggerFactoryProvider = mock(LoggerFactoryProvider.class);
     ILoggerFactory mockLoggerFactory = mock(ILoggerFactory.class);
@@ -99,15 +100,19 @@ public void testGetLogger_loggingEnabled_noBinding() {
   @Test
   public void testIsLoggingEnabled_true() {
     testEnvironmentProvider.setEnv("GOOGLE_SDK_JAVA_LOGGING", "true");
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
     assertTrue(LoggingConfigs.isLoggingEnabled());
     testEnvironmentProvider.setEnv("GOOGLE_SDK_JAVA_LOGGING", "TRUE");
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
     assertTrue(LoggingConfigs.isLoggingEnabled());
     testEnvironmentProvider.setEnv("GOOGLE_SDK_JAVA_LOGGING", "True");
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
     assertTrue(LoggingConfigs.isLoggingEnabled());
   }
 
   @Test
   public void testIsLoggingEnabled_defaultToFalse() {
+    LoggingConfigs.setEnvironmentProvider(testEnvironmentProvider);
     assertFalse(LoggingConfigs.isLoggingEnabled());
   }