bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
| 
3549.1.1
by Martin Pool
 rename push --reference to --stacked-on  | 
1  | 
# Copyright (C) 2005, 2007, 2008 Canonical Ltd
 | 
| 
1887.1.1
by Adeodato Simó
 Do not separate paragraphs in the copyright statement with blank lines,  | 
2  | 
#
 | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
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.
 | 
|
| 
1887.1.1
by Adeodato Simó
 Do not separate paragraphs in the copyright statement with blank lines,  | 
7  | 
#
 | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
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.
 | 
|
| 
1887.1.1
by Adeodato Simó
 Do not separate paragraphs in the copyright statement with blank lines,  | 
12  | 
#
 | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
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
 | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
16  | 
|
17  | 
||
| 
1692.3.1
by Robert Collins
 Fix push to work with just a branch, no need for a working tree.  | 
18  | 
"""Black-box tests for bzr push."""
 | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
19  | 
|
| 
3066.3.2
by jml at canonical
 Add tests to check the handling of TooManyRedirections.  | 
20  | 
import re  | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
21  | 
|
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
22  | 
from bzrlib import (  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
23  | 
branch,  | 
24  | 
bzrdir,  | 
|
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
25  | 
errors,  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
26  | 
osutils,  | 
27  | 
tests,  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
28  | 
transport,  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
29  | 
uncommit,  | 
| 
2220.2.9
by Martin Pool
 Add specific tests for push -d and pull -d  | 
30  | 
urlutils,  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
31  | 
    workingtree
 | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
32  | 
    )
 | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
33  | 
from bzrlib.repofmt import knitrepo  | 
34  | 
from bzrlib.tests import http_server  | 
|
35  | 
from bzrlib.transport import memory  | 
|
36  | 
||
37  | 
||
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
38  | 
def load_tests(standard_tests, module, loader):  | 
39  | 
"""Multiply tests for the push command."""  | 
|
40  | 
result = loader.suiteClass()  | 
|
41  | 
||
42  | 
    # one for each king of change
 | 
|
43  | 
changes_tests, remaining_tests = tests.split_suite_by_condition(  | 
|
44  | 
standard_tests, tests.condition_isinstance((  | 
|
45  | 
TestPushStrictWithChanges,  | 
|
46  | 
                )))
 | 
|
47  | 
changes_scenarios = [  | 
|
48  | 
('uncommitted',  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
49  | 
dict(_changes_type= '_uncommitted_changes')),  | 
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
50  | 
('pending-merges',  | 
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
51  | 
dict(_changes_type= '_pending_merges')),  | 
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
52  | 
('out-of-sync-trees',  | 
53  | 
dict(_changes_type= '_out_of_sync_trees')),  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
54  | 
        ]
 | 
55  | 
tests.multiply_tests(changes_tests, changes_scenarios, result)  | 
|
56  | 
    # No parametrization for the remaining tests
 | 
|
57  | 
result.addTests(remaining_tests)  | 
|
58  | 
||
59  | 
return result  | 
|
60  | 
||
61  | 
||
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
62  | 
class TestPush(tests.TestCaseWithTransport):  | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
63  | 
|
| 
4017.2.3
by Robert Collins
 Review feedback.  | 
64  | 
def test_push_error_on_vfs_http(self):  | 
65  | 
""" pushing a branch to a HTTP server fails cleanly. """  | 
|
66  | 
        # the trunk is published on a web server
 | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
67  | 
self.transport_readonly_server = http_server.HttpServer  | 
| 
4017.2.3
by Robert Collins
 Review feedback.  | 
68  | 
self.make_branch('source')  | 
69  | 
public_url = self.get_readonly_url('target')  | 
|
70  | 
self.run_bzr_error(['http does not support mkdir'],  | 
|
71  | 
['push', public_url],  | 
|
72  | 
working_dir='source')  | 
|
73  | 
||
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
74  | 
def test_push_remember(self):  | 
75  | 
"""Push changes from one branch to another and test push location."""  | 
|
| 
1614.2.16
by Olaf Conradi
 Modified blackbox test cases to use bzrlib API.  | 
76  | 
transport = self.get_transport()  | 
77  | 
tree_a = self.make_branch_and_tree('branch_a')  | 
|
78  | 
branch_a = tree_a.branch  | 
|
79  | 
self.build_tree(['branch_a/a'])  | 
|
80  | 
tree_a.add('a')  | 
|
81  | 
tree_a.commit('commit a')  | 
|
| 
1666.1.4
by Robert Collins
 * 'Metadir' is now the default disk format. This improves behaviour in  | 
82  | 
tree_b = branch_a.bzrdir.sprout('branch_b').open_workingtree()  | 
83  | 
branch_b = tree_b.branch  | 
|
84  | 
tree_c = branch_a.bzrdir.sprout('branch_c').open_workingtree()  | 
|
85  | 
branch_c = tree_c.branch  | 
|
| 
1614.2.16
by Olaf Conradi
 Modified blackbox test cases to use bzrlib API.  | 
86  | 
self.build_tree(['branch_a/b'])  | 
87  | 
tree_a.add('b')  | 
|
88  | 
tree_a.commit('commit b')  | 
|
89  | 
self.build_tree(['branch_b/c'])  | 
|
90  | 
tree_b.add('c')  | 
|
91  | 
tree_b.commit('commit c')  | 
|
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
92  | 
        # initial push location must be empty
 | 
| 
1614.2.16
by Olaf Conradi
 Modified blackbox test cases to use bzrlib API.  | 
