/*
 * Copyright 2002-2017 the original author or authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.springframework.security.web.authentication.logout;

import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.mock;

import javax.servlet.http.Cookie;

import org.junit.Test;
import org.springframework.mock.web.MockHttpServletRequest;
import org.springframework.mock.web.MockHttpServletResponse;
import org.springframework.security.core.Authentication;

/**
 * @author Luke Taylor
 */
public class CookieClearingLogoutHandlerTests {

	// SEC-2036
	@Test
	public void emptyContextRootIsConverted() {
		MockHttpServletResponse response = new MockHttpServletResponse();
		MockHttpServletRequest request = new MockHttpServletRequest();
		request.setContextPath("");
		CookieClearingLogoutHandler handler = new CookieClearingLogoutHandler("my_cookie");
		handler.logout(request, response, mock(Authentication.class));
		assertThat(response.getCookies()).hasSize(1);
		for (Cookie c : response.getCookies()) {
			assertThat(c.getPath()).isEqualTo("/");
			assertThat(c.getMaxAge()).isZero();
		}
	}

	@Test
	public void configuredCookiesAreCleared() {
		MockHttpServletResponse response = new MockHttpServletResponse();
		MockHttpServletRequest request = new MockHttpServletRequest();
		request.setContextPath("/app");
		CookieClearingLogoutHandler handler = new CookieClearingLogoutHandler(
				"my_cookie", "my_cookie_too");
		handler.logout(request, response, mock(Authentication.class));
		assertThat(response.getCookies()).hasSize(2);
		for (Cookie c : response.getCookies()) {
			// gh-2325
			assertThat(c.getPath()).isEqualTo("/app/");
			assertThat(c.getMaxAge()).isZero();
		}
	}
	
	@Test
	public void passedInCookiesAreCleared() {
		MockHttpServletResponse response = new MockHttpServletResponse();
		MockHttpServletRequest request = new MockHttpServletRequest();
		request.setContextPath("/foo/bar");
		Cookie cookie1 = new Cookie("my_cookie",null);
		cookie1.setPath("/foo");
		cookie1.setMaxAge(0);
		Cookie cookie2 = new Cookie("my_cookie_too",null);
		cookie2.setPath("/foo");
		cookie2.setMaxAge(0);
		CookieClearingLogoutHandler handler = new CookieClearingLogoutHandler(cookie1, cookie2);
		handler.logout(request, response, mock(Authentication.class));
		assertThat(response.getCookies()).hasSize(2);
		for (Cookie c : response.getCookies()) {
			assertThat(c.getPath()).isEqualTo("/foo");
			assertThat(c.getMaxAge()).isZero();
		}
	}
	
	@Test(expected=IllegalArgumentException.class)
	public void invalidAge() {
		MockHttpServletResponse response = new MockHttpServletResponse();
		MockHttpServletRequest request = new MockHttpServletRequest();
		request.setContextPath("/foo/bar");
		Cookie cookie1 = new Cookie("my_cookie",null);
		cookie1.setPath("/foo");
		cookie1.setMaxAge(100);
		CookieClearingLogoutHandler handler = new CookieClearingLogoutHandler(cookie1);
		handler.logout(request, response, mock(Authentication.class));
	}
}

@@ -2241,26 +2241,25 @@
 o();%0A%09%09%7D%0A%09%7D%0A
-%09
 %0A
+
 %09@Test%0A%09publ
@@ -2501,32 +2501,33 @@
 kie(%22my_cookie%22,
+ 
 null);%0A%09%09cookie1
@@ -2614,16 +2614,17 @@
 ie_too%22,
+ 
 null);%0A%09
@@ -3016,10 +3016,9 @@
 %0A%09%7D%0A
-%09
 %0A
+
 %09@Te
@@ -3252,32 +3252,32 @@
 th(%22/foo/bar%22);%0A
-
 %09%09Cookie cookie1
@@ -3302,16 +3302,17 @@
 cookie%22,
+ 
 null);%0A%09

Clean up code to conform to basic checkstyle Issue: gh-6078