Added test for missing cookie header value (dotnet#103969)

* Test for missing cookie header value * Disable this unit test on net48
cshung · Jun 26, 2024 · 9d008f4 · 9d008f4
1 parent ce02b53
commit 9d008f4
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTest.Cookies.cs b/src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTest.Cookies.cs
@@ -148,6 +148,18 @@ await LoopbackServerFactory.CreateClientAndServerAsync(
                 });
         }
 
+        [Theory]
+        [InlineData(null)]
+        [InlineData("")]
+        [SkipOnTargetFramework(TargetFrameworkMonikers.NetFramework, ".NET Framework allows empty value of cookie header while net7+ behaves by the specs and prohibit it")]
+        public void AddCookieHeader_MissingValue_Throws(string? cookieValue)
+        {
+            var requestMessage = new HttpRequestMessage(HttpMethod.Get, "http://foo/bar") { Version = UseVersion };
+            Assert.Throws<FormatException>(
+                () => requestMessage.Headers.Add("Cookie", cookieValue)
+            );
+        }
+
         [Fact]
         public async Task GetAsync_AddMultipleCookieHeaders_CookiesSent()
         {