/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.
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
137
        self.thisFailsStrictLockCheck()
1692.3.1 by Robert Collins
Fix push to work with just a branch, no need for a working tree.
138
        b = self.make_branch('.')
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
139
        out, err = self.run_bzr('push pushed-location')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
140
        self.assertEqual('', out)
141
        self.assertEqual('Created new branch.\n', err)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
142
        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.
143
        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).
144
145
    def test_push_new_branch_revision_count(self):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
146
        # bzr push of a branch with revisions to a new location
147
        # 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).
148
        # local branch.
4523.4.18 by John Arbash Meinel
tracked down a few more blackbox test cases that were using push.
149
        self.thisFailsStrictLockCheck()
1692.3.6 by Robert Collins
Show the correct number of revisions pushed when pushing a new branch (Robert Collins).
150
        t = self.make_branch_and_tree('tree')
151
        self.build_tree(['tree/file'])
152
        t.add('file')
153
        t.commit('commit 1')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
154
        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.
155
        self.assertEqual('', out)
156
        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???)
157
158
    def test_push_only_pushes_history(self):
159
        # Knit branches should only push the history for the current revision.
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
160
        self.thisFailsStrictLockCheck()
4420.1.1 by Vincent Ladeuil
Cleanup imports.
161
        format = bzrdir.BzrDirMetaFormat1()
162
        format.repository_format = knitrepo.RepositoryFormatKnit1()
1711.2.3 by John Arbash Meinel
Fix push to only push revisions in the current ancestry. (bug???)
163
        shared_repo = self.make_repository('repo', format=format, shared=True)
164
        shared_repo.set_make_working_trees(True)
165
166
        def make_shared_tree(path):
167
            shared_repo.bzrdir.root_transport.mkdir(path)
168
            shared_repo.bzrdir.create_branch_convenience('repo/' + path)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
169
            return workingtree.WorkingTree.open('repo/' + path)
1711.2.3 by John Arbash Meinel
Fix push to only push revisions in the current ancestry. (bug???)
170
        tree_a = make_shared_tree('a')
171
        self.build_tree(['repo/a/file'])
172
        tree_a.add('file')
173
        tree_a.commit('commit a-1', rev_id='a-1')
174
        f = open('repo/a/file', 'ab')
175
        f.write('more stuff\n')
176
        f.close()
177
        tree_a.commit('commit a-2', rev_id='a-2')
178
179
        tree_b = make_shared_tree('b')
180
        self.build_tree(['repo/b/file'])
181
        tree_b.add('file')
182
        tree_b.commit('commit b-1', rev_id='b-1')
183
184
        self.assertTrue(shared_repo.has_revision('a-1'))
185
        self.assertTrue(shared_repo.has_revision('a-2'))
186
        self.assertTrue(shared_repo.has_revision('b-1'))
187
188
        # Now that we have a repository with shared files, make sure
189
        # that things aren't copied out by a 'push'
4420.1.1 by Vincent Ladeuil
Cleanup imports.
190
        self.run_bzr('push ../../push-b', working_dir='repo/b')
191
        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???)
192
        pushed_repo = pushed_tree.branch.repository
193
        self.assertFalse(pushed_repo.has_revision('a-1'))
194
        self.assertFalse(pushed_repo.has_revision('a-2'))
195
        self.assertTrue(pushed_repo.has_revision('b-1'))
196
1843.2.1 by Aaron Bentley
Add failing tests for funky ids
197
    def test_push_funky_id(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
198
        self.thisFailsStrictLockCheck()
1843.2.1 by Aaron Bentley
Add failing tests for funky ids
199
        t = self.make_branch_and_tree('tree')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
200
        self.build_tree(['tree/filename'])
1843.2.1 by Aaron Bentley
Add failing tests for funky ids
201
        t.add('filename', 'funky-chars<>%&;"\'')
202
        t.commit('commit filename')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
203
        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
204
2220.2.9 by Martin Pool
Add specific tests for push -d and pull -d
205
    def test_push_dash_d(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
206
        self.thisFailsStrictLockCheck()
2220.2.9 by Martin Pool
Add specific tests for push -d and pull -d
207
        t = self.make_branch_and_tree('from')
208
        t.commit(allow_pointless=True,
209
                message='first commit')
2530.3.1 by Martin Pool
Cleanup old variations on run_bzr in the test suite
210
        self.run_bzr('push -d from to-one')
2220.2.9 by Martin Pool
Add specific tests for push -d and pull -d
211
        self.failUnlessExists('to-one')
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
212
        self.run_bzr('push -d %s %s'
2220.2.9 by Martin Pool
Add specific tests for push -d and pull -d
213
            % tuple(map(urlutils.local_path_to_url, ['from', 'to-two'])))
214
        self.failUnlessExists('to-two')
2279.3.1 by mbp at sourcefrog
Add a -d option to push, pull, merge (ported from tags branch)
215
4011.2.1 by Robert Collins
Add ratchet style blackbox effort tests for push over bzr+ssh. (Andrew Bennetts, Robert Collins)
216
    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.
217
        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)
218
        t = self.make_branch_and_tree('from')
219
        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.
220
        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)
