From e187a42bfca79b75548cdf6dd5c46298ccfa8134 Mon Sep 17 00:00:00 2001 From: Sam Brannen Date: Sun, 21 Apr 2019 15:17:40 +0200 Subject: [PATCH] Introduce failing tests for HttpHeaders See gh-22821 --- .../http/HttpHeadersTests.java | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/spring-web/src/test/java/org/springframework/http/HttpHeadersTests.java b/spring-web/src/test/java/org/springframework/http/HttpHeadersTests.java index 1a97828e70..1e609fc7f2 100644 --- a/spring-web/src/test/java/org/springframework/http/HttpHeadersTests.java +++ b/spring-web/src/test/java/org/springframework/http/HttpHeadersTests.java @@ -35,6 +35,7 @@ import java.util.Locale; import java.util.TimeZone; import org.hamcrest.Matchers; +import org.junit.Ignore; import org.junit.Test; import static org.hamcrest.Matchers.*; @@ -558,4 +559,34 @@ public class HttpHeadersTests { assertEquals("Bearer foo", authorization); } + @Test + @Ignore("Disabled until gh-22821 is resolved") + public void removalFromKeySetRemovesEntryFromUnderlyingMap() { + String headerName = "MyHeader"; + String headerValue = "value"; + + assertTrue(headers.isEmpty()); + headers.add(headerName, headerValue); + assertTrue(headers.containsKey(headerName)); + headers.keySet().removeIf(key -> key.equals(headerName)); + assertTrue(headers.isEmpty()); + headers.add(headerName, headerValue); + assertEquals(headerValue, headers.get(headerName)); + } + + @Test + @Ignore("Disabled until gh-22821 is resolved") + public void removalFromEntrySetRemovesEntryFromUnderlyingMap() { + String headerName = "MyHeader"; + String headerValue = "value"; + + assertTrue(headers.isEmpty()); + headers.add(headerName, headerValue); + assertTrue(headers.containsKey(headerName)); + headers.entrySet().removeIf(entry -> entry.getKey().equals(headerName)); + assertTrue(headers.isEmpty()); + headers.add(headerName, headerValue); + assertEquals(headerValue, headers.get(headerName)); + } + }