From 27a85be4e074d763e157d3393c72f34fa372df44 Mon Sep 17 00:00:00 2001 From: Sam Brannen <104798+sbrannen@users.noreply.github.com> Date: Thu, 13 Aug 2026 12:41:41 +0200 Subject: [PATCH] Polish MockCookieTests See gh-37134 --- .../springframework/mock/web/MockCookieTests.java | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/spring-test/src/test/java/org/springframework/mock/web/MockCookieTests.java b/spring-test/src/test/java/org/springframework/mock/web/MockCookieTests.java index 749f12192e6..1f032efb64d 100644 --- a/spring-test/src/test/java/org/springframework/mock/web/MockCookieTests.java +++ b/spring-test/src/test/java/org/springframework/mock/web/MockCookieTests.java @@ -22,6 +22,7 @@ import java.time.format.DateTimeParseException; import org.junit.jupiter.api.Test; import org.junit.jupiter.params.ParameterizedTest; +import org.junit.jupiter.params.provider.NullAndEmptySource; import org.junit.jupiter.params.provider.ValueSource; import static org.assertj.core.api.Assertions.assertThat; @@ -114,16 +115,10 @@ class MockCookieTests { assertThat(cookie.getValue()).isEqualTo(value); } - @Test - void parseNullHeader() { - assertThatIllegalArgumentException() - .isThrownBy(() -> MockCookie.parse(null)) - .withMessageContaining("Set-Cookie header must not be null or empty"); - } - @ParameterizedTest - @ValueSource(strings = {"", " "}) - void parseEmptyHeader(String header) { + @NullAndEmptySource + @ValueSource(strings = " ") + void parseNullOrEmptyHeader(String header) { assertThatIllegalArgumentException() .isThrownBy(() -> MockCookie.parse(header)) .withMessageContaining("Set-Cookie header must not be null or empty");