BUrl unit tests fail

[KapiX]

Repro: UnitTester UrlTest

- FAILURES: 3
    TestCaller UrlTest::ParseTest: Expected: http://user:pass@www.foo.com:80/path?, but was: http://user:pass@www.foo.com:80/path.
    TestCaller UrlTest::TestIsValid: Set to an invalid host
    TestCaller UrlTest::RelativeUriTest: Expected: foo:/c, but was: foo:a/c. Base: foo:a/b Relative: ../c

• ParseTest: // Test that parsing a valid URL and converting back to string doesn't alter it [1]
• TestIsValid: looks like assert condition is reversed [2]
• RelativeUriTest: // However, foo:/c is what the RFC says we should get. [3]

[1] ​https://git.haiku-os.org/haiku/tree/src/tests/kits/net/service/UrlTest.cpp#n31

[2] ​https://git.haiku-os.org/haiku/tree/src/tests/kits/net/service/UrlTest.cpp#n71

[3] ​https://git.haiku-os.org/haiku/tree/src/tests/kits/net/service/UrlTest.cpp#n271

[KapiX]

Oops, correction: TestIsValid condition is OK, it appears BUrl considers "<invalid>" URL valid.

[pulkomandy]

Reassigning to Andrew as he rewrote the parser, but IIRC the tests were still working back then. I'll try to have a look too.

[apl-haiku]

Hi; I am a bit busy at the moment but will see what I can do. I'm looking at Set to an invalid host first.

[waddlesplash]

Last fixes merged in hrev52332. Thanks, apl!