Elijah Newren <[email protected]> writes:

> +test_expect_success 'rebase --interactive: NO directory rename' '
> +     test_when_finished "git -C no-dir-rename rebase --abort" &&
> +     (
> +             cd no-dir-rename &&
> +
> +             git checkout B^0 &&
> +
> +             set_fake_editor &&
> +             FAKE_LINES="1" test_must_fail git rebase --interactive A &&

Is this a single-shot environment assignment?  That would have been
caught with the test linter.

Perhaps squshing this in would be sufficient fix?

 t/t3401-rebase-and-am-rename.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/t/t3401-rebase-and-am-rename.sh b/t/t3401-rebase-and-am-rename.sh
index 94bdfbd69c..13e09afec0 100755
--- a/t/t3401-rebase-and-am-rename.sh
+++ b/t/t3401-rebase-and-am-rename.sh
@@ -141,7 +141,7 @@ test_expect_success 'rebase --interactive: NO directory 
rename' '
                git checkout B^0 &&
 
                set_fake_editor &&
-               FAKE_LINES="1" test_must_fail git rebase --interactive A &&
+               test_must_fail env FAKE_LINES="1" git rebase --interactive A &&
 
                git ls-files -s >out &&
                test_line_count = 6 out &&
@@ -160,7 +160,7 @@ test_expect_success 'rebase (am): NO directory rename' '
                git checkout B^0 &&
 
                set_fake_editor &&
-               FAKE_LINES="1" test_must_fail git rebase A &&
+               test_must_fail env FAKE_LINES="1" git rebase A &&
 
                git ls-files -s >out &&
                test_line_count = 6 out &&
@@ -179,7 +179,7 @@ test_expect_success 'rebase --merge: NO directory rename' '
                git checkout B^0 &&
 
                set_fake_editor &&
-               FAKE_LINES="1" test_must_fail git rebase --merge A &&
+               test_must_fail env FAKE_LINES="1" git rebase --merge A &&
 
                git ls-files -s >out &&
                test_line_count = 6 out &&

Reply via email to