t3404: replace test with test_line_count()

Refactor t3404 to replace instances of `test` with `test_line_count()`
for checking line counts. This improves readability and aligns with Git's
current test practices.

Signed-off-by: Usman Akinyemi <usmanakinyemi202@gmail.com>
Signed-off-by: Taylor Blau <me@ttaylorr.com>

diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh
index 96a65783c47a2fa162c1d2ea986ffc5d9f463f57..2ab660ef30f00654effe50886e4065df8db13f7a 100755 (executable)
--- a/t/t3404-rebase-interactive.sh
+++ b/t/t3404-rebase-interactive.sh
@@ -281,8 +281,9 @@ test_expect_success 'stop on conflicting pick' '
        test_cmp expect2 file1 &&
        test "$(git diff --name-status |
                sed -n -e "/^U/s/^U[^a-z]*//p")" = file1 &&
-       test 4 = $(grep -v "^#" < .git/rebase-merge/done | wc -l) &&
-       test 0 = $(grep -c "^[^#]" < .git/rebase-merge/git-rebase-todo)
+       grep -v "^#" <.git/rebase-merge/done >actual &&
+       test_line_count = 4 actual &&
+       test 0 = $(grep -c "^[^#]" <.git/rebase-merge/git-rebase-todo)
 '
 
 test_expect_success 'show conflicted patch' '
@@ -401,8 +402,8 @@ test_expect_success 'multi-squash only fires up editor once' '
        ) &&
        test $base = $(git rev-parse HEAD^) &&
        git show >output &&
-       count=$(grep ONCE output | wc -l) &&
-       test 1 = $count
+       grep ONCE output >actual &&
+       test_line_count = 1 actual
 '
 
 test_expect_success 'multi-fixup does not fire up editor' '
@@ -416,8 +417,7 @@ test_expect_success 'multi-fixup does not fire up editor' '
        ) &&
        test $base = $(git rev-parse HEAD^) &&
        git show >output &&
-       count=$(grep NEVER output | wc -l) &&
-       test 0 = $count &&
+       ! grep NEVER output &&
        git checkout @{-1} &&
        git branch -D multi-fixup
 '
@@ -436,8 +436,8 @@ test_expect_success 'commit message used after conflict' '
        ) &&
        test $base = $(git rev-parse HEAD^) &&
        git show >output &&
-       count=$(grep ONCE output | wc -l) &&
-       test 1 = $count &&
+       grep ONCE output >actual &&
+       test_line_count = 1 actual &&
        git checkout @{-1} &&
        git branch -D conflict-fixup
 '
@@ -456,8 +456,8 @@ test_expect_success 'commit message retained after conflict' '
        ) &&
        test $base = $(git rev-parse HEAD^) &&
        git show >output &&
-       count=$(grep TWICE output | wc -l) &&
-       test 2 = $count &&
+       grep TWICE output >actual &&
+       test_line_count = 2 actual &&
        git checkout @{-1} &&
        git branch -D conflict-squash
 '
@@ -501,8 +501,8 @@ test_expect_success 'squash ignores comments' '
        ) &&
        test $base = $(git rev-parse HEAD^) &&
        git show >output &&
-       count=$(grep ONCE output | wc -l) &&
-       test 1 = $count &&
+       grep ONCE output >actual &&
+       test_line_count = 1 actual &&
        git checkout @{-1} &&
        git branch -D skip-comments
 '
@@ -519,8 +519,8 @@ test_expect_success 'squash ignores blank lines' '
        ) &&
        test $base = $(git rev-parse HEAD^) &&
        git show >output &&
-       count=$(grep ONCE output | wc -l) &&
-       test 1 = $count &&
+       grep ONCE output >actual &&
+       test_line_count = 1 actual &&
        git checkout @{-1} &&
        git branch -D skip-blank-lines
 '