/brz/remove-bazaar

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