]> git.cworth.org Git - notmuch/commitdiff
test: factor out test_diff_file_
authorDavid Bremner <david@tethera.net>
Wed, 4 Aug 2021 10:42:31 +0000 (07:42 -0300)
committerDavid Bremner <david@tethera.net>
Sat, 21 Aug 2021 21:55:27 +0000 (14:55 -0700)
A following commit will use this new function in a public test
function.

test/test-lib.sh

index 1f696c44701ab76092f89444aa0fe07ab761faa3..6f4bb47fb88c681d015f0b879f3c7b148c34f70f 100644 (file)
@@ -402,17 +402,8 @@ test_expect_equal () {
     fi
 }
 
-# Like test_expect_equal, but takes two filenames.
-test_expect_equal_file () {
+test_diff_file_ () {
     local file1 file2 testname basename1 basename2
-    exec 1>&6 2>&7             # Restore stdout and stderr
-    if [ -z "$inside_subtest" ]; then
-       error "bug in the test script: test_expect_equal_file without test_begin_subtest"
-    fi
-    inside_subtest=
-    test "$#" = 2 ||
-       error "bug in the test script: not 2 parameters to test_expect_equal_file"
-
     file1="$1"
     file2="$2"
     if ! test_skip "$test_subtest_name"
@@ -430,6 +421,19 @@ test_expect_equal_file () {
     fi
 }
 
+# Like test_expect_equal, but takes two filenames.
+test_expect_equal_file () {
+    exec 1>&6 2>&7             # Restore stdout and stderr
+    if [ -z "$inside_subtest" ]; then
+       error "bug in the test script: test_expect_equal_file without test_begin_subtest"
+    fi
+    inside_subtest=
+    test "$#" = 2 ||
+       error "bug in the test script: not 2 parameters to test_expect_equal_file"
+
+    test_diff_file_ "$1" "$2"
+}
+
 # Like test_expect_equal, but arguments are JSON expressions to be
 # canonicalized before diff'ing.  If an argument cannot be parsed, it
 # is used unchanged so that there's something to diff against.