Skip to content

Commit 546f352

Browse files
Denton-Lgitster
authored andcommitted
t9819: don't use test_must_fail with p4
We were using `test_must_fail p4` to test that the p4 command failed as expected. However, test_must_fail() is used to ensure that commands fail in an expected way, not due to something like a segv. Since we are not in the business of verifying the sanity of the external world, replace `test_must_fail p4` with `! p4` and assume that the `p4` command does not die unexpectedly. Signed-off-by: Denton Liu <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent bf12181 commit 546f352

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

t/t9819-git-p4-case-folding.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ test_expect_success 'Check p4 is in case-folding mode' '
3030
cd "$cli" &&
3131
>lc/FILE.TXT &&
3232
p4 add lc/FILE.TXT &&
33-
test_must_fail p4 submit -d "Cannot add file differing only in case" lc/FILE.TXT
33+
! p4 submit -d "Cannot add file differing only in case" lc/FILE.TXT
3434
)
3535
'
3636

0 commit comments

Comments
 (0)