93  | 
self.assertEqual(None, branch_b.get_push_location())  | 
| 
1785.1.2
by John Arbash Meinel
 Push should only save the location if it can actually connect (doesn't need to succeed)  | 
94  | 
|
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
95  | 
        # test push for failure without push location set
 | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
96  | 
out = self.run_bzr('push', working_dir='branch_a', retcode=3)  | 
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
97  | 
self.assertEquals(out,  | 
98  | 
('','bzr: ERROR: No push location known or specified.\n'))  | 
|
| 
1785.1.2
by John Arbash Meinel
 Push should only save the location if it can actually connect (doesn't need to succeed)  | 
99  | 
|
100  | 
        # test not remembered if cannot actually push
 | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
101  | 
self.run_bzr('push path/which/doesnt/exist',  | 
102  | 
working_dir='branch_a', retcode=3)  | 
|
103  | 
out = self.run_bzr('push', working_dir='branch_a', retcode=3)  | 
|
| 
1785.1.2
by John Arbash Meinel
 Push should only save the location if it can actually connect (doesn't need to succeed)  | 
104  | 
self.assertEquals(  | 
105  | 
('', 'bzr: ERROR: No push location known or specified.\n'),  | 
|
106  | 
out)  | 
|
107  | 
||
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
108  | 
        # test implicit --remember when no push location set, push fails
 | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
109  | 
out = self.run_bzr('push ../branch_b',  | 
110  | 
working_dir='branch_a', retcode=3)  | 
|
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
111  | 
self.assertEquals(out,  | 
112  | 
('','bzr: ERROR: These branches have diverged. '  | 
|
| 
4441.1.1
by Aaron Bentley
 Merge nmb's diverged-branch docs.  | 
113  | 
'See "bzr help diverged-branches" for more information.\n'))  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
114  | 
self.assertEquals(osutils.abspath(branch_a.get_push_location()),  | 
115  | 
osutils.abspath(branch_b.bzrdir.root_transport.base))  | 
|
| 
1785.1.2
by John Arbash Meinel
 Push should only save the location if it can actually connect (doesn't need to succeed)  | 
116  | 
|
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
117  | 
        # test implicit --remember after resolving previous failure
 | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
118  | 
uncommit.uncommit(branch=branch_b, tree=tree_b)  | 
| 
1614.2.16
by Olaf Conradi
 Modified blackbox test cases to use bzrlib API.  | 
119  | 
transport.delete('branch_b/c')  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
120  | 
out, err = self.run_bzr('push', working_dir='branch_a')  | 
| 
1785.1.1
by John Arbash Meinel
 Fix the output of 'bzr push' so that it prints the location correctly.  | 
121  | 
path = branch_a.get_push_location()  | 
| 
2220.2.38
by Martin Pool
 Tag conflicts from push go to stdout.  | 
122  | 
self.assertEquals(out,  | 
| 
3978.2.5
by Jelmer Vernooij
 Fix trailing whitespace.  | 
123  | 
'Using saved push location: %s\n'  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
124  | 
% urlutils.local_path_from_url(path))  | 
| 
2220.2.38
by Martin Pool
 Tag conflicts from push go to stdout.  | 
125  | 
self.assertEqual(err,  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
126  | 
'All changes applied successfully.\n'  | 
127  | 
'Pushed up to revision 2.\n')  | 
|
| 
1785.1.1
by John Arbash Meinel
 Fix the output of 'bzr push' so that it prints the location correctly.  | 
128  | 
self.assertEqual(path,  | 
129  | 
branch_b.bzrdir.root_transport.base)  | 
|
| 
1614.2.9
by Olaf Conradi
 Added testcases for using push with --remember. Moved remember code to  | 
130  | 
        # test explicit --remember
 | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
131  | 
self.run_bzr('push ../branch_c --remember', working_dir='branch_a')  | 
| 
1785.1.1
by John Arbash Meinel
 Fix the output of 'bzr push' so that it prints the location correctly.  | 
132  | 
self.assertEquals(branch_a.get_push_location(),  | 
133  | 
branch_c.bzrdir.root_transport.base)  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
134  | 
|
| 
1692.3.1
by Robert Collins
 Fix push to work with just a branch, no need for a working tree.  | 
135  | 
def test_push_without_tree(self):  | 
136  | 
        # bzr push from a branch that does not have a checkout should work.
 | 
|
137  | 
b = self.make_branch('.')  | 
|
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
138  | 
out, err = self.run_bzr('push pushed-location')  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
139  | 
self.assertEqual('', out)  | 
140  | 
self.assertEqual('Created new branch.\n', err)  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
141  | 
b2 = branch.Branch.open('pushed-location')  | 
| 
1692.3.1
by Robert Collins
 Fix push to work with just a branch, no need for a working tree.  | 
142  | 
self.assertEndsWith(b2.base, 'pushed-location/')  | 
| 
1692.3.6
by Robert Collins
 Show the correct number of revisions pushed when pushing a new branch (Robert Collins).  | 
143  | 
|
144  | 
def test_push_new_branch_revision_count(self):  | 
|
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
145  | 
        # bzr push of a branch with revisions to a new location
 | 
146  | 
        # should print the number of revisions equal to the length of the
 | 
|
| 
1692.3.6
by Robert Collins
 Show the correct number of revisions pushed when pushing a new branch (Robert Collins).  | 
147  | 
        # local branch.
 | 
148  | 
t = self.make_branch_and_tree('tree')  | 
|
149  | 
self.build_tree(['tree/file'])  | 
|
150  | 
t.add('file')  | 
|
151  | 
t.commit('commit 1')  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
152  | 
out, err = self.run_bzr('push -d tree pushed-to')  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
153  | 
self.assertEqual('', out)  | 
154  | 
self.assertEqual('Created new branch.\n', err)  | 
|
| 
1711.2.3
by John Arbash Meinel
 Fix push to only push revisions in the current ancestry. (bug???)  | 
155  | 
|
156  | 
def test_push_only_pushes_history(self):  | 
|
157  | 
        # Knit branches should only push the history for the current revision.
 | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
158  | 
format = bzrdir.BzrDirMetaFormat1()  | 
159  | 
format.repository_format = knitrepo.RepositoryFormatKnit1()  | 
|
| 
1711.2.3
by John Arbash Meinel
 Fix push to only push revisions in the current ancestry. (bug???)  | 
160  | 
shared_repo = self.make_repository('repo', format=format, shared=True)  | 
161  | 
shared_repo.set_make_working_trees(True)  | 
|
162  | 
||
163  | 
def make_shared_tree(path):  | 
|
164  | 
shared_repo.bzrdir.root_transport.mkdir(path)  | 
|
165  | 
shared_repo.bzrdir.create_branch_convenience('repo/' + path)  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
166  | 
return workingtree.WorkingTree.open('repo/' + path)  | 
| 
1711.2.3
by John Arbash Meinel
 Fix push to only push revisions in the current ancestry. (bug???)  | 
167  | 
tree_a = make_shared_tree('a')  | 
168  | 
self.build_tree(['repo/a/file'])  | 
|
169  | 
tree_a.add('file')  | 
|
170  | 
tree_a.commit('commit a-1', rev_id='a-1')  | 
|
171  | 
f = open('repo/a/file', 'ab')  | 
|
172  | 
f.write('more stuff\n')  | 
|
173  | 
f.close()  | 
|
174  | 
tree_a.commit('commit a-2', rev_id='a-2')  | 
|
175  | 
||
176  | 
tree_b = make_shared_tree('b')  | 
|
177  | 
self.build_tree(['repo/b/file'])  | 
|
178  | 
tree_b.add('file')  | 
|
179  | 
tree_b.commit('commit b-1', rev_id='b-1')  | 
|
180  | 
||
181  | 
self.assertTrue(shared_repo.has_revision('a-1'))  | 
|
182  | 
self.assertTrue(shared_repo.has_revision('a-2'))  | 
|
183  | 
self.assertTrue(shared_repo.has_revision('b-1'))  | 
|
184  | 
||
185  | 
        # Now that we have a repository with shared files, make sure
 | 
|
186  | 
        # that things aren't copied out by a 'push'
 | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
187  | 
self.run_bzr('push ../../push-b', working_dir='repo/b')  | 
188  | 
pushed_tree = workingtree.WorkingTree.open('push-b')  | 
|
| 
1711.2.3
by John Arbash Meinel
 Fix push to only push revisions in the current ancestry. (bug???)  | 
189  | 
pushed_repo = pushed_tree.branch.repository  | 
190  | 
self.assertFalse(pushed_repo.has_revision('a-1'))  | 
|
191  | 
self.assertFalse(pushed_repo.has_revision('a-2'))  | 
|
192  | 
self.assertTrue(pushed_repo.has_revision('b-1'))  | 
|
193  | 
||
| 
1843.2.1
by Aaron Bentley
 Add failing tests for funky ids  | 
194  | 
def test_push_funky_id(self):  | 
195  | 
t = self.make_branch_and_tree('tree')  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
196  | 
self.build_tree(['tree/filename'])  | 
| 
1843.2.1
by Aaron Bentley
 Add failing tests for funky ids  | 
197  | 
t.add('filename', 'funky-chars<>%&;"\'')  | 
198  | 
t.commit('commit filename')  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
199  | 
self.run_bzr('push -d tree new-tree')  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
200  | 
|
| 
2220.2.9
by Martin Pool
 Add specific tests for push -d and pull -d  | 
201  | 
def test_push_dash_d(self):  | 
202  | 
t = self.make_branch_and_tree('from')  | 
|
203  | 
t.commit(allow_pointless=True,  | 
|
204  | 
message='first commit')  | 
|
| 
2530.3.1
by Martin Pool
 Cleanup old variations on run_bzr in the test suite  | 
205  | 
self.run_bzr('push -d from to-one')  | 
| 
2220.2.9
by Martin Pool
 Add specific tests for push -d and pull -d  | 
206  | 
self.failUnlessExists('to-one')  | 
| 
3943.8.1
by Marius Kruger
 remove all trailing whitespace from bzr source  | 
207  | 
self.run_bzr('push -d %s %s'  | 
| 
2220.2.9
by Martin Pool
 Add specific tests for push -d and pull -d  | 
208  | 
% tuple(map(urlutils.local_path_to_url, ['from', 'to-two'])))  | 
209  | 
self.failUnlessExists('to-two')  | 
|
| 
2279.3.1
by mbp at sourcefrog
 Add a -d option to push, pull, merge (ported from tags branch)  | 
210  | 
|
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
211  | 
def test_push_smart_non_stacked_streaming_acceptance(self):  | 
| 
4017.2.1
by Robert Collins
 Add BzrDirFormatMeta1 test for the amount of rpc calls made initializing over the network.  | 
212  | 
self.setup_smart_server_with_call_log()  | 
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
213  | 
t = self.make_branch_and_tree('from')  | 
214  | 
t.commit(allow_pointless=True, message='first commit')  | 
|
| 
4017.2.1
by Robert Collins
 Add BzrDirFormatMeta1 test for the amount of rpc calls made initializing over the network.  | 
215  | 
self.reset_smart_call_log()  | 
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
216  | 
self.run_bzr(['push', self.get_url('to-one')], working_dir='from')  | 
| 
4011.2.4
by Robert Collins
 Make the ratchet aspect of the blackbox smart server push acceptance tests clearer.  | 
217  | 
        # This figure represent the amount of work to perform this use case. It
 | 
218  | 
        # is entirely ok to reduce this number if a test fails due to rpc_count
 | 
|
219  | 
        # being too low. If rpc_count increases, more network roundtrips have
 | 
|
220  | 
        # become necessary for this use case. Please do not adjust this number
 | 
|
221  | 
        # upwards without agreement from bzr's network support maintainers.
 | 
|
| 
4307.2.2
by Robert Collins
 Lock repositories created by BzrDirFormat.initialize_on_transport_ex.  | 
222  | 
self.assertLength(9, self.hpss_calls)  | 
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
223  | 
|
224  | 
def test_push_smart_stacked_streaming_acceptance(self):  | 
|
| 
4017.2.1
by Robert Collins
 Add BzrDirFormatMeta1 test for the amount of rpc calls made initializing over the network.  | 
225  | 
self.setup_smart_server_with_call_log()  | 
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
226  | 
parent = self.make_branch_and_tree('parent', format='1.9')  | 
227  | 
parent.commit(message='first commit')  | 
|
228  | 
local = parent.bzrdir.sprout('local').open_workingtree()  | 
|
229  | 
local.commit(message='local commit')  | 
|
| 
4017.2.1
by Robert Collins
 Add BzrDirFormatMeta1 test for the amount of rpc calls made initializing over the network.  | 
230  | 
self.reset_smart_call_log()  | 
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
231  | 
self.run_bzr(['push', '--stacked', '--stacked-on', '../parent',  | 
232  | 
self.get_url('public')], working_dir='local')  | 
|
| 
4011.2.4
by Robert Collins
 Make the ratchet aspect of the blackbox smart server push acceptance tests clearer.  | 
233  | 
        # This figure represent the amount of work to perform this use case. It
 | 
234  | 
        # is entirely ok to reduce this number if a test fails due to rpc_count
 | 
|
235  | 
        # being too low. If rpc_count increases, more network roundtrips have
 | 
|
236  | 
        # become necessary for this use case. Please do not adjust this number
 | 
|
237  | 
        # upwards without agreement from bzr's network support maintainers.
 | 
|
| 
4307.2.5
by Robert Collins
 Remove too-early checks for revisions adding unnecessary round trips, at the cost of actually reading revision data when pulling (because we currently don't have a hint as about whats local for fetch).  | 
238  | 
self.assertLength(14, self.hpss_calls)  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
239  | 
remote = branch.Branch.open('public')  | 
| 
4011.2.1
by Robert Collins
 Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)  | 
240  | 
self.assertEndsWith(remote.get_stacked_on_url(), '/parent')  | 
241  | 
||
| 
4556.2.1
by Andrew Bennetts
 Add Branch.set_tags_bytes RPC, with HPSS call count acceptance test. Also fixes serialisation of LockDir, and uses external_url() in LockDir's repr and contention message.  | 
242  | 
def test_push_smart_tags_streaming_acceptance(self):  | 
243  | 
self.setup_smart_server_with_call_log()  | 
|
244  | 
t = self.make_branch_and_tree('from')  | 
|
245  | 
rev_id = t.commit(allow_pointless=True, message='first commit')  | 
|
246  | 
t.branch.tags.set_tag('new-tag', rev_id)  | 
|
247  | 
self.reset_smart_call_log()  | 
|
248  | 
self.run_bzr(['push', self.get_url('to-one')], working_dir='from')  | 
|
249  | 
        # This figure represent the amount of work to perform this use case. It
 | 
|
250  | 
        # is entirely ok to reduce this number if a test fails due to rpc_count
 | 
|
251  | 
        # being too low. If rpc_count increases, more network roundtrips have
 | 
|
252  | 
        # become necessary for this use case. Please do not adjust this number
 | 
|
253  | 
        # upwards without agreement from bzr's network support maintainers.
 | 
|
254  | 
self.assertLength(11, self.hpss_calls)  | 
|
255  | 
||
| 
4634.47.3
by Andrew Bennetts
 Add a BzrDir.open_2.1 verb that indicates if there is a workingtree present. Removes the last 2 VFS calls from incremental pushes.  | 
256  | 
def test_push_smart_incremental_acceptance(self):  | 
257  | 
self.setup_smart_server_with_call_log()  | 
|
258  | 
t = self.make_branch_and_tree('from')  | 
|
259  | 
rev_id1 = t.commit(allow_pointless=True, message='first commit')  | 
|
260  | 
rev_id2 = t.commit(allow_pointless=True, message='second commit')  | 
|
261  | 
self.run_bzr(  | 
|
262  | 
['push', self.get_url('to-one'), '-r1'], working_dir='from')  | 
|
263  | 
self.reset_smart_call_log()  | 
|
264  | 
self.run_bzr(['push', self.get_url('to-one')], working_dir='from')  | 
|
265  | 
        # This figure represent the amount of work to perform this use case. It
 | 
|
266  | 
        # is entirely ok to reduce this number if a test fails due to rpc_count
 | 
|
267  | 
        # being too low. If rpc_count increases, more network roundtrips have
 | 
|
268  | 
        # become necessary for this use case. Please do not adjust this number
 | 
|
269  | 
        # upwards without agreement from bzr's network support maintainers.
 | 
|
270  | 
self.assertLength(11, self.hpss_calls)  | 
|
271  | 
||
| 
4416.3.1
by Jonathan Lange
 Add two tests, the second of which demonstrates bug 385132.  | 
272  | 
def test_push_smart_with_default_stacking_url_path_segment(self):  | 
| 
4416.3.2
by Jonathan Lange
 Comment the blackbox tests, for my own sanity.  | 
273  | 
        # If the default stacked-on location is a path element then branches
 | 
274  | 
        # we push there over the smart server are stacked and their
 | 
|
275  | 
        # stacked_on_url is that exact path segment. Added to nail bug 385132.
 | 
|
| 
4416.3.1
by Jonathan Lange
 Add two tests, the second of which demonstrates bug 385132.  | 
276  | 
self.setup_smart_server_with_call_log()  | 
277  | 
self.make_branch('stack-on', format='1.9')  | 
|
| 
4416.3.7
by Jonathan Lange
 Update the test to fail correctly.  | 
278  | 
self.make_bzrdir('.').get_config().set_default_stack_on(  | 
| 
4416.3.9
by Jonathan Lange
 Put the test back the way it was.  | 
279  | 
'/stack-on')  | 
| 
4416.3.1
by Jonathan Lange
 Add two tests, the second of which demonstrates bug 385132.  | 
280  | 
self.make_branch('from', format='1.9')  | 
281  | 
out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])  | 
|
| 
4416.3.10
by Jonathan Lange
 Merge trunk  | 
282  | 
b = branch.Branch.open(self.get_url('to'))  | 
| 
4416.3.12
by Jonathan Lange
 This makes the test pass, but it's a bit ick.  | 
283  | 
self.assertEqual('/extra/stack-on', b.get_stacked_on_url())  | 
| 
4416.3.1
by Jonathan Lange
 Add two tests, the second of which demonstrates bug 385132.  | 
284  | 
|
285  | 
def test_push_smart_with_default_stacking_relative_path(self):  | 
|
| 
4416.3.2
by Jonathan Lange
 Comment the blackbox tests, for my own sanity.  | 
286  | 
        # If the default stacked-on location is a relative path then branches
 | 
287  | 
        # we push there over the smart server are stacked and their
 | 
|
288  | 
        # stacked_on_url is a relative path. Added to nail bug 385132.
 | 
|
| 
4416.3.1
by Jonathan Lange
 Add two tests, the second of which demonstrates bug 385132.  | 
289  | 
self.setup_smart_server_with_call_log()  | 
290  | 
self.make_branch('stack-on', format='1.9')  | 
|
291  | 
self.make_bzrdir('.').get_config().set_default_stack_on('stack-on')  | 
|
292  | 
self.make_branch('from', format='1.9')  | 
|
293  | 
out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])  | 
|
| 
4416.3.10
by Jonathan Lange
 Merge trunk  | 
