~bzr-pqm/bzr/bzr.dev

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