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