From 73f8da8d5a490f248a0f7eea64dc2797ee4792f5 Mon Sep 17 00:00:00 2001
From: houxiufeng <houxiufeng@didiglobal.com>
Date: Mon, 7 Mar 2022 17:10:15 +0800
Subject: [PATCH] modify AbstractSingleSignOnTest error

---
 .gitignore                                                   | 1 +
 .../kafka/manager/account/AbstractSingleSignOnTest.java      | 5 +++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/.gitignore b/.gitignore
index 77560587..9bdbddbe 100644
--- a/.gitignore
+++ b/.gitignore
@@ -111,3 +111,4 @@ dist/
 dist/*
 kafka-manager-web/src/main/resources/templates/
 .DS_Store
+kafka-manager-console/package-lock.json
diff --git a/kafka-manager-extends/kafka-manager-account/src/test/java/com/xiaojukeji/kafka/manager/account/AbstractSingleSignOnTest.java b/kafka-manager-extends/kafka-manager-account/src/test/java/com/xiaojukeji/kafka/manager/account/AbstractSingleSignOnTest.java
index c65ae280..ec75fe9e 100644
--- a/kafka-manager-extends/kafka-manager-account/src/test/java/com/xiaojukeji/kafka/manager/account/AbstractSingleSignOnTest.java
+++ b/kafka-manager-extends/kafka-manager-account/src/test/java/com/xiaojukeji/kafka/manager/account/AbstractSingleSignOnTest.java
@@ -25,6 +25,7 @@ import org.testng.annotations.Test;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import java.lang.reflect.Field;
+import java.util.HashMap;
 
 /**
  * @author wyc
@@ -142,7 +143,7 @@ public class AbstractSingleSignOnTest extends BaseTest {
         Field accountLdapEnabled = abstractSingleSignOn.getClass().getDeclaredField("accountLdapEnabled");
         FieldSetter.setField(abstractSingleSignOn, accountLdapEnabled, true);
 
-        Mockito.when(ldapAuthentication.authenticate(Mockito.anyString(), Mockito.anyString())).thenReturn(false);
+        Mockito.when(ldapAuthentication.authenticate(Mockito.anyString(), Mockito.anyString())).thenReturn(null);
         Assert.assertEquals(abstractSingleSignOn.loginAndGetLdap(request, response, dto).toString(), Result.buildFrom(ResultStatus.LDAP_AUTHENTICATION_FAILED).toString());
     }
 
@@ -158,7 +159,7 @@ public class AbstractSingleSignOnTest extends BaseTest {
         Field accountLdapEnabled = abstractSingleSignOn.getClass().getDeclaredField("accountLdapEnabled");
         FieldSetter.setField(abstractSingleSignOn, accountLdapEnabled, true);
 
-        Mockito.when(ldapAuthentication.authenticate(Mockito.anyString(), Mockito.anyString())).thenReturn(true);
+        Mockito.when(ldapAuthentication.authenticate(Mockito.anyString(), Mockito.anyString())).thenReturn(new HashMap<>());
 
         // 通过反射初始化成员变量，防止出现空指针异常
         Field authUserRegistrationRole = abstractSingleSignOn.getClass().getDeclaredField("authUserRegistrationRole");