[PATCH] t5801 (remote-helpers): cleanup environment sets
Commit 512477b (tests: use env to run commands with temporary env-var settings) missed some variables in the remote-helpers test. Also standardize these. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- t/t5801-remote-helpers.sh | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/t/t5801-remote-helpers.sh b/t/t5801-remote-helpers.sh index 25fd2e7..42d1c66 100755 --- a/t/t5801-remote-helpers.sh +++ b/t/t5801-remote-helpers.sh @@ -212,19 +212,17 @@ test_expect_success 'push update refs failure' ' echo update fail file git commit -a -m update fail git rev-parse --verify testgit/origin/heads/update expect - GIT_REMOTE_TESTGIT_PUSH_ERROR=non-fast forward - export GIT_REMOTE_TESTGIT_PUSH_ERROR - test_expect_code 1 git push origin update + test_expect_code 1 env GIT_REMOTE_TESTGIT_FAILURE=non-fast forward \ + git push origin update git rev-parse --verify testgit/origin/heads/update actual test_cmp expect actual ) ' test_expect_success 'proper failure checks for fetching' ' - (GIT_REMOTE_TESTGIT_FAILURE=1 - export GIT_REMOTE_TESTGIT_FAILURE - cd local - test_must_fail git fetch 2 error + (cd local + test_must_fail env GIT_REMOTE_TESTGIT_FAILURE=1 \ + git fetch 2 error cat error grep -q Error while running fast-import error ) @@ -232,7 +230,8 @@ test_expect_success 'proper failure checks for fetching' ' test_expect_success 'proper failure checks for pushing' ' (cd local - test_must_fail env GIT_REMOTE_TESTGIT_FAILURE=1 git push --all + test_must_fail env GIT_REMOTE_TESTGIT_FAILURE=1 \ + git push --all ) ' -- 1.9.1+fc3.9.gc73078e -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] t5801 (remote-helpers): cleanup environment sets
Felipe Contreras felipe.contre...@gmail.com writes: Commit 512477b (tests: use env to run commands with temporary env-var settings) missed some variables in the remote-helpers test. Also standardize these. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com Good. I was wondering about these myself when juggling some series that touched this file the other day. Will queue. Thanks. --- t/t5801-remote-helpers.sh | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/t/t5801-remote-helpers.sh b/t/t5801-remote-helpers.sh index 25fd2e7..42d1c66 100755 --- a/t/t5801-remote-helpers.sh +++ b/t/t5801-remote-helpers.sh @@ -212,19 +212,17 @@ test_expect_success 'push update refs failure' ' echo update fail file git commit -a -m update fail git rev-parse --verify testgit/origin/heads/update expect - GIT_REMOTE_TESTGIT_PUSH_ERROR=non-fast forward - export GIT_REMOTE_TESTGIT_PUSH_ERROR - test_expect_code 1 git push origin update + test_expect_code 1 env GIT_REMOTE_TESTGIT_FAILURE=non-fast forward \ + git push origin update git rev-parse --verify testgit/origin/heads/update actual test_cmp expect actual ) ' test_expect_success 'proper failure checks for fetching' ' - (GIT_REMOTE_TESTGIT_FAILURE=1 - export GIT_REMOTE_TESTGIT_FAILURE - cd local - test_must_fail git fetch 2 error + (cd local + test_must_fail env GIT_REMOTE_TESTGIT_FAILURE=1 \ + git fetch 2 error cat error grep -q Error while running fast-import error ) @@ -232,7 +230,8 @@ test_expect_success 'proper failure checks for fetching' ' test_expect_success 'proper failure checks for pushing' ' (cd local - test_must_fail env GIT_REMOTE_TESTGIT_FAILURE=1 git push --all + test_must_fail env GIT_REMOTE_TESTGIT_FAILURE=1 \ + git push --all ) ' -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html