294  | 
b = branch.Branch.open(self.get_url('to'))  | 
295  | 
self.assertEqual('../stack-on', b.get_stacked_on_url())  | 
|
| 
4416.3.1
by Jonathan Lange
 Add two tests, the second of which demonstrates bug 385132.  | 
296  | 
|
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
297  | 
def create_simple_tree(self):  | 
298  | 
tree = self.make_branch_and_tree('tree')  | 
|
299  | 
self.build_tree(['tree/a'])  | 
|
300  | 
tree.add(['a'], ['a-id'])  | 
|
301  | 
tree.commit('one', rev_id='r1')  | 
|
302  | 
return tree  | 
|
303  | 
||
304  | 
def test_push_create_prefix(self):  | 
|
305  | 
"""'bzr push --create-prefix' will create leading directories."""  | 
|
306  | 
tree = self.create_simple_tree()  | 
|
307  | 
||
308  | 
self.run_bzr_error(['Parent directory of ../new/tree does not exist'],  | 
|
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
309  | 
'push ../new/tree',  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
310  | 
working_dir='tree')  | 
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
311  | 
self.run_bzr('push ../new/tree --create-prefix',  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
312  | 
working_dir='tree')  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
313  | 
new_tree = workingtree.WorkingTree.open('new/tree')  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
314  | 
self.assertEqual(tree.last_revision(), new_tree.last_revision())  | 
315  | 
self.failUnlessExists('new/tree/a')  | 
|
316  | 
||
317  | 
def test_push_use_existing(self):  | 
|
| 
2227.3.4
by John Arbash Meinel
 Switch back to --use-existing-dir  | 
