Factor out logic in test_diff_funcname() into two helper functions,
these will be useful in a follow-up commit where we'll do this munging
in more than one place.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
---
t/t4018-diff-funcname.sh | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/t/t4018-diff-funcname.sh b/t/t4018-diff-funcname.sh
index 3e4c07e42b..7a830ec57f 100755
--- a/t/t4018-diff-funcname.sh
+++ b/t/t4018-diff-funcname.sh
@@ -47,6 +47,17 @@ test_expect_success 'last regexp must not be negated' '
test_i18ngrep ": Last expression must not be negated:" msg
'
+do_change_me () {
+ file=$1
+ sed -e "s/ChangeMe/IWasChanged/" <"$file" >tmp &&
+ mv tmp "$file"
+}
+
+last_diff_context_line () {
+ file=$1
+ sed -n -e "s/^.*@@\( \|$\)//p" <$file
+}
+
test_diff_funcname () {
desc=$1
cat <&8 >arg.header &&
@@ -57,13 +68,12 @@ test_diff_funcname () {
cp arg.test "$what" &&
cp arg.header expected &&
git add "$what" &&
- sed -e "s/ChangeMe/IWasChanged/" <"$what" >tmp &&
- mv tmp "$what"
+ do_change_me "$what"
' &&
test_expect_success "$desc" '
git diff -U1 "$what" >diff &&
- sed -n -e "s/^.*@@\( \|$\)//p" <diff >actual &&
+ last_diff_context_line diff >actual &&
test_cmp expected actual
'
}
--
2.30.0.284.gd98b1dd5eaa7
