-update submodule ref

-change exception message
redis · Nov 17, 2024 · f037440 · f037440
1 parent 392b3b0
commit f037440
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 7 deletions.
diff --git a/redis-authx b/redis-authx
diff --git a/src/main/java/redis/clients/jedis/authentication/JedisAuthXManager.java b/src/main/java/redis/clients/jedis/authentication/JedisAuthXManager.java
@@ -79,7 +79,7 @@ public void authenticateConnections(Token token) {
 
     public void onError(Exception reason) {
         throw new JedisAuthenticationException(
-                "Token request/renewal failed with message:" + reason.getMessage(), reason);
+                "Token manager failed to acquire new token!", reason);
     }
 
     public Connection addConnection(Connection connection) {

diff --git a/...est/java/redis/clients/jedis/authentication/TokenBasedAuthenticationIntegrationTests.java b/...est/java/redis/clients/jedis/authentication/TokenBasedAuthenticationIntegrationTests.java
@@ -54,7 +54,7 @@ public void testJedisPooledAuth() {
 
         TokenAuthConfig tokenAuthConfig = TokenAuthConfig.builder()
                 .identityProviderConfig(idProviderConfig).expirationRefreshRatio(0.8F)
-                .lowerRefreshBoundMillis(10000).tokenRequestExecutionTimeoutInMs(1000).build();
+                .lowerRefreshBoundMillis(10000).tokenRequestExecTimeoutInMs(1000).build();
 
         JedisClientConfig clientConfig = DefaultJedisClientConfig.builder()
                 .tokenAuthConfig(tokenAuthConfig).build();

diff --git a/src/test/java/redis/clients/jedis/authentication/TokenBasedAuthenticationUnitTests.java b/src/test/java/redis/clients/jedis/authentication/TokenBasedAuthenticationUnitTests.java
@@ -180,7 +180,8 @@ public void testCalculateRenewalDelay() {
   }
 
   @Test
-  public void testAuthXManagerReceivesNewToken() throws InterruptedException, ExecutionException, TimeoutException {
+  public void testAuthXManagerReceivesNewToken()
+      throws InterruptedException, ExecutionException, TimeoutException {
 
     IdentityProvider identityProvider = () -> new SimpleToken("tokenVal",
         System.currentTimeMillis() + 5 * 1000, System.currentTimeMillis(),
@@ -204,8 +205,9 @@ public void testAuthXManagerReceivesNewToken() throws InterruptedException, Exec
 
   @Test
   public void testBlockForInitialToken() {
+    String exceptionMessage = "Test exception from identity provider!";
     IdentityProvider identityProvider = () -> {
-      throw new RuntimeException("Test exception from identity provider!");
+      throw new RuntimeException(exceptionMessage);
     };
 
     TokenManager tokenManager = new TokenManager(identityProvider,
@@ -214,8 +216,8 @@ public void testBlockForInitialToken() {
     JedisAuthXManager manager = new JedisAuthXManager(tokenManager);
     ExecutionException e = assertThrows(ExecutionException.class, () -> manager.start(true));
 
-    assertEquals("java.lang.RuntimeException: Test exception from identity provider!",
-      e.getCause().getCause().getMessage());
+    assertEquals(exceptionMessage,
+      e.getCause().getCause().getCause().getCause().getMessage());
   }
 
   @Test