/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
5557.1.7 by John Arbash Meinel
Merge in the bzr.dev 5582
1
# Copyright (C) 2006-2011 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
4925.1.1 by Jelmer Vernooij
Print a proper error when attempting to push to a foreign VCS for which
34
from bzrlib.tests import (
35
    blackbox,
36
    http_server,
5559.2.2 by Martin Pool
Change to using standard load_tests_apply_scenarios.
37
    scenarios,
6175.2.4 by Vincent Ladeuil
Add script tests and documentation.
38
    script,
4925.1.1 by Jelmer Vernooij
Print a proper error when attempting to push to a foreign VCS for which
39
    test_foreign,
5017.3.37 by Vincent Ladeuil
-s bb.test_push passing
40
    test_server,
4925.1.1 by Jelmer Vernooij
Print a proper error when attempting to push to a foreign VCS for which
41
    )
4420.1.1 by Vincent Ladeuil
Cleanup imports.
42
from bzrlib.transport import memory
43
44
5559.2.2 by Martin Pool
Change to using standard load_tests_apply_scenarios.
45
load_tests = scenarios.load_tests_apply_scenarios
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
46
47
4420.1.1 by Vincent Ladeuil
Cleanup imports.
48
class TestPush(tests.TestCaseWithTransport):
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
49
4017.2.3 by Robert Collins
Review feedback.
50
    def test_push_error_on_vfs_http(self):
51
        """ pushing a branch to a HTTP server fails cleanly. """
52
        # the trunk is published on a web server
4420.1.1 by Vincent Ladeuil
Cleanup imports.
53
        self.transport_readonly_server = http_server.HttpServer
4017.2.3 by Robert Collins
Review feedback.
54
        self.make_branch('source')
55
        public_url = self.get_readonly_url('target')
56
        self.run_bzr_error(['http does not support mkdir'],
57
                           ['push', public_url],
58
                           working_dir='source')
59
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
60
    def test_push_remember(self):