318  | 
"""'bzr push --use-existing-dir' can push into an existing dir.  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
319  | 
|
320  | 
        By default, 'bzr push' will not use an existing, non-versioned dir.
 | 
|
321  | 
        """
 | 
|
322  | 
tree = self.create_simple_tree()  | 
|
323  | 
self.build_tree(['target/'])  | 
|
324  | 
||
325  | 
self.run_bzr_error(['Target directory ../target already exists',  | 
|
| 
2227.3.4
by John Arbash Meinel
 Switch back to --use-existing-dir  | 
326  | 
'Supply --use-existing-dir',  | 
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
327  | 
                           ],
 | 
328  | 
'push ../target', working_dir='tree')  | 
|
| 
2227.3.4
by John Arbash Meinel
 Switch back to --use-existing-dir  | 
329  | 
|
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
330  | 
self.run_bzr('push --use-existing-dir ../target',  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
331  | 
working_dir='tree')  | 
332  | 
||
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
333  | 
new_tree = workingtree.WorkingTree.open('target')  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
334  | 
self.assertEqual(tree.last_revision(), new_tree.last_revision())  | 
335  | 
        # The push should have created target/a
 | 
|
336  | 
self.failUnlessExists('target/a')  | 
|
337  | 
||
| 
4634.105.1
by Andrew Bennetts
 Fix traceback when doing 'bzr push --use-existing-dir' into a dir with an invalid .bzr directory.  | 
338  | 
def test_push_use_existing_into_empty_bzrdir(self):  | 
339  | 
"""'bzr push --use-existing-dir' into a dir with an empty .bzr dir  | 
|
340  | 
        fails.
 | 
|
341  | 
        """
 | 
|
342  | 
tree = self.create_simple_tree()  | 
|
343  | 
self.build_tree(['target/', 'target/.bzr/'])  | 
|
344  | 
self.run_bzr_error(  | 
|
| 
4634.105.2
by Andrew Bennetts
 Clearer code and error message.  | 
345  | 
['Target directory ../target already contains a .bzr directory, '  | 
346  | 
'but it is not valid.'],  | 
|
| 
4634.105.1
by Andrew Bennetts
 Fix traceback when doing 'bzr push --use-existing-dir' into a dir with an invalid .bzr directory.  | 
347  | 
'push ../target --use-existing-dir', working_dir='tree')  | 
348  | 
||
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
349  | 
def test_push_onto_repo(self):  | 
350  | 
"""We should be able to 'bzr push' into an existing bzrdir."""  | 
|
351  | 
tree = self.create_simple_tree()  | 
|
352  | 
repo = self.make_repository('repo', shared=True)  | 
|
353  | 
||
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
354  | 
self.run_bzr('push ../repo',  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
355  | 
working_dir='tree')  | 
356  | 
||
357  | 
        # Pushing onto an existing bzrdir will create a repository and
 | 
|
358  | 
        # branch as needed, but will only create a working tree if there was
 | 
|
359  | 
        # no BzrDir before.
 | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
360  | 
self.assertRaises(errors.NoWorkingTree,  | 
361  | 
workingtree.WorkingTree.open, 'repo')  | 
|
362  | 
new_branch = branch.Branch.open('repo')  | 
|
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
363  | 
self.assertEqual(tree.last_revision(), new_branch.last_revision())  | 
364  | 
||
365  | 
def test_push_onto_just_bzrdir(self):  | 
|
366  | 
"""We don't handle when the target is just a bzrdir.  | 
|
367  | 
||
368  | 
        Because you shouldn't be able to create *just* a bzrdir in the wild.
 | 
|
369  | 
        """
 | 
|
370  | 
        # TODO: jam 20070109 Maybe it would be better to create the repository
 | 
|
371  | 
        #       if at this point
 | 
|
372  | 
tree = self.create_simple_tree()  | 
|
373  | 
a_bzrdir = self.make_bzrdir('dir')  | 
|
374  | 
||
375  | 
self.run_bzr_error(['At ../dir you have a valid .bzr control'],  | 
|
| 
2552.2.3
by Vincent Ladeuil
 Deprecate the varargs syntax and fix the tests.  | 
376  | 
'push ../dir',  | 
| 
2227.3.1
by John Arbash Meinel
 Allow push to create Branch when necessary, and add --use-existing  | 
377  | 
working_dir='tree')  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
378  | 
|
| 
3256.1.1
by Daniel Watkins
 Added test for push with a revspec.  | 
379  | 
def test_push_with_revisionspec(self):  | 
| 
3256.1.3
by Daniel Watkins
 Clarified test intent.  | 