221
        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.
222
        # This figure represent the amount of work to perform this use case. It
223
        # is entirely ok to reduce this number if a test fails due to rpc_count
224
        # being too low. If rpc_count increases, more network roundtrips have
225
        # become necessary for this use case. Please do not adjust this number
226
        # upwards without agreement from bzr's network support maintainers.
4307.2.2 by Robert Collins
Lock repositories created by BzrDirFormat.initialize_on_transport_ex.
227
        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)
228
229
    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.
230
        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)
231
        parent = self.make_branch_and_tree('parent', format='1.9')
232
        parent.commit(message='first commit')
233
        local = parent.bzrdir.sprout('local').open_workingtree()
234
        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.
235
        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)
236
        self.run_bzr(['push', '--stacked', '--stacked-on', '../parent',
237
            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.
238
        # This figure represent the amount of work to perform this use case. It
239
        # is entirely ok to reduce this number if a test fails due to rpc_count
240
        # being too low. If rpc_count increases, more network roundtrips have
241
        # become necessary for this use case. Please do not adjust this number
242
        # 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).
243
        self.assertLength(14, self.hpss_calls)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
244
        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)
245
        self.assertEndsWith(remote.get_stacked_on_url(), '/parent')
246
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.
247
    def test_push_smart_tags_streaming_acceptance(self):
248
        self.setup_smart_server_with_call_log()
249
        t = self.make_branch_and_tree('from')
250
        rev_id = t.commit(allow_pointless=True, message='first commit')
251
        t.branch.tags.set_tag('new-tag', rev_id)
252
        self.reset_smart_call_log()
253
        self.run_bzr(['push', self.get_url('to-one')], working_dir='from')
254
        # This figure represent the amount of work to perform this use case. It
255
        # is entirely ok to reduce this number if a test fails due to rpc_count
256
        # being too low. If rpc_count increases, more network roundtrips have
257
        # become necessary for this use case. Please do not adjust this number
258
        # upwards without agreement from bzr's network support maintainers.
259
        self.assertLength(11, self.hpss_calls)
260
4416.3.1 by Jonathan Lange
Add two tests, the second of which demonstrates bug 385132.
261
    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.
262
        # If the default stacked-on location is a path element then branches
263
        # we push there over the smart server are stacked and their
264
        # 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.
265
        self.setup_smart_server_with_call_log()
266
        self.make_branch('stack-on', format='1.9')
4416.3.7 by Jonathan Lange
Update the test to fail correctly.
267
        self.make_bzrdir('.').get_config().set_default_stack_on(
4416.3.9 by Jonathan Lange
Put the test back the way it was.
268
            '/stack-on')
4416.3.1 by Jonathan Lange
Add two tests, the second of which demonstrates bug 385132.
269
        self.make_branch('from', format='1.9')
270
        out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])
4416.3.10 by Jonathan Lange
Merge trunk
271
        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.
272
        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.
273
274
    def test_push_smart_with_default_stacking_relative_path(self):
4416.3.2 by Jonathan Lange
Comment the blackbox tests, for my own sanity.
275
        # If the default stacked-on location is a relative path then branches
276
        # we push there over the smart server are stacked and their
277
        # 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.
278
        self.setup_smart_server_with_call_log()
279
        self.make_branch('stack-on', format='1.9')
280
        self.make_bzrdir('.').get_config().set_default_stack_on('stack-on')
281
        self.make_branch('from', format='1.9')
282
        out, err = self.run_bzr(['push', '-d', 'from', self.get_url('to')])
4416.3.10 by Jonathan Lange
Merge trunk
283
        b = branch.Branch.open(self.get_url('to'))
284
        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.
285
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
286
    def create_simple_tree(self):
