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) 2007-2012 Canonical Ltd
|
2999.1.4
by Ian Clatworthy
more review tweaks including commit of blackbox tests |
2 |
# -*- coding: utf-8 -*-
|
3 |
#
|
|
4 |
# This program is free software; you can redistribute it and/or modify
|
|
5 |
# it under the terms of the GNU General Public License as published by
|
|
6 |
# the Free Software Foundation; either version 2 of the License, or
|
|
7 |
# (at your option) any later version.
|
|
8 |
#
|
|
9 |
# This program is distributed in the hope that it will be useful,
|
|
10 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
11 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
12 |
# GNU General Public License for more details.
|
|
13 |
#
|
|
14 |
# You should have received a copy of the GNU General Public License
|
|
15 |
# along with this program; if not, write to the Free Software
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
16 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
2999.1.4
by Ian Clatworthy
more review tweaks including commit of blackbox tests |
17 |
|
18 |
||
19 |
"""Tests for the switch command of bzr."""
|
|
20 |
||
21 |
import os |
|
22 |
||
5268.8.12
by Jelmer Vernooij
support creating new colocated branches from 'bzr switch -b'. |
23 |
from bzrlib.bzrdir import BzrDir |
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
24 |
from bzrlib import ( |
25 |
osutils, |
|
5816.6.11
by A. S. Budden
Refactored assertParentCorrect to check the full path. |
26 |
urlutils, |
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
27 |
branch, |
28 |
)
|
|
3565.6.7
by Marius Kruger
* checkouts now use master nick when no explicit nick is set. |
29 |
from bzrlib.workingtree import WorkingTree |
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
30 |
from bzrlib.tests import ( |
31 |
TestCaseWithTransport, |
|
32 |
script, |
|
33 |
)
|
|
5268.8.18
by Jelmer Vernooij
Use UnicodeFilenameFeature. |
34 |
from bzrlib.tests.features import UnicodeFilenameFeature |
4879.2.1
by Neil Martinsen-Burrell
switch should use directory services when creating a branch |
35 |
from bzrlib.directory_service import directories |
2999.1.4
by Ian Clatworthy
more review tweaks including commit of blackbox tests |
36 |
|
6366.1.5
by Jelmer Vernooij
add test for hpss call / connection count of 'bzr switch'. |
37 |
from bzrlib.tests.matchers import ContainsNoVfsCalls |
38 |
||
5816.6.15
by A. S. Budden
Added erroneously removed blank line. |
39 |
|
5283.4.5
by Martin Pool
Update remaining subclasses of ExternalBase |
40 |
class TestSwitch(TestCaseWithTransport): |
2999.1.4
by Ian Clatworthy
more review tweaks including commit of blackbox tests |
41 |
|
3984.5.14
by Daniel Watkins
Extracted common setup code. |
42 |
def _create_sample_tree(self): |
43 |
tree = self.make_branch_and_tree('branch-1') |
|
44 |
self.build_tree(['branch-1/file-1', 'branch-1/file-2']) |
|
45 |
tree.add('file-1') |
|
46 |
tree.commit('rev1') |
|
47 |
tree.add('file-2') |
|
48 |
tree.commit('rev2') |
|
49 |
return tree |
|
50 |
||
2999.1.4
by Ian Clatworthy
more review tweaks including commit of blackbox tests |
51 |
def test_switch_up_to_date_light_checkout(self): |
52 |
self.make_branch_and_tree('branch') |
|
53 |
self.run_bzr('branch branch branch2') |
|
54 |
self.run_bzr('checkout --lightweight branch checkout') |
|
55 |
os.chdir('checkout') |
|
56 |
out, err = self.run_bzr('switch ../branch2') |
|
57 |
self.assertContainsRe(err, 'Tree is up to date at revision 0.\n') |
|
58 |
self.assertContainsRe(err, 'Switched to branch: .*/branch2.\n') |
|
59 |
self.assertEqual('', out) |
|
60 |
||
61 |
def test_switch_out_of_date_light_checkout(self): |
|
62 |
self.make_branch_and_tree('branch') |
|
63 |
self.run_bzr('branch branch branch2') |
|
64 |
self.build_tree(['branch2/file']) |
|
65 |
self.run_bzr('add branch2/file') |
|
66 |
self.run_bzr('commit -m add-file branch2') |
|
67 |
self.run_bzr('checkout --lightweight branch checkout') |
|
68 |
os.chdir('checkout') |
|
69 |
out, err = self.run_bzr('switch ../branch2') |
|
70 |
#self.assertContainsRe(err, '\+N file')
|
|
71 |
self.assertContainsRe(err, 'Updated to revision 1.\n') |
|
72 |
self.assertContainsRe(err, 'Switched to branch: .*/branch2.\n') |
|
73 |
self.assertEqual('', out) |
|
3246.5.1
by Robert Collins
* ``bzr switch`` will attempt to find branches to switch to relative to the |
74 |
|
3565.6.7
by Marius Kruger
* checkouts now use master nick when no explicit nick is set. |
75 |
def _test_switch_nick(self, lightweight): |
76 |
"""Check that the nick gets switched too."""
|
|
77 |
tree1 = self.make_branch_and_tree('branch1') |
|
78 |
tree2 = self.make_branch_and_tree('branch2') |
|
79 |
tree2.pull(tree1.branch) |
|
80 |
checkout = tree1.branch.create_checkout('checkout', |
|
81 |
lightweight=lightweight) |
|
82 |
self.assertEqual(checkout.branch.nick, tree1.branch.nick) |
|
83 |
self.assertEqual(checkout.branch.get_config().has_explicit_nickname(), |
|
84 |
False) |
|
85 |
self.run_bzr('switch branch2', working_dir='checkout') |
|
86 |
||
87 |
# we need to get the tree again, otherwise we don't get the new branch
|
|
88 |
checkout = WorkingTree.open('checkout') |
|
89 |
self.assertEqual(checkout.branch.nick, tree2.branch.nick) |
|
90 |
self.assertEqual(checkout.branch.get_config().has_explicit_nickname(), |
|
91 |
False) |
|
92 |
||
3565.6.1
by Marius Kruger
Let 'bzr switch' update the nick too. |
93 |
def test_switch_nick(self): |
3565.6.7
by Marius Kruger
* checkouts now use master nick when no explicit nick is set. |
94 |
self._test_switch_nick(lightweight=False) |
95 |
||
96 |
def test_switch_nick_lightweight(self): |
|
97 |
self._test_switch_nick(lightweight=True) |
|
98 |
||
99 |
def _test_switch_explicit_nick(self, lightweight): |
|
3565.6.1
by Marius Kruger
Let 'bzr switch' update the nick too. |
100 |
"""Check that the nick gets switched too."""
|
101 |
tree1 = self.make_branch_and_tree('branch1') |
|
102 |
tree2 = self.make_branch_and_tree('branch2') |
|
103 |
tree2.pull(tree1.branch) |
|
3565.6.7
by Marius Kruger
* checkouts now use master nick when no explicit nick is set. |
104 |
checkout = tree1.branch.create_checkout('checkout', |
105 |
lightweight=lightweight) |
|
106 |
self.assertEqual(checkout.branch.nick, tree1.branch.nick) |
|
107 |
checkout.branch.nick = "explicit_nick" |
|
108 |
self.assertEqual(checkout.branch.nick, "explicit_nick") |
|
109 |
self.assertEqual(checkout.branch.get_config()._get_explicit_nickname(), |
|
110 |
"explicit_nick") |
|
111 |
self.run_bzr('switch branch2', working_dir='checkout') |
|
112 |
||
113 |
# we need to get the tree again, otherwise we don't get the new branch
|
|
114 |
checkout = WorkingTree.open('checkout') |
|
115 |
self.assertEqual(checkout.branch.nick, tree2.branch.nick) |
|
116 |
self.assertEqual(checkout.branch.get_config()._get_explicit_nickname(), |
|
117 |
tree2.branch.nick) |
|
118 |
||
119 |
def test_switch_explicit_nick(self): |
|
120 |
self._test_switch_explicit_nick(lightweight=False) |
|
121 |
||
122 |
def test_switch_explicit_nick_lightweight(self): |
|
123 |
self._test_switch_explicit_nick(lightweight=True) |
|
3565.6.1
by Marius Kruger
Let 'bzr switch' update the nick too. |
124 |
|
3246.5.1
by Robert Collins
* ``bzr switch`` will attempt to find branches to switch to relative to the |
125 |
def test_switch_finds_relative_branch(self): |
3565.6.1
by Marius Kruger
Let 'bzr switch' update the nick too. |
126 |
"""Switch will find 'foo' relative to the branch the checkout is of."""
|
3246.5.1
by Robert Collins
* ``bzr switch`` will attempt to find branches to switch to relative to the |
127 |
self.build_tree(['repo/']) |
128 |
tree1 = self.make_branch_and_tree('repo/brancha') |
|
129 |
tree1.commit('foo') |
|
130 |
tree2 = self.make_branch_and_tree('repo/branchb') |
|
131 |
tree2.pull(tree1.branch) |
|
132 |
branchb_id = tree2.commit('bar') |
|
133 |
checkout = tree1.branch.create_checkout('checkout', lightweight=True) |
|
134 |
self.run_bzr(['switch', 'branchb'], working_dir='checkout') |
|
135 |
self.assertEqual(branchb_id, checkout.last_revision()) |
|
136 |
checkout = checkout.bzrdir.open_workingtree() |
|
137 |
self.assertEqual(tree2.branch.base, checkout.branch.base) |
|
3602.3.1
by Adrian Wilkins
Test that `bzr switch` finds the sibling of the bound branch of heavy checkout. |
138 |
|
139 |
def test_switch_finds_relative_bound_branch(self): |
|
3602.3.4
by Adrian Wilkins
Improved comments and documentation |
140 |
"""Using switch on a heavy checkout should find master sibling
|
141 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
142 |
The behaviour of lighweight and heavy checkouts should be
|
5816.6.1
by A. S. Budden
Test harness to stimulate Bug #513709: parent being set incorrectly in switch --create-branch. |
143 |
consistent when using the convenient "switch to sibling" feature
|
3602.3.4
by Adrian Wilkins
Improved comments and documentation |
144 |
Both should switch to a sibling of the branch
|
145 |
they are bound to, and not a sibling of themself"""
|
|
146 |
||
3602.3.1
by Adrian Wilkins
Test that `bzr switch` finds the sibling of the bound branch of heavy checkout. |
147 |
self.build_tree(['repo/', |
148 |
'heavyco/']) |
|
149 |
tree1 = self.make_branch_and_tree('repo/brancha') |
|
150 |
tree1.commit('foo') |
|
151 |
tree2 = self.make_branch_and_tree('repo/branchb') |
|
152 |
tree2.pull(tree1.branch) |
|
153 |
branchb_id = tree2.commit('bar') |
|
154 |
checkout = tree1.branch.create_checkout('heavyco/a', lightweight=False) |
|
155 |
self.run_bzr(['switch', 'branchb'], working_dir='heavyco/a') |
|
6404.6.6
by Vincent Ladeuil
Use idioms coherently and add comments to make their purpose clearer. |
156 |
# Refresh checkout as 'switch' modified it
|
157 |
checkout = checkout.bzrdir.open_workingtree() |
|
3602.3.1
by Adrian Wilkins
Test that `bzr switch` finds the sibling of the bound branch of heavy checkout. |
158 |
self.assertEqual(branchb_id, checkout.last_revision()) |
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. |
159 |
self.assertEqual(tree2.branch.base, |
160 |
checkout.branch.get_bound_location()) |
|
4354.2.2
by Aaron Bentley
Enable switch --force for lightweight checkouts after moves. |
161 |
|
5268.8.15
by Jelmer Vernooij
Fix switching to unicode branch names. |
162 |
def test_switch_finds_relative_unicode_branch(self): |
163 |
"""Switch will find 'foo' relative to the branch the checkout is of."""
|
|
5268.8.18
by Jelmer Vernooij
Use UnicodeFilenameFeature. |
164 |
self.requireFeature(UnicodeFilenameFeature) |
5268.8.15
by Jelmer Vernooij
Fix switching to unicode branch names. |
165 |
self.build_tree(['repo/']) |
166 |
tree1 = self.make_branch_and_tree('repo/brancha') |
|
167 |
tree1.commit('foo') |
|
168 |
tree2 = self.make_branch_and_tree(u'repo/branch\xe9') |
|
169 |
tree2.pull(tree1.branch) |
|
170 |
branchb_id = tree2.commit('bar') |
|
171 |
checkout = tree1.branch.create_checkout('checkout', lightweight=True) |
|
172 |
self.run_bzr(['switch', u'branch\xe9'], working_dir='checkout') |
|
173 |
self.assertEqual(branchb_id, checkout.last_revision()) |
|
174 |
checkout = checkout.bzrdir.open_workingtree() |
|
175 |
self.assertEqual(tree2.branch.base, checkout.branch.base) |
|
176 |
||
3984.5.2
by Daniel Watkins
Added blackbox test. |
177 |
def test_switch_revision(self): |
3984.5.14
by Daniel Watkins
Extracted common setup code. |
178 |
tree = self._create_sample_tree() |
3984.5.2
by Daniel Watkins
Added blackbox test. |
179 |
checkout = tree.branch.create_checkout('checkout', lightweight=True) |
180 |
self.run_bzr(['switch', 'branch-1', '-r1'], working_dir='checkout') |
|
5784.1.3
by Martin Pool
Switch away from using failUnlessExists and failIfExists |
181 |
self.assertPathExists('checkout/file-1') |
182 |
self.assertPathDoesNotExist('checkout/file-2') |
|
3984.5.9
by Daniel Watkins
Added test for allowing only revisions to be passed to switch. |
183 |
|
6437.7.5
by Jelmer Vernooij
"bzr switch -b" in a standalone tree will now create a colocated branch. |
184 |
def test_switch_into_colocated(self): |
185 |
# Create a new colocated branch from an existing non-colocated branch.
|
|
186 |
tree = self.make_branch_and_tree('.', format='development-colo') |
|
187 |
self.build_tree(['file-1', 'file-2']) |
|
188 |
tree.add('file-1') |
|
189 |
revid1 = tree.commit('rev1') |
|
190 |
tree.add('file-2') |
|
191 |
revid2 = tree.commit('rev2') |
|
192 |
self.run_bzr(['switch', '-b', 'anotherbranch']) |
|
193 |
self.assertEquals( |
|
194 |
['', 'anotherbranch'], |
|
195 |
tree.branch.bzrdir.get_branches().keys()) |
|
196 |
||
6437.7.6
by Jelmer Vernooij
Add test for --force argument used in standalone branch. |
197 |
def test_switch_into_unrelated_colocated(self): |
198 |
# Create a new colocated branch from an existing non-colocated branch.
|
|
199 |
tree = self.make_branch_and_tree('.', format='development-colo') |
|
200 |
self.build_tree(['file-1', 'file-2']) |
|
201 |
tree.add('file-1') |
|
202 |
revid1 = tree.commit('rev1') |
|
203 |
tree.add('file-2') |
|
204 |
revid2 = tree.commit('rev2') |
|
205 |
tree.bzrdir.create_branch(name='foo') |
|
206 |
self.run_bzr_error(['Cannot switch a branch, only a checkout.'], |
|
207 |
'switch foo') |
|
208 |
self.run_bzr(['switch', '--force', 'foo']) |
|
209 |
||
5268.8.12
by Jelmer Vernooij
support creating new colocated branches from 'bzr switch -b'. |
210 |
def test_switch_existing_colocated(self): |
5268.8.6
by Jelmer Vernooij
More fixes to test. |
211 |
# Create a branch branch-1 that initially is a checkout of 'foo'
|
212 |
# Use switch to change it to 'anotherbranch'
|
|
213 |
repo = self.make_repository('branch-1', format='development-colo') |
|
214 |
target_branch = repo.bzrdir.create_branch(name='foo') |
|
6437.7.3
by Jelmer Vernooij
Use ControlDir.set_branch_reference. |
215 |
repo.bzrdir.set_branch_reference(target_branch) |
5268.8.6
by Jelmer Vernooij
More fixes to test. |
216 |
tree = repo.bzrdir.create_workingtree() |
5268.8.5
by Jelmer Vernooij
Add tests for switching to colocated branches. |
217 |
self.build_tree(['branch-1/file-1', 'branch-1/file-2']) |
218 |
tree.add('file-1') |
|
219 |
revid1 = tree.commit('rev1') |
|
220 |
tree.add('file-2') |
|
221 |
revid2 = tree.commit('rev2') |
|
5268.8.6
by Jelmer Vernooij
More fixes to test. |
222 |
otherbranch = tree.bzrdir.create_branch(name='anotherbranch') |
5268.8.5
by Jelmer Vernooij
Add tests for switching to colocated branches. |
223 |
otherbranch.generate_revision_history(revid1) |
224 |
self.run_bzr(['switch', 'anotherbranch'], working_dir='branch-1') |
|
5268.8.10
by Jelmer Vernooij
bzr switch can now switch to colocated branches. |
225 |
tree = WorkingTree.open("branch-1") |
5268.8.5
by Jelmer Vernooij
Add tests for switching to colocated branches. |
226 |
self.assertEquals(tree.last_revision(), revid1) |
5268.8.10
by Jelmer Vernooij
bzr switch can now switch to colocated branches. |
227 |
self.assertEquals(tree.branch.control_url, otherbranch.control_url) |
5268.8.5
by Jelmer Vernooij
Add tests for switching to colocated branches. |
228 |
|
5268.8.12
by Jelmer Vernooij
support creating new colocated branches from 'bzr switch -b'. |
229 |
def test_switch_new_colocated(self): |
230 |
# Create a branch branch-1 that initially is a checkout of 'foo'
|
|
231 |
# Use switch to create 'anotherbranch' which derives from that
|
|
232 |
repo = self.make_repository('branch-1', format='development-colo') |
|
233 |
target_branch = repo.bzrdir.create_branch(name='foo') |
|
6437.7.3
by Jelmer Vernooij
Use ControlDir.set_branch_reference. |
234 |
repo.bzrdir.set_branch_reference(target_branch) |
5268.8.12
by Jelmer Vernooij
support creating new colocated branches from 'bzr switch -b'. |
235 |
tree = repo.bzrdir.create_workingtree() |
236 |
self.build_tree(['branch-1/file-1', 'branch-1/file-2']) |
|
237 |
tree.add('file-1') |
|
238 |
revid1 = tree.commit('rev1') |
|
239 |
self.run_bzr(['switch', '-b', 'anotherbranch'], working_dir='branch-1') |
|
240 |
bzrdir = BzrDir.open("branch-1") |
|
241 |
self.assertEquals( |
|
242 |
set([b.name for b in bzrdir.list_branches()]), |
|
243 |
set(["foo", "anotherbranch"])) |
|
244 |
self.assertEquals(bzrdir.open_branch().name, "anotherbranch") |
|
245 |
self.assertEquals(bzrdir.open_branch().last_revision(), revid1) |
|
246 |
||
5268.8.17
by Jelmer Vernooij
Support switching to new unicode colocated branch. |
247 |
def test_switch_new_colocated_unicode(self): |
248 |
# Create a branch branch-1 that initially is a checkout of 'foo'
|
|
249 |
# Use switch to create 'branch\xe9' which derives from that
|
|
5268.8.18
by Jelmer Vernooij
Use UnicodeFilenameFeature. |
250 |
self.requireFeature(UnicodeFilenameFeature) |
5268.8.17
by Jelmer Vernooij
Support switching to new unicode colocated branch. |
251 |
repo = self.make_repository('branch-1', format='development-colo') |
252 |
target_branch = repo.bzrdir.create_branch(name='foo') |
|
6437.7.3
by Jelmer Vernooij
Use ControlDir.set_branch_reference. |
253 |
repo.bzrdir.set_branch_reference(target_branch) |
5268.8.17
by Jelmer Vernooij
Support switching to new unicode colocated branch. |
254 |
tree = repo.bzrdir.create_workingtree() |
255 |
self.build_tree(['branch-1/file-1', 'branch-1/file-2']) |
|
256 |
tree.add('file-1') |
|
257 |
revid1 = tree.commit('rev1') |
|
258 |
self.run_bzr(['switch', '-b', u'branch\xe9'], working_dir='branch-1') |
|
259 |
bzrdir = BzrDir.open("branch-1") |
|
260 |
self.assertEquals( |
|
261 |
set([b.name for b in bzrdir.list_branches()]), |
|
262 |
set(["foo", u"branch\xe9"])) |
|
263 |
self.assertEquals(bzrdir.open_branch().name, u"branch\xe9") |
|
264 |
self.assertEquals(bzrdir.open_branch().last_revision(), revid1) |
|
265 |
||
3984.5.9
by Daniel Watkins
Added test for allowing only revisions to be passed to switch. |
266 |
def test_switch_only_revision(self): |
3984.5.14
by Daniel Watkins
Extracted common setup code. |
267 |
tree = self._create_sample_tree() |
3984.5.9
by Daniel Watkins
Added test for allowing only revisions to be passed to switch. |
268 |
checkout = tree.branch.create_checkout('checkout', lightweight=True) |
5784.1.3
by Martin Pool
Switch away from using failUnlessExists and failIfExists |
269 |
self.assertPathExists('checkout/file-1') |
270 |
self.assertPathExists('checkout/file-2') |
|
3984.5.9
by Daniel Watkins
Added test for allowing only revisions to be passed to switch. |
271 |
self.run_bzr(['switch', '-r1'], working_dir='checkout') |
5784.1.3
by Martin Pool
Switch away from using failUnlessExists and failIfExists |
272 |
self.assertPathExists('checkout/file-1') |
273 |
self.assertPathDoesNotExist('checkout/file-2') |
|
3984.5.15
by Daniel Watkins
Add to test to ensure that we don't accept a range of revisions. |
274 |
# Check that we don't accept a range
|
3984.5.17
by Daniel Watkins
Fixed incorrect call of run_bzr_error. |
275 |
self.run_bzr_error( |
276 |
['bzr switch --revision takes exactly one revision identifier'], |
|
277 |
['switch', '-r0..2'], working_dir='checkout') |
|
3984.5.19
by Andrew Bennetts
Merge lp:bzr, resolving conflicts. |
278 |
|
4354.2.2
by Aaron Bentley
Enable switch --force for lightweight checkouts after moves. |
279 |
def prepare_lightweight_switch(self): |
280 |
branch = self.make_branch('branch') |
|
281 |
branch.create_checkout('tree', lightweight=True) |
|
5186.2.2
by Martin Pool
wrap os.rename to insert the source and destination filenames in any exception that may be raised |
282 |
osutils.rename('branch', 'branch1') |
4354.2.2
by Aaron Bentley
Enable switch --force for lightweight checkouts after moves. |
283 |
|
284 |
def test_switch_lightweight_after_branch_moved(self): |
|
285 |
self.prepare_lightweight_switch() |
|
286 |
self.run_bzr('switch --force ../branch1', working_dir='tree') |
|
287 |
branch_location = WorkingTree.open('tree').branch.base |
|
288 |
self.assertEndsWith(branch_location, 'branch1/') |
|
289 |
||
290 |
def test_switch_lightweight_after_branch_moved_relative(self): |
|
291 |
self.prepare_lightweight_switch() |
|
292 |
self.run_bzr('switch --force branch1', working_dir='tree') |
|
293 |
branch_location = WorkingTree.open('tree').branch.base |
|
294 |
self.assertEndsWith(branch_location, 'branch1/') |
|
4520.1.1
by John Arbash Meinel
'bzr switch -b' can now be used to create the branch while you switch to it. |
295 |
|
296 |
def test_create_branch_no_branch(self): |
|
297 |
self.prepare_lightweight_switch() |
|
298 |
self.run_bzr_error(['cannot create branch without source branch'], |
|
299 |
'switch --create-branch ../branch2', working_dir='tree') |
|
300 |
||
301 |
def test_create_branch(self): |
|
302 |
branch = self.make_branch('branch') |
|
303 |
tree = branch.create_checkout('tree', lightweight=True) |
|
304 |
tree.commit('one', rev_id='rev-1') |
|
305 |
self.run_bzr('switch --create-branch ../branch2', working_dir='tree') |
|
306 |
tree = WorkingTree.open('tree') |
|
307 |
self.assertEndsWith(tree.branch.base, '/branch2/') |
|
308 |
||
309 |
def test_create_branch_local(self): |
|
310 |
branch = self.make_branch('branch') |
|
311 |
tree = branch.create_checkout('tree', lightweight=True) |
|
312 |
tree.commit('one', rev_id='rev-1') |
|
313 |
self.run_bzr('switch --create-branch branch2', working_dir='tree') |
|
314 |
tree = WorkingTree.open('tree') |
|
315 |
# The new branch should have been created at the same level as
|
|
316 |
# 'branch', because we did not have a '/' segment
|
|
317 |
self.assertEqual(branch.base[:-1] + '2/', tree.branch.base) |
|
318 |
||
319 |
def test_create_branch_short_name(self): |
|
320 |
branch = self.make_branch('branch') |
|
321 |
tree = branch.create_checkout('tree', lightweight=True) |
|
322 |
tree.commit('one', rev_id='rev-1') |
|
323 |
self.run_bzr('switch -b branch2', working_dir='tree') |
|
324 |
tree = WorkingTree.open('tree') |
|
325 |
# The new branch should have been created at the same level as
|
|
326 |
# 'branch', because we did not have a '/' segment
|
|
327 |
self.assertEqual(branch.base[:-1] + '2/', tree.branch.base) |
|
4879.2.1
by Neil Martinsen-Burrell
switch should use directory services when creating a branch |
328 |
|
329 |
def test_create_branch_directory_services(self): |
|
330 |
branch = self.make_branch('branch') |
|
331 |
tree = branch.create_checkout('tree', lightweight=True) |
|
332 |
class FooLookup(object): |
|
333 |
def look_up(self, name, url): |
|
334 |
return 'foo-'+name |
|
335 |
directories.register('foo:', FooLookup, 'Create branches named foo-') |
|
4879.2.2
by Neil Martinsen-Burrell
add test cleanup per JAMs review |
336 |
self.addCleanup(directories.remove, 'foo:') |
4879.2.1
by Neil Martinsen-Burrell
switch should use directory services when creating a branch |
337 |
self.run_bzr('switch -b foo:branch2', working_dir='tree') |
338 |
tree = WorkingTree.open('tree') |
|
339 |
self.assertEndsWith(tree.branch.base, 'foo-branch2/') |
|
5107.3.6
by Marco Pantaleoni
Documented behaviour of 'post_branch_init' for lightweight checkouts. |
340 |
|
341 |
def test_switch_with_post_switch_hook(self): |
|
342 |
from bzrlib import branch as _mod_branch |
|
343 |
calls = [] |
|
344 |
_mod_branch.Branch.hooks.install_named_hook('post_switch', |
|
345 |
calls.append, None) |
|
346 |
self.make_branch_and_tree('branch') |
|
347 |
self.run_bzr('branch branch branch2') |
|
348 |
self.run_bzr('checkout branch checkout') |
|
349 |
os.chdir('checkout') |
|
350 |
self.assertLength(0, calls) |
|
351 |
out, err = self.run_bzr('switch ../branch2') |
|
352 |
self.assertLength(1, calls) |
|
353 |
||
354 |
def test_switch_lightweight_co_with_post_switch_hook(self): |
|
355 |
from bzrlib import branch as _mod_branch |
|
356 |
calls = [] |
|
357 |
_mod_branch.Branch.hooks.install_named_hook('post_switch', |
|
358 |
calls.append, None) |
|
359 |
self.make_branch_and_tree('branch') |
|
360 |
self.run_bzr('branch branch branch2') |
|
361 |
self.run_bzr('checkout --lightweight branch checkout') |
|
362 |
os.chdir('checkout') |
|
363 |
self.assertLength(0, calls) |
|
364 |
out, err = self.run_bzr('switch ../branch2') |
|
365 |
self.assertLength(1, calls) |
|
5171.3.13
by Martin von Gagern
Add --directory option to 7 more commands. |
366 |
|
367 |
def test_switch_lightweight_directory(self): |
|
368 |
"""Test --directory option"""
|
|
369 |
||
370 |
# create a source branch
|
|
371 |
a_tree = self.make_branch_and_tree('a') |
|
372 |
self.build_tree_contents([('a/a', 'initial\n')]) |
|
373 |
a_tree.add('a') |
|
374 |
a_tree.commit(message='initial') |
|
375 |
||
376 |
# clone and add a differing revision
|
|
377 |
b_tree = a_tree.bzrdir.sprout('b').open_workingtree() |
|
378 |
self.build_tree_contents([('b/a', 'initial\nmore\n')]) |
|
379 |
b_tree.commit(message='more') |
|
380 |
||
381 |
self.run_bzr('checkout --lightweight a checkout') |
|
382 |
self.run_bzr('switch --directory checkout b') |
|
383 |
self.assertFileEqual('initial\nmore\n', 'checkout/a') |
|
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
384 |
|
6015.7.2
by John Arbash Meinel
Bug #812285. Add an effort test to show that things have improved. |
385 |
|
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
386 |
class TestSwitchParentLocationBase(TestCaseWithTransport): |
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
387 |
|
388 |
def setUp(self): |
|
389 |
"""Set up a repository and branch ready for testing."""
|
|
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
390 |
super(TestSwitchParentLocationBase, self).setUp() |
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
391 |
self.script_runner = script.ScriptRunner() |
392 |
self.script_runner.run_script(self, ''' |
|
393 |
$ bzr init-repo --no-trees repo
|
|
394 |
Shared repository...
|
|
395 |
Location:
|
|
396 |
shared repository: repo
|
|
397 |
$ bzr init repo/trunk
|
|
398 |
Created a repository branch...
|
|
399 |
Using shared repository: ...
|
|
400 |
''') |
|
401 |
||
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
402 |
def assertParent(self, expected_parent, branch): |
5816.6.11
by A. S. Budden
Refactored assertParentCorrect to check the full path. |
403 |
"""Verify that the parent is not None and is set correctly."""
|
404 |
actual_parent = branch.get_parent() |
|
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
405 |
self.assertIsSameRealPath(urlutils.local_path_to_url(expected_parent), |
406 |
branch.get_parent()) |
|
407 |
||
408 |
||
409 |
class TestSwitchParentLocation(TestSwitchParentLocationBase): |
|
410 |
||
411 |
def _checkout_and_switch(self, option=''): |
|
5816.6.8
by A. S. Budden
Refactored to use common generation code for lightweight and heavyweight. |
412 |
self.script_runner.run_script(self, ''' |
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
413 |
$ bzr checkout %(option)s repo/trunk checkout |
414 |
$ cd checkout
|
|
415 |
$ bzr switch --create-branch switched
|
|
5816.6.8
by A. S. Budden
Refactored to use common generation code for lightweight and heavyweight. |
416 |
2>Tree is up to date at revision 0.
|
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
417 |
2>Switched to branch:...switched...
|
5816.6.11
by A. S. Budden
Refactored assertParentCorrect to check the full path. |
418 |
$ cd ..
|
5816.6.9
by A. S. Budden
Use locals() instead of kwargs so that parameters are more explicit. |
419 |
''' % locals()) |
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
420 |
bound_branch = branch.Branch.open_containing('checkout')[0] |
421 |
master_branch = branch.Branch.open_containing('repo/switched')[0] |
|
5816.6.12
by A. S. Budden
Check parent branch of both the checkout (light or heavy) and the branch to which it is connected. |
422 |
return (bound_branch, master_branch) |
5816.6.8
by A. S. Budden
Refactored to use common generation code for lightweight and heavyweight. |
423 |
|
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
424 |
def test_switch_parent_lightweight(self): |
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
425 |
"""Lightweight checkout using bzr switch."""
|
426 |
bb, mb = self._checkout_and_switch(option='--lightweight') |
|
427 |
self.assertParent('repo/trunk', bb) |
|
428 |
self.assertParent('repo/trunk', mb) |
|
5816.6.7
by A. S. Budden
Moved test harnesses into test_switch and test_branch as these are the commands that are being tested. |
429 |
|
430 |
def test_switch_parent_heavyweight(self): |
|
5816.7.1
by Vincent Ladeuil
Remove duplication and simplify. |
431 |
"""Heavyweight checkout using bzr switch."""
|
432 |
bb, mb = self._checkout_and_switch() |
|
433 |
self.assertParent('repo/trunk', bb) |
|
434 |
self.assertParent('repo/trunk', mb) |
|
5816.6.11
by A. S. Budden
Refactored assertParentCorrect to check the full path. |
435 |
|
6015.7.2
by John Arbash Meinel
Bug #812285. Add an effort test to show that things have improved. |
436 |
|
437 |
class TestSwitchDoesntOpenMasterBranch(TestCaseWithTransport): |
|
438 |
# See https://bugs.launchpad.net/bzr/+bug/812285
|
|
439 |
# "bzr switch --create-branch" can point the new branch's parent to the
|
|
440 |
# master branch, but it doesn't have to open it to do so.
|
|
441 |
||
442 |
def test_switch_create_doesnt_open_master_branch(self): |
|
443 |
master = self.make_branch_and_tree('master') |
|
444 |
master.commit('one') |
|
445 |
# Note: not a lightweight checkout
|
|
446 |
checkout = master.branch.create_checkout('checkout') |
|
447 |
opened = [] |
|
448 |
def open_hook(branch): |
|
449 |
# Just append the final directory of the branch
|
|
450 |
name = branch.base.rstrip('/').rsplit('/', 1)[1] |
|
451 |
opened.append(name) |
|
452 |
branch.Branch.hooks.install_named_hook('open', open_hook, |
|
453 |
'open_hook_logger') |
|
454 |
self.run_bzr('switch --create-branch -d checkout feature') |
|
455 |
# We only open the master branch 1 time.
|
|
456 |
# This test should be cleaner to write, but see bug:
|
|
457 |
# https://bugs.launchpad.net/bzr/+bug/812295
|
|
458 |
self.assertEqual(1, opened.count('master')) |
|
6366.1.5
by Jelmer Vernooij
add test for hpss call / connection count of 'bzr switch'. |
459 |
|
460 |
||
461 |
class TestSmartServerSwitch(TestCaseWithTransport): |
|
462 |
||
463 |
def test_switch_lightweight(self): |
|
464 |
self.setup_smart_server_with_call_log() |
|
465 |
t = self.make_branch_and_tree('from') |
|
466 |
for count in range(9): |
|
467 |
t.commit(message='commit %d' % count) |
|
468 |
out, err = self.run_bzr(['checkout', '--lightweight', self.get_url('from'), |
|
469 |
'target']) |
|
470 |
self.reset_smart_call_log() |
|
471 |
self.run_bzr(['switch', self.get_url('from')], working_dir='target') |
|
472 |
# This figure represent the amount of work to perform this use case. It
|
|
473 |
# is entirely ok to reduce this number if a test fails due to rpc_count
|
|
474 |
# being too low. If rpc_count increases, more network roundtrips have
|
|
475 |
# become necessary for this use case. Please do not adjust this number
|
|
476 |
# upwards without agreement from bzr's network support maintainers.
|
|
6366.1.6
by Jelmer Vernooij
Merge bzr.dev. |
477 |
self.assertLength(24, self.hpss_calls) |
478 |
self.assertLength(5, self.hpss_connections) |
|
6366.1.5
by Jelmer Vernooij
add test for hpss call / connection count of 'bzr switch'. |
479 |
self.assertThat(self.hpss_calls, ContainsNoVfsCalls) |