Use a pattern from the nul_match() function in t7008-grep-binary.sh to
make sure that we don't just fall through to the "else" if there's an
unknown parameter.

This is something I added in commit 77f6f4406f ("grep: add a test
helper function for less verbose -f \0 tests", 2017-05-20) to grep
tests, which were modeled on these wildmatch tests, and I'm now
porting back to the original wildmatch tests.

Signed-off-by: Ævar Arnfjörð Bjarmason <ava...@gmail.com>
---
 t/t3070-wildmatch.sh | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/t/t3070-wildmatch.sh b/t/t3070-wildmatch.sh
index 4d589d1f9a..71d6ea0f56 100755
--- a/t/t3070-wildmatch.sh
+++ b/t/t3070-wildmatch.sh
@@ -10,10 +10,13 @@ match() {
                test_expect_success "wildmatch:     match '$3' '$4'" "
                        test-wildmatch wildmatch '$3' '$4'
                "
-       else
+       elif test "$1" = 0
+       then
                test_expect_success "wildmatch:  no match '$3' '$4'" "
                        ! test-wildmatch wildmatch '$3' '$4'
                "
+       else
+               test_expect_success "PANIC: Test framework error. Unknown 
matches value $1" 'false'
        fi
 }
 
@@ -23,10 +26,13 @@ imatch() {
                test_expect_success "iwildmatch:    match '$2' '$3'" "
                        test-wildmatch iwildmatch '$2' '$3'
                "
-       else
+       elif test "$1" = 0
+       then
                test_expect_success "iwildmatch: no match '$2' '$3'" "
                        ! test-wildmatch iwildmatch '$2' '$3'
                "
+       else
+               test_expect_success "PANIC: Test framework error. Unknown 
matches value $1" 'false'
        fi
 }
 
@@ -36,10 +42,13 @@ pathmatch() {
                test_expect_success "pathmatch:     match '$2' '$3'" "
                        test-wildmatch pathmatch '$2' '$3'
                "
-       else
+       elif test "$1" = 0
+       then
                test_expect_success "pathmatch:  no match '$2' '$3'" "
                        ! test-wildmatch pathmatch '$2' '$3'
                "
+       else
+               test_expect_success "PANIC: Test framework error. Unknown 
matches value $1" 'false'
        fi
 }
 
-- 
2.15.1.424.g9478a66081

Reply via email to