61
        """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.
62
        transport = self.get_transport()
63
        tree_a = self.make_branch_and_tree('branch_a')
64
        branch_a = tree_a.branch
65
        self.build_tree(['branch_a/a'])
66
        tree_a.add('a')
67
        tree_a.commit('commit a')
1666.1.4 by Robert Collins
* 'Metadir' is now the default disk format. This improves behaviour in
68
        tree_b = branch_a.bzrdir.sprout('branch_b').open_workingtree()
69
        branch_b = tree_b.branch
70
        tree_c = branch_a.bzrdir.sprout('branch_c').open_workingtree()
71
        branch_c = tree_c.branch
1614.2.16 by Olaf Conradi
Modified blackbox test cases to use bzrlib API.
72
        self.build_tree(['branch_a/b'])
73
        tree_a.add('b')
74
        tree_a.commit('commit b')
75
        self.build_tree(['branch_b/c'])
76
        tree_b.add('c')
77
        tree_b.commit('commit c')
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
78
        # initial push location must be empty
1614.2.16 by Olaf Conradi
Modified blackbox test cases to use bzrlib API.
79
        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)
80
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
81
        # test push for failure without push location set
4420.1.1 by Vincent Ladeuil
Cleanup imports.
82
        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
83
        self.assertEquals(out,
84
                ('','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)
85
86
        # test not remembered if cannot actually push
4420.1.1 by Vincent Ladeuil
Cleanup imports.
87
        self.run_bzr('push path/which/doesnt/exist',
88
                     working_dir='branch_a', retcode=3)
89
        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)
90
        self.assertEquals(
91
                ('', 'bzr: ERROR: No push location known or specified.\n'),
92
                out)
93
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
94
        # test implicit --remember when no push location set, push fails
4420.1.1 by Vincent Ladeuil
Cleanup imports.
95
        out = self.run_bzr('push ../branch_b',
96
                           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: These branches have diverged.  '
4441.1.1 by Aaron Bentley
Merge nmb's diverged-branch docs.
99
                 'See "bzr help diverged-branches" for more information.\n'))
4420.1.1 by Vincent Ladeuil
Cleanup imports.
100
        self.assertEquals(osutils.abspath(branch_a.get_push_location()),
101
                          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)
102
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
103
        # test implicit --remember after resolving previous failure
4420.1.1 by Vincent Ladeuil
Cleanup imports.
104
        uncommit.uncommit(branch=branch_b, tree=tree_b)
1614.2.16 by Olaf Conradi
Modified blackbox test cases to use bzrlib API.
105
        transport.delete('branch_b/c')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
106
        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.
107
        path = branch_a.get_push_location()
2220.2.38 by Martin Pool
Tag conflicts from push go to stdout.
108
        self.assertEqual(err,
5998.1.3 by Jonathan Riddell
add a test for bzr push -q
109
                         'Using saved push location: %s\n'
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
110
                         'All changes applied successfully.\n'
5998.1.3 by Jonathan Riddell
add a test for bzr push -q
111
                         'Pushed up to revision 2.\n'
112
                         % urlutils.local_path_from_url(path))
1785.1.1 by John Arbash Meinel
Fix the output of 'bzr push' so that it prints the location correctly.
113
        self.assertEqual(path,
114
                         branch_b.bzrdir.root_transport.base)
1614.2.9 by Olaf Conradi
Added testcases for using push with --remember. Moved remember code to
115
        # test explicit --remember
4420.1.1 by Vincent Ladeuil
Cleanup imports.
116
        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.
117
        self.assertEquals(branch_a.get_push_location(),
118
                          branch_c.bzrdir.root_transport.base)
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
119
1692.3.1 by Robert Collins
Fix push to work with just a branch, no need for a working tree.
120
    def test_push_without_tree(self):
121
        # bzr push from a branch that does not have a checkout should work.
122
        b = self.make_branch('.')
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
123
        out, err = self.run_bzr('push pushed-location')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
124
        self.assertEqual('', out)
125
        self.assertEqual('Created new branch.\n', err)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
126
        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.
127
        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).
128
5448.6.5 by Matthew Gordon
Added test for push --no-tree.
129
    def test_push_no_tree(self):
130
        # bzr push --no-tree of a branch with working trees
131
        b = self.make_branch_and_tree('push-from')
132
        self.build_tree(['push-from/file'])
133
        b.add('file')
134
        b.commit('commit 1')
135
        out, err = self.run_bzr('push --no-tree -d push-from push-to')
136
        self.assertEqual('', out)
137
        self.assertEqual('Created new branch.\n', err)
5784.1.3 by Martin Pool
Switch away from using failUnlessExists and failIfExists
138
        self.assertPathDoesNotExist('push-to/file')
5448.6.5 by Matthew Gordon
Added test for push --no-tree.
139
1692.3.6 by Robert Collins
Show the correct number of revisions pushed when pushing a new branch (Robert Collins).
140
    def test_push_new_branch_revision_count(self):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
141
        # bzr push of a branch with revisions to a new location
142
        # 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).
143
        # local branch.
144
        t = self.make_branch_and_tree('tree')
145
        self.build_tree(['tree/file'])
146
        t.add('file')
147
        t.commit('commit 1')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
148
        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.
149
        self.assertEqual('', out)
150
        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???)
151
5998.1.3 by Jonathan Riddell
add a test for bzr push -q
152
    def test_push_quiet(self):
153
        # test that using -q makes output quiet
154
        t = self.make_branch_and_tree('tree')
155
        self.build_tree(['tree/file'])
156
        t.add('file')
157
        t.commit('commit 1')
158
        self.run_bzr('push -d tree pushed-to')
159
        path = t.branch.get_push_location()
160
        out, err = self.run_bzr('push', working_dir="tree")
6112.4.3 by Jelmer Vernooij
Fix push tests.
161
        self.assertEqual('Using saved push location: %s\n'
162
                         'No new revisions or tags to push.\n' %
163
                         urlutils.local_path_from_url(path), err)
5998.1.3 by Jonathan Riddell
add a test for bzr push -q
164
        out, err = self.run_bzr('push -q', working_dir="tree")
165
        self.assertEqual('', out)
166
        self.assertEqual('', err)
167
1711.2.3 by John Arbash Meinel
Fix push to only push revisions in the current ancestry. (bug???)
168
    def test_push_only_pushes_history(self):
169
        # Knit branches should only push the history for the current revision.
4420.1.1 by Vincent Ladeuil
Cleanup imports.
170
        format = bzrdir.BzrDirMetaFormat1()
171
        format.repository_format = knitrepo.RepositoryFormatKnit1()
1711.2.3 by John Arbash Meinel
Fix push to only push revisions in the current ancestry. (bug???)
172
        shared_repo = self.make_repository('repo', format=format, shared=True)
173
        shared_repo.set_make_working_trees(True)
174
175
        def make_shared_tree(path):
176
            shared_repo.bzrdir.root_transport.mkdir(path)
177
            shared_repo.bzrdir.create_branch_convenience('repo/' + path)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
178
            return workingtree.WorkingTree.open('repo/' + path)
1711.2.3 by John Arbash Meinel
Fix push to only push revisions in the current ancestry. (bug???)
179
        tree_a = make_shared_tree('a')
180
        self.build_tree(['repo/a/file'])
181
        tree_a.add('file')
182
        tree_a.commit('commit a-1', rev_id='a-1')
183
        f = open('repo/a/file', 'ab')
184
        f.write('more stuff\n')
185
        f.close()
186
        tree_a.commit('commit a-2', rev_id='a-2')
187
188
        tree_b = make_shared_tree('b')
189
        self.build_tree(['repo/b/file'])
190
        tree_b.add('file')
191
        tree_b.commit('commit b-1', rev_id='b-1')
192
193
        self.assertTrue(shared_repo.has_revision('a-1'))
194
        self.assertTrue(shared_repo.has_revision('a-2'))
195
        self.assertTrue(shared_repo.has_revision('b-1'))
196
197
        # Now that we have a repository with shared files, make sure
198
        # that things aren't copied out by a 'push'
4420.1.1 by Vincent Ladeuil
Cleanup imports.
199
        self.run_bzr('push ../../push-b', working_dir='repo/b')
200
        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???)
201
        pushed_repo = pushed_tree.branch.repository
202
        self.assertFalse(pushed_repo.has_revision('a-1'))
203
        self.assertFalse(pushed_repo.has_revision('a-2'))
204
        self.assertTrue(pushed_repo.has_revision('b-1'))
205
1843.2.1 by Aaron Bentley
Add failing tests for funky ids
206
    def test_push_funky_id(self):
207
        t = self.make_branch_and_tree('tree')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
208
        self.build_tree(['tree/filename'])
1843.2.1 by Aaron Bentley
Add failing tests for funky ids
209
        t.add('filename', 'funky-chars<>%&;"\'')
210
        t.commit('commit filename')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
211
        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
212
2220.2.9 by Martin Pool
Add specific tests for push -d and pull -d
213
    def test_push_dash_d(self):
214
        t = self.make_branch_and_tree('from')
215
        t.commit(allow_pointless=True,
216
                message='first commit')
2530.3.1 by Martin Pool
Cleanup old variations on run_bzr in the test suite
217
        self.run_bzr('push -d from to-one')
5784.1.3 by Martin Pool
Switch away from using failUnlessExists and failIfExists
218
        self.assertPathExists('to-one')
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
219
        self.run_bzr('push -d %s %s'
2220.2.9 by Martin Pool
Add specific tests for push -d and pull -d
220
            % tuple(map(urlutils.local_path_to_url, ['from', 'to-two'])))
5784.1.3 by Martin Pool
Switch away from using failUnlessExists and failIfExists
221
        self.assertPathExists('to-two')
2279.3.1 by mbp at sourcefrog
Add a -d option to push, pull, merge (ported from tags branch)
222
5609.26.1 by John Arbash Meinel
Fix bug #465517, 'bzr push' to a target with a repo but no branch
223
    def test_push_repository_no_branch_doesnt_fetch_all_revs(self):
224
        # See https://bugs.launchpad.net/bzr/+bug/465517
225
        target_repo = self.make_repository('target')
226
        source = self.make_branch_builder('source')
227
        source.start_series()
228
        source.build_snapshot('A', None, [
229
            ('add', ('', 'root-id', 'directory', None))])
230
        source.build_snapshot('B', ['A'], [])
231
        source.build_snapshot('C', ['A'], [])
232
        source.finish_series()
233
        self.run_bzr('push target -d source')
234
        self.addCleanup(target_repo.lock_read().unlock)
235
        # We should have pushed 'C', but not 'B', since it isn't in the
236
        # ancestry
237
        self.assertEqual([('A',), ('C',)], sorted(target_repo.revisions.keys()))
238
4011.2.1 by Robert Collins
Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)
239
    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.
240
        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)
241
        t = self.make_branch_and_tree('from')
242
        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.
243
        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)
244
        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.
245
        # This figure represent the amount of work to perform this use case. It
246
        # is entirely ok to reduce this number if a test fails due to rpc_count
247
        # being too low. If rpc_count increases, more network roundtrips have
248
        # become necessary for this use case. Please do not adjust this number
249
        # upwards without agreement from bzr's network support maintainers.
4307.2.2 by Robert Collins
Lock repositories created by BzrDirFormat.initialize_on_transport_ex.
250
        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)
251
252
    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.
253
        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)
254
        parent = self.make_branch_and_tree('parent', format='1.9')
255
        parent.commit(message='first commit')
256
        local = parent.bzrdir.sprout('local').open_workingtree()
257
        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.
258
        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)
259
        self.run_bzr(['push', '--stacked', '--stacked-on', '../parent',
260
            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.
261
        # This figure represent the amount of work to perform this use case. It
262
        # is entirely ok to reduce this number if a test fails due to rpc_count
263
        # being too low. If rpc_count increases, more network roundtrips have
264
        # become necessary for this use case. Please do not adjust this number
265
        # upwards without agreement from bzr's network support maintainers.
5566.1.1 by Andrew Bennetts
Remove redundant parent inventories calculation.
266
        self.assertLength(13, self.hpss_calls)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
267
        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)
268
        self.assertEndsWith(remote.get_stacked_on_url(), '/parent')
269
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.
270
    def test_push_smart_tags_streaming_acceptance(self):
271
        self.setup_smart_server_with_call_log()
272
        t = self.make_branch_and_tree('from')
273
        rev_id = t.commit(allow_pointless=True, message='first commit')
274
        t.branch.tags.set_tag('new-tag', rev_id)
275
        self.reset_smart_call_log()
276
        self.run_bzr(['push', self.get_url('to-one')], working_dir='from')
277
        # This figure represent the amount of work to perform this use case. It
278
        # is entirely ok to reduce this number if a test fails due to rpc_count
279
        # being too low. If rpc_count increases, more network roundtrips have
280
        # become necessary for this use case. Please do not adjust this number
281
        # upwards without agreement from bzr's network support maintainers.
282
        self.assertLength(11, self.hpss_calls)
283
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.
284
    def test_push_smart_incremental_acceptance(self):
285
        self.setup_smart_server_with_call_log()
286
        t = self.make_branch_and_tree('from')
287
        rev_id1 = t.commit(allow_pointless=True, message='first commit')
288
        rev_id2 = t.commit(allow_pointless=True, message='second commit')
289
        self.run_bzr(
290
            ['push', self.get_url('to-one'), '-r1'], working_dir='from')
291
        self.reset_smart_call_log()
292
        self.run_bzr(['push', self.get_url('to-one')], working_dir='from')
293
        # This figure represent the amount of work to perform this use case. It
294
        # is entirely ok to reduce this number if a test fails due to rpc_count
295
        # being too low. If rpc_count increases, more network roundtrips have
296
        # become necessary for this use case. Please do not adjust this number
297
        # upwards without agreement from bzr's network support maintainers.
298
        self.assertLength(11, self.hpss_calls)
299
4416.3.1 by Jonathan Lange
Add two tests, the second of which demonstrates bug 385132.
300
    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.
301
        # If the default stacked-on location is a path element then branches
302
        # we push there over the smart server are stacked and their
303
        # 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.
304
        self.setup_smart_server_with_call_log()
305
        self.make_branch('stack-on', format='1.9')
4416.3.7 by Jonathan Lange
Update the test to fail correctly.
306
        self.make_bzrdir('.').get_config().set_default_stack_on(
4416.3.9 by Jonathan Lange
Put the test back the way it was.
307
            '/stack-on')
4416.3.1 by Jonathan Lange
Add two tests, the second of which demonstrates bug 385132.
308
        self.make_branch('from', format='1.9')
309
        out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])
4416.3.10 by Jonathan Lange
Merge trunk
310
        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.
311
        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.
312
313
    def test_push_smart_with_default_stacking_relative_path(self):
4416.3.2 by Jonathan Lange
Comment the blackbox tests, for my own sanity.
314
        # If the default stacked-on location is a relative path then branches
315
        # we push there over the smart server are stacked and their
316
        # 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.
317
        self.setup_smart_server_with_call_log()
318
        self.make_branch('stack-on', format='1.9')
319
        self.make_bzrdir('.').get_config().set_default_stack_on('stack-on')
320
        self.make_branch('from', format='1.9')
321
        out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])
4416.3.10 by Jonathan Lange
Merge trunk
322
        b = branch.Branch.open(self.get_url('to'))
323
        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.
324
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
325
    def create_simple_tree(self):
326
        tree = self.make_branch_and_tree('tree')
327
        self.build_tree(['tree/a'])
328
        tree.add(['a'], ['a-id'])
329
        tree.commit('one', rev_id='r1')
330
        return tree
331
332
    def test_push_create_prefix(self):
333
        """'bzr push --create-prefix' will create leading directories."""
334
        tree = self.create_simple_tree()
335
336
        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.
337
                           'push ../new/tree',
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
338
                           working_dir='tree')
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
339
        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
340
                     working_dir='tree')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
341
        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
342
        self.assertEqual(tree.last_revision(), new_tree.last_revision())
5784.1.3 by Martin Pool
Switch away from using failUnlessExists and failIfExists
343
        self.assertPathExists('new/tree/a')
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
344
345
    def test_push_use_existing(self):
2227.3.4 by John Arbash Meinel
Switch back to --use-existing-dir
346
        """'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
