]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t5551: drop redundant grep for Accept-Language
authorJeff King <peff@peff.net>
Thu, 23 Feb 2023 10:52:10 +0000 (05:52 -0500)
committerJunio C Hamano <gitster@pobox.com>
Thu, 23 Feb 2023 21:01:14 +0000 (13:01 -0800)
Commit b0c4adcdd7 (remote-curl: send Accept-Language header to server,
2022-07-11) added tests to make sure the header is sent via HTTP.
However, it checks in two places:

  1. In the expected trace output, we check verbatim for the header and
     its value.

  2. Afterwards, we grep for the header again in the trace file.

This (2) is probably cargo-culted from the earlier grep for
Accept-Encoding. It is needed for the encoding because we smudge the
value of that header when doing the verbatim check; see 1a53e692af
(remote-curl: accept all encodings supported by curl, 2018-05-22).

But we don't do so for the language header, so any problem that the
"grep" would catch in (2) would already have been caught by (1).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t5551-http-fetch-smart.sh

index 10b7e7cda28a062121bd89359e00cfd2513486e6..29d489768e1c6dacc519acf73e01d8c734e139c7 100755 (executable)
@@ -103,10 +103,7 @@ test_expect_success 'clone http repository' '
                test_cmp exp actual.smudged &&
 
                grep "Accept-Encoding:.*gzip" actual >actual.gzip &&
-               test_line_count = 2 actual.gzip &&
-
-               grep "Accept-Language: ko-KR, *" actual >actual.language &&
-               test_line_count = 2 actual.language
+               test_line_count = 2 actual.gzip
        fi
 '