380  | 
"""We should be able to push a revision older than the tip."""  | 
| 
3256.1.1
by Daniel Watkins
 Added test for push with a revspec.  | 
381  | 
tree_from = self.make_branch_and_tree('from')  | 
382  | 
tree_from.commit("One.", rev_id="from-1")  | 
|
383  | 
tree_from.commit("Two.", rev_id="from-2")  | 
|
384  | 
||
385  | 
self.run_bzr('push -r1 ../to', working_dir='from')  | 
|
386  | 
||
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
387  | 
tree_to = workingtree.WorkingTree.open('to')  | 
| 
3256.1.1
by Daniel Watkins
 Added test for push with a revspec.  | 
388  | 
repo_to = tree_to.branch.repository  | 
389  | 
self.assertTrue(repo_to.has_revision('from-1'))  | 
|
390  | 
self.assertFalse(repo_to.has_revision('from-2'))  | 
|
391  | 
self.assertEqual(tree_to.branch.last_revision_info()[1], 'from-1')  | 
|
392  | 
||
| 
3256.1.4
by Daniel Watkins
 Added test to ensure that passing a range of revisions errors.  | 
393  | 
self.run_bzr_error(  | 
| 
4325.4.6
by Vincent Ladeuil
 Fixed as per John's and Markus reviews.  | 
394  | 
['bzr: ERROR: bzr push --revision '  | 
395  | 
'takes exactly one revision identifier\n'],  | 
|
| 
3256.1.4
by Daniel Watkins
 Added test to ensure that passing a range of revisions errors.  | 
396  | 
'push -r0..2 ../to', working_dir='from')  | 
397  | 
||
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
398  | 
def create_trunk_and_feature_branch(self):  | 
| 
3221.11.12
by Robert Collins
 Basic push --reference support, requires url, slow.  | 
399  | 
        # We have a mainline
 | 
400  | 
trunk_tree = self.make_branch_and_tree('target',  | 
|
| 
4241.6.8
by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
 Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)  | 
401  | 
format='1.9')  | 
| 
3221.11.12
by Robert Collins
 Basic push --reference support, requires url, slow.  | 
402  | 
trunk_tree.commit('mainline')  | 
403  | 
        # and a branch from it
 | 
|
404  | 
branch_tree = self.make_branch_and_tree('branch',  | 
|
| 
4241.6.8
by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
 Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)  | 
405  | 
format='1.9')  | 
| 
3221.11.12
by Robert Collins
 Basic push --reference support, requires url, slow.  | 
406  | 
branch_tree.pull(trunk_tree.branch)  | 
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
407  | 
branch_tree.branch.set_parent(trunk_tree.branch.base)  | 
| 
3221.11.12
by Robert Collins
 Basic push --reference support, requires url, slow.  | 
408  | 
        # with some work on it
 | 
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
409  | 
branch_tree.commit('moar work plz')  | 
410  | 
return trunk_tree, branch_tree  | 
|
411  | 
||
| 
3221.11.14
by Robert Collins
 Refactor to reduce duplication.  | 
412  | 
def assertPublished(self, branch_revid, stacked_on):  | 
413  | 
"""Assert that the branch 'published' has been published correctly."""  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
414  | 
published_branch = branch.Branch.open('published')  | 
| 
3221.11.14
by Robert Collins
 Refactor to reduce duplication.  | 
415  | 
        # The published branch refers to the mainline
 | 
| 
3537.3.1
by Martin Pool
 Rename branch.get_stacked_on to get_stacked_on_url  | 
416  | 
self.assertEqual(stacked_on, published_branch.get_stacked_on_url())  | 
| 
3221.11.14
by Robert Collins
 Refactor to reduce duplication.  | 
417  | 
        # and the branch's work was pushed
 | 
418  | 
self.assertTrue(published_branch.repository.has_revision(branch_revid))  | 
|
419  | 
||
| 
3549.1.1
by Martin Pool
 rename push --reference to --stacked-on  | 
420  | 
def test_push_new_branch_stacked_on(self):  | 
421  | 
"""Pushing a new branch with --stacked-on creates a stacked branch."""  | 
|
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
422  | 
trunk_tree, branch_tree = self.create_trunk_and_feature_branch()  | 
423  | 
        # we publish branch_tree with a reference to the mainline.
 | 
|
| 
3549.1.1
by Martin Pool
 rename push --reference to --stacked-on  | 
424  | 
out, err = self.run_bzr(['push', '--stacked-on', trunk_tree.branch.base,  | 
| 
3221.11.12
by Robert Collins
 Basic push --reference support, requires url, slow.  | 
425  | 
self.get_url('published')], working_dir='branch')  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
426  | 
self.assertEqual('', out)  | 
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
427  | 
self.assertEqual('Created new stacked branch referring to %s.\n' %  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
428  | 
trunk_tree.branch.base, err)  | 
| 
3221.11.14
by Robert Collins
 Refactor to reduce duplication.  | 
429  | 
self.assertPublished(branch_tree.last_revision(),  | 
430  | 
trunk_tree.branch.base)  | 
|
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
431  | 
|
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
432  | 
def test_push_new_branch_stacked_uses_parent_when_no_public_url(self):  | 
| 
3221.11.17
by Robert Collins
 no public location causes the parent to be used directly with push --shallow.  | 
433  | 
"""When the parent has no public url the parent is used as-is."""  | 
434  | 
trunk_tree, branch_tree = self.create_trunk_and_feature_branch()  | 
|
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
435  | 
        # now we do a stacked push, which should determine the public location
 | 
| 
3221.11.17
by Robert Collins
 no public location causes the parent to be used directly with push --shallow.  | 
436  | 
        # for us.
 | 
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
437  | 
out, err = self.run_bzr(['push', '--stacked',  | 
| 
3221.11.17
by Robert Collins
 no public location causes the parent to be used directly with push --shallow.  | 
438  | 
self.get_url('published')], working_dir='branch')  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
439  | 
self.assertEqual('', out)  | 
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
440  | 
self.assertEqual('Created new stacked branch referring to %s.\n' %  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
441  | 
trunk_tree.branch.base, err)  | 
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
442  | 
self.assertPublished(branch_tree.last_revision(),  | 
443  | 
trunk_tree.branch.base)  | 
|
| 
3221.11.17
by Robert Collins
 no public location causes the parent to be used directly with push --shallow.  | 
444  | 
|
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
445  | 
def test_push_new_branch_stacked_uses_parent_public(self):  | 
446  | 
"""Pushing a new branch with --stacked creates a stacked branch."""  | 
|
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
447  | 
trunk_tree, branch_tree = self.create_trunk_and_feature_branch()  | 
448  | 
        # the trunk is published on a web server
 | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
449  | 
self.transport_readonly_server = http_server.HttpServer  | 
| 
4241.6.8
by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
 Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)  | 
450  | 
trunk_public = self.make_branch('public_trunk', format='1.9')  | 
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
451  | 
trunk_public.pull(trunk_tree.branch)  | 
452  | 
trunk_public_url = self.get_readonly_url('public_trunk')  | 
|
453  | 
trunk_tree.branch.set_public_branch(trunk_public_url)  | 
|
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
454  | 
        # now we do a stacked push, which should determine the public location
 | 
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
455  | 
        # for us.
 | 
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
456  | 
out, err = self.run_bzr(['push', '--stacked',  | 
| 
3221.11.13
by Robert Collins
 Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.  | 
457  | 
self.get_url('published')], working_dir='branch')  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
458  | 
self.assertEqual('', out)  | 
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
459  | 
self.assertEqual('Created new stacked branch referring to %s.\n' %  | 
| 
3978.2.2
by Jelmer Vernooij
 Write status messages during push to stderr rather than stdout.  | 
460  | 
trunk_public_url, err)  | 
| 
3221.11.14
by Robert Collins
 Refactor to reduce duplication.  | 