287
        tree = self.make_branch_and_tree('tree')
288
        self.build_tree(['tree/a'])
289
        tree.add(['a'], ['a-id'])
290
        tree.commit('one', rev_id='r1')
291
        return tree
292
293
    def test_push_create_prefix(self):
294
        """'bzr push --create-prefix' will create leading directories."""
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
295
        self.thisFailsStrictLockCheck()
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
296
        tree = self.create_simple_tree()
297
298
        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.
299
                           'push ../new/tree',
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
300
                           working_dir='tree')
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
301
        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
302
                     working_dir='tree')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
303
        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
304
        self.assertEqual(tree.last_revision(), new_tree.last_revision())
305
        self.failUnlessExists('new/tree/a')
306
307
    def test_push_use_existing(self):
2227.3.4 by John Arbash Meinel
Switch back to --use-existing-dir
308
        """'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
309
310
        By default, 'bzr push' will not use an existing, non-versioned dir.
311
        """
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
312
        self.thisFailsStrictLockCheck()
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
313
        tree = self.create_simple_tree()
314
        self.build_tree(['target/'])
315
316
        self.run_bzr_error(['Target directory ../target already exists',
2227.3.4 by John Arbash Meinel
Switch back to --use-existing-dir
317
                            'Supply --use-existing-dir',
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
318
                           ],
319
                           'push ../target', working_dir='tree')
2227.3.4 by John Arbash Meinel
Switch back to --use-existing-dir
320
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
321
        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
322
                     working_dir='tree')
323
4420.1.1 by Vincent Ladeuil
Cleanup imports.
324
        new_tree = workingtree.WorkingTree.open('target')
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
325
        self.assertEqual(tree.last_revision(), new_tree.last_revision())
326
        # The push should have created target/a
327
        self.failUnlessExists('target/a')
328
329
    def test_push_onto_repo(self):
330
        """We should be able to 'bzr push' into an existing bzrdir."""
331
        tree = self.create_simple_tree()
332
        repo = self.make_repository('repo', shared=True)
333
2552.2.3 by Vincent Ladeuil
Deprecate the varargs syntax and fix the tests.
334
        self.run_bzr('push ../repo',
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
335
                     working_dir='tree')
336
337
        # Pushing onto an existing bzrdir will create a repository and
338
        # branch as needed, but will only create a working tree if there was
339
        # no BzrDir before.
4420.1.1 by Vincent Ladeuil
Cleanup imports.
340
        self.assertRaises(errors.NoWorkingTree,
341
                          workingtree.WorkingTree.open, 'repo')
342
        new_branch = branch.Branch.open('repo')
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
343
        self.assertEqual(tree.last_revision(), new_branch.last_revision())
344
345
    def test_push_onto_just_bzrdir(self):
346
        """We don't handle when the target is just a bzrdir.
347
348
        Because you shouldn't be able to create *just* a bzrdir in the wild.
349
        """
350
        # TODO: jam 20070109 Maybe it would be better to create the repository
351
        #       if at this point
352
        tree = self.create_simple_tree()
353
        a_bzrdir = self.make_bzrdir('dir')
354
355
        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.
356
                'push ../dir',
2227.3.1 by John Arbash Meinel
Allow push to create Branch when necessary, and add --use-existing
357
                working_dir='tree')
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
358
3256.1.1 by Daniel Watkins
Added test for push with a revspec.
359
    def test_push_with_revisionspec(self):
3256.1.3 by Daniel Watkins
Clarified test intent.
360
        """We should be able to push a revision older than the tip."""
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
361
        self.thisFailsStrictLockCheck()
3256.1.1 by Daniel Watkins
Added test for push with a revspec.
362
        tree_from = self.make_branch_and_tree('from')
363
        tree_from.commit("One.", rev_id="from-1")
364
        tree_from.commit("Two.", rev_id="from-2")
365
366
        self.run_bzr('push -r1 ../to', working_dir='from')
367
4420.1.1 by Vincent Ladeuil
Cleanup imports.
368
        tree_to = workingtree.WorkingTree.open('to')
3256.1.1 by Daniel Watkins
Added test for push with a revspec.
369
        repo_to = tree_to.branch.repository
370
        self.assertTrue(repo_to.has_revision('from-1'))
371
        self.assertFalse(repo_to.has_revision('from-2'))
372
        self.assertEqual(tree_to.branch.last_revision_info()[1], 'from-1')