347
348
        By default, 'bzr push' will not use an existing, non-versioned dir.
349
        """
350
        tree = self.create_simple_tree()
351
        self.build_tree(['target/'])
352
353
        self.run_bzr_error(['Target directory ../target already exists',
2227.3.4 by John Arbash Meinel
Switch back to --use-existing-dir
354
                            'Supply --use-existing-dir',
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
355
                           ],
356
                           'push ../target', working_dir='tree')
2227.3.4 by John Arbash Meinel
Switch back to --use-existing-dir
357
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
358
        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
359
                     working_dir='tree')
360
4420.1.1 by Vincent Ladeuil
Cleanup imports.
361
        new_tree = workingtree.WorkingTree.open('target')
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
362
        self.assertEqual(tree.last_revision(), new_tree.last_revision())
363
        # The push should have created target/a
5784.1.3 by Martin Pool
Switch away from using failUnlessExists and failIfExists
364
        self.assertPathExists('target/a')
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
365
4634.105.1 by Andrew Bennetts
Fix traceback when doing 'bzr push --use-existing-dir' into a dir with an invalid .bzr directory.
366
    def test_push_use_existing_into_empty_bzrdir(self):
367
        """'bzr push --use-existing-dir' into a dir with an empty .bzr dir
368
        fails.
