TestGxReadEnvVar was failing because it was calling internal JVM methods via reflection, which can change between versions.

The test has been changed so that it no longer calls internal JVM methods to modify environment variables.

Author: iroqueta

java/pom.xml

@@ -164,6 +164,12 @@
             <artifactId>commons-dbcp2</artifactId>
             <version>2.9.0</version>
         </dependency>
+        <dependency>
+            <groupId>com.github.stefanbirkner</groupId>
+            <artifactId>system-lambda</artifactId>
+            <version>1.2.1</version>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
 	<build>

java/src/test/java/com/genexus/TestGxReadEnvVar.java

@@ -4,6 +4,7 @@
 import com.genexus.sampleapp.GXcfg;
 import com.genexus.specific.java.Connect;
 import com.genexus.util.EnvVarReader;
+import com.github.stefanbirkner.systemlambda.SystemLambda;
 import org.junit.Assert;
 import org.junit.Test;
 
@@ -22,55 +23,34 @@ public class TestGxReadEnvVar {
 	private static final String SECOND_VALUE = "SecondDB_URL";
 
 	@Test
-	public void testGxReadEnvVar()
-	{
+	public void testGxReadEnvVar() throws Exception {
 		Connect.init();
 		ModelContext modelContext = ModelContext.getModelContext(GXcfg.class);
-		try {
-			Map<String, String> newenv = new HashMap<>();
-			newenv.put(FIRST_ENV_VAR, FIRST_VALUE);
-			setEnvVar(newenv);
-			String envVarValue = EnvVarReader.getEnvironmentVar(GX_DATASTORE, GX_PROP, null);
-			Assert.assertEquals(FIRST_VALUE, envVarValue);
 
-			newenv.put(SECOND_ENV_VAR, SECOND_VALUE);
-			setEnvVar(newenv);
-			envVarValue = EnvVarReader.getEnvironmentVar(GX_DATASTORE, GX_PROP, null);
-			Assert.assertEquals(SECOND_VALUE, envVarValue);
-			newenv.put(FIRST_ENV_VAR, null);
-			setEnvVar(newenv);
-			newenv.put(SECOND_ENV_VAR, null);
-			setEnvVar(newenv);
-		} catch (Exception e) {
-			e.printStackTrace();
-		}
-	}
-
-	private void setEnvVar(Map<String, String> newenv) throws Exception{
-		try {
-			Class<?> processEnvironmentClass = Class.forName("java.lang.ProcessEnvironment");
-			Field theEnvironmentField = processEnvironmentClass.getDeclaredField("theEnvironment");
-			theEnvironmentField.setAccessible(true);
-			Map<String, String> env = (Map<String, String>) theEnvironmentField.get(null);
-			env.putAll(newenv);
-			Field theCaseInsensitiveEnvironmentField = processEnvironmentClass.getDeclaredField("theCaseInsensitiveEnvironment");
-			theCaseInsensitiveEnvironmentField.setAccessible(true);
-			Map<String, String> cienv = (Map<String, String>)     theCaseInsensitiveEnvironmentField.get(null);
-			cienv.putAll(newenv);
-		} catch (NoSuchFieldException e)
-		{
-			Class[] classes = Collections.class.getDeclaredClasses();
-			Map<String, String> env = System.getenv();
-			for (Class cl : classes) {
-				if ("java.util.Collections$UnmodifiableMap".equals(cl.getName())) {
-					Field field = cl.getDeclaredField("m");
-					field.setAccessible(true);
-					Object obj = field.get(env);
-					Map<String, String> map = (Map<String, String>) obj;
-					map.clear();
-					map.putAll(newenv);
-				}
-			}
-		}
+		SystemLambda.withEnvironmentVariable(FIRST_ENV_VAR, FIRST_VALUE)
+			.execute(() -> {
+				String envVarValue = EnvVarReader.getEnvironmentVar(GX_DATASTORE, GX_PROP, null);
+				Assert.assertEquals(FIRST_VALUE, envVarValue);
+			});
+
+		SystemLambda.withEnvironmentVariable(SECOND_ENV_VAR, SECOND_VALUE)
+			.execute(() -> {
+				String envVarValue = EnvVarReader.getEnvironmentVar(GX_DATASTORE, GX_PROP, null);
+				Assert.assertEquals(SECOND_VALUE, envVarValue);
+			});
+
+		SystemLambda.withEnvironmentVariable(FIRST_ENV_VAR, FIRST_VALUE)
+			.and(SECOND_ENV_VAR, SECOND_VALUE)
+			.execute(() -> {
+				String envVarValue1 = EnvVarReader.getEnvironmentVar(GX_DATASTORE, GX_PROP, null);
+				Assert.assertEquals(SECOND_VALUE, envVarValue1); // Prioridad según tu EnvVarReader
+			});
+
+		SystemLambda.withEnvironmentVariable(FIRST_ENV_VAR, null)
+			.and(SECOND_ENV_VAR, null)
+			.execute(() -> {
+				String envVarValue = EnvVarReader.getEnvironmentVar(GX_DATASTORE, GX_PROP, null);
+				Assert.assertNull(envVarValue);
+			});
 	}
 }