Merge branch 'jk/test-chain-lint'
People often forget to chain the commands in their test together with &&, leaving a failure from an earlier command in the test go unnoticed. The new GIT_TEST_CHAIN_LINT mechanism allows you to catch such a mistake more easily. * jk/test-chain-lint: (36 commits) t9001: drop save_confirm helper t0020: use test_* helpers instead of hand-rolled messages t: simplify loop exit-code status variables t: fix some trivial cases of ignored exit codes in loops t7701: fix ignored exit code inside loop t3305: fix ignored exit code inside loop t0020: fix ignored exit code inside loops perf-lib: fix ignored exit code inside loop t6039: fix broken && chain t9158, t9161: fix broken &&-chain in git-svn tests t9104: fix test for following larger parents t4104: drop hand-rolled error reporting t0005: fix broken &&-chains t7004: fix embedded single-quotes t0050: appease --chain-lint t9001: use test_when_finished t4117: use modern test_* helpers t6034: use modern test_* helpers t1301: use modern test_* helpers t0020: use modern test_* helpers ...
This commit is contained in:
@ -94,7 +94,7 @@ EOF
|
||||
while read verb expect cmd args
|
||||
do
|
||||
test_expect_success "$cmd $verb COLUMNS (big change)" '
|
||||
COLUMNS=200 git $cmd $args >output
|
||||
COLUMNS=200 git $cmd $args >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect" actual
|
||||
'
|
||||
@ -102,7 +102,7 @@ do
|
||||
case "$cmd" in diff|show) continue;; esac
|
||||
|
||||
test_expect_success "$cmd --graph $verb COLUMNS (big change)" '
|
||||
COLUMNS=200 git $cmd $args --graph >output
|
||||
COLUMNS=200 git $cmd $args --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect-graph" actual
|
||||
'
|
||||
@ -122,7 +122,7 @@ EOF
|
||||
while read verb expect cmd args
|
||||
do
|
||||
test_expect_success "$cmd $verb not enough COLUMNS (big change)" '
|
||||
COLUMNS=40 git $cmd $args >output
|
||||
COLUMNS=40 git $cmd $args >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect" actual
|
||||
'
|
||||
@ -130,7 +130,7 @@ do
|
||||
case "$cmd" in diff|show) continue;; esac
|
||||
|
||||
test_expect_success "$cmd --graph $verb not enough COLUMNS (big change)" '
|
||||
COLUMNS=40 git $cmd $args --graph >output
|
||||
COLUMNS=40 git $cmd $args --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect-graph" actual
|
||||
'
|
||||
@ -150,7 +150,7 @@ EOF
|
||||
while read verb expect cmd args
|
||||
do
|
||||
test_expect_success "$cmd $verb statGraphWidth config" '
|
||||
git -c diff.statGraphWidth=26 $cmd $args >output
|
||||
git -c diff.statGraphWidth=26 $cmd $args >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect" actual
|
||||
'
|
||||
@ -158,7 +158,7 @@ do
|
||||
case "$cmd" in diff|show) continue;; esac
|
||||
|
||||
test_expect_success "$cmd --graph $verb statGraphWidth config" '
|
||||
git -c diff.statGraphWidth=26 $cmd $args --graph >output
|
||||
git -c diff.statGraphWidth=26 $cmd $args --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect-graph" actual
|
||||
'
|
||||
@ -179,19 +179,19 @@ EOF
|
||||
while read cmd args
|
||||
do
|
||||
test_expect_success "$cmd --stat=width with big change" '
|
||||
git $cmd $args --stat=40 >output
|
||||
git $cmd $args --stat=40 >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success "$cmd --stat-width=width with big change" '
|
||||
git $cmd $args --stat-width=40 >output
|
||||
git $cmd $args --stat-width=40 >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success "$cmd --stat-graph-width with big change" '
|
||||
git $cmd $args --stat-graph-width=26 >output
|
||||
git $cmd $args --stat-graph-width=26 >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
@ -199,13 +199,13 @@ do
|
||||
case "$cmd" in diff|show) continue;; esac
|
||||
|
||||
test_expect_success "$cmd --stat-width=width --graph with big change" '
|
||||
git $cmd $args --stat-width=40 --graph >output
|
||||
git $cmd $args --stat-width=40 --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect-graph actual
|
||||
'
|
||||
|
||||
test_expect_success "$cmd --stat-graph-width --graph with big change" '
|
||||
git $cmd $args --stat-graph-width=26 --graph >output
|
||||
git $cmd $args --stat-graph-width=26 --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect-graph actual
|
||||
'
|
||||
@ -265,7 +265,7 @@ EOF
|
||||
while read verb expect cmd args
|
||||
do
|
||||
test_expect_success "$cmd $verb COLUMNS (long filename)" '
|
||||
COLUMNS=200 git $cmd $args >output
|
||||
COLUMNS=200 git $cmd $args >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect" actual
|
||||
'
|
||||
@ -273,7 +273,7 @@ do
|
||||
case "$cmd" in diff|show) continue;; esac
|
||||
|
||||
test_expect_success "$cmd --graph $verb COLUMNS (long filename)" '
|
||||
COLUMNS=200 git $cmd $args --graph >output
|
||||
COLUMNS=200 git $cmd $args --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect-graph" actual
|
||||
'
|
||||
@ -294,7 +294,7 @@ while read verb expect cmd args
|
||||
do
|
||||
test_expect_success COLUMNS_CAN_BE_1 \
|
||||
"$cmd $verb prefix greater than COLUMNS (big change)" '
|
||||
COLUMNS=1 git $cmd $args >output
|
||||
COLUMNS=1 git $cmd $args >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect" actual
|
||||
'
|
||||
@ -303,7 +303,7 @@ do
|
||||
|
||||
test_expect_success COLUMNS_CAN_BE_1 \
|
||||
"$cmd --graph $verb prefix greater than COLUMNS (big change)" '
|
||||
COLUMNS=1 git $cmd $args --graph >output
|
||||
COLUMNS=1 git $cmd $args --graph >output &&
|
||||
grep " | " output >actual &&
|
||||
test_cmp "$expect-graph" actual
|
||||
'
|
||||
@ -320,7 +320,7 @@ EOF
|
||||
test_expect_success 'merge --stat respects COLUMNS (big change)' '
|
||||
git checkout -b branch HEAD^^ &&
|
||||
COLUMNS=100 git merge --stat --no-ff master^ >output &&
|
||||
grep " | " output >actual
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
|
||||
@ -329,7 +329,7 @@ cat >expect <<'EOF'
|
||||
EOF
|
||||
test_expect_success 'merge --stat respects COLUMNS (long filename)' '
|
||||
COLUMNS=100 git merge --stat --no-ff master >output &&
|
||||
grep " | " output >actual
|
||||
grep " | " output >actual &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
|
||||
|
||||
Reference in New Issue
Block a user