373
3256.1.4 by Daniel Watkins
Added test to ensure that passing a range of revisions errors.
374
        self.run_bzr_error(
4325.4.6 by Vincent Ladeuil
Fixed as per John's and Markus reviews.
375
            ['bzr: ERROR: bzr push --revision '
376
             'takes exactly one revision identifier\n'],
3256.1.4 by Daniel Watkins
Added test to ensure that passing a range of revisions errors.
377
            'push -r0..2 ../to', working_dir='from')
378
3221.11.13 by Robert Collins
Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.
379
    def create_trunk_and_feature_branch(self):
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
380
        # We have a mainline
381
        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)
382
            format='1.9')
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
383
        trunk_tree.commit('mainline')
384
        # and a branch from it
385
        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)
386
            format='1.9')
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
387
        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.
388
        branch_tree.branch.set_parent(trunk_tree.branch.base)
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
389
        # 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.
390
        branch_tree.commit('moar work plz')
391
        return trunk_tree, branch_tree
392
3221.11.14 by Robert Collins
Refactor to reduce duplication.
393
    def assertPublished(self, branch_revid, stacked_on):
394
        """Assert that the branch 'published' has been published correctly."""
4420.1.1 by Vincent Ladeuil
Cleanup imports.
395
        published_branch = branch.Branch.open('published')
3221.11.14 by Robert Collins
Refactor to reduce duplication.
396
        # The published branch refers to the mainline
3537.3.1 by Martin Pool
Rename branch.get_stacked_on to get_stacked_on_url
397
        self.assertEqual(stacked_on, published_branch.get_stacked_on_url())
3221.11.14 by Robert Collins
Refactor to reduce duplication.
398
        # and the branch's work was pushed
399
        self.assertTrue(published_branch.repository.has_revision(branch_revid))
400
3549.1.1 by Martin Pool
rename push --reference to --stacked-on
401
    def test_push_new_branch_stacked_on(self):
402
        """Pushing a new branch with --stacked-on creates a stacked branch."""
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
403
        self.thisFailsStrictLockCheck()
3221.11.13 by Robert Collins
Allow push --shallow to just work, and fix the testing HTTPServer to not be affected by chdir() calls.
404
        trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
405
        # we publish branch_tree with a reference to the mainline.
3549.1.1 by Martin Pool
rename push --reference to --stacked-on
406
        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.
407
            self.get_url('published')], working_dir='branch')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
408
        self.assertEqual('', out)
3221.19.4 by Ian Clatworthy
shallow -> stacked
409
        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.
410
            trunk_tree.branch.base, err)
3221.11.14 by Robert Collins
Refactor to reduce duplication.
411
        self.assertPublished(branch_tree.last_revision(),
412
            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.
413
3221.19.4 by Ian Clatworthy
shallow -> stacked
414
    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.
415
        """When the parent has no public url the parent is used as-is."""
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
416
        self.thisFailsStrictLockCheck()
3221.11.17 by Robert Collins
no public location causes the parent to be used directly with push --shallow.
417
        trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
3221.19.4 by Ian Clatworthy
shallow -> stacked
418
        # 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.
419
        # for us.
3221.19.4 by Ian Clatworthy
shallow -> stacked
420
        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.
421
            self.get_url('published')], working_dir='branch')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
422
        self.assertEqual('', out)
3221.19.4 by Ian Clatworthy
shallow -> stacked
423
        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.
424
            trunk_tree.branch.base, err)
4420.1.1 by Vincent Ladeuil
Cleanup imports.
425
        self.assertPublished(branch_tree.last_revision(),
426
                             trunk_tree.branch.base)
3221.11.17 by Robert Collins
no public location causes the parent to be used directly with push --shallow.
427
3221.19.4 by Ian Clatworthy
shallow -> stacked
428
    def test_push_new_branch_stacked_uses_parent_public(self):
429
        """Pushing a new branch with --stacked creates a stacked branch."""
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
430
        self.thisFailsStrictLockCheck()
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
        trunk_tree, branch_tree = self.create_trunk_and_feature_branch()
432
        # the trunk is published on a web server
4420.1.1 by Vincent Ladeuil
Cleanup imports.
433
        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)
434
        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.
435
        trunk_public.pull(trunk_tree.branch)
436
        trunk_public_url = self.get_readonly_url('public_trunk')
437
        trunk_tree.branch.set_public_branch(trunk_public_url)
