From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Tue, 17 Jan 2017 12:53:47 +0000 (+0000)
Subject: test suite: gitattributes: Break out sums_check_broken
X-Git-Tag: archive/debian/3.5~9
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=005625309ea693fcb6e95b595c2395a9c533dd3d;p=dgit.git

test suite: gitattributes: Break out sums_check_broken

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---

diff --git a/tests/tests/gitattributes b/tests/tests/gitattributes
index dec85b56..83fa6102 100755
--- a/tests/tests/gitattributes
+++ b/tests/tests/gitattributes
@@ -121,6 +121,24 @@ sums_check () {
 	diff -U0 ../sums ../$sums
 }
 
+sums_check_broken () {
+	# caller should cd into working directory, set
+	#   $sums
+	# and check out the broken branch
+
+	sha256sum af/* >../$sums.broken
+
+	for s in ../sums ../$sums.broken; do
+		sed 's/[0-9a-f]* //' $s >$s.nosums
+	done
+	diff -U0 ../sums.nosums ../$sums.broken.nosums
+	set +e
+	diff -U0 ../sums ../$sums.broken
+	rc=$?
+	set -e
+	test $rc = 1
+}
+
 t-dgit-warn-check () {
 	local warnok=$1; shift
 	# warnok should be  0 if the warning is expected
@@ -153,17 +171,8 @@ sfmt_import () {
 	t-dgit-warn-check 0 $dgitargs import-dsc ../$dscf +$branch.broken
 
 	git checkout $branch.broken
-	sha256sum af/* >../$sums.broken
 
-	for s in ../sums ../$sums.broken; do
-		sed 's/[0-9a-f]* //' $s >$s.nosums
-	done
-	diff -U0 ../sums.nosums ../$sums.broken.nosums
-	set +e
-	diff -U0 ../sums ../$sums.broken
-	rc=$?
-	set -e
-	test $rc = 1
+	sums_check_broken
 
 	t-dgit setup-new-tree