Code

t4014: "no-add-headers" is actually called "no-add-header"
authorThomas Rast <trast@student.ethz.ch>
Mon, 29 Aug 2011 20:10:48 +0000 (22:10 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 29 Aug 2011 22:27:05 +0000 (15:27 -0700)
Since c426003 (format-patch: add --no-cc, --no-to, and
--no-add-headers, 2010-03-07) the tests have checked for an option
called --no-add-headers introduced by letting the user negate
--add-header.

However, the parseopt machinery does not automatically pluralize
anything, so it is in fact called --no-add-header.

Since the option never worked, is not documented anywhere, and
implementing an actual --no-add-headers would lead to silly code
complications, we just adapt the test to the code.

Signed-off-by: Thomas Rast <trast@student.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t4014-format-patch.sh

index a45d4fbd03e76f6c5283623cfe4d08cb02f5ea20..5cbc066e68e15b1015d9b8838b44a8042d4f04fd 100755 (executable)
@@ -219,11 +219,11 @@ test_expect_success '--no-cc overrides config.cc' '
        ! grep "^Cc: C. E. Cipient <rcipient@example.com>\$" patch12
 '
 
-test_expect_failure '--no-add-headers overrides config.headers' '
+test_expect_success '--no-add-header overrides config.headers' '
 
        git config --replace-all format.headers \
                "Header1: B. E. Cipient <rcipient@example.com>" &&
-       git format-patch --no-add-headers --stdout master..side |
+       git format-patch --no-add-header --stdout master..side |
        sed -e "/^\$/q" >patch13 &&
        check_patch patch13 &&
        ! grep "^Header1: B. E. Cipient <rcipient@example.com>\$" patch13