3221.19.4 by Ian Clatworthy
shallow -> stacked
438
        # 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.
439
        # for us.
3221.19.4 by Ian Clatworthy
shallow -> stacked
440
        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.
441
            self.get_url('published')], working_dir='branch')
3978.2.2 by Jelmer Vernooij
Write status messages during push to stderr rather than stdout.
442
        self.assertEqual('', out)
3221.19.4 by Ian Clatworthy
shallow -> stacked
443
        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.
444
            trunk_public_url, err)
3221.11.14 by Robert Collins
Refactor to reduce duplication.
445
        self.assertPublished(branch_tree.last_revision(), trunk_public_url)
3221.11.12 by Robert Collins
Basic push --reference support, requires url, slow.
446
3221.19.4 by Ian Clatworthy
shallow -> stacked
447
    def test_push_new_branch_stacked_no_parent(self):
448
        """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)
449
        branch = self.make_branch_and_tree('branch', format='1.9')
3221.19.4 by Ian Clatworthy
shallow -> stacked
450
        # 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.
451
        # cannot be determined.
452
        out, err = self.run_bzr_error(
3221.19.4 by Ian Clatworthy
shallow -> stacked
453
            ['Could not determine branch to refer to\\.'], ['push', '--stacked',
3221.11.15 by Robert Collins
no parent branch causes an error on push --shallow.
454
            self.get_url('published')], working_dir='branch')
455
        self.assertEqual('', out)
456
        self.assertFalse(self.get_transport('published').has('.'))
457
3606.8.3 by John Arbash Meinel
push doesn't notify because it doesn't notice by default.
458
    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.
459
        self.make_branch('stack_on', format='1.6')
3242.3.34 by Aaron Bentley
Add notification of default stacking
460
        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.
461
        self.make_branch('from', format='1.6')
3242.3.34 by Aaron Bentley
Add notification of default stacking
462
        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
463
        self.assertContainsRe(err,
464
                              'Using default stacking branch stack_on at .*')
3242.3.34 by Aaron Bentley
Add notification of default stacking
465
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.
466
    def test_push_stacks_with_default_stacking_if_target_is_stackable(self):
467
        self.make_branch('stack_on', format='1.6')
468
        self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
469
        self.make_branch('from', format='pack-0.92')
470
        out, err = self.run_bzr('push -d from to')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
471
        b = branch.Branch.open('to')
472
        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.
473
474
    def test_push_does_not_change_format_with_default_if_target_cannot(self):
475
        self.make_branch('stack_on', format='pack-0.92')
476
        self.make_bzrdir('.').get_config().set_default_stack_on('stack_on')
477
        self.make_branch('from', format='pack-0.92')
478
        out, err = self.run_bzr('push -d from to')
4420.1.1 by Vincent Ladeuil
Cleanup imports.
479
        b = branch.Branch.open('to')
480
        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.
481
3904.3.2 by Andrew Bennetts
Blackbox test that triggers the bug. Should get replaced with a unit test.
482
    def test_push_doesnt_create_broken_branch(self):
3904.3.7 by Andrew Bennetts
Comment the new tests.
483
        """Pushing a new standalone branch works even when there's a default
484
        stacking policy at the destination.
485
486
        The new branch will preserve the repo format (even if it isn't the
487
        default for the branch), and will be stacked when the repo format
