Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(84)

Side by Side Diff: tests/push-basic.sh

Issue 1160273005: Fix push_basic.sh (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env bash 1 #!/usr/bin/env bash
2 2
3 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 3 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 set -e 7 set -e
8 8
9 . ./test-lib.sh 9 . ./test-lib.sh
10 10
(...skipping 26 matching lines...) Expand all
37 "$GIT_CL_STATUS | grep -q 'Issue number'" 37 "$GIT_CL_STATUS | grep -q 'Issue number'"
38 38
39 # Push a description to this URL. 39 # Push a description to this URL.
40 URL=$($GIT_CL_STATUS | sed -ne '/Issue number/s/[^(]*(\(.*\))/\1/p') 40 URL=$($GIT_CL_STATUS | sed -ne '/Issue number/s/[^(]*(\(.*\))/\1/p')
41 curl --cookie dev_appserver_login="test@example.com:False" \ 41 curl --cookie dev_appserver_login="test@example.com:False" \
42 --data-urlencode subject="test" \ 42 --data-urlencode subject="test" \
43 --data-urlencode description="foo-quux" \ 43 --data-urlencode description="foo-quux" \
44 --data-urlencode xsrf_token="$(print_xsrf_token)" \ 44 --data-urlencode xsrf_token="$(print_xsrf_token)" \
45 $URL/edit 45 $URL/edit
46 46
47 API=$(echo $URL | sed -e 's/\([0-9]\+\)$/api\/\1/')
47 test_expect_success "Base URL contains branch name" \ 48 test_expect_success "Base URL contains branch name" \
48 "curl -s $($GIT_CL_STATUS --field=url) | grep 'URL:[[:space:]]*[^<]' | gre p -q '@master'" 49 "curl -s $API | python -mjson.tool | grep base_url | grep -q '@master'"
49 50
50 test_expect_success "git-cl land ok" \ 51 test_expect_success "git-cl land ok" \
51 "$GIT_CL land -f --no-oauth2" 52 "$GIT_CL land -f --no-oauth2"
52 53
53 git checkout -q master > /dev/null 2>&1 54 git checkout -q master > /dev/null 2>&1
54 git pull -q > /dev/null 2>&1 55 git pull -q > /dev/null 2>&1
55 56
56 test_expect_success "committed code has proper description" \ 57 test_expect_success "committed code has proper description" \
57 "git show | grep -q 'foo-quux'" 58 "git show | grep -q 'foo-quux'"
58 59
59 test_expect_success "issue no longer has a branch" \ 60 test_expect_success "issue no longer has a branch" \
60 "$GIT_CL_STATUS | grep -q 'work : None'" 61 "$GIT_CL_STATUS | grep -q 'work : None'"
61 62
62 cd $GITREPO_PATH 63 cd $GITREPO_PATH
63 test_expect_success "upstream repo has our commit" \ 64 test_expect_success "upstream repo has our commit" \
64 "git log master 2>/dev/null | grep -q 'foo-quux'" 65 "git log master 2>/dev/null | grep -q 'foo-quux'"
65 ) 66 )
66 SUCCESS=$? 67 SUCCESS=$?
67 68
68 cleanup 69 cleanup
69 70
70 if [ $SUCCESS == 0 ]; then 71 if [ $SUCCESS == 0 ]; then
71 echo PASS 72 echo PASS
72 fi 73 fi
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698