461  | 
self.assertPublished(branch_tree.last_revision(), trunk_public_url)  | 
| 
3221.11.12
by Robert Collins
 Basic push --reference support, requires url, slow.  | 
462  | 
|
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
463  | 
def test_push_new_branch_stacked_no_parent(self):  | 
464  | 
"""Pushing with --stacked and no parent branch errors."""  | 
|
| 
4241.6.8
by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
 Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)  | 
465  | 
branch = self.make_branch_and_tree('branch', format='1.9')  | 
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
466  | 
        # now we do a stacked push, which should fail as the place to refer too
 | 
| 
3221.11.15
by Robert Collins
 no parent branch causes an error on push --shallow.  | 
467  | 
        # cannot be determined.
 | 
468  | 
out, err = self.run_bzr_error(  | 
|
| 
3221.19.4
by Ian Clatworthy
 shallow -> stacked  | 
469  | 
['Could not determine branch to refer to\\.'], ['push', '--stacked',  | 
| 
3221.11.15
by Robert Collins
 no parent branch causes an error on push --shallow.  | 
470  | 
self.get_url('published')], working_dir='branch')  | 
471  | 
self.assertEqual('', out)  | 
|
472  | 
self.assertFalse(self.get_transport('published').has('.'))  | 
|
473  | 
||
| 
3606.8.3
by John Arbash Meinel
 push doesn't notify because it doesn't notice by default.  | 
474  | 
def test_push_notifies_default_stacking(self):  | 
| 
3735.1.2
by Robert Collins
 Remove 1.5 series dev formats and document development2 a little better.  | 
475  | 
self.make_branch('stack_on', format='1.6')  | 
| 
3242.3.34
by Aaron Bentley
 Add notification of default stacking  | 
476  | 
self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')  | 
| 
3735.1.2
by Robert Collins
 Remove 1.5 series dev formats and document development2 a little better.  | 
477  | 
self.make_branch('from', format='1.6')  | 
| 
3242.3.34
by Aaron Bentley
 Add notification of default stacking  | 
478  | 
out, err = self.run_bzr('push -d from to')  | 
| 
3641.1.1
by John Arbash Meinel
 Merge in 1.6rc5 and revert disabling default stack on policy  | 
479  | 
self.assertContainsRe(err,  | 
480  | 
'Using default stacking branch stack_on at .*')  | 
|
| 
3242.3.34
by Aaron Bentley
 Add notification of default stacking  | 
481  | 
|
| 
4165.2.1
by Robert Collins
 Fix bzr failing to stack when a server requests it and the branch it is pushing from cannot stack but the branch it should stack on can.  | 
482  | 
def test_push_stacks_with_default_stacking_if_target_is_stackable(self):  | 
483  | 
self.make_branch('stack_on', format='1.6')  | 
|
484  | 
self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')  | 
|
485  | 
self.make_branch('from', format='pack-0.92')  | 
|
486  | 
out, err = self.run_bzr('push -d from to')  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
487  | 
b = branch.Branch.open('to')  | 
488  | 
self.assertEqual('../stack_on', b.get_stacked_on_url())  | 
|
| 
4165.2.1
by Robert Collins
 Fix bzr failing to stack when a server requests it and the branch it is pushing from cannot stack but the branch it should stack on can.  | 
489  | 
|
490  | 
def test_push_does_not_change_format_with_default_if_target_cannot(self):  | 
|
491  | 
self.make_branch('stack_on', format='pack-0.92')  | 
|
492  | 
self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')  | 
|
493  | 
self.make_branch('from', format='pack-0.92')  | 
|
494  | 
out, err = self.run_bzr('push -d from to')  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
495  | 
b = branch.Branch.open('to')  | 
496  | 
self.assertRaises(errors.UnstackableBranchFormat, b.get_stacked_on_url)  | 
|
| 
4165.2.1
by Robert Collins
 Fix bzr failing to stack when a server requests it and the branch it is pushing from cannot stack but the branch it should stack on can.  | 
497  | 
|
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
498  | 
def test_push_doesnt_create_broken_branch(self):  | 
| 
3904.3.7
by Andrew Bennetts
 Comment the new tests.  | 
499  | 
"""Pushing a new standalone branch works even when there's a default  | 
500  | 
        stacking policy at the destination.
 | 
|
501  | 
||
502  | 
        The new branch will preserve the repo format (even if it isn't the
 | 
|
503  | 
        default for the branch), and will be stacked when the repo format
 | 
|
504  | 
        allows (which means that the branch format isn't necessarly preserved).
 | 
|
505  | 
        """
 | 
|
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
506  | 
self.make_repository('repo', shared=True, format='1.6')  | 
507  | 
builder = self.make_branch_builder('repo/local', format='pack-0.92')  | 
|
508  | 
builder.start_series()  | 
|
509  | 
builder.build_snapshot('rev-1', None, [  | 
|
510  | 
('add', ('', 'root-id', 'directory', '')),  | 
|
| 
3904.3.3
by Andrew Bennetts
 Simplify test slightly.  | 
511  | 
('add', ('filename', 'f-id', 'file', 'content\n'))])  | 
512  | 
builder.build_snapshot('rev-2', ['rev-1'], [])  | 
|
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
513  | 
builder.build_snapshot('rev-3', ['rev-2'],  | 
| 
3904.3.3
by Andrew Bennetts
 Simplify test slightly.  | 
514  | 
[('modify', ('f-id', 'new-content\n'))])  | 
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
515  | 
builder.finish_series()  | 
516  | 
branch = builder.get_branch()  | 
|
| 
3904.3.7
by Andrew Bennetts
 Comment the new tests.  | 
517  | 
        # Push rev-1 to "trunk", so that we can stack on it.
 | 
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
518  | 
self.run_bzr('push -d repo/local trunk -r 1')  | 
| 
3904.3.7
by Andrew Bennetts
 Comment the new tests.  | 
519  | 
        # Set a default stacking policy so that new branches will automatically
 | 
520  | 
        # stack on trunk.
 | 
|
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
521  | 
self.make_bzrdir('.').get_config().set_default_stack_on('trunk')  | 
| 
3904.3.7
by Andrew Bennetts
 Comment the new tests.  | 
522  | 
        # Push rev-2 to a new branch "remote".  It will be stacked on "trunk".
 | 
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
523  | 
out, err = self.run_bzr('push -d repo/local remote -r 2')  | 
524  | 
self.assertContainsRe(  | 
|
525  | 
err, 'Using default stacking branch trunk at .*')  | 
|
| 
3904.3.7
by Andrew Bennetts
 Comment the new tests.  | 
526  | 
        # Push rev-3 onto "remote".  If "remote" not stacked and is missing the
 | 
527  | 
        # fulltext record for f-id @ rev-1, then this will fail.
 | 
|
| 
3904.3.2
by Andrew Bennetts
 Blackbox test that triggers the bug. Should get replaced with a unit test.  | 
528  | 
out, err = self.run_bzr('push -d repo/local remote -r 3')  | 
529  | 
||
| 
3848.1.19
by Aaron Bentley
 Show log for non-initial push -v  | 
530  | 
def test_push_verbose_shows_log(self):  | 
531  | 
tree = self.make_branch_and_tree('source')  | 
|
532  | 
tree.commit('rev1')  | 
|
533  | 
out, err = self.run_bzr('push -v -d source target')  | 
|
534  | 
        # initial push contains log
 | 
|
535  | 
self.assertContainsRe(out, 'rev1')  | 
|
536  | 
tree.commit('rev2')  | 
|
537  | 
out, err = self.run_bzr('push -v -d source target')  | 
|
538  | 
        # subsequent push contains log
 | 
