Skip to content

Commit 357b97a

Browse files
polish test assertions
1 parent 77e37f9 commit 357b97a

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

server/src/test/java/org/cloudfoundry/identity/uaa/provider/saml/UaaRelyingPartyRegistrationResolverTests.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,8 @@ void resolveWhenEntityBaseUrlIsNullOrEmptyUsesRequestUrl(String baseUrl) {
139139
request.setContextPath("/uaa-security");
140140
request.setRequestURI("/uaa-security/saml/metadata/test-idp");
141141

142+
String expectedBaseUrl = "https://uaa.example.com/uaa-security";
143+
142144
RelyingPartyRegistration testRegistration = RelyingPartyRegistration.withRegistrationId("test-idp")
143145
.entityId("{baseUrl}/saml/metadata")
144146
.assertionConsumerServiceLocation("{baseUrl}/saml/SSO")
@@ -160,8 +162,8 @@ void resolveWhenEntityBaseUrlIsNullOrEmptyUsesRequestUrl(String baseUrl) {
160162
assertThat(result).isNotNull();
161163
assertThat(result.getEntityId()).startsWith("https://uaa.example.com");
162164
assertThat(result.getAssertionConsumerServiceLocation()).startsWith("https://uaa.example.com");
163-
assertThat(result.getEntityId()).isEqualTo("https://uaa.example.com/uaa-security/saml/metadata");
164-
assertThat(result.getAssertionConsumerServiceLocation()).isEqualTo("https://uaa.example.com/uaa-security/saml/SSO");
165+
assertThat(result.getEntityId()).isEqualTo(expectedBaseUrl +"/saml/metadata");
166+
assertThat(result.getAssertionConsumerServiceLocation()).isEqualTo(expectedBaseUrl +"/saml/SSO");
165167
}
166168

167169
@Test
@@ -198,8 +200,8 @@ void resolveWhenEntityBaseUrlIsSetUsesConfiguredEntityBaseUrl() {
198200
assertThat(result).isNotNull();
199201
assertThat(result.getEntityId()).startsWith("https://custom.domain.com");
200202
assertThat(result.getAssertionConsumerServiceLocation()).startsWith("https://custom.domain.com");
201-
assertThat(result.getEntityId()).isEqualTo("https://custom.domain.com/uaa/saml/metadata");
202-
assertThat(result.getAssertionConsumerServiceLocation()).isEqualTo("https://custom.domain.com/uaa/saml/SSO");
203-
assertThat(result.getSingleLogoutServiceLocation()).isEqualTo("https://custom.domain.com/uaa/saml/SingleLogout");
203+
assertThat(result.getEntityId()).isEqualTo(configuredBaseUrl + "/saml/metadata");
204+
assertThat(result.getAssertionConsumerServiceLocation()).isEqualTo(configuredBaseUrl + "/saml/SSO");
205+
assertThat(result.getSingleLogoutServiceLocation()).isEqualTo(configuredBaseUrl + "/saml/SingleLogout");
204206
}
205207
}

0 commit comments

Comments
 (0)