path: root/t
diff options
authorJeff King <>2017-10-13 17:26:02 (GMT)
committerJunio C Hamano <>2017-10-17 06:10:13 (GMT)
commitb521fd122865dca88b99d05344ec189d39efcefb (patch)
tree6e1dbd9e849dfdc6d1bf00c0a4251f3f3e01758e /t
parent33c643bb083133376f3fdcb190ebc58f9eef12bb (diff)
tag: respect color.ui config
Since 11b087adfd (ref-filter: consult want_color() before emitting colors, 2017-07-13), we expect that setting "color.ui" to "always" will enable color tag formats even without a tty. As that commit was built on top of 136c8c8b8f (color: check color.ui in git_default_config(), 2017-07-13) from the same series, we didn't need to touch tag's config parsing at all. However, since we reverted 136c8c8b8f, we now need to explicitly call git_color_default_config() to make this work. Let's do so, and also restore the test dropped in 0c88bf5050 (provide --color option for all ref-filter users, 2017-10-03). That commit swapped out our "color.ui=always" test for "--color" in preparation for "always" going away. But since it is here to stay, we should test both cases. Note that for-each-ref also lost its color.ui support as part of reverting 136c8c8b8f. But as a plumbing command, it should _not_ respect the color.ui config. Since it also gained a --color option in 0c88bf5050, that's the correct way to ask it for color. We'll continue to test that, and confirm that "color.ui" is not respected. Signed-off-by: Jeff King <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't')
2 files changed, 11 insertions, 0 deletions
diff --git a/t/ b/t/
index 6358134..09f2b77 100755
--- a/t/
+++ b/t/
@@ -441,6 +441,11 @@ test_expect_success '--color can override tty check' '
test_cmp expected.color actual
+test_expect_success 'color.ui=always does not override tty check' '
+ git -c color.ui=always for-each-ref --format="$color_format" >actual &&
+ test_cmp expected.bare actual
cat >expected <<\EOF
diff --git a/t/ b/t/
index b90db1b..62aa322 100755
--- a/t/
+++ b/t/
@@ -1925,6 +1925,12 @@ test_expect_success '--color overrides auto-color' '
test_cmp expect.color actual
+test_expect_success 'color.ui=always overrides auto-color' '
+ git -c color.ui=always tag $color_args >actual.raw &&
+ test_decode_color <actual.raw >actual &&
+ test_cmp expect.color actual
test_expect_success 'setup --merged test tags' '
git tag mergetest-1 HEAD~2 &&
git tag mergetest-2 HEAD~1 &&