Mårten Kongstad <marten.kongstad@xxxxxxxxx> writes: > v3: change how tests count (part of) the dirstat number of lines: instead of > 'grep -c', use 'grep >filename && test_line_count'. Thanks to Torsten > Bögershausen and SZEDER Gábor for pointing out how to improve the tests. Thanks. I'd squash the following on top before queuing it. The overlong lines that ignores the exit status from "git diff" looked problematic to me. t/t4047-diff-dirstat.sh | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/t/t4047-diff-dirstat.sh b/t/t4047-diff-dirstat.sh index 065d74f..0d50dce 100755 --- a/t/t4047-diff-dirstat.sh +++ b/t/t4047-diff-dirstat.sh @@ -974,13 +974,16 @@ test_expect_success 'diff.dirstat=future_param,0,lines should warn, but still wo ' test_expect_success '--shortstat --dirstat should output only one dirstat' ' - git diff --shortstat --dirstat=changes HEAD^..HEAD | grep " dst/copy/changed/$" >actual_diff_shortstat_dirstat_changes && + git diff --shortstat --dirstat=changes HEAD^..HEAD >out && + grep " dst/copy/changed/$" out >actual_diff_shortstat_dirstat_changes && test_line_count = 1 actual_diff_shortstat_dirstat_changes && - git diff --shortstat --dirstat=lines HEAD^..HEAD | grep " dst/copy/changed/$" >actual_diff_shortstat_dirstat_lines && + git diff --shortstat --dirstat=lines HEAD^..HEAD >out && + grep " dst/copy/changed/$" out >actual_diff_shortstat_dirstat_lines && test_line_count = 1 actual_diff_shortstat_dirstat_lines && - git diff --shortstat --dirstat=files HEAD^..HEAD | grep " dst/copy/changed/$" >actual_diff_shortstat_dirstat_files && + git diff --shortstat --dirstat=files HEAD^..HEAD >out && + grep " dst/copy/changed/$" out >actual_diff_shortstat_dirstat_files && test_line_count = 1 actual_diff_shortstat_dirstat_files ' -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html