Add check for missing IdentityProvider configuration.

Author: ggivo

entraid/src/main/java/redis/clients/authentication/entraid/EntraIDTokenAuthConfigBuilder.java

@@ -118,6 +118,11 @@ public TokenAuthConfig build() {
             throw new RedisEntraIDException(
                     "Cannot have both customEntraIdAuthenticationSupplier and ServicePrincipal/ManagedIdentity!");
         }
+        if (this.customEntraIdAuthenticationSupplier == null && spi == null && mii == null) {
+            throw new RedisEntraIDException(
+                "Missing configuration. One of customEntraIdAuthenticationSupplier, ServicePrincipal or ManagedIdentity must be configured!");
+        }
+
         if (spi != null) {
             super.identityProviderConfig(
                 new EntraIDIdentityProviderConfig(spi, scopes, tokenRequestExecTimeoutInMs));

entraid/src/test/java/redis/clients/authentication/EntraIDUnitTests.java

@@ -37,6 +37,8 @@
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.function.Supplier;
 
+import com.microsoft.aad.msal4j.IAccount;
+import com.microsoft.aad.msal4j.ITenantProfile;
 import org.awaitility.Awaitility;
 import org.awaitility.Durations;
 import org.junit.Test;
@@ -64,6 +66,7 @@
 import redis.clients.authentication.entraid.EntraIDTokenAuthConfigBuilder;
 import redis.clients.authentication.entraid.JWToken;
 import redis.clients.authentication.entraid.ManagedIdentityInfo;
+import redis.clients.authentication.entraid.RedisEntraIDException;
 import redis.clients.authentication.entraid.ServicePrincipalInfo;
 import redis.clients.authentication.entraid.ManagedIdentityInfo.UserManagedIdentityType;
 
@@ -143,6 +146,28 @@ public void testConfigBuilder() {
         }
     }
 
+    @Test
+    public void testConfigBuilderThrowsErrorIfMissconfigured() {
+
+        // Missing Configuration
+        assertThrows(RedisEntraIDException.class,() -> EntraIDTokenAuthConfigBuilder.builder().build());
+
+        // spi & mpi configured
+        assertThrows(RedisEntraIDException.class,() -> EntraIDTokenAuthConfigBuilder.builder()
+            .clientId("clientid")
+            .secret("secret")
+            .systemAssignedManagedIdentity()
+            .build());
+
+        // spi || mpi  && customEntraIdAuthenticationSupplier configured
+        assertThrows(RedisEntraIDException.class,() -> EntraIDTokenAuthConfigBuilder.builder()
+            .clientId("clientid")
+            .secret("secret")
+            .customEntraIdAuthenticationSupplier(() -> mock(IAuthenticationResult.class))
+            .build());
+    }
+
+
     // T.1.2
     // Implement a stubbed IdentityProvider and verify that the TokenManager works normally and handles:
     // network errors or other exceptions thrown from the IdentityProvider