summaryrefslogtreecommitdiff
path: root/t/t5541-http-push-smart.sh
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2016-07-20 11:32:26 (GMT)
committerJunio C Hamano <gitster@pobox.com>2016-07-20 22:08:40 (GMT)
commit68f3c079fe87caab2e116fd9a48bbdedc54be026 (patch)
tree86985af3416b4c262bf19b6dfa1ca96683b1e619 /t/t5541-http-push-smart.sh
parent882d49ca5cb6a584a37ab0266b8720b5b0bf2101 (diff)
downloadgit-68f3c079fe87caab2e116fd9a48bbdedc54be026.zip
git-68f3c079fe87caab2e116fd9a48bbdedc54be026.tar.gz
git-68f3c079fe87caab2e116fd9a48bbdedc54be026.tar.bz2
t5541: fix url scrubbing test when GPG is not set
When the GPG prereq is not set, we do not run test 34. That test changes the directory of the test script as a side effect (something we usually frown on, but which matches the style of the rest of this script). When test 35 (the url-scrubbing test) runs, it expects to be in the directory from test 34. If it's not, the test fails; we are in a different sub-repo, our test-commit is built on a different history, and the push becomes a non-fast-forward. We can fix this by unconditionally moving to the directory we expect (again, against our usual style but matching how the rest of the script operates). As an additional protection, let's also switch from "make a new commit and push to master" to just "push to a new branch". We don't care about the branch name; we just want _some_ ref update to trigger the status output. Pushing to a new branch is less likely to run into problems with force-updates, changing the checked-out branch, etc. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5541-http-push-smart.sh')
-rwxr-xr-xt/t5541-http-push-smart.sh6
1 files changed, 4 insertions, 2 deletions
diff --git a/t/t5541-http-push-smart.sh b/t/t5541-http-push-smart.sh
index 8d08e06..9593fc1 100755
--- a/t/t5541-http-push-smart.sh
+++ b/t/t5541-http-push-smart.sh
@@ -369,8 +369,10 @@ test_expect_success GPG 'push with post-receive to inspect certificate' '
'
test_expect_success 'push status output scrubs password' '
- test_commit scrub &&
- git push --porcelain "$HTTPD_URL_USER_PASS/smart/test_repo.git" >status &&
+ cd "$ROOT_PATH/test_repo_clone" &&
+ git push --porcelain \
+ "$HTTPD_URL_USER_PASS/smart/test_repo.git" \
+ +HEAD:scrub >status &&
# should have been scrubbed down to vanilla URL
grep "^To $HTTPD_URL/smart/test_repo.git" status
'