488
        allows (which means that the branch format isn't necessarly preserved).
489
        """
3904.3.2 by Andrew Bennetts
Blackbox test that triggers the bug. Should get replaced with a unit test.
490
        self.make_repository('repo', shared=True, format='1.6')
491
        builder = self.make_branch_builder('repo/local', format='pack-0.92')
492
        builder.start_series()
493
        builder.build_snapshot('rev-1', None, [
494
            ('add', ('', 'root-id', 'directory', '')),
3904.3.3 by Andrew Bennetts
Simplify test slightly.
495
            ('add', ('filename', 'f-id', 'file', 'content\n'))])
496
        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.
497
        builder.build_snapshot('rev-3', ['rev-2'],
3904.3.3 by Andrew Bennetts
Simplify test slightly.
498
            [('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.
499
        builder.finish_series()
500
        branch = builder.get_branch()
3904.3.7 by Andrew Bennetts
Comment the new tests.
501
        # 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.
502
        self.run_bzr('push -d repo/local trunk -r 1')
3904.3.7 by Andrew Bennetts
Comment the new tests.
503
        # Set a default stacking policy so that new branches will automatically
504
        # stack on trunk.
3904.3.2 by Andrew Bennetts
Blackbox test that triggers the bug. Should get replaced with a unit test.
505
        self.make_bzrdir('.').get_config().set_default_stack_on('trunk')
3904.3.7 by Andrew Bennetts
Comment the new tests.
506
        # 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.
507
        out, err = self.run_bzr('push -d repo/local remote -r 2')
508
        self.assertContainsRe(
509
            err, 'Using default stacking branch trunk at .*')
3904.3.7 by Andrew Bennetts
Comment the new tests.
510
        # Push rev-3 onto "remote".  If "remote" not stacked and is missing the
511
        # 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.
512
        out, err = self.run_bzr('push -d repo/local remote -r 3')
513
3848.1.19 by Aaron Bentley
Show log for non-initial push -v
514
    def test_push_verbose_shows_log(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
515
        self.thisFailsStrictLockCheck()
3848.1.19 by Aaron Bentley
Show log for non-initial push -v
516
        tree = self.make_branch_and_tree('source')
517
        tree.commit('rev1')
518
        out, err = self.run_bzr('push -v -d source target')
519
        # initial push contains log
520
        self.assertContainsRe(out, 'rev1')
521
        tree.commit('rev2')
522
        out, err = self.run_bzr('push -v -d source target')
523
        # subsequent push contains log
524
        self.assertContainsRe(out, 'rev2')
525
        # subsequent log is accurate
526
        self.assertNotContainsRe(out, 'rev1')
527
4453.1.2 by Matthew Fuller
Add a test for pushing from subdir.
528
    def test_push_from_subdir(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
529
        self.thisFailsStrictLockCheck()
4453.1.2 by Matthew Fuller
Add a test for pushing from subdir.
530
        t = self.make_branch_and_tree('tree')
531
        self.build_tree(['tree/dir/', 'tree/dir/file'])
532
        t.add('dir', 'dir/file')
533
        t.commit('r1')
534
        out, err = self.run_bzr('push ../../pushloc', working_dir='tree/dir')
535
        self.assertEqual('', out)
536
        self.assertEqual('Created new branch.\n', err)
537
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
538
4420.1.1 by Vincent Ladeuil
Cleanup imports.
539
class RedirectingMemoryTransport(memory.MemoryTransport):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
540
3878.4.4 by Vincent Ladeuil
Cleanup.
541
    def mkdir(self, relpath, mode=None):
542
        if self._cwd == '/source/':
543
            raise errors.RedirectRequested(self.abspath(relpath),
544
                                           self.abspath('../target'),
545
                                           is_permanent=True)
546
        elif self._cwd == '/infinite-loop/':
547
            raise errors.RedirectRequested(self.abspath(relpath),
548
                                           self.abspath('../infinite-loop'),
549
                                           is_permanent=True)
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
550
        else:
551
            return super(RedirectingMemoryTransport, self).mkdir(
3878.4.4 by Vincent Ladeuil
Cleanup.
552
                relpath, mode)
553
4547.2.2 by Andrew Bennetts
Add test for read_mergeable_from_transport raising NotABundle when TooManyRedirections happens.
554
    def get(self, relpath):
555
        if self.clone(relpath)._cwd == '/infinite-loop/':
556
            raise errors.RedirectRequested(self.abspath(relpath),
557
                                           self.abspath('../infinite-loop'),
558
                                           is_permanent=True)
559
        else:
560
            return super(RedirectingMemoryTransport, self).get(relpath)
561
3878.4.5 by Vincent Ladeuil
Don't use the exception as a parameter for _redirected_to.
562
    def _redirected_to(self, source, target):
3878.4.4 by Vincent Ladeuil
Cleanup.
563
        # We do accept redirections
3878.4.5 by Vincent Ladeuil
Don't use the exception as a parameter for _redirected_to.
564
        return transport.get_transport(target)
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
565
566
4420.1.1 by Vincent Ladeuil
Cleanup imports.
567
class RedirectingMemoryServer(memory.MemoryServer):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
568
569
    def setUp(self):
570
        self._dirs = {'/': None}
571
        self._files = {}
572
        self._locks = {}
573
        self._scheme = 'redirecting-memory+%s:///' % id(self)
3878.4.4 by Vincent Ladeuil
Cleanup.
574
        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.
575
576
    def _memory_factory(self, url):
577
        result = RedirectingMemoryTransport(url)
578
        result._dirs = self._dirs
579
        result._files = self._files
580
        result._locks = self._locks
581
        return result
582
583
    def tearDown(self):
3878.4.4 by Vincent Ladeuil
Cleanup.
584
        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
585
586
4420.1.1 by Vincent Ladeuil
Cleanup imports.
587
class TestPushRedirect(tests.TestCaseWithTransport):
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
588
589
    def setUp(self):
4420.1.1 by Vincent Ladeuil
Cleanup imports.
590
        tests.TestCaseWithTransport.setUp(self)
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
591
        self.memory_server = RedirectingMemoryServer()
592
        self.memory_server.setUp()
593
        self.addCleanup(self.memory_server.tearDown)
594
3066.3.2 by jml at canonical
Add tests to check the handling of TooManyRedirections.
595
        # Make the branch and tree that we'll be pushing.
596
        t = self.make_branch_and_tree('tree')
597
        self.build_tree(['tree/file'])
598
        t.add('file')
599
        t.commit('commit 1')
600
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
601
    def test_push_redirects_on_mkdir(self):
602
        """If the push requires a mkdir, push respects redirect requests.
603
604
        This is added primarily to handle lp:/ URI support, so that users can
605
        push to new branches by specifying lp:/ URIs.
606
        """
607
        destination_url = self.memory_server.get_url() + 'source'
3878.4.4 by Vincent Ladeuil
Cleanup.
608
        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
609
4420.1.1 by Vincent Ladeuil
Cleanup imports.
610
        local_revision = branch.Branch.open('tree').last_revision()
611
        remote_revision = branch.Branch.open(
3066.3.1 by jml at canonical
Catch redirects raised by mkdir() in the push command. This is primarily
612
            self.memory_server.get_url() + 'target').last_revision()
613
        self.assertEqual(remote_revision, local_revision)
3066.3.2 by jml at canonical
Add tests to check the handling of TooManyRedirections.
614
615
    def test_push_gracefully_handles_too_many_redirects(self):
616
        """Push fails gracefully if the mkdir generates a large number of
617
        redirects.
618
        """
619
        destination_url = self.memory_server.get_url() + 'infinite-loop'
620
        out, err = self.run_bzr_error(
621
            ['Too many redirections trying to make %s\\.\n'
622
             % re.escape(destination_url)],
3878.4.4 by Vincent Ladeuil
Cleanup.
623
            ['push', '-d', 'tree', destination_url], retcode=3)
3066.3.2 by jml at canonical
Add tests to check the handling of TooManyRedirections.
624
        self.assertEqual('', out)
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
625
626
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
627
class TestPushStrictMixin(object):
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
628
629
    def make_local_branch_and_tree(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
630
        self.tree = self.make_branch_and_tree('local')
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
631
        self.build_tree_contents([('local/file', 'initial')])
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
632
        self.tree.add('file')
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
633
        self.tree.commit('adding file', rev_id='added')
634
        self.build_tree_contents([('local/file', 'modified')])
635
        self.tree.commit('modify file', rev_id='modified')
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
636
637
    def set_config_push_strict(self, value):
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
638
        # set config var (any of bazaar.conf, locations.conf, branch.conf
639
        # should do)
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
640
        conf = self.tree.branch.get_config()
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
641
        conf.set_user_option('push_strict', value)
642
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
643
    _default_command = ['push', '../to']
644
    _default_wd = 'local'
645
    _default_errors = ['Working tree ".*/local/" has uncommitted '
646
                       'changes \(See bzr status\)\.',]
647
    _default_pushed_revid = 'modified'
648
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
649
    def assertPushFails(self, args):
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
650
        self.run_bzr_error(self._default_errors, self._default_command + args,
651
                           working_dir=self._default_wd, retcode=3)
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
652
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
653
    def assertPushSucceeds(self, args, pushed_revid=None):
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
654
        self.run_bzr(self._default_command + args,
655
                     working_dir=self._default_wd)
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
656
        if pushed_revid is None:
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
657
            pushed_revid = self._default_pushed_revid
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
658
        tree_to = workingtree.WorkingTree.open('to')
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
659
        repo_to = tree_to.branch.repository
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
660
        self.assertTrue(repo_to.has_revision(pushed_revid))
661
        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.
662
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
663
664
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
665
class TestPushStrictWithoutChanges(tests.TestCaseWithTransport,
666
                                   TestPushStrictMixin):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
667
668
    def setUp(self):
669
        super(TestPushStrictWithoutChanges, self).setUp()
670
        self.make_local_branch_and_tree()
671
672
    def test_push_default(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
673
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
674
        self.assertPushSucceeds([])
675
676
    def test_push_strict(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
677
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
678
        self.assertPushSucceeds(['--strict'])
679
680
    def test_push_no_strict(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
681
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
682
        self.assertPushSucceeds(['--no-strict'])
683
684
    def test_push_config_var_strict(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
685
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
686
        self.set_config_push_strict('true')
687
        self.assertPushSucceeds([])
688
689
    def test_push_config_var_no_strict(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
690
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
691
        self.set_config_push_strict('false')
692
        self.assertPushSucceeds([])
693
694
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
695
class TestPushStrictWithChanges(tests.TestCaseWithTransport,
696
                                TestPushStrictMixin):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
697
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
698
    _changes_type = None # Set by load_tests
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
699
700
    def setUp(self):
701
        super(TestPushStrictWithChanges, self).setUp()
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
702
        getattr(self, self._changes_type)()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
703
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
704
    def _uncommitted_changes(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
705
        self.make_local_branch_and_tree()
706
        # Make a change without committing it
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
707
        self.build_tree_contents([('local/file', 'in progress')])
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
708
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
709
    def _pending_merges(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
710
        self.make_local_branch_and_tree()
711
        # Create 'other' branch containing a new file
712
        other_bzrdir = self.tree.bzrdir.sprout('other')
713
        other_tree = other_bzrdir.open_workingtree()
714
        self.build_tree_contents([('other/other-file', 'other')])
715
        other_tree.add('other-file')
716
        other_tree.commit('other commit', rev_id='other')
717
        # Merge and revert, leaving a pending merge
718
        self.tree.merge_from_branch(other_tree.branch)
719
        self.tree.revert(filenames=['other-file'], backups=False)
720
4487.2.6 by Vincent Ladeuil
Fixed as per jam's review.
721
    def _out_of_sync_trees(self):
722
        self.make_local_branch_and_tree()
723
        self.run_bzr(['checkout', '--lightweight', 'local', 'checkout'])
724
        # Make a change and commit it
725
        self.build_tree_contents([('local/file', 'modified in local')])
726
        self.tree.commit('modify file', rev_id='modified-in-local')
727
        # Exercise commands from the checkout directory
728
        self._default_wd = 'checkout'
729
        self._default_errors = ["Working tree is out of date, please run"
730
                                " 'bzr update'\.",]
731
        self._default_pushed_revid = 'modified-in-local'
732
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
733
    def test_push_default(self):
734
        self.assertPushFails([])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
735
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
736
    def test_push_with_revision(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
737
        self.thisFailsStrictLockCheck()
4487.2.3 by Vincent Ladeuil
--strict is meaningless when --revision is specified.
738
        self.assertPushSucceeds(['-r', 'revid:added'], pushed_revid='added')
739
740
    def test_push_no_strict(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
741
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
742
        self.assertPushSucceeds(['--no-strict'])
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
743
744
    def test_push_strict_with_changes(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
745
        self.assertPushFails(['--strict'])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
746
747
    def test_push_respect_config_var_strict(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
748
        self.set_config_push_strict('true')
749
        self.assertPushFails([])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
750
4420.1.6 by Vincent Ladeuil
Fixed as per John's review feedback.
751
    def test_push_bogus_config_var_ignored(self):
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
752
        self.set_config_push_strict("I don't want you to be strict")
753
        self.assertPushFails([])
4420.1.6 by Vincent Ladeuil
Fixed as per John's review feedback.
754
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
755
    def test_push_no_strict_command_line_override_config(self):
4523.4.15 by John Arbash Meinel
First pass at marking tests that are known to fail.
756
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
757
        self.set_config_push_strict('yES')
758
        self.assertPushFails([])
759
        self.assertPushSucceeds(['--no-strict'])
4420.1.5 by Vincent Ladeuil
Start implementing jam's review feedback.
760
761
    def test_push_strict_command_line_override_config(self):
4523.4.18 by John Arbash Meinel
tracked down a few more blackbox test cases that were using push.
762
        self.thisFailsStrictLockCheck()
4487.2.2 by Vincent Ladeuil
Add tests for push --strict against pending merges.
763
        self.set_config_push_strict('oFF')
764
        self.assertPushFails(['--strict'])
765
        self.assertPushSucceeds([])