Skipped real service tests when required environment variables are not set

Author: jaaufauvre

library/src/test/java/com/mastercard/developer/oauth2/test/fixtures/BaseClientTest.java

@@ -11,7 +11,9 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Named;
 import org.junit.jupiter.api.extension.RegisterExtension;
+import org.junit.jupiter.api.function.ThrowingSupplier;
 import org.junit.jupiter.params.provider.Arguments;
+import org.opentest4j.TestAbortedException;
 
 public abstract class BaseClientTest extends BaseTest {
 
@@ -68,63 +70,27 @@ protected static Stream<Arguments> serverAndKeyProvider() {
     }
 
     protected static TestConfig getMastercardConfig() {
-        try {
-            return TestConfig.getMastercardApiConfig();
-        } catch (IllegalStateException e) {
-            throw e;
-        } catch (Exception e) {
-            throw new IllegalStateException(e);
-        }
+        return runOrWrap(TestConfig::getMastercardApiConfig);
     }
 
     private static TestConfig getMastercardConfigWithEcKey() {
-        try {
-            return TestConfig.getMastercardApiConfig(StaticKeys.EC_KEY_PAIR);
-        } catch (IllegalStateException e) {
-            throw e;
-        } catch (Exception e) {
-            throw new IllegalStateException(e);
-        }
+        return runOrWrap(() -> TestConfig.getMastercardApiConfig(StaticKeys.EC_KEY_PAIR));
     }
 
     private static TestConfig getMastercardConfigWithRsaKey() {
-        try {
-            return TestConfig.getMastercardApiConfig(StaticKeys.RSA_KEY_PAIR);
-        } catch (IllegalStateException e) {
-            throw e;
-        } catch (Exception e) {
-            throw new IllegalStateException(e);
-        }
+        return runOrWrap(() -> TestConfig.getMastercardApiConfig(StaticKeys.RSA_KEY_PAIR));
     }
 
     protected static TestConfig getFakeConfig() {
-        try {
-            return TestConfig.getFakeApiConfig(authorizationServer, resourceServer);
-        } catch (IllegalStateException e) {
-            throw e;
-        } catch (Exception e) {
-            throw new IllegalStateException(e);
-        }
+        return runOrWrap(() -> TestConfig.getFakeApiConfig(authorizationServer, resourceServer));
     }
 
     private static TestConfig getFakeConfigWithEcKey() {
-        try {
-            return TestConfig.getFakeApiConfig(authorizationServer, resourceServer, StaticKeys.EC_KEY_PAIR);
-        } catch (IllegalStateException e) {
-            throw e;
-        } catch (Exception e) {
-            throw new IllegalStateException(e);
-        }
+        return runOrWrap(() -> TestConfig.getFakeApiConfig(authorizationServer, resourceServer, StaticKeys.EC_KEY_PAIR));
     }
 
     private static TestConfig getFakeConfigWithRsaKey() {
-        try {
-            return TestConfig.getFakeApiConfig(authorizationServer, resourceServer, StaticKeys.RSA_KEY_PAIR);
-        } catch (IllegalStateException e) {
-            throw e;
-        } catch (Exception e) {
-            throw new IllegalStateException(e);
-        }
+        return runOrWrap(() -> TestConfig.getFakeApiConfig(authorizationServer, resourceServer, StaticKeys.RSA_KEY_PAIR));
     }
 
     private static Arguments createConfigArgument(String name, Supplier<TestConfig> configSupplier) {
@@ -135,4 +101,14 @@ protected static String readResourceId(String resource) throws OAuth2ClientJsonE
         Map<String, Object> jsonMap = JsonProvider.getInstance().parse(resource);
         return (String) jsonMap.get("id");
     }
+
+    private static <T> T runOrWrap(ThrowingSupplier<T> action) {
+        try {
+            return action.get();
+        } catch (TestAbortedException | IllegalStateException e) {
+            throw e;
+        } catch (Throwable e) {
+            throw new IllegalStateException(e);
+        }
+    }
 }

library/src/test/java/com/mastercard/developer/oauth2/test/fixtures/TestConfig.java

@@ -1,5 +1,7 @@
 package com.mastercard.developer.oauth2.test.fixtures;
 
+import static org.junit.jupiter.api.Assumptions.assumeTrue;
+
 import com.mastercard.developer.oauth2.config.OAuth2Config;
 import com.mastercard.developer.oauth2.config.SecurityProfile;
 import com.mastercard.developer.oauth2.core.dpop.StaticDPoPKeyProvider;
@@ -51,14 +53,19 @@ public static TestConfig getMastercardApiConfig(KeyPair dpopKeyPair) throws Exce
         String readScopes = getEnv("READ_SCOPES");
         String writeScopes = getEnv("WRITE_SCOPES");
 
-        validateEnvVariable("PRIVATE_KEY", privateKeyContent);
-        validateEnvVariable("CLIENT_ID", clientId);
-        validateEnvVariable("KID", kid);
-        validateEnvVariable("TOKEN_ENDPOINT", tokenEndpoint);
-        validateEnvVariable("ISSUER", issuer);
-        validateEnvVariable("API_BASE_URL", apiBaseUrlEnv);
-        validateEnvVariable("READ_SCOPES", readScopes);
-        validateEnvVariable("WRITE_SCOPES", readScopes);
+        try {
+            validateEnvVariable("PRIVATE_KEY", privateKeyContent);
+            validateEnvVariable("CLIENT_ID", clientId);
+            validateEnvVariable("KID", kid);
+            validateEnvVariable("TOKEN_ENDPOINT", tokenEndpoint);
+            validateEnvVariable("ISSUER", issuer);
+            validateEnvVariable("API_BASE_URL", apiBaseUrlEnv);
+            validateEnvVariable("READ_SCOPES", readScopes);
+            validateEnvVariable("WRITE_SCOPES", readScopes);
+        } catch (Exception e) {
+            // Disable the calling test when environment variables are missing
+            assumeTrue(false, "Test disabled: %s".formatted(e.getMessage()));
+        }
 
         OAuth2Config oauth2Config = OAuth2Config.builder()
             .securityProfile(SecurityProfile.FAPI2SP_PRIVATE_KEY_DPOP)