Skip to content

Commit

Permalink
t5551: Remove header from curl cookie file
Browse files Browse the repository at this point in the history
The URL included in the header appears to vary from curl version to
curl version.  Since we only care about the final few lines, only test
them.  However, make sure the blank line after the header is still
included to make sure there are no extra cookie lines.

Signed-off-by: Brian Gernhardt <brian@gernhardtsoftware.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Brian Gernhardt authored and Junio C Hamano committed Aug 5, 2013
1 parent 912b2ac commit 580cf0a
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions t/t5551-http-fetch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -191,9 +191,6 @@ cat >cookies.txt <<EOF
127.0.0.1 FALSE /smart_cookies/ FALSE 0 othername othervalue
EOF
cat >expect_cookies.txt <<EOF
# Netscape HTTP Cookie File
# http://curl.haxx.se/docs/http-cookies.html
# This file was generated by libcurl! Edit at your own risk.
127.0.0.1 FALSE /smart_cookies/ FALSE 0 othername othervalue
127.0.0.1 FALSE /smart_cookies/repo.git/info/ FALSE 0 name value
Expand All @@ -202,7 +199,8 @@ test_expect_success 'cookies stored in http.cookiefile when http.savecookies set
git config http.cookiefile cookies.txt &&
git config http.savecookies true &&
git ls-remote $HTTPD_URL/smart_cookies/repo.git master &&
test_cmp expect_cookies.txt cookies.txt
tail -3 cookies.txt > cookies_tail.txt
test_cmp expect_cookies.txt cookies_tail.txt
'

test -n "$GIT_TEST_LONG" && test_set_prereq EXPENSIVE
Expand Down

0 comments on commit 580cf0a

Please sign in to comment.