369
        """
370
        tree = self.create_simple_tree()
371
        self.build_tree(['target/', 'target/.bzr/'])
372
        self.run_bzr_error(
4634.105.2 by Andrew Bennetts
Clearer code and error message.
373
            ['Target directory ../target already contains a .bzr directory, '
374
             '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.
375
            'push ../target --use-existing-dir', working_dir='tree')
376
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
377
    def test_push_onto_repo(self):
378
        """We should be able to 'bzr push' into an existing bzrdir."""
379
        tree = self.create_simple_tree()
380
        repo = self.make_repository('repo', shared=True)
381
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
382
        self.run_bzr('push ../repo',
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
383
                     working_dir='tree')
384
385
        # Pushing onto an existing bzrdir will create a repository and
386
        # branch as needed, but will only create a working tree if there was
387
        # no BzrDir before.
4420.1.1 by Vincent Ladeuil
Cleanup imports.
388
        self.assertRaises(errors.NoWorkingTree,
389
                          workingtree.WorkingTree.open, 'repo')
390
        new_branch = branch.Branch.open('repo')
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
391
        self.assertEqual(tree.last_revision(), new_branch.last_revision())
392
393
    def test_push_onto_just_bzrdir(self):
394
        """We don't handle when the target is just a bzrdir.
395
396
        Because you shouldn't be able to create *just* a bzrdir in the wild.
397
        """
398
        # TODO: jam 20070109 Maybe it would be better to create the repository
399
        #       if at this point
400
        tree = self.create_simple_tree()
401
        a_bzrdir = self.make_bzrdir('dir')
402
403
        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.
404
                'push ../dir',
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
405
                working_dir='tree')
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
406
3256.1.1 by Daniel Watkins
Added test for push with a revspec.
407
    def test_push_with_revisionspec(self):
3256.1.3 by Daniel Watkins
Clarified test intent.
408
        """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.
409
        tree_from = self.make_branch_and_tree('from')
410
        tree_from.commit("One.", rev_id="from-1")
411
        tree_from.commit("Two.", rev_id="from-2")
412
413
        self.run_bzr('push -r1 ../to', working_dir='from')
414
4420.1.1 by Vincent Ladeuil
Cleanup imports.
415
        tree_to = workingtree.WorkingTree.open('to')
3256.1.1 by Daniel Watkins
Added test for push with a revspec.
416
        repo_to = tree_to.branch.repository
417
        self.assertTrue(repo_to.has_revision('from-1'))
418
        self.assertFalse(repo_to.has_revision('from-2'))
419
        self.assertEqual(tree_to.branch.last_revision_info()[1], 'from-1')
420
3256.1.4 by Daniel Watkins
Added test to ensure that passing a range of revisions errors.
421
        self.run_bzr_error(
4325.4.6 by Vincent Ladeuil
Fixed as per John's and Markus reviews.
422
            ['bzr: ERROR: bzr push --revision '
423
             'takes exactly one revision identifier\n'],
3256.1.4 by Daniel Watkins
Added test to ensure that passing a range of revisions errors.
424
            'push -r0..2 ../to', working_dir='from')
425
3221.11.13 by Robert Collins
Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.
426
    def create_trunk_and_feature_branch(self):
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
427
        # We have a mainline
428
        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)
429
            format='1.9')
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
430
        trunk_tree.commit('mainline')
431
        # and a branch from it
432
        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)
433
            format='1.9')
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
434
        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.
435
        branch_tree.branch.set_parent(trunk_tree.branch.base)
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
436
        # 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.
437
        branch_tree.commit('moar work plz')
438
        return trunk_tree, branch_tree
439
3221.11.14 by Robert Collins
Refactor to reduce duplication.
440
    def assertPublished(self, branch_revid, stacked_on):
441
        """Assert that the branch 'published' has been published correctly."""
