From 547dee2307b800b83dbc43b331671638a5bedaf3 Mon Sep 17 00:00:00 2001
From: Michael Marshall <mmarshall@apache.org>
Date: Thu, 16 Feb 2023 22:40:31 -0600
Subject: [PATCH] [fix][broker] Correct MockAlwaysExpiredAuthenticationState
 test impl

In c0f38a06a699b2243ee9ce1dc979a9e6a825e8ea, we cherry picked a commit that relied on
a feature not yet implemented in this branch. This commit fixes what that commit
broke.
---
 .../broker/auth/MockAlwaysExpiredAuthenticationState.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockAlwaysExpiredAuthenticationState.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockAlwaysExpiredAuthenticationState.java
index 3ed85067a3a5b..b1c4a4d1f756a 100644
--- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockAlwaysExpiredAuthenticationState.java
+++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockAlwaysExpiredAuthenticationState.java
@@ -52,6 +52,8 @@ public String getAuthRole() throws AuthenticationException {
 
     @Override
     public AuthData authenticate(AuthData authData) throws AuthenticationException {
+        authenticationDataSource = new AuthenticationDataCommand(new String(authData.getBytes(), UTF_8));
+        authRole = provider.authenticate(authenticationDataSource);
         return null;
     }
 
@@ -76,7 +78,7 @@ public AuthenticationDataSource getAuthDataSource() {
 
     @Override
     public boolean isComplete() {
-        return true;
+        return authRole != null;
     }
 
     @Override