ci: Add env vars for split units in sdk-platform-java

Author: lqiu96

.github/workflows/ci.yaml

@@ -147,6 +147,16 @@ jobs:
         BUILD_SUBDIR: ${{matrix.package}}
         JOB_TYPE: test
         JOB_NAME: units-${{matrix.package}}-${{matrix.java}}
+    - name: Env Var Tests
+      if: matrix.package == 'sdk-platform-java'
+      run: .kokoro/build.sh
+      env:
+        BUILD_SUBDIR: ${{matrix.package}}
+        JOB_TYPE: test
+        SUREFIRE_JVM_OPT: '-PenvVarTest'
+        GOOGLE_CLOUD_UNIVERSE_DOMAIN: random.com
+        GOOGLE_CLOUD_ENABLE_DIRECT_PATH_XDS: true
+        GOOGLE_SDK_JAVA_LOGGING: true
   split-units-8:
     runs-on: ubuntu-latest
     name: "split-units"
@@ -198,6 +208,25 @@ jobs:
         JOB_TYPE: test
         JOB_NAME: units-8-runtime-${{matrix.java}}
       working-directory: ${{matrix.package}}
+    - name: Run Env Var Tests in Java ${{matrix.java}} with the source compiled in Java 11
+      if: matrix.package == 'sdk-platform-java'
+      run: |
+        mvn test \
+          -B -ntp \
+          -Pquick-build \
+          -PenvVarTest \
+          -Dorg.slf4j.simpleLogger.showDateTime=true \
+          -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \
+          -Dmaven.wagon.http.retryHandler.count=5 \
+          --also-make \
+          -T 1C
+      env:
+        BUILD_SUBDIR: ${{matrix.package}}
+        JOB_TYPE: test
+        GOOGLE_CLOUD_UNIVERSE_DOMAIN: random.com
+        GOOGLE_CLOUD_ENABLE_DIRECT_PATH_XDS: true
+        GOOGLE_SDK_JAVA_LOGGING: true
+      working-directory: ${{matrix.package}}
   windows:
     runs-on: windows-latest
     steps:

sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/logging/Slf4jUtils.java

@@ -47,7 +47,6 @@
 class Slf4jUtils {
 
   private static final Logger NO_OP_LOGGER = org.slf4j.helpers.NOPLogger.NOP_LOGGER;
-  private static final boolean loggingEnabled = LoggingUtils.isLoggingEnabled();
 
   private static final boolean isSLF4J2x;
 
@@ -70,7 +69,7 @@ static Logger getLogger(Class<?> clazz) {
 
   // constructor with LoggerFactoryProvider to make testing easier
   static Logger getLogger(Class<?> clazz, LoggerFactoryProvider factoryProvider) {
-    if (loggingEnabled) {
+    if (LoggingUtils.isLoggingEnabled()) {
       ILoggerFactory loggerFactory = factoryProvider.getLoggerFactory();
       return loggerFactory.getLogger(clazz.getName());
     } else {

sdk-platform-java/gax-java/gax/src/test/java/com/google/api/gax/logging/Slf4jUtilsTest.java

@@ -57,10 +57,16 @@ class Slf4jUtilsTest {
   // This test should only run GOOGLE_SDK_JAVA_LOGGING != true
   @Test
   void testGetLogger_loggingDisabled_shouldGetNOPLogger() {
-    Logger logger = Slf4jUtils.getLogger(Slf4jUtilsTest.class);
-    assertEquals(NOPLogger.class, logger.getClass());
-    assertFalse(logger.isInfoEnabled());
-    assertFalse(logger.isDebugEnabled());
+    boolean originalValue = LoggingUtils.isLoggingEnabled();
+    try {
+      LoggingUtils.setLoggingEnabled(false);
+      Logger logger = Slf4jUtils.getLogger(Slf4jUtilsTest.class);
+      assertEquals(NOPLogger.class, logger.getClass());
+      assertFalse(logger.isInfoEnabled());
+      assertFalse(logger.isDebugEnabled());
+    } finally {
+      LoggingUtils.setLoggingEnabled(originalValue);
+    }
   }
 
   // These tests does not require GOOGLE_SDK_JAVA_LOGGING