Polish tests

config/src/test/java/org/springframework/security/config/InvalidConfigurationTests.java

@@ -16,6 +16,7 @@
 package org.springframework.security.config;
 
 import static org.assertj.core.api.Assertions.*;
+import static org.junit.Assert.fail;
 
 import org.junit.After;
 import org.junit.Test;
@@ -57,14 +58,15 @@ public class InvalidConfigurationTests {
 	public void missingAuthenticationManagerGivesSensibleErrorMessage() {
 		try {
 			setContext("<http auto-config='true' />");
+			fail();
 		}
 		catch (BeanCreationException e) {
 			Throwable cause = ultimateCause(e);
 			assertThat(cause instanceof NoSuchBeanDefinitionException).isTrue();
 			NoSuchBeanDefinitionException nsbe = (NoSuchBeanDefinitionException) cause;
 			assertThat(nsbe.getBeanName()).isEqualTo(BeanIds.AUTHENTICATION_MANAGER);
-			assertThat(nsbe.getMessage().endsWith(
+			assertThat(nsbe.getMessage()).endsWith(
-					AuthenticationManagerFactoryBean.MISSING_BEAN_ERROR_MESSAGE));
+					AuthenticationManagerFactoryBean.MISSING_BEAN_ERROR_MESSAGE);
 		}
 	}
 

oauth2/oauth2-client/src/test/java/org/springframework/security/oauth2/client/registration/ClientRegistrationsTest.java

@@ -140,7 +140,7 @@ public class ClientRegistrationsTest {
 	@Test
 	public void issuerWhenContainsTrailingSlashThenSuccess() throws Exception {
 		assertThat(registration("")).isNotNull();
-		assertThat(this.issuer.endsWith("/")).isTrue();
+		assertThat(this.issuer).endsWith("/");
 	}
 
 	/**

oauth2/oauth2-jose/src/test/java/org/springframework/security/oauth2/jwt/JwtDecodersTests.java

@@ -97,7 +97,7 @@ public class JwtDecodersTests {
 		prepareOpenIdConfigurationResponse();
 		this.server.enqueue(new MockResponse().setBody(JWK_SET));
 		assertThat(JwtDecoders.fromOidcIssuerLocation(this.issuer)).isNotNull();
-		assertThat(this.issuer.endsWith("/")).isTrue();
+		assertThat(this.issuer).endsWith("/");
 	}
 
 	@Test

web/src/test/java/org/springframework/security/web/authentication/rememberme/AbstractRememberMeServicesTests.java

@@ -92,7 +92,7 @@ public class AbstractRememberMeServicesTests {
 
 		String encoded = services.encodeCookie(cookie);
 		// '=' aren't allowed in version 0 cookies.
-		assertThat(encoded.endsWith("=")).isFalse();
+		assertThat(encoded).doesNotEndWith("=");
 		String[] decoded = services.decodeCookie(encoded);
 
 		assertThat(decoded).containsExactly("name:with:colon", "cookie", "tokens", "blah");