modernize t9300: use test_must_be_empty

Instead of comparing actual output to an empty file, use
test_must_be_empty. In addition to the better error message provided by
the helper, allocation of an empty file during the setup sequence can be
avoided.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Jeff King <peff@peff.net>

diff --git a/t/t9300-fast-import.sh b/t/t9300-fast-import.sh
index e9c760284950ee990224d45a22eae72f3e65205c..ceb3db3100e76a4d7c6e2e85b9fe93f364741bed 100755 (executable)
--- a/t/t9300-fast-import.sh
+++ b/t/t9300-fast-import.sh
@@ -47,8 +47,6 @@ file5_data='an inline file.
 file6_data='#!/bin/sh
 echo "$@"'
 
 file6_data='#!/bin/sh
 echo "$@"'
 

->empty
-

 ###
 ### series A
 ###
 ###
 ### series A
 ###


@@ -2320,12 +2318,12 @@ test_expect_success !MINGW 'R: in-stream cat-blob-fd not respected' '
        cat-blob $blob
        EOF
        test_cmp expect actual.3 &&
        cat-blob $blob
        EOF
        test_cmp expect actual.3 &&

-       test_cmp empty actual.1 &&
+       test_must_be_empty actual.1 &&

        git fast-import 3>actual.3 >actual.1 <<-EOF &&
        option cat-blob-fd=3
        cat-blob $blob
        EOF
        git fast-import 3>actual.3 >actual.1 <<-EOF &&
        option cat-blob-fd=3
        cat-blob $blob
        EOF

-       test_cmp empty actual.3 &&
+       test_must_be_empty actual.3 &&

        test_cmp expect actual.1
 '
 
        test_cmp expect actual.1
 '
 


@@ -2549,7 +2547,7 @@ EOF
 
 test_expect_success 'R: quiet option results in no stats being output' '
        cat input | git fast-import 2> output &&
 
 test_expect_success 'R: quiet option results in no stats being output' '
        cat input | git fast-import 2> output &&

-       test_cmp empty output
+       test_must_be_empty output

 '
 
 test_expect_success 'R: feature done means terminating "done" is mandatory' '
 '
 
 test_expect_success 'R: feature done means terminating "done" is mandatory' '