4420.1.1 by Vincent Ladeuil
Cleanup imports.
442
        published_branch = branch.Branch.open('published')
3221.11.14 by Robert Collins
Refactor to reduce duplication.
443
        # The published branch refers to the mainline
3537.3.1 by Martin Pool
Rename branch.get_stacked_on to get_stacked_on_url
444
        self.assertEqual(stacked_on, published_branch.get_stacked_on_url())
3221.11.14 by Robert Collins
Refactor to reduce duplication.
445
        # and the branch's work was pushed
446
        self.assertTrue(published_branch.repository.has_revision(branch_revid))
447
3549.1.1 by Martin Pool
rename push --reference to --stacked-on
448
    def test_push_new_branch_stacked_on(self):
449
        """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.
450
        trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
451
        # we publish branch_tree with a reference to the mainline.
3549.1.1 by Martin Pool
rename push --reference to --stacked-on
452
        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.
453
            self.get_url('published')], working_dir='branch')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
454
        self.assertEqual('', out)
3221.19.4 by Ian Clatworthy
shallow -> stacked
455
        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.
456
            trunk_tree.branch.base, err)
3221.11.14 by Robert Collins
Refactor to reduce duplication.
457
        self.assertPublished(branch_tree.last_revision(),
458
            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.
459
3221.19.4 by Ian Clatworthy
shallow -> stacked
460
    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.
461
        """When the parent has no public url the parent is used as-is."""
462
        trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
3221.19.4 by Ian Clatworthy
shallow -> stacked
463
        # 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.
464
        # for us.
3221.19.4 by Ian Clatworthy
shallow -> stacked
465
        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.
466
            self.get_url('published')], working_dir='branch')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
467
        self.assertEqual('', out)
3221.19.4 by Ian Clatworthy
shallow -> stacked
468
        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.
469
            trunk_tree.branch.base, err)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
470
        self.assertPublished(branch_tree.last_revision(),
471
                             trunk_tree.branch.base)
3221.11.17 by Robert Collins
no public location causes the parent to be used directly with push --shallow.
472
3221.19.4 by Ian Clatworthy
shallow -> stacked
473
    def test_push_new_branch_stacked_uses_parent_public(self):
474
        """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.
475
        trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
476
        # the trunk is published on a web server
4420.1.1 by Vincent Ladeuil
Cleanup imports.
477
        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)
478
        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.
479
        trunk_public.pull(trunk_tree.branch)
480
        trunk_public_url = self.get_readonly_url('public_trunk')
481
        trunk_tree.branch.set_public_branch(trunk_public_url)
3221.19.4 by Ian Clatworthy
shallow -> stacked
482
        # 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.
483
        # for us.
3221.19.4 by Ian Clatworthy
shallow -> stacked
484
        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.
485
            self.get_url('published')], working_dir='branch')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
486
        self.assertEqual('', out)
3221.19.4 by Ian Clatworthy
shallow -> stacked
487
        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.
488
            trunk_public_url, err)
3221.11.14 by Robert Collins
Refactor to reduce duplication.
489
        self.assertPublished(branch_tree.last_revision(), trunk_public_url)
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
490
3221.19.4 by Ian Clatworthy
shallow -> stacked
491
    def test_push_new_branch_stacked_no_parent(self):
492
        """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)
493
        branch = self.make_branch_and_tree('branch', format='1.9')
3221.19.4 by Ian Clatworthy
shallow -> stacked
494
        # 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.
495
        # cannot be determined.
496
        out, err = self.run_bzr_error(
3221.19.4 by Ian Clatworthy
shallow -> stacked
497
            ['Could not determine branch to refer to\\.'], ['push', '--stacked',
3221.11.15 by Robert Collins
no parent branch causes an error on push --shallow.
498
            self.get_url('published')], working_dir='branch')
499
        self.assertEqual('', out)
500
        self.assertFalse(self.get_transport('published').has('.'))
501
3606.8.3 by John Arbash Meinel
push doesn't notify because it doesn't notice by default.
502
    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.
503
        self.make_branch('stack_on', format='1.6')
3242.3.34 by Aaron Bentley
Add notification of default stacking
504
        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.
505
        self.make_branch('from', format='1.6')
3242.3.34 by Aaron Bentley
Add notification of default stacking
506
        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
507
        self.assertContainsRe(err,
508
                              'Using default stacking branch stack_on at .*')
3242.3.34 by Aaron Bentley
Add notification of default stacking
509
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.
510
    def test_push_stacks_with_default_stacking_if_target_is_stackable(self):
511
        self.make_branch('stack_on', format='1.6')
512
        self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
513
        self.make_branch('from', format='pack-0.92')
514
        out, err = self.run_bzr('push -d from to')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
515
        b = branch.Branch.open('to')
516
        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.
517
518
    def test_push_does_not_change_format_with_default_if_target_cannot(self):
519
        self.make_branch('stack_on', format='pack-0.92')
520
        self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
521
        self.make_branch('from', format='pack-0.92')
522
        out, err = self.run_bzr('push -d from to')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
523
        b = branch.Branch.open('to')
524
        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.
525
3904.3.2 by Andrew Bennetts
Blackbox test that triggers the bug. Should get replaced with a unit test.
526
    def test_push_doesnt_create_broken_branch(self):
3904.3.7 by Andrew Bennetts
Comment the new tests.
527
        """Pushing a new standalone branch works even when there's a default
528
        stacking policy at the destination.
529
530
        The new branch will preserve the repo format (even if it isn't the
531
        default for the branch), and will be stacked when the repo format
