6404.6.1
by Vincent Ladeuil
Tests passing for a first rough version of a cached branch config store. The changes here are too invasive and several parallel proposals have been made. |
1 |
# Copyright (C) 2009, 2010, 2012 Canonical Ltd
|
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
2 |
#
|
3 |
# This program is free software; you can redistribute it and/or modify
|
|
4 |
# it under the terms of the GNU General Public License as published by
|
|
5 |
# the Free Software Foundation; either version 2 of the License, or
|
|
6 |
# (at your option) any later version.
|
|
7 |
#
|
|
8 |
# This program is distributed in the hope that it will be useful,
|
|
9 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
10 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
11 |
# GNU General Public License for more details.
|
|
12 |
#
|
|
13 |
# You should have received a copy of the GNU General Public License
|
|
14 |
# along with this program; if not, write to the Free Software
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
15 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
16 |
|
17 |
"""Tests for the launchpad-open command."""
|
|
18 |
||
19 |
from bzrlib.tests import TestCaseWithTransport |
|
20 |
||
21 |
||
22 |
class TestLaunchpadOpen(TestCaseWithTransport): |
|
23 |
||
4867.2.3
by Vincent Ladeuil
Fixed as per John's hawk eye review. |
24 |
def run_open(self, location, retcode=0, working_dir='.'): |
25 |
out, err = self.run_bzr(['launchpad-open', '--dry-run', location], |
|
26 |
retcode=retcode, |
|
27 |
working_dir=working_dir) |
|
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
28 |
return err.splitlines() |
29 |
||
30 |
def test_non_branch(self): |
|
4031.2.4
by Jonathan Lange
Improve some comments. |
31 |
# If given a branch with no public or push locations, lp-open will try
|
32 |
# to guess the Launchpad page for the given URL / path. If it cannot
|
|
33 |
# find one, it will raise an error.
|
|
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
34 |
self.assertEqual( |
4031.2.8
by Jonathan Lange
Say "registered on", not "hosted on". |
35 |
['bzr: ERROR: . is not registered on Launchpad.'], |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
36 |
self.run_open('.', retcode=3)) |
37 |
||
4031.2.1
by Jonathan Lange
Rename tests in anticipation of adding more. |
38 |
def test_no_public_location_no_push_location(self): |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
39 |
self.make_branch('not-public') |
40 |
self.assertEqual( |
|
4031.2.8
by Jonathan Lange
Say "registered on", not "hosted on". |
41 |
['bzr: ERROR: not-public is not registered on Launchpad.'], |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
42 |
self.run_open('not-public', retcode=3)) |
43 |
||
44 |
def test_non_launchpad_branch(self): |
|
45 |
branch = self.make_branch('non-lp') |
|
46 |
url = 'http://example.com/non-lp' |
|
6404.6.7
by Vincent Ladeuil
Change set/remove to require a lock for the branch config files. |
47 |
branch.set_public_branch(url) |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
48 |
self.assertEqual( |
4031.2.8
by Jonathan Lange
Say "registered on", not "hosted on". |
49 |
['bzr: ERROR: %s is not registered on Launchpad.' % url], |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
50 |
self.run_open('non-lp', retcode=3)) |
51 |
||
4031.2.1
by Jonathan Lange
Rename tests in anticipation of adding more. |
52 |
def test_launchpad_branch_with_public_location(self): |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
53 |
branch = self.make_branch('lp') |
6404.6.7
by Vincent Ladeuil
Change set/remove to require a lock for the branch config files. |
54 |
branch.set_public_branch('bzr+ssh://bazaar.launchpad.net/~foo/bar/baz') |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
55 |
self.assertEqual( |
5243.1.1
by Martin
Use the production server in the launchpad plugin rather than edge |
56 |
['Opening https://code.launchpad.net/~foo/bar/baz in web ' |
3955.3.10
by Jonathan Lange
Blackbox tests, forgot to add these earlier. |
57 |
'browser'], |
58 |
self.run_open('lp')) |
|
4031.2.2
by Jonathan Lange
Fall-back to push location. |
59 |
|
4031.2.10
by Jonathan Lange
Add one precedence test. |
60 |
def test_launchpad_branch_with_public_and_push_location(self): |
61 |
branch = self.make_branch('lp') |
|
6404.6.1
by Vincent Ladeuil
Tests passing for a first rough version of a cached branch config store. The changes here are too invasive and several parallel proposals have been made. |
62 |
branch.lock_write() |
6404.6.6
by Vincent Ladeuil
Use idioms coherently and add comments to make their purpose clearer. |
63 |
try: |
64 |
branch.set_public_branch( |
|
65 |
'bzr+ssh://bazaar.launchpad.net/~foo/bar/public') |
|
66 |
branch.set_push_location( |
|
67 |
'bzr+ssh://bazaar.launchpad.net/~foo/bar/push') |
|
68 |
finally: |
|
69 |
branch.unlock() |
|
4031.2.10
by Jonathan Lange
Add one precedence test. |
70 |
self.assertEqual( |
5243.1.1
by Martin
Use the production server in the launchpad plugin rather than edge |
71 |
['Opening https://code.launchpad.net/~foo/bar/public in web ' |
4031.2.10
by Jonathan Lange
Add one precedence test. |
72 |
'browser'], |
73 |
self.run_open('lp')) |
|
74 |
||
4031.2.2
by Jonathan Lange
Fall-back to push location. |
75 |
def test_launchpad_branch_with_no_public_but_with_push(self): |
76 |
# lp-open falls back to the push location if it cannot find a public
|
|
77 |
# location.
|
|
78 |
branch = self.make_branch('lp') |
|
6404.6.7
by Vincent Ladeuil
Change set/remove to require a lock for the branch config files. |
79 |
branch.set_push_location('bzr+ssh://bazaar.launchpad.net/~foo/bar/baz') |
4031.2.2
by Jonathan Lange
Fall-back to push location. |
80 |
self.assertEqual( |
5243.1.1
by Martin
Use the production server in the launchpad plugin rather than edge |
81 |
['Opening https://code.launchpad.net/~foo/bar/baz in web ' |
4031.2.2
by Jonathan Lange
Fall-back to push location. |
82 |
'browser'], |
83 |
self.run_open('lp')) |
|
4031.2.3
by Jonathan Lange
Try to open the page for the branch location on the command line if |
84 |
|
85 |
def test_launchpad_branch_with_no_public_no_push(self): |
|
86 |
# If lp-open is given a branch URL and that branch has no public
|
|
87 |
# location and no push location, then just try to look up the
|
|
88 |
# Launchpad page for that URL.
|
|
89 |
self.assertEqual( |
|
5243.1.1
by Martin
Use the production server in the launchpad plugin rather than edge |
90 |
['Opening https://code.launchpad.net/~foo/bar/baz in web ' |
4031.2.3
by Jonathan Lange
Try to open the page for the branch location on the command line if |
91 |
'browser'], |
92 |
self.run_open('bzr+ssh://bazaar.launchpad.net/~foo/bar/baz')) |
|
4867.2.1
by Neil Martinsen-Burrell
launchpad-open works from a subdirectory of a branch |
93 |
|
94 |
def test_launchpad_branch_subdirectory(self): |
|
95 |
# lp-open in a subdirectory of a registered branch should work
|
|
96 |
wt = self.make_branch_and_tree('lp') |
|
6404.6.7
by Vincent Ladeuil
Change set/remove to require a lock for the branch config files. |
97 |
wt.branch.set_push_location( |
98 |
'bzr+ssh://bazaar.launchpad.net/~foo/bar/baz') |
|
4867.2.1
by Neil Martinsen-Burrell
launchpad-open works from a subdirectory of a branch |
99 |
self.build_tree(['lp/a/']) |
100 |
self.assertEqual( |
|
5243.1.1
by Martin
Use the production server in the launchpad plugin rather than edge |
101 |
['Opening https://code.launchpad.net/~foo/bar/baz in web ' |
4867.2.1
by Neil Martinsen-Burrell
launchpad-open works from a subdirectory of a branch |
102 |
'browser'], |
4867.2.3
by Vincent Ladeuil
Fixed as per John's hawk eye review. |
103 |
self.run_open('.', working_dir='lp/a')) |