|
539  | 
self.assertContainsRe(out, 'rev2')  | 
|
540  | 
        # subsequent log is accurate
 | 
|
541  | 
self.assertNotContainsRe(out, 'rev1')  | 
|
542  | 
||
| 
4453.1.2
by Matthew Fuller
 Add a test for pushing from subdir.  | 
543  | 
def test_push_from_subdir(self):  | 
544  | 
t = self.make_branch_and_tree('tree')  | 
|
545  | 
self.build_tree(['tree/dir/', 'tree/dir/file'])  | 
|
546  | 
t.add('dir', 'dir/file')  | 
|
547  | 
t.commit('r1')  | 
|
548  | 
out, err = self.run_bzr('push ../../pushloc', working_dir='tree/dir')  | 
|
549  | 
self.assertEqual('', out)  | 
|
550  | 
self.assertEqual('Created new branch.\n', err)  | 
|
551  | 
||
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
552  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
553  | 
class RedirectingMemoryTransport(memory.MemoryTransport):  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
554  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
555  | 
def mkdir(self, relpath, mode=None):  | 
556  | 
if self._cwd == '/source/':  | 
|
557  | 
raise errors.RedirectRequested(self.abspath(relpath),  | 
|
558  | 
self.abspath('../target'),  | 
|
559  | 
is_permanent=True)  | 
|
560  | 
elif self._cwd == '/infinite-loop/':  | 
|
561  | 
raise errors.RedirectRequested(self.abspath(relpath),  | 
|
562  | 
self.abspath('../infinite-loop'),  | 
|
563  | 
is_permanent=True)  | 
|
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
564  | 
else:  | 
565  | 
return super(RedirectingMemoryTransport, self).mkdir(  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
566  | 
relpath, mode)  | 
567  | 
||
| 
4547.2.2
by Andrew Bennetts
 Add test for read_mergeable_from_transport raising NotABundle when TooManyRedirections happens.  | 
568  | 
def get(self, relpath):  | 
569  | 
if self.clone(relpath)._cwd == '/infinite-loop/':  | 
|
570  | 
raise errors.RedirectRequested(self.abspath(relpath),  | 
|
571  | 
self.abspath('../infinite-loop'),  | 
|
572  | 
is_permanent=True)  | 
|
573  | 
else:  | 
|
574  | 
return super(RedirectingMemoryTransport, self).get(relpath)  | 
|
575  | 
||
| 
3878.4.5
by Vincent Ladeuil
 Don't use the exception as a parameter for _redirected_to.  | 
576  | 
def _redirected_to(self, source, target):  | 
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
577  | 
        # We do accept redirections
 | 
| 
3878.4.5
by Vincent Ladeuil
 Don't use the exception as a parameter for _redirected_to.  | 
578  | 
return transport.get_transport(target)  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
579  | 
|
580  | 
||
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
581  | 
class RedirectingMemoryServer(memory.MemoryServer):  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
582  | 
|
583  | 
def setUp(self):  | 
|
584  | 
self._dirs = {'/': None}  | 
|
585  | 
self._files = {}  | 
|
586  | 
self._locks = {}  | 
|
587  | 
self._scheme = 'redirecting-memory+%s:///' % id(self)  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
588  | 
transport.register_transport(self._scheme, self._memory_factory)  | 
| 
3066.3.3
by jml at canonical
 Unregister the test transport in order to be clean and to make test_selftest pass.  | 
589  | 
|
590  | 
def _memory_factory(self, url):  | 
|
591  | 
result = RedirectingMemoryTransport(url)  | 
|
592  | 
result._dirs = self._dirs  | 
|
593  | 
result._files = self._files  | 
|
594  | 
result._locks = self._locks  | 
|
595  | 
return result  | 
|
596  | 
||
597  | 
def tearDown(self):  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
598  | 
transport.unregister_transport(self._scheme, self._memory_factory)  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
599  | 
|
600  | 
||
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
601  | 
class TestPushRedirect(tests.TestCaseWithTransport):  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
602  | 
|
603  | 
def setUp(self):  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
604  | 
tests.TestCaseWithTransport.setUp(self)  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
605  | 
self.memory_server = RedirectingMemoryServer()  | 
| 
4659.1.2
by Robert Collins
 Refactor creation and shutdown of test servers to use a common helper,  | 
606  | 
self.start_server(self.memory_server)  | 
| 
3066.3.2
by jml at canonical
 Add tests to check the handling of TooManyRedirections.  | 
607  | 
        # Make the branch and tree that we'll be pushing.
 | 
608  | 
t = self.make_branch_and_tree('tree')  | 
|
609  | 
self.build_tree(['tree/file'])  | 
|
610  | 
t.add('file')  | 
|
611  | 
t.commit('commit 1')  | 
|
612  | 
||
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
613  | 
def test_push_redirects_on_mkdir(self):  | 
614  | 
"""If the push requires a mkdir, push respects redirect requests.  | 
|
615  | 
||
616  | 
        This is added primarily to handle lp:/ URI support, so that users can
 | 
|
617  | 
        push to new branches by specifying lp:/ URIs.
 | 
|
618  | 
        """
 | 
|
619  | 
destination_url = self.memory_server.get_url() + 'source'  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
620  | 
self.run_bzr(['push', '-d', 'tree', destination_url])  | 
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
621  | 
|
| 
4420.1.1
by Vincent Ladeuil
 Cleanup imports.  | 
622  | 
local_revision = branch.Branch.open('tree').last_revision()  | 
623  | 
remote_revision = branch.Branch.open(  | 
|
| 
3066.3.1
by jml at canonical
 Catch redirects raised by mkdir() in the push command. This is primarily  | 
624  | 
self.memory_server.get_url() + 'target').last_revision()  | 
625  | 
self.assertEqual(remote_revision, local_revision)  | 
|
| 
3066.3.2
by jml at canonical
 Add tests to check the handling of TooManyRedirections.  | 
626  | 
|
627  | 
def test_push_gracefully_handles_too_many_redirects(self):  | 
|
628  | 
"""Push fails gracefully if the mkdir generates a large number of  | 
|
629  | 
        redirects.
 | 
|
630  | 
        """
 | 
|
631  | 
destination_url = self.memory_server.get_url() + 'infinite-loop'  | 
|
632  | 
out, err = self.run_bzr_error(  | 
|
633  | 
['Too many redirections trying to make %s\\.\n'  | 
|
634  | 
% re.escape(destination_url)],  | 
|
| 
3878.4.4
by Vincent Ladeuil
 Cleanup.  | 
635  | 
['push', '-d', 'tree', destination_url], retcode=3)  | 
| 
3066.3.2
by jml at canonical
 Add tests to check the handling of TooManyRedirections.  | 
636  | 
self.assertEqual('', out)  | 
| 
4420.1.2
by Vincent Ladeuil
 Fix bug #284038 by adding a --strict option to push.  | 
637  | 
|
638  | 
||
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
639  | 
class TestPushStrictMixin(object):  | 
| 
4420.1.2
by Vincent Ladeuil
 Fix bug #284038 by adding a --strict option to push.  | 
640  | 
|
641  | 
def make_local_branch_and_tree(self):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
642  | 
self.tree = self.make_branch_and_tree('local')  | 
| 
4420.1.2
by Vincent Ladeuil
 Fix bug #284038 by adding a --strict option to push.  | 
643  | 
self.build_tree_contents([('local/file', 'initial')])  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
644  | 
self.tree.add('file')  | 
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
645  | 
self.tree.commit('adding file', rev_id='added')  | 
646  | 
self.build_tree_contents([('local/file', 'modified')])  | 
|
647  | 
self.tree.commit('modify file', rev_id='modified')  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
648  | 
|
649  | 
def set_config_push_strict(self, value):  | 
|
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
650  | 
        # set config var (any of bazaar.conf, locations.conf, branch.conf
 | 
