]> git.ipfire.org Git - thirdparty/git.git/commitdiff
remote-mediawiki tests: use test_cmp in tests
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Mon, 21 Sep 2020 10:39:48 +0000 (12:39 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 21 Sep 2020 19:37:38 +0000 (12:37 -0700)
Change code that used an ad-hoc "diff -b" invocation to use our
test_cmp helper instead. I'm also changing the order of arguments to
be the standard "test_cmp <expected> <actual>".

Using test_cmp has different semantics since the "-b" option to diff
causes it to ignore whitespace, but in these cases the use of "-b" was
just meaningless boilerplate. The desired semantics here are to
compare "git log" lines with know-good data, so we don't want to
ignore whitespace.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
contrib/mw-to-git/t/t9360-mw-to-git-clone.sh

index 9106833578e0e23c43157d1b6291296bab6e6c13..4c39bda7bf4da0f36db289c3aecdd5e9e40c9330 100755 (executable)
@@ -28,7 +28,7 @@ test_expect_success 'Git clone creates the expected git log with one file' '
                git log --format=%s HEAD^..HEAD >log.tmp
        ) &&
        echo "this must be the same" >msg.tmp &&
-       diff -b mw_dir_1/log.tmp msg.tmp
+       test_cmp msg.tmp mw_dir_1/log.tmp
 '
 
 
@@ -50,8 +50,8 @@ test_expect_success 'Git clone creates the expected git log with multiple files'
        echo "this must be the same" >>msgDaddy.tmp &&
        echo "identical too" >msgDj.tmp &&
        echo "identical" >>msgDj.tmp &&
-       diff -b mw_dir_2/logDaddy.tmp msgDaddy.tmp &&
-       diff -b mw_dir_2/logDj.tmp msgDj.tmp
+       test_cmp msgDaddy.tmp mw_dir_2/logDaddy.tmp &&
+       test_cmp msgDj.tmp mw_dir_2/logDj.tmp
 '
 
 
@@ -135,7 +135,7 @@ test_expect_success 'Git clone works with one specific page cloned ' '
                cd mw_dir_8 &&
                echo "this log must stay" >msg.tmp &&
                git log --format=%s >log.tmp &&
-               diff -b msg.tmp log.tmp
+               test_cmp msg.tmp log.tmp
        ) &&
        wiki_check_content mw_dir_8/Namnam.mw Namnam
 '