From 364db6762e51d2f46b065843f692afc2ed14196c Mon Sep 17 00:00:00 2001 From: Ruben Dijkstra Date: Wed, 1 Jun 2016 11:53:48 +0200 Subject: [PATCH] Add failing test for #3905 Fix Assert usage --- .../AbstractAuthenticationProcessingFilterTests.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/web/src/test/java/org/springframework/security/web/authentication/AbstractAuthenticationProcessingFilterTests.java b/web/src/test/java/org/springframework/security/web/authentication/AbstractAuthenticationProcessingFilterTests.java index 8252e70ceb..a8a258595d 100644 --- a/web/src/test/java/org/springframework/security/web/authentication/AbstractAuthenticationProcessingFilterTests.java +++ b/web/src/test/java/org/springframework/security/web/authentication/AbstractAuthenticationProcessingFilterTests.java @@ -418,6 +418,15 @@ public class AbstractAuthenticationProcessingFilterTests { assertThat(response.getStatus()).isEqualTo(HttpServletResponse.SC_UNAUTHORIZED); } + /** + * https://github.com/spring-projects/spring-security/pull/3905 + */ + @Test(expected = IllegalArgumentException.class) + public void setRememberMeServicesShouldntAllowNulls() { + AbstractAuthenticationProcessingFilter filter = new MockAuthenticationFilter(); + filter.setRememberMeServices(null); + } + // ~ Inner Classes // ==================================================================================================