diff --git a/internal/url/url_test.go b/internal/url/url_test.go index 51e48cfc3..29c5fbb6d 100644 --- a/internal/url/url_test.go +++ b/internal/url/url_test.go @@ -33,13 +33,6 @@ func TestString(t *testing.T) { "https://foo.com/i haz spaces?q=i haz spaces", "https://foo.com/i%20haz%20spaces?q=i%20haz%20spaces", }, - { - // TODO(b/123017837): Go escapes only certain chars in fragments. - "fragment not entirely reescaped", // This is intrinsic Go URL behavior. - "https://example.com/amp.html#htmlURL=http%3A%2F%2Fbar.com%2Fbaz", - //does not produce: "https://example.com/amp.html#htmlURL=http%3A%2F%2Fbar.com%2Fbaz", - "https://example.com/amp.html#htmlURL=http://bar.com/baz", - }, { "fragment with space and quote reescaped", "https://example.com/amp.html#fragment-\" ", diff --git a/transformer/internal/amphtml/urls_test.go b/transformer/internal/amphtml/urls_test.go index 49f89c778..3515fa1e7 100644 --- a/transformer/internal/amphtml/urls_test.go +++ b/transformer/internal/amphtml/urls_test.go @@ -134,15 +134,6 @@ func TestToAbsoluteURL(t *testing.T) { documentURL: otherURL, expected: rootURL + "#dogs", }, - { - // TODO(b/123017837): Go escapes only certain chars in fragments. - desc: "fragment not entirely reescaped", // This is intrinsic Go URL behavior. - input: "https://example.com/amp.html#htmlURL=http%3A%2F%2Fbar.com%2Fbaz", - baseURL: rootURL, - documentURL: rootURL, - //expected: "https://example.com/amp.html#htmlURL=http%3A%2F%2Fbar.com%2Fbaz", - expected: "https://example.com/amp.html#htmlURL=http://bar.com/baz", - }, { desc: "fragment with space and quote reescaped", input: "https://example.com/amp.html#fragment-\" ",