t4026: test "normal" color

peff · gitster · commit cb357221a402 · 2014-11-20T10:54:10.000-08:00
If the user specifiers "normal" for a foreground color, this
should be a noop (while this may sound useless, it is the
only way to specify an unchanged foreground color followed
by a specific background color).

We also check that color "-1" does the same thing. This is
not documented, but has worked forever, so let's make sure
we keep supporting it.

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/t/t4026-color.sh b/t/t4026-color.sh
@@ -53,6 +53,14 @@ test_expect_success '256 colors' '
 	color "254 bold 255" "[1;38;5;254;48;5;255m"
 '
 
+test_expect_success '"normal" yields no color at all"' '
+	color "normal black" "[40m"
+'
+
+test_expect_success '-1 is a synonym for "normal"' '
+	color "-1 black" "[40m"
+'
+
 test_expect_success 'color too small' '
 	invalid_color "-2"
 '

Original file line number	Diff line number	Diff line change
`@@ -53,6 +53,14 @@ test_expect_success '256 colors' '`
`53`	`53`	`color "254 bold 255" "[1;38;5;254;48;5;255m"`
`54`	`54`	`'`
`55`	`55`
	`56`	`+test_expect_success '"normal" yields no color at all"' '`
	`57`	`+ color "normal black" "[40m"`
	`58`	`+'`
	`59`	`+`
	`60`	`+test_expect_success '-1 is a synonym for "normal"' '`
	`61`	`+ color "-1 black" "[40m"`
	`62`	`+'`
	`63`	`+`
`56`	`64`	`test_expect_success 'color too small' '`
`57`	`65`	`invalid_color "-2"`
`58`	`66`	`'`