mirror of
https://github.com/didi/KnowStreaming.git
synced 2025-12-24 11:52:08 +08:00
Merge pull request #474 from houxiufeng/dev_v2.5.0_addtest
modify AbstractSingleSignOnTest error
This commit is contained in:
1
.gitignore
vendored
1
.gitignore
vendored
@@ -111,3 +111,4 @@ dist/
|
|||||||
dist/*
|
dist/*
|
||||||
kafka-manager-web/src/main/resources/templates/
|
kafka-manager-web/src/main/resources/templates/
|
||||||
.DS_Store
|
.DS_Store
|
||||||
|
kafka-manager-console/package-lock.json
|
||||||
|
|||||||
@@ -25,6 +25,7 @@ import org.testng.annotations.Test;
|
|||||||
import javax.servlet.http.HttpServletRequest;
|
import javax.servlet.http.HttpServletRequest;
|
||||||
import javax.servlet.http.HttpServletResponse;
|
import javax.servlet.http.HttpServletResponse;
|
||||||
import java.lang.reflect.Field;
|
import java.lang.reflect.Field;
|
||||||
|
import java.util.HashMap;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @author wyc
|
* @author wyc
|
||||||
@@ -142,7 +143,7 @@ public class AbstractSingleSignOnTest extends BaseTest {
|
|||||||
Field accountLdapEnabled = abstractSingleSignOn.getClass().getDeclaredField("accountLdapEnabled");
|
Field accountLdapEnabled = abstractSingleSignOn.getClass().getDeclaredField("accountLdapEnabled");
|
||||||
FieldSetter.setField(abstractSingleSignOn, accountLdapEnabled, true);
|
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());
|
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");
|
Field accountLdapEnabled = abstractSingleSignOn.getClass().getDeclaredField("accountLdapEnabled");
|
||||||
FieldSetter.setField(abstractSingleSignOn, accountLdapEnabled, true);
|
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");
|
Field authUserRegistrationRole = abstractSingleSignOn.getClass().getDeclaredField("authUserRegistrationRole");
|
||||||
|
|||||||
Reference in New Issue
Block a user