Cherry pick branch 'genexuslabs:FixTestGxReadEnvVar' into beta

Author: iroqueta

java/pom.xml

@@ -159,6 +159,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);
+			});
 	}
 }