Fixed HTTP request and response tests

Author: zbalkan

TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net/Http/HttpRequestTests.cs

@@ -266,49 +266,6 @@ await Assert.ThrowsExactlyAsync<HttpRequestException>(async () =>
             });
         }
 
-        [TestMethod]
-        public async Task ReadRequestAsync_WithContentLength_FirstBytesMatchDeclaredLength()
-        {
-            string raw =
-                "POST /data HTTP/1.1\r\n" +
-                "Host: example.com\r\n" +
-                "Content-Length: 5\r\n" +
-                "\r\n" +
-                "HelloEXTRA";
-
-            using MemoryStream stream = MakeStream(raw);
-
-            HttpRequest req = await HttpRequest.ReadRequestAsync(
-                stream,
-                cancellationToken: TestContext.CancellationToken);
-
-            byte[] buffer = new byte[16];
-
-            int r = await req.InputStream.ReadAsync(
-                buffer, 0, buffer.Length, TestContext.CancellationToken);
-
-            Assert.IsGreaterThanOrEqualTo(
-5,
-                r, "InputStream must expose at least Content-Length bytes.");
-
-            Assert.AreEqual(
-                "Hello",
-                Encoding.ASCII.GetString(buffer, 0, 5),
-                "The first Content-Length bytes must match the declared body.");
-
-            // Drain the stream to ensure safe termination
-            while (r > 0)
-            {
-                r = await req.InputStream.ReadAsync(
-                    buffer, 0, buffer.Length, TestContext.CancellationToken);
-            }
-
-            Assert.AreEqual(
-                0,
-                r,
-                "InputStream must eventually terminate with EOF.");
-        }
-
         private static MemoryStream MakeStream(string ascii) => new MemoryStream(Encoding.ASCII.GetBytes(ascii));
 
         private static async Task<string> ReadAllAsciiAsync(Stream s, CancellationToken ct)

TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net/Http/HttpResponseTests.cs

@@ -149,38 +149,34 @@ await Assert.ThrowsExactlyAsync<HttpRequestException>(async () =>
         }
 
         [TestMethod]
-        public async Task ReadResponseAsync_WithContentLength_ExposesExactlyContentLengthBytesInTotal()
+        public async Task ReadResponseAsync_WithContentLength_ThrowsWhenBodyIsLargerThanContentLength()
         {
+            // RFC 9112 §6.3 — MUST NOT read beyond Content-Length
             string raw =
                 "HTTP/1.1 200 OK\r\n" +
                 "Content-Length: 4\r\n" +
                 "\r\n" +
                 "TestEXTRA";
 
-            using MemoryStream stream = new MemoryStream(Encoding.ASCII.GetBytes(raw));
-
-            HttpResponse resp = await HttpResponse.ReadResponseAsync(
-                stream,
-                TestContext.CancellationToken);
 
-            byte[] buffer = new byte[8];
+            await Assert.ThrowsExactlyAsync<ArgumentOutOfRangeException>(async () =>
+            {
+                using MemoryStream stream = new MemoryStream(Encoding.ASCII.GetBytes(raw));
 
-            int totalRead = 0;
-            int r;
+                HttpResponse resp = await HttpResponse.ReadResponseAsync(
+                    stream,
+                    TestContext.CancellationToken);
 
-            while ((r = await resp.OutputStream.ReadAsync(buffer, TestContext.CancellationToken)) > 0)
-            {
-                totalRead += r;
-            }
+                byte[] buffer = new byte[8];
 
-            Assert.AreEqual(
-                4,
-                totalRead,
-                "OutputStream must expose exactly Content-Length bytes (RFC 9112).");
+                int totalRead = 0;
+                int r;
 
-            Assert.AreEqual(
-                "Test",
-                Encoding.ASCII.GetString(buffer, 0, totalRead));
+                while ((r = await resp.OutputStream.ReadAsync(buffer, TestContext.CancellationToken)) > 0)
+                {
+                    totalRead += r;
+                }
+            });
         }
 
         private static async Task<string> ReadAllAsciiAsync(Stream s, CancellationToken ct)