AK: Strip leading/trailing C0-control-or-space in URLs correctly

We have to stop scanning once we hit a non-strippable character.
Add some tests to cover this.

1 files changed, 21 insertions, 0 deletions

diff --git a/Tests/AK/TestURL.cpp b/Tests/AK/TestURL.cpp
index 3fd1413f1d..9fff681525 100644
--- a/Tests/AK/TestURL.cpp
+++ b/Tests/AK/TestURL.cpp
@@ -307,3 +307,24 @@ TEST_CASE(complete_url)
 
     EXPECT(base_url.complete_url("../index.html#fragment").equals(base_url));
 }
+
+TEST_CASE(leading_whitespace)
+{
+    URL url { "   https://foo.com/" };
+    EXPECT(url.is_valid());
+    EXPECT_EQ(url.to_string(), "https://foo.com/");
+}
+
+TEST_CASE(trailing_whitespace)
+{
+    URL url { "https://foo.com/   " };
+    EXPECT(url.is_valid());
+    EXPECT_EQ(url.to_string(), "https://foo.com/");
+}
+
+TEST_CASE(leading_and_trailing_whitespace)
+{
+    URL url { "      https://foo.com/   " };
+    EXPECT(url.is_valid());
+    EXPECT_EQ(url.to_string(), "https://foo.com/");
+}