Skip to content

Commit

Permalink
t/t5528-push-default: remove redundant test_config lines
Browse files Browse the repository at this point in the history
The line

  test_config push.default upstream

appears unnecessarily in two tests, as the final test_push_failure sets
push.default before pushing anyway.

Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Ramkumar Ramachandra authored and Junio C Hamano committed Jun 20, 2013
1 parent fb7dfaa commit 3cb8a5f
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions t/t5528-push-default.sh
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ test_expect_success '"upstream" pushes to configured upstream' '
test_expect_success '"upstream" does not push on unconfigured remote' '
git checkout master &&
test_unconfig branch.master.remote &&
test_config push.default upstream &&
test_commit three &&
test_push_failure upstream
'
Expand All @@ -57,7 +56,6 @@ test_expect_success '"upstream" does not push on unconfigured branch' '
git checkout master &&
test_config branch.master.remote parent1 &&
test_unconfig branch.master.merge &&
test_config push.default upstream
test_commit four &&
test_push_failure upstream
'
Expand Down

0 comments on commit 3cb8a5f

Please sign in to comment.