Skip to content

Commit 182b6f0

Browse files
committed
Fix OAuth2 client tests as per upstream changes
1 parent 66b55de commit 182b6f0

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/security/oauth2/client/OAuth2ClientPropertiesRegistrationAdapterTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ public void getClientRegistrationsWhenUsingCommonProviderShouldAdapt()
111111
assertThat(adapted.getAuthorizationGrantType()).isEqualTo(
112112
org.springframework.security.oauth2.core.AuthorizationGrantType.AUTHORIZATION_CODE);
113113
assertThat(adapted.getRedirectUri()).isEqualTo(
114-
"{scheme}://{serverName}:{serverPort}{contextPath}/login/oauth2/authorize/code/{registrationId}");
114+
"{scheme}://{serverName}:{serverPort}{contextPath}/login/oauth2/{registrationId}");
115115
assertThat(adapted.getScopes()).containsExactly("openid", "profile", "email",
116116
"address", "phone");
117117
assertThat(adapted.getClientName()).isEqualTo("Google");
@@ -195,7 +195,7 @@ public void getClientRegistrationsWhenProviderNotSpecifiedShouldUseRegistrationI
195195
assertThat(adapted.getAuthorizationGrantType()).isEqualTo(
196196
org.springframework.security.oauth2.core.AuthorizationGrantType.AUTHORIZATION_CODE);
197197
assertThat(adapted.getRedirectUri()).isEqualTo(
198-
"{scheme}://{serverName}:{serverPort}{contextPath}/login/oauth2/authorize/code/{registrationId}");
198+
"{scheme}://{serverName}:{serverPort}{contextPath}/login/oauth2/{registrationId}");
199199
assertThat(adapted.getScopes()).containsExactly("openid", "profile", "email",
200200
"address", "phone");
201201
assertThat(adapted.getClientName()).isEqualTo("Google");

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/security/oauth2/client/OAuth2WebSecurityConfigurationTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@
3535
import org.springframework.security.oauth2.client.registration.ClientRegistration;
3636
import org.springframework.security.oauth2.client.registration.ClientRegistrationRepository;
3737
import org.springframework.security.oauth2.client.registration.InMemoryClientRegistrationRepository;
38-
import org.springframework.security.oauth2.client.web.AuthorizationCodeAuthenticationFilter;
3938
import org.springframework.security.oauth2.client.web.AuthorizationRequestRedirectFilter;
39+
import org.springframework.security.oauth2.client.web.OAuth2LoginAuthenticationFilter;
4040
import org.springframework.security.oauth2.core.AuthorizationGrantType;
4141
import org.springframework.security.web.FilterChainProxy;
4242
import org.springframework.security.web.SecurityFilterChain;
@@ -96,11 +96,11 @@ private List<Filter> getAuthCodeFilters(AssertableApplicationContext context) {
9696
List<Filter> filters = (List<Filter>) ReflectionTestUtils
9797
.getField(filterChains.get(0), "filters");
9898
List<Filter> oauth2Filters = filters.stream()
99-
.filter((f) -> f instanceof AuthorizationCodeAuthenticationFilter
99+
.filter((f) -> f instanceof OAuth2LoginAuthenticationFilter
100100
|| f instanceof AuthorizationRequestRedirectFilter)
101101
.collect(Collectors.toList());
102102
return oauth2Filters.stream()
103-
.filter((f) -> f instanceof AuthorizationCodeAuthenticationFilter)
103+
.filter((f) -> f instanceof OAuth2LoginAuthenticationFilter)
104104
.collect(Collectors.toList());
105105
}
106106

0 commit comments

Comments
 (0)