From: Johannes Sixt <j6t@kdbg.org>
Date: Thu, 19 Nov 2015 19:09:44 +0000 (+0100)
Subject: modernize t9300: use test_must_fail
X-Git-Tag: v2.7.0-rc0~23^2~5
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b08d82f0e8c373bfa463fc8df1419db24fbfc494;p=git

modernize t9300: use test_must_fail

One test case open-codes a test for an expected failure. Replace it by
test_must_fail.

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 566f7bdd30..e9c7602849 100755
--- a/t/t9300-fast-import.sh
+++ b/t/t9300-fast-import.sh
@@ -630,20 +630,9 @@ from refs/heads/branch
 
 INPUT_END
 test_expect_success 'F: non-fast-forward update skips' '
-	if git fast-import <input
-	then
-		echo BAD gfi did not fail
-		return 1
-	else
-		if test $old_branch = `git rev-parse --verify branch^0`
-		then
-			: branch unaffected and failure returned
-			return 0
-		else
-			echo BAD gfi changed branch $old_branch
-			return 1
-		fi
-	fi
+	test_must_fail git fast-import <input &&
+	# branch must remain unaffected
+	test $old_branch = `git rev-parse --verify branch^0`
 '
 
 test_expect_success 'F: verify pack' '