cweb/url-testing — pull request #4 by sideshowbarker (2012-12-03 08:44:35+00:00)
Title: Removed/moved some misplaced tests in the file-http-base group, fixed pathless URL test.

Tests 9 and 22-24 of the "file-http-base" appear to have maybe been
intended to test that a relative URL starting with "//" and that has a base
URL with "http:" would for some reason get changed into a "file:" URL. But
that's incorrect. That case is just a normal relative URL case, so
shouldn't be in the "file-http-base" group of cases. And the "relative"
group already has tests for those (48, 50, and 52, I think) with correct
expected results. So I think tests 9 and 22-24 in the "file-http-base"
group can just be dropped.

The existing expected result for test 8 in the "file-http-base" was also
wrong for the same reason, but it had something worth including, which is
that "c|" in a hostname should have an expected result with "c|" as-is.
WebKit at least gets that wrong (changing it to "c%7C"), so it seems like a
case worth retaining. But since that part is really just a host-parsing
test, I simplified it and moved it to the "host" group.

Also, fixed the "pathless URL" test (trailing slash should be appended).