t4049: refocus tests
authorJunio C Hamano <gitster@pobox.com>
Thu, 29 Nov 2012 17:46:30 +0000 (09:46 -0800)
committerJunio C Hamano <gitster@pobox.com>
Thu, 29 Nov 2012 17:46:30 +0000 (09:46 -0800)
The primary thing Linus's patch wanted to change was to make sure
that 0-line change appears for a mode-only change. Update the
first test to chmod a file that we can see in the output (limited
by --stat-count) to demonstrate it. Also make sure to use test_chmod
and compare the index and the tree, so that we can run this test
even on a filesystem without permission bits.

Later two tests are about fixes to separate issues that were
introduced and/or uncovered by Linus's patch as a side effect, but
the issues are not related to mode-only changes. Remove chmod from
the tests.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t4049-diff-stat-count.sh
index 37f50cdef9d87209a52502eb80e9f69081368930..5b594e878f7561560fb2a8563b7355a32d4d3914 100755 (executable)
@@ -13,32 +13,31 @@ test_expect_success 'setup' '
        git commit -m initial
 '
 
-test_expect_success 'limit output to 2 (simple)' '
+test_expect_success 'mode-only change show as a 0-line change' '
        git reset --hard &&
-       chmod +x c d &&
+       test_chmod +x b d &&
        echo a >a &&
-       echo b >b &&
+       echo c >c &&
        cat >expect <<-\EOF
         a | 1 +
-        b | 1 +
+        b | 0
         ...
         4 files changed, 2 insertions(+)
        EOF
-       git diff --stat --stat-count=2 >actual &&
+       git diff --stat --stat-count=2 HEAD >actual &&
        test_i18ncmp expect actual
 '
 
 test_expect_success 'binary changes do not count in lines' '
        git reset --hard &&
-       chmod +x c d &&
        echo a >a &&
-       echo b >b &&
+       echo c >c &&
        cat "$TEST_DIRECTORY"/test-binary-1.png >d &&
        cat >expect <<-\EOF
         a | 1 +
-        b | 1 +
+        c | 1 +
         ...
-        4 files changed, 2 insertions(+)
+        3 files changed, 2 insertions(+)
        EOF
        git diff --stat --stat-count=2 >actual &&
        test_i18ncmp expect actual
@@ -56,12 +55,11 @@ test_expect_success 'exclude unmerged entries from total file count' '
        done |
        git update-index --index-info &&
        echo d >d &&
-       chmod +x c d &&
        cat >expect <<-\EOF
         a | 1 +
         b | 1 +
         ...
-        4 files changed, 3 insertions(+)
+        3 files changed, 3 insertions(+)
        EOF
        git diff --stat --stat-count=2 >actual &&
        test_i18ncmp expect actual