~bzr-pqm/bzr/bzr.dev

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