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