Skip to content

Commit

Permalink
URL: Unbreak the serialization test
Browse files Browse the repository at this point in the history
http:https:// URLs no longer include ":80" when serialized, since port 80 is
implied by the protocol. Non-standard ports are still serialized.
  • Loading branch information
awesomekling committed Oct 21, 2019
1 parent 04b94a7 commit b74a433
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion AK/Tests/TestURL.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,8 @@ TEST_CASE(some_bad_urls)

TEST_CASE(serialization)
{
EXPECT_EQ(URL("http:https://www.serenityos.org/").to_string(), "http:https://www.serenityos.org:80/");
EXPECT_EQ(URL("http:https://www.serenityos.org/").to_string(), "http:https://www.serenityos.org/");
EXPECT_EQ(URL("http:https://www.serenityos.org:81/").to_string(), "http:https://www.serenityos.org:81/");
}

TEST_MAIN(URL)

0 comments on commit b74a433

Please sign in to comment.