651  | 
        # should do)
 | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
652  | 
conf = self.tree.branch.get_config()  | 
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
653  | 
conf.set_user_option('push_strict', value)  | 
654  | 
||
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
655  | 
_default_command = ['push', '../to']  | 
656  | 
_default_wd = 'local'  | 
|
657  | 
_default_errors = ['Working tree ".*/local/" has uncommitted '  | 
|
658  | 
'changes \(See bzr status\)\.',]  | 
|
659  | 
_default_pushed_revid = 'modified'  | 
|
660  | 
||
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
661  | 
def assertPushFails(self, args):  | 
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
662  | 
self.run_bzr_error(self._default_errors, self._default_command + args,  | 
663  | 
working_dir=self._default_wd, retcode=3)  | 
|
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
664  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
665  | 
def assertPushSucceeds(self, args, pushed_revid=None):  | 
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
666  | 
self.run_bzr(self._default_command + args,  | 
667  | 
working_dir=self._default_wd)  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
668  | 
if pushed_revid is None:  | 
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
669  | 
pushed_revid = self._default_pushed_revid  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
670  | 
tree_to = workingtree.WorkingTree.open('to')  | 
| 
4420.1.2
by Vincent Ladeuil
 Fix bug #284038 by adding a --strict option to push.  | 
671  | 
repo_to = tree_to.branch.repository  | 
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
672  | 
self.assertTrue(repo_to.has_revision(pushed_revid))  | 
673  | 
self.assertEqual(tree_to.branch.last_revision_info()[1], pushed_revid)  | 
|
| 
4420.1.2
by Vincent Ladeuil
 Fix bug #284038 by adding a --strict option to push.  | 
674  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
675  | 
|
676  | 
||
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
677  | 
class TestPushStrictWithoutChanges(tests.TestCaseWithTransport,  | 
678  | 
TestPushStrictMixin):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
679  | 
|
680  | 
def setUp(self):  | 
|
681  | 
super(TestPushStrictWithoutChanges, self).setUp()  | 
|
682  | 
self.make_local_branch_and_tree()  | 
|
683  | 
||
684  | 
def test_push_default(self):  | 
|
685  | 
self.assertPushSucceeds([])  | 
|
686  | 
||
687  | 
def test_push_strict(self):  | 
|
688  | 
self.assertPushSucceeds(['--strict'])  | 
|
689  | 
||
690  | 
def test_push_no_strict(self):  | 
|
691  | 
self.assertPushSucceeds(['--no-strict'])  | 
|
692  | 
||
693  | 
def test_push_config_var_strict(self):  | 
|
694  | 
self.set_config_push_strict('true')  | 
|
695  | 
self.assertPushSucceeds([])  | 
|
696  | 
||
697  | 
def test_push_config_var_no_strict(self):  | 
|
698  | 
self.set_config_push_strict('false')  | 
|
699  | 
self.assertPushSucceeds([])  | 
|
700  | 
||
701  | 
||
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
702  | 
class TestPushStrictWithChanges(tests.TestCaseWithTransport,  | 
703  | 
TestPushStrictMixin):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
704  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
705  | 
_changes_type = None # Set by load_tests  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
706  | 
|
707  | 
def setUp(self):  | 
|
708  | 
super(TestPushStrictWithChanges, self).setUp()  | 
|
| 
4721.2.6
by Vincent Ladeuil
 More agressive test sharing between push and dpush.  | 
709  | 
        # Apply the changes defined in load_tests: one of _uncommitted_changes,
 | 
710  | 
        # _pending_merges or _out_of_sync_trees
 | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
711  | 
getattr(self, self._changes_type)()  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
712  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
713  | 
def _uncommitted_changes(self):  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
714  | 
self.make_local_branch_and_tree()  | 
715  | 
        # Make a change without committing it
 | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
716  | 
self.build_tree_contents([('local/file', 'in progress')])  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
717  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
718  | 
def _pending_merges(self):  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
719  | 
self.make_local_branch_and_tree()  | 
720  | 
        # Create 'other' branch containing a new file
 | 
|
721  | 
other_bzrdir = self.tree.bzrdir.sprout('other')  | 
|
722  | 
other_tree = other_bzrdir.open_workingtree()  | 
|
723  | 
self.build_tree_contents([('other/other-file', 'other')])  | 
|
724  | 
other_tree.add('other-file')  | 
|
725  | 
other_tree.commit('other commit', rev_id='other')  | 
|
726  | 
        # Merge and revert, leaving a pending merge
 | 
|
727  | 
self.tree.merge_from_branch(other_tree.branch)  | 
|
728  | 
self.tree.revert(filenames=['other-file'], backups=False)  | 
|
729  | 
||
| 
4487.2.6
by Vincent Ladeuil
 Fixed as per jam's review.  | 
730  | 
def _out_of_sync_trees(self):  | 
731  | 
self.make_local_branch_and_tree()  | 
|
732  | 
self.run_bzr(['checkout', '--lightweight', 'local', 'checkout'])  | 
|
733  | 
        # Make a change and commit it
 | 
|
734  | 
self.build_tree_contents([('local/file', 'modified in local')])  | 
|
735  | 
self.tree.commit('modify file', rev_id='modified-in-local')  | 
|
736  | 
        # Exercise commands from the checkout directory
 | 
|
737  | 
self._default_wd = 'checkout'  | 
|
738  | 
self._default_errors = ["Working tree is out of date, please run"  | 
|
739  | 
" 'bzr update'\.",]  | 
|
740  | 
self._default_pushed_revid = 'modified-in-local'  | 
|
741  | 
||
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
742  | 
def test_push_default(self):  | 
743  | 
self.assertPushFails([])  | 
|
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
744  | 
|
| 
4487.2.3
by Vincent Ladeuil
 --strict is meaningless when --revision is specified.  | 
745  | 
def test_push_with_revision(self):  | 
746  | 
self.assertPushSucceeds(['-r', 'revid:added'], pushed_revid='added')  | 
|
747  | 
||
748  | 
def test_push_no_strict(self):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
749  | 
self.assertPushSucceeds(['--no-strict'])  | 
| 
4420.1.2
by Vincent Ladeuil
 Fix bug #284038 by adding a --strict option to push.  | 
750  | 
|
751  | 
def test_push_strict_with_changes(self):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
752  | 
self.assertPushFails(['--strict'])  | 
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
753  | 
|
754  | 
def test_push_respect_config_var_strict(self):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
755  | 
self.set_config_push_strict('true')  | 
756  | 
self.assertPushFails([])  | 
|
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
757  | 
|
| 
4420.1.6
by Vincent Ladeuil
 Fixed as per John's review feedback.  | 
758  | 
def test_push_bogus_config_var_ignored(self):  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
759  | 
self.set_config_push_strict("I don't want you to be strict")  | 
760  | 
self.assertPushFails([])  | 
|
| 
4420.1.6
by Vincent Ladeuil
 Fixed as per John's review feedback.  | 
761  | 
|
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
762  | 
def test_push_no_strict_command_line_override_config(self):  | 
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
763  | 
self.set_config_push_strict('yES')  | 
764  | 
self.assertPushFails([])  | 
|
765  | 
self.assertPushSucceeds(['--no-strict'])  | 
|
| 
4420.1.5
by Vincent Ladeuil
 Start implementing jam's review feedback.  | 
766  | 
|
767  | 
def test_push_strict_command_line_override_config(self):  | 
|
| 
4487.2.2
by Vincent Ladeuil
 Add tests for push --strict against pending merges.  | 
768  | 
self.set_config_push_strict('oFF')  | 
769  | 
self.assertPushFails(['--strict'])  | 
|
770  | 
self.assertPushSucceeds([])  |