532
        allows (which means that the branch format isn't necessarly preserved).
533
        """
3904.3.2 by Andrew Bennetts
Blackbox test that triggers the bug. Should get replaced with a unit test.
534
        self.make_repository('repo', shared=True, format='1.6')
535
        builder = self.make_branch_builder('repo/local', format='pack-0.92')
536
        builder.start_series()
537
        builder.build_snapshot('rev-1', None, [
538
            ('add', ('', 'root-id', 'directory', '')),
3904.3.3 by Andrew Bennetts
Simplify test slightly.
539
            ('add', ('filename', 'f-id', 'file', 'content\n'))])
540
        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.
541
        builder.build_snapshot('rev-3', ['rev-2'],
3904.3.3 by Andrew Bennetts
Simplify test slightly.
542
            [('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.
543
        builder.finish_series()
544
        branch = builder.get_branch()
3904.3.7 by Andrew Bennetts
Comment the new tests.
545
        # 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.
546
        self.run_bzr('push -d repo/local trunk -r 1')
3904.3.7 by Andrew Bennetts
Comment the new tests.
547
        # Set a default stacking policy so that new branches will automatically
548
        # stack on trunk.
3904.3.2 by Andrew Bennetts
Blackbox test that triggers the bug. Should get replaced with a unit test.
549
        self.make_bzrdir('.').get_config().set_default_stack_on('trunk')
3904.3.7 by Andrew Bennetts
Comment the new tests.
550
        # 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.
551
        out, err = self.run_bzr('push -d repo/local remote -r 2')
552
        self.assertContainsRe(
553
            err, 'Using default stacking branch trunk at .*')
3904.3.7 by Andrew Bennetts
Comment the new tests.
554
        # Push rev-3 onto "remote".  If "remote" not stacked and is missing the
555
        # 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.
556
        out, err = self.run_bzr('push -d repo/local remote -r 3')
557
3848.1.19 by Aaron Bentley
Show log for non-initial push -v
558
    def test_push_verbose_shows_log(self):
559
        tree = self.make_branch_and_tree('source')
560
        tree.commit('rev1')
561
        out, err = self.run_bzr('push -v -d source target')
562
        # initial push contains log
563
        self.assertContainsRe(out, 'rev1')
564
        tree.commit('rev2')
565
        out, err = self.run_bzr('push -v -d source target')
566
        # subsequent push contains log
567
        self.assertContainsRe(out, 'rev2')
568
        # subsequent log is accurate
569
        self.assertNotContainsRe(out, 'rev1')
570
4453.1.2 by Matthew Fuller
Add a test for pushing from subdir.
571
    def test_push_from_subdir(self):
572
        t = self.make_branch_and_tree('tree')
573
        self.build_tree(['tree/dir/', 'tree/dir/file'])
574
        t.add('dir', 'dir/file')
575
        t.commit('r1')
576
        out, err = self.run_bzr('push ../../pushloc', working_dir='tree/dir')
577
        self.assertEqual('', out)
578
        self.assertEqual('Created new branch.\n', err)
579
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
580
4420.1.1 by Vincent Ladeuil
Cleanup imports.
581
class RedirectingMemoryTransport(memory.MemoryTransport):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
582
3878.4.4 by Vincent Ladeuil
Cleanup.
583
    def mkdir(self, relpath, mode=None):
584
        if self._cwd == '/source/':
585
            raise errors.RedirectRequested(self.abspath(relpath),
586
                                           self.abspath('../target'),
587
                                           is_permanent=True)
588
        elif self._cwd == '/infinite-loop/':
589
            raise errors.RedirectRequested(self.abspath(relpath),
590
                                           self.abspath('../infinite-loop'),
591
                                           is_permanent=True)
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
592
        else:
593
            return super(RedirectingMemoryTransport, self).mkdir(
3878.4.4 by Vincent Ladeuil
Cleanup.
594
                relpath, mode)
595
4547.2.2 by Andrew Bennetts
Add test for read_mergeable_from_transport raising NotABundle when TooManyRedirections happens.
596
    def get(self, relpath):
597
        if self.clone(relpath)._cwd == '/infinite-loop/':
598
            raise errors.RedirectRequested(self.abspath(relpath),
599
                                           self.abspath('../infinite-loop'),
600
                                           is_permanent=True)
601
        else:
602
            return super(RedirectingMemoryTransport, self).get(relpath)
603
3878.4.5 by Vincent Ladeuil
Don't use the exception as a parameter for _redirected_to.
604
    def _redirected_to(self, source, target):
3878.4.4 by Vincent Ladeuil
Cleanup.
605
        # We do accept redirections
3878.4.5 by Vincent Ladeuil
Don't use the exception as a parameter for _redirected_to.
606
        return transport.get_transport(target)
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
607
608
5017.3.45 by Vincent Ladeuil
Move MemoryServer back into bzrlib.transport.memory as it's needed as soon as a MemoryTransport is used. Add a NEWS entry.
609
class RedirectingMemoryServer(memory.MemoryServer):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
610
4934.3.3 by Martin Pool
Rename Server.setUp to Server.start_server
611
    def start_server(self):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
612
        self._dirs = {'/': None}
613
        self._files = {}
614
        self._locks = {}
615
        self._scheme = 'redirecting-memory+%s:///' % id(self)
3878.4.4 by Vincent Ladeuil
Cleanup.
616
        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.
617
618
    def _memory_factory(self, url):
619
        result = RedirectingMemoryTransport(url)
620
        result._dirs = self._dirs
621
        result._files = self._files
622
        result._locks = self._locks
623
        return result
624
4934.3.1 by Martin Pool
Rename Server.tearDown to .stop_server
625
    def stop_server(self):
3878.4.4 by Vincent Ladeuil
Cleanup.
626
        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
627
628
4420.1.1 by Vincent Ladeuil
Cleanup imports.
629
class TestPushRedirect(tests.TestCaseWithTransport):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
630
631
    def setUp(self):
4420.1.1 by Vincent Ladeuil
Cleanup imports.
632
        tests.TestCaseWithTransport.setUp(self)
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
633
        self.memory_server = RedirectingMemoryServer()
4659.1.2 by Robert Collins
Refactor creation and shutdown of test servers to use a common helper,
634
        self.start_server(self.memory_server)
3066.3.2 by jml at canonical
Add tests to check the handling of TooManyRedirections.
635
        # Make the branch and tree that we'll be pushing.
636
        t = self.make_branch_and_tree('tree')
637
        self.build_tree(['tree/file'])
638
        t.add('file')
639
        t.commit('commit 1')
640
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
641
    def test_push_redirects_on_mkdir(self):
642
        """If the push requires a mkdir, push respects redirect requests.
643
644
        This is added primarily to handle lp:/ URI support, so that users can
645
        push to new branches by specifying lp:/ URIs.
646
        """
647
        destination_url = self.memory_server.get_url() + 'source'
3878.4.4 by Vincent Ladeuil
Cleanup.
648
        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
649
4420.1.1 by Vincent Ladeuil
Cleanup imports.
650
        local_revision = branch.Branch.open('tree').last_revision()
651
        remote_revision = branch.Branch.open(
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
652
            self.memory_server.get_url() + 'target').last_revision()
653
        self.assertEqual(remote_revision, local_revision)
3066.3.2 by jml at canonical
Add tests to check the handling of TooManyRedirections.
654
655
    def test_push_gracefully_handles_too_many_redirects(self):
656
        """Push fails gracefully if the mkdir generates a large number of
657
        redirects.
658
        """
659
        destination_url = self.memory_server.get_url() + 'infinite-loop'
660
        out, err = self.run_bzr_error(
661
            ['Too many redirections trying to make %s\\.\n'
662
             % re.escape(destination_url)],
3878.4.4 by Vincent Ladeuil
Cleanup.
663
            ['push', '-d', 'tree', destination_url], retcode=3)
3066.3.2 by jml at canonical
Add tests to check the handling of TooManyRedirections.
664
        self.assertEqual('', out)
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
665
666
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
667
class TestPushStrictMixin(object):
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
668
669
    def make_local_branch_and_tree(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
670
        self.tree = self.make_branch_and_tree('local')
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
671
        self.build_tree_contents([('local/file', 'initial')])
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
672
        self.tree.add('file')
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
673
        self.tree.commit('adding file', rev_id='added')
674
        self.build_tree_contents([('local/file', 'modified')])
675
        self.tree.commit('modify file', rev_id='modified')
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
676
677
    def set_config_push_strict(self, value):
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
678
        # set config var (any of bazaar.conf, locations.conf, branch.conf
679
        # should do)
6155.2.1 by Vincent Ladeuil
Migrate dpush_strict, push_strict and send_strict options to the stack based config design, introducing get_config_stack for branches.
680
        conf = self.tree.branch.get_config_stack()
681
        conf.set('push_strict', value)
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
682
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
683
    _default_command = ['push', '../to']
684
    _default_wd = 'local'
685
    _default_errors = ['Working tree ".*/local/" has uncommitted '
686
                       'changes \(See bzr status\)\.',]
5171.2.3 by Vincent Ladeuil
Fixed as per Andrew's review.
687
    _default_additional_error = 'Use --no-strict to force the push.\n'
688
    _default_additional_warning = 'Uncommitted changes will not be pushed.'
689
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
690
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
691
    def assertPushFails(self, args):
5171.2.3 by Vincent Ladeuil
Fixed as per Andrew's review.
692
        out, err = self.run_bzr_error(self._default_errors,
693
                                      self._default_command + args,
694
                                      working_dir=self._default_wd, retcode=3)
695
        self.assertContainsRe(err, self._default_additional_error)
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
696
5171.2.3 by Vincent Ladeuil
Fixed as per Andrew's review.
697
    def assertPushSucceeds(self, args, with_warning=False, revid_to_push=None):
5147.2.1 by Vincent Ladeuil
Failing tests for bug #519319.
698
        if with_warning:
699
            error_regexes = self._default_errors
700
        else:
701
            error_regexes = []
5171.2.3 by Vincent Ladeuil
Fixed as per Andrew's review.
702
        out, err = self.run_bzr(self._default_command + args,
703
                                working_dir=self._default_wd,
704
                                error_regexes=error_regexes)
705
        if with_warning:
706
            self.assertContainsRe(err, self._default_additional_warning)
707
        else:
708
            self.assertNotContainsRe(err, self._default_additional_warning)
709
        branch_from = branch.Branch.open(self._default_wd)
710
        if revid_to_push is None:
711
            revid_to_push = branch_from.last_revision()
712
        branch_to = branch.Branch.open('to')
713
        repo_to = branch_to.repository
714
        self.assertTrue(repo_to.has_revision(revid_to_push))
715
        self.assertEqual(revid_to_push, branch_to.last_revision())
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
716
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
717
718
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
719
class TestPushStrictWithoutChanges(tests.TestCaseWithTransport,
720
                                   TestPushStrictMixin):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
721
722
    def setUp(self):
723
        super(TestPushStrictWithoutChanges, self).setUp()
724
        self.make_local_branch_and_tree()
725
726
    def test_push_default(self):
727
        self.assertPushSucceeds([])
728
729
    def test_push_strict(self):
730
        self.assertPushSucceeds(['--strict'])
731
732
    def test_push_no_strict(self):
733
        self.assertPushSucceeds(['--no-strict'])
734
735
    def test_push_config_var_strict(self):
736
        self.set_config_push_strict('true')
737
        self.assertPushSucceeds([])
738
739
    def test_push_config_var_no_strict(self):
740
        self.set_config_push_strict('false')
741
        self.assertPushSucceeds([])
742
743
5559.2.2 by Martin Pool
Change to using standard load_tests_apply_scenarios.
744
strict_push_change_scenarios = [
745
    ('uncommitted',
746
        dict(_changes_type= '_uncommitted_changes')),
747
    ('pending-merges',
748
        dict(_changes_type= '_pending_merges')),
749
    ('out-of-sync-trees',
750
        dict(_changes_type= '_out_of_sync_trees')),
751
    ]
752
753
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
754
class TestPushStrictWithChanges(tests.TestCaseWithTransport,
755
                                TestPushStrictMixin):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
756
5559.2.2 by Martin Pool
Change to using standard load_tests_apply_scenarios.
757
    scenarios = strict_push_change_scenarios 
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
758
    _changes_type = None # Set by load_tests
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
759
760
    def setUp(self):
761
        super(TestPushStrictWithChanges, self).setUp()
4721.2.6 by Vincent Ladeuil
More agressive test sharing between push and dpush.
762
        # Apply the changes defined in load_tests: one of _uncommitted_changes,
763
        # _pending_merges or _out_of_sync_trees
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
764
        getattr(self, self._changes_type)()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
765
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
766
    def _uncommitted_changes(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
767
        self.make_local_branch_and_tree()
768
        # Make a change without committing it
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
769
        self.build_tree_contents([('local/file', 'in progress')])
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
770
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
771
    def _pending_merges(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
772
        self.make_local_branch_and_tree()
773
        # Create 'other' branch containing a new file
774
        other_bzrdir = self.tree.bzrdir.sprout('other')
775
        other_tree = other_bzrdir.open_workingtree()
776
        self.build_tree_contents([('other/other-file', 'other')])
777
        other_tree.add('other-file')
778
        other_tree.commit('other commit', rev_id='other')
779
        # Merge and revert, leaving a pending merge
780
        self.tree.merge_from_branch(other_tree.branch)
781
        self.tree.revert(filenames=['other-file'], backups=False)
782
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
783
    def _out_of_sync_trees(self):
784
        self.make_local_branch_and_tree()
785
        self.run_bzr(['checkout', '--lightweight', 'local', 'checkout'])
786
        # Make a change and commit it
787
        self.build_tree_contents([('local/file', 'modified in local')])
788
        self.tree.commit('modify file', rev_id='modified-in-local')
789
        # Exercise commands from the checkout directory
790
        self._default_wd = 'checkout'
791
        self._default_errors = ["Working tree is out of date, please run"
792
                                " 'bzr update'\.",]
793
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
794
    def test_push_default(self):
5147.2.1 by Vincent Ladeuil
Failing tests for bug #519319.
795
        self.assertPushSucceeds([], with_warning=True)
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
796
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
797
    def test_push_with_revision(self):
5171.2.3 by Vincent Ladeuil
Fixed as per Andrew's review.
798
        self.assertPushSucceeds(['-r', 'revid:added'], revid_to_push='added')
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
799
800
    def test_push_no_strict(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
801
        self.assertPushSucceeds(['--no-strict'])
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
802
803
    def test_push_strict_with_changes(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
804
        self.assertPushFails(['--strict'])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
805
806
    def test_push_respect_config_var_strict(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
807
        self.set_config_push_strict('true')
808
        self.assertPushFails([])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
809
4420.1.6 by Vincent Ladeuil
Fixed as per John's review feedback.
810
    def test_push_bogus_config_var_ignored(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
811
        self.set_config_push_strict("I don't want you to be strict")
5147.2.1 by Vincent Ladeuil
Failing tests for bug #519319.
812
        self.assertPushSucceeds([], with_warning=True)
4420.1.6 by Vincent Ladeuil
Fixed as per John's review feedback.
813
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
814
    def test_push_no_strict_command_line_override_config(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
815
        self.set_config_push_strict('yES')
816
        self.assertPushFails([])
817
        self.assertPushSucceeds(['--no-strict'])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
818
819
    def test_push_strict_command_line_override_config(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
820
        self.set_config_push_strict('oFF')
821
        self.assertPushFails(['--strict'])
822
        self.assertPushSucceeds([])
4925.1.1 by Jelmer Vernooij
Print a proper error when attempting to push to a foreign VCS for which
823
824
5283.4.5 by Martin Pool
Update remaining subclasses of ExternalBase
825
class TestPushForeign(tests.TestCaseWithTransport):
4925.1.1 by Jelmer Vernooij
Print a proper error when attempting to push to a foreign VCS for which
826
827
    def setUp(self):
828
        super(TestPushForeign, self).setUp()
829
        test_foreign.register_dummy_foreign_for_test(self)
830
831
    def make_dummy_builder(self, relpath):
832
        builder = self.make_branch_builder(
833
            relpath, format=test_foreign.DummyForeignVcsDirFormat())
834
        builder.build_snapshot('revid', None,
835
            [('add', ('', 'TREE_ROOT', 'directory', None)),
836
             ('add', ('foo', 'fooid', 'file', 'bar'))])
837
        return builder
838
839
    def test_no_roundtripping(self):
840
        target_branch = self.make_dummy_builder('dp').get_branch()
841
        source_tree = self.make_branch_and_tree("dc")
842
        output, error = self.run_bzr("push -d dc dp", retcode=3)
843
        self.assertEquals("", output)
844
        self.assertEquals(error, "bzr: ERROR: It is not possible to losslessly"
845
            " push to dummy. You may want to use dpush instead.\n")
6175.2.4 by Vincent Ladeuil
Add script tests and documentation.
846
847
848
class TestPushOutput(script.TestCaseWithTransportAndScript):
849
850
    def test_push_log_format(self):
851
        self.run_script("""
852
            $ bzr init trunk
853
            Created a standalone tree (format: 2a)
854
            $ cd trunk
855
            $ echo foo > file
856
            $ bzr add
857
            adding file
858
            $ bzr commit -m 'we need some foo'
859
            2>Committing to:...trunk/
860
            2>added file
861
            2>Committed revision 1.
862
            $ bzr init ../feature
863
            Created a standalone tree (format: 2a)
864
            $ bzr push -v ../feature -Olog_format=line
865
            Added Revisions:
866
            1: jrandom@example.com ...we need some foo
867
            2>All changes applied successfully.
868
            2>Pushed up to revision 1.
869
            """)