~bzr-pqm/bzr/bzr.dev

5609.26.1 by John Arbash Meinel
Fix bug #465517, 'bzr push' to a target with a repo but no branch
1
# Copyright (C) 2007-2011 Canonical Ltd
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
2
#
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.
7
#
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.
12
#
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
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
16
17
"""Tests for branch.push behaviour."""
18
3904.3.5 by Andrew Bennetts
Improve the test; now 4/7 passing.
19
from cStringIO import StringIO
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
20
import os
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
21
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
22
from bzrlib import (
23
    branch,
24
    builtins,
25
    bzrdir,
4332.3.35 by Robert Collins
Fix failing tests.
26
    check,
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
27
    errors,
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
28
    memorytree,
3904.3.5 by Andrew Bennetts
Improve the test; now 4/7 passing.
29
    push,
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
30
    revision,
5348.1.2 by Martin Pool
Deprecate casting PushResult and PullResult to int to get the relative revno change
31
    symbol_versioning,
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
32
    tests,
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
33
    transport,
34
    )
35
from bzrlib.smart import (
36
    client,
37
    )
5017.3.29 by Vincent Ladeuil
-s bt.per_branch.test_push passing
38
from bzrlib.tests import (
39
    per_branch,
40
    test_server,
41
    )
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
42
43
44
class TestPush(per_branch.TestCaseWithBranch):
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
45
46
    def test_push_convergence_simple(self):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
47
        # when revisions are pushed, the left-most accessible parents must
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
48
        # become the revision-history.
49
        mine = self.make_branch_and_tree('mine')
50
        mine.commit('1st post', rev_id='P1', allow_pointless=True)
51
        other = mine.bzrdir.sprout('other').open_workingtree()
52
        other.commit('my change', rev_id='M1', allow_pointless=True)
53
        mine.merge_from_branch(other.branch)
54
        mine.commit('merge my change', rev_id='P2')
2297.1.4 by Martin Pool
Push now returns a PushResult rather than just an integer.
55
        result = mine.branch.push(other.branch)
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
56
        self.assertEqual(['P1', 'P2'], other.branch.revision_history())
2297.1.4 by Martin Pool
Push now returns a PushResult rather than just an integer.
57
        # result object contains some structured data
58
        self.assertEqual(result.old_revid, 'M1')
59
        self.assertEqual(result.new_revid, 'P2')
60
        # and it can be treated as an integer for compatibility
5348.1.2 by Martin Pool
Deprecate casting PushResult and PullResult to int to get the relative revno change
61
        self.assertEqual(self.applyDeprecated(
62
            symbol_versioning.deprecated_in((2, 3, 0)),
63
            result.__int__),
64
            0)
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
65
66
    def test_push_merged_indirect(self):
67
        # it should be possible to do a push from one branch into another
68
        # when the tip of the target was merged into the source branch
69
        # via a third branch - so its buried in the ancestry and is not
70
        # directly accessible.
71
        mine = self.make_branch_and_tree('mine')
72
        mine.commit('1st post', rev_id='P1', allow_pointless=True)
73
        target = mine.bzrdir.sprout('target').open_workingtree()
74
        target.commit('my change', rev_id='M1', allow_pointless=True)
75
        other = mine.bzrdir.sprout('other').open_workingtree()
76
        other.merge_from_branch(target.branch)
77
        other.commit('merge my change', rev_id='O2')
78
        mine.merge_from_branch(other.branch)
79
        mine.commit('merge other', rev_id='P2')
80
        mine.branch.push(target.branch)
81
        self.assertEqual(['P1', 'P2'], target.branch.revision_history())
82
83
    def test_push_to_checkout_updates_master(self):
84
        """Pushing into a checkout updates the checkout and the master branch"""
85
        master_tree = self.make_branch_and_tree('master')
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
86
        checkout = self.make_branch_and_tree('checkout')
87
        try:
88
            checkout.branch.bind(master_tree.branch)
89
        except errors.UpgradeRequired:
90
            # cant bind this format, the test is irrelevant.
91
            return
92
        rev1 = checkout.commit('master')
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
93
94
        other = master_tree.branch.bzrdir.sprout('other').open_workingtree()
95
        rev2 = other.commit('other commit')
96
        # now push, which should update both checkout and master.
97
        other.branch.push(checkout.branch)
98
        self.assertEqual([rev1, rev2], checkout.branch.revision_history())
99
        self.assertEqual([rev1, rev2], master_tree.branch.revision_history())
100
101
    def test_push_raises_specific_error_on_master_connection_error(self):
102
        master_tree = self.make_branch_and_tree('master')
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
103
        checkout = self.make_branch_and_tree('checkout')
104
        try:
105
            checkout.branch.bind(master_tree.branch)
106
        except errors.UpgradeRequired:
107
            # cant bind this format, the test is irrelevant.
108
            return
2245.2.1 by Robert Collins
Split branch pushing out of branch pulling.
109
        other = master_tree.branch.bzrdir.sprout('other').open_workingtree()
110
        # move the branch out of the way on disk to cause a connection
111
        # error.
112
        os.rename('master', 'master_gone')
113
        # try to push, which should raise a BoundBranchConnectionFailure.
114
        self.assertRaises(errors.BoundBranchConnectionFailure,
115
                other.branch.push, checkout.branch)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
116
5609.25.1 by Andrew Bennetts
Add failing test for bug #733350.
117
    def test_push_new_tag_to_bound_branch(self):
118
        master = self.make_branch('master')
119
        bound = self.make_branch('bound')
120
        try:
121
            bound.bind(master)
122
        except errors.UpgradeRequired:
123
            raise tests.TestNotApplicable(
124
                'Format does not support bound branches')
125
        other = bound.bzrdir.sprout('other').open_branch()
126
        try:
127
            other.tags.set_tag('new-tag', 'some-rev')
128
        except errors.TagsNotSupported:
129
            raise tests.TestNotApplicable('Format does not support tags')
130
        other.push(bound)
5609.25.2 by Andrew Bennetts
Hackish fix: pass some extra state to our _basic_push implementation via a private instance variable, rather than changing the API of _basic_push (which bzr-svn and possibly other plugins implement).
131
        self.assertEqual({'new-tag': 'some-rev'}, bound.tags.get_tag_dict())
132
        self.assertEqual({'new-tag': 'some-rev'}, master.tags.get_tag_dict())
5609.25.1 by Andrew Bennetts
Add failing test for bug #733350.
133
2279.1.1 by John Arbash Meinel
Branch.push() only needs a read lock.
134
    def test_push_uses_read_lock(self):
135
        """Push should only need a read lock on the source side."""
136
        source = self.make_branch_and_tree('source')
137
        target = self.make_branch('target')
138
2381.1.3 by Robert Collins
Review feedback.
139
        self.build_tree(['source/a'])
2279.1.1 by John Arbash Meinel
Branch.push() only needs a read lock.
140
        source.add(['a'])
141
        source.commit('a')
142
143
        source.branch.lock_read()
144
        try:
145
            target.lock_write()
146
            try:
147
                source.branch.push(target, stop_revision=source.last_revision())
148
            finally:
149
                target.unlock()
150
        finally:
151
            source.branch.unlock()
152
2279.1.3 by John Arbash Meinel
Switch the test to being a branch_implementation test.
153
    def test_push_within_repository(self):
154
        """Push from one branch to another inside the same repository."""
155
        try:
156
            repo = self.make_repository('repo', shared=True)
157
        except (errors.IncompatibleFormat, errors.UninitializableFormat):
158
            # This Branch format cannot create shared repositories
159
            return
160
        # This is a little bit trickier because make_branch_and_tree will not
161
        # re-use a shared repository.
162
        a_bzrdir = self.make_bzrdir('repo/tree')
163
        try:
164
            a_branch = self.branch_format.initialize(a_bzrdir)
165
        except (errors.UninitializableFormat):
166
            # Cannot create these branches
167
            return
2018.5.97 by Andrew Bennetts
Fix more tests.
168
        try:
169
            tree = a_branch.bzrdir.create_workingtree()
170
        except errors.NotLocalUrl:
5017.3.29 by Vincent Ladeuil
-s bt.per_branch.test_push passing
171
            if self.vfs_transport_factory is test_server.LocalURLServer:
2018.5.130 by Robert Collins
Make all branch_implementations tests pass.
172
                # the branch is colocated on disk, we cannot create a checkout.
173
                # hopefully callers will expect this.
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
174
                local_controldir= bzrdir.BzrDir.open(
175
                    self.get_vfs_only_url('repo/tree'))
2018.5.130 by Robert Collins
Make all branch_implementations tests pass.
176
                tree = local_controldir.create_workingtree()
177
            else:
178
                tree = a_branch.create_checkout('repo/tree', lightweight=True)
2381.1.3 by Robert Collins
Review feedback.
179
        self.build_tree(['repo/tree/a'])
2279.1.3 by John Arbash Meinel
Switch the test to being a branch_implementation test.
180
        tree.add(['a'])
181
        tree.commit('a')
182
183
        to_bzrdir = self.make_bzrdir('repo/branch')
184
        to_branch = self.branch_format.initialize(to_bzrdir)
185
        tree.branch.push(to_branch)
186
187
        self.assertEqual(tree.branch.last_revision(),
188
                         to_branch.last_revision())
189
5611.2.1 by Jelmer Vernooij
Fix 'bzr push --overwrite -rOLD_MAINLINE_REV'.
190
    def test_push_overwrite_with_older_mainline_rev(self):
191
        """Pushing an older mainline revision with overwrite.
192
193
        This was <https://bugs.launchpad.net/bzr/+bug/386576>.
194
        """
195
        source = self.make_branch_and_tree('source')
196
        target = self.make_branch('target')
197
198
        source.commit('1st commit')
199
        source.commit('2nd commit', rev_id='rev-2')
200
        source.commit('3rd commit')
201
        source.branch.push(target)
202
        source.branch.push(target, stop_revision='rev-2', overwrite=True)
203
        self.assertEqual('rev-2', target.last_revision())
204
3449.1.2 by Andrew Bennetts
Add test and NEWS entry.
205
    def test_push_overwrite_of_non_tip_with_stop_revision(self):
206
        """Combining the stop_revision and overwrite options works.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
207
3449.1.2 by Andrew Bennetts
Add test and NEWS entry.
208
        This was <https://bugs.launchpad.net/bzr/+bug/234229>.
209
        """
210
        source = self.make_branch_and_tree('source')
211
        target = self.make_branch('target')
212
213
        source.commit('1st commit')
214
        source.branch.push(target)
215
        source.commit('2nd commit', rev_id='rev-2')
216
        source.commit('3rd commit')
217
218
        source.branch.push(target, stop_revision='rev-2', overwrite=True)
219
        self.assertEqual('rev-2', target.last_revision())
220
5609.26.1 by John Arbash Meinel
Fix bug #465517, 'bzr push' to a target with a repo but no branch
221
    def test_push_repository_no_branch_doesnt_fetch_all_revs(self):
222
        # See https://bugs.launchpad.net/bzr/+bug/465517
223
        t = self.get_transport('target')
224
        t.ensure_base()
225
        bzrdir = self.bzrdir_format.initialize_on_transport(t)
226
        try:
227
            bzrdir.open_branch()
228
        except errors.NotBranchError:
229
            pass
230
        else:
231
            raise tests.TestNotApplicable('older formats can\'t have a repo'
232
                                          ' without a branch')
233
        try:
234
            source = self.make_branch_builder('source',
235
                                              format=self.bzrdir_format)
236
        except errors.UninitializableFormat:
237
            raise tests.TestNotApplicable('cannot initialize this format')
238
        source.start_series()
239
        source.build_snapshot('A', None, [
240
            ('add', ('', 'root-id', 'directory', None))])
241
        source.build_snapshot('B', ['A'], [])
242
        source.build_snapshot('C', ['A'], [])
243
        source.finish_series()
244
        b = source.get_branch()
245
        # Note: We can't read lock the source branch. Some formats take a write
246
        # lock to 'set_push_location', which breaks
247
        self.addCleanup(b.lock_write().unlock)
248
        repo = bzrdir.create_repository()
249
        # This means 'push the source branch into this dir'
250
        bzrdir.push_branch(b)
251
        self.addCleanup(repo.lock_read().unlock)
252
        # We should have pushed 'C', but not 'B', since it isn't in the
253
        # ancestry
254
        self.assertEqual([('A',), ('C',)], sorted(repo.revisions.keys()))
255
3904.3.5 by Andrew Bennetts
Improve the test; now 4/7 passing.
256
    def test_push_with_default_stacking_does_not_create_broken_branch(self):
3904.3.7 by Andrew Bennetts
Comment the new tests.
257
        """Pushing a new standalone branch works even when there's a default
258
        stacking policy at the destination.
259
260
        The new branch will preserve the repo format (even if it isn't the
261
        default for the branch), and will be stacked when the repo format
262
        allows (which means that the branch format isn't necessarly preserved).
263
        """
5673.1.3 by Jelmer Vernooij
Change flexible_components to fixed_components.
264
        if self.bzrdir_format.fixed_components:
3904.3.6 by Andrew Bennetts
Skip test for two formats, and fix format 5 by avoiding a full history sync with non-format5 branches.
265
            raise tests.TestNotApplicable('Not a metadir format.')
266
        if isinstance(self.branch_format, branch.BranchReferenceFormat):
3904.3.7 by Andrew Bennetts
Comment the new tests.
267
            # This test could in principle apply to BranchReferenceFormat, but
268
            # make_branch_builder doesn't support it.
3904.3.6 by Andrew Bennetts
Skip test for two formats, and fix format 5 by avoiding a full history sync with non-format5 branches.
269
            raise tests.TestSkipped(
270
                "BranchBuilder can't make reference branches.")
3904.3.7 by Andrew Bennetts
Comment the new tests.
271
        # Make a branch called "local" in a stackable repository
272
        # The branch has 3 revisions:
273
        #   - rev-1, adds a file
274
        #   - rev-2, no changes
275
        #   - rev-3, modifies the file.
3904.3.5 by Andrew Bennetts
Improve the test; now 4/7 passing.
276
        repo = self.make_repository('repo', shared=True, format='1.6')
277
        builder = self.make_branch_builder('repo/local')
3904.3.4 by Andrew Bennetts
First cut of a branch_implementations test. It fails.
278
        builder.start_series()
279
        builder.build_snapshot('rev-1', None, [
280
            ('add', ('', 'root-id', 'directory', '')),
281
            ('add', ('filename', 'f-id', 'file', 'content\n'))])
282
        builder.build_snapshot('rev-2', ['rev-1'], [])
283
        builder.build_snapshot('rev-3', ['rev-2'],
284
            [('modify', ('f-id', 'new-content\n'))])
285
        builder.finish_series()
3904.3.6 by Andrew Bennetts
Skip test for two formats, and fix format 5 by avoiding a full history sync with non-format5 branches.
286
        trunk = builder.get_branch()
3904.3.7 by Andrew Bennetts
Comment the new tests.
287
        # Sprout rev-1 to "trunk", so that we can stack on it.
3904.3.6 by Andrew Bennetts
Skip test for two formats, and fix format 5 by avoiding a full history sync with non-format5 branches.
288
        trunk.bzrdir.sprout(self.get_url('trunk'), revision_id='rev-1')
3904.3.7 by Andrew Bennetts
Comment the new tests.
289
        # Set a default stacking policy so that new branches will automatically
290
        # stack on trunk.
3904.3.4 by Andrew Bennetts
First cut of a branch_implementations test. It fails.
291
        self.make_bzrdir('.').get_config().set_default_stack_on('trunk')
3904.3.7 by Andrew Bennetts
Comment the new tests.
292
        # Push rev-2 to a new branch "remote".  It will be stacked on "trunk".
3904.3.5 by Andrew Bennetts
Improve the test; now 4/7 passing.
293
        output = StringIO()
3904.3.6 by Andrew Bennetts
Skip test for two formats, and fix format 5 by avoiding a full history sync with non-format5 branches.
294
        push._show_push_branch(trunk, 'rev-2', self.get_url('remote'), output)
3904.3.7 by Andrew Bennetts
Comment the new tests.
295
        # Push rev-3 onto "remote".  If "remote" not stacked and is missing the
296
        # fulltext record for f-id @ rev-1, then this will fail.
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
297
        remote_branch = branch.Branch.open(self.get_url('remote'))
3904.3.6 by Andrew Bennetts
Skip test for two formats, and fix format 5 by avoiding a full history sync with non-format5 branches.
298
        trunk.push(remote_branch)
4332.3.35 by Robert Collins
Fix failing tests.
299
        check.check_dwim(remote_branch.base, False, True, True)
3904.3.4 by Andrew Bennetts
First cut of a branch_implementations test. It fails.
300
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
301
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
302
class TestPushHook(per_branch.TestCaseWithBranch):
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
303
304
    def setUp(self):
305
        self.hook_calls = []
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
306
        super(TestPushHook, self).setUp()
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
307
2297.1.4 by Martin Pool
Push now returns a PushResult rather than just an integer.
308
    def capture_post_push_hook(self, result):
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
309
        """Capture post push hook calls to self.hook_calls.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
310
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
311
        The call is logged, as is some state of the two branches.
312
        """
2297.1.6 by Martin Pool
Add docs for Results, give some members cleaner names
313
        if result.local_branch:
314
            local_locked = result.local_branch.is_locked()
315
            local_base = result.local_branch.base
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
316
        else:
317
            local_locked = None
318
            local_base = None
319
        self.hook_calls.append(
2297.1.6 by Martin Pool
Add docs for Results, give some members cleaner names
320
            ('post_push', result.source_branch, local_base,
321
             result.master_branch.base,
2297.1.4 by Martin Pool
Push now returns a PushResult rather than just an integer.
322
             result.old_revno, result.old_revid,
2297.1.6 by Martin Pool
Add docs for Results, give some members cleaner names
323
             result.new_revno, result.new_revid,
324
             result.source_branch.is_locked(), local_locked,
325
             result.master_branch.is_locked()))
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
326
327
    def test_post_push_empty_history(self):
328
        target = self.make_branch('target')
329
        source = self.make_branch('source')
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
330
        branch.Branch.hooks.install_named_hook(
331
            'post_push', self.capture_post_push_hook, None)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
332
        source.push(target)
333
        # with nothing there we should still get a notification, and
334
        # have both branches locked at the notification time.
335
        self.assertEqual([
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
336
            ('post_push', source, None, target.base, 0, revision.NULL_REVISION,
337
             0, revision.NULL_REVISION, True, None, True)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
338
            ],
339
            self.hook_calls)
340
341
    def test_post_push_bound_branch(self):
342
        # pushing to a bound branch should pass in the master branch to the
343
        # hook, allowing the correct number of emails to be sent, while still
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
344
        # allowing hooks that want to modify the target to do so to both
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
345
        # instances.
346
        target = self.make_branch('target')
347
        local = self.make_branch('local')
348
        try:
349
            local.bind(target)
350
        except errors.UpgradeRequired:
2477.1.2 by Martin Pool
Rename push/pull back to 'run_hooks' (jameinel)
351
            # We can't bind this format to itself- typically it is the local
352
            # branch that doesn't support binding.  As of May 2007
353
            # remotebranches can't be bound.  Let's instead make a new local
354
            # branch of the default type, which does allow binding.
355
            # See https://bugs.launchpad.net/bzr/+bug/112020
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
356
            local = bzrdir.BzrDir.create_branch_convenience('local2')
2477.1.9 by Martin Pool
Review cleanups from John, mostly docs
357
            local.bind(target)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
358
        source = self.make_branch('source')
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
359
        branch.Branch.hooks.install_named_hook(
360
            'post_push', self.capture_post_push_hook, None)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
361
        source.push(local)
362
        # with nothing there we should still get a notification, and
363
        # have both branches locked at the notification time.
364
        self.assertEqual([
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
365
            ('post_push', source, local.base, target.base, 0,
366
             revision.NULL_REVISION, 0, revision.NULL_REVISION,
367
             True, True, True)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
368
            ],
369
            self.hook_calls)
370
371
    def test_post_push_nonempty_history(self):
372
        target = self.make_branch_and_memory_tree('target')
373
        target.lock_write()
374
        target.add('')
375
        rev1 = target.commit('rev 1')
376
        target.unlock()
377
        sourcedir = target.bzrdir.clone(self.get_url('source'))
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
378
        source = memorytree.MemoryTree.create_on_branch(sourcedir.open_branch())
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
379
        rev2 = source.commit('rev 2')
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
380
        branch.Branch.hooks.install_named_hook(
381
            'post_push', self.capture_post_push_hook, None)
2246.1.3 by Robert Collins
New branch hooks: post_push, post_pull, post_commit, post_uncommit. These
382
        source.branch.push(target.branch)
383
        # with nothing there we should still get a notification, and
384
        # have both branches locked at the notification time.
385
        self.assertEqual([
386
            ('post_push', source.branch, None, target.branch.base, 1, rev1,
387
             2, rev2, True, None, True)
388
            ],
389
            self.hook_calls)
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
390
391
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
392
class EmptyPushSmartEffortTests(per_branch.TestCaseWithBranch):
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
393
    """Tests that a push of 0 revisions should make a limited number of smart
394
    protocol RPCs.
395
    """
396
397
    def setUp(self):
398
        # Skip some scenarios that don't apply to these tests.
5247.3.38 by Vincent Ladeuil
Fix the last remaining failures.
399
        if (self.transport_server is not None
400
            and issubclass(self.transport_server,
401
                           test_server.SmartTCPServer_for_testing)):
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
402
            raise tests.TestNotApplicable(
403
                'Does not apply when remote backing branch is also '
404
                'a smart branch')
5674.1.1 by Jelmer Vernooij
Add supports_leave_lock flag to BranchFormat and RepositoryFormat.
405
        if not self.branch_format.supports_leaving_lock():
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
406
            raise tests.TestNotApplicable(
5674.1.1 by Jelmer Vernooij
Add supports_leave_lock flag to BranchFormat and RepositoryFormat.
407
                'Branch format is not usable via HPSS.')
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
408
        super(EmptyPushSmartEffortTests, self).setUp()
409
        # Create a smart server that publishes whatever the backing VFS server
410
        # does.
5017.3.29 by Vincent Ladeuil
-s bt.per_branch.test_push passing
411
        self.smart_server = test_server.SmartTCPServer_for_testing()
4659.1.2 by Robert Collins
Refactor creation and shutdown of test servers to use a common helper,
412
        self.start_server(self.smart_server, self.get_server())
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
413
        # Make two empty branches, 'empty' and 'target'.
414
        self.empty_branch = self.make_branch('empty')
415
        self.make_branch('target')
416
        # Log all HPSS calls into self.hpss_calls.
417
        client._SmartClient.hooks.install_named_hook(
418
            'call', self.capture_hpss_call, None)
419
        self.hpss_calls = []
420
421
    def capture_hpss_call(self, params):
422
        self.hpss_calls.append(params.method)
423
424
    def test_empty_branch_api(self):
425
        """The branch_obj.push API should make a limited number of HPSS calls.
426
        """
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
427
        t = transport.get_transport(self.smart_server.get_url()).clone('target')
428
        target = branch.Branch.open_from_transport(t)
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
429
        self.empty_branch.push(target)
430
        self.assertEqual(
4634.47.8 by Andrew Bennetts
Fix per_branch.test_push effort test to expect new verb.
431
            ['BzrDir.open_2.1',
4734.4.13 by Andrew Bennetts
Fix trivial test failure.
432
             'BzrDir.open_branchV3',
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
433
             'BzrDir.find_repositoryV3',
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
434
             'Branch.get_stacked_on_url',
435
             'Branch.lock_write',
436
             'Branch.last_revision_info',
437
             'Branch.unlock'],
438
            self.hpss_calls)
439
440
    def test_empty_branch_command(self):
441
        """The 'bzr push' command should make a limited number of HPSS calls.
442
        """
443
        cmd = builtins.cmd_push()
444
        cmd.outf = tests.StringIOWrapper()
445
        cmd.run(
4420.1.2 by Vincent Ladeuil
Fix bug #284038 by adding a --strict option to push.
446
            directory=self.get_url('empty'),
4420.1.4 by Vincent Ladeuil
Cleanup.
447
            location=self.smart_server.get_url() + 'target')
3703.3.7 by Andrew Bennetts
Move empty push effort tests to branch_implementations.
448
        # HPSS calls as of 2008/09/22:
449
        # [BzrDir.open, BzrDir.open_branch, BzrDir.find_repositoryV2,
450
        # Branch.get_stacked_on_url, get, get, Branch.lock_write,
451
        # Branch.last_revision_info, Branch.unlock]
452
        self.assertTrue(len(self.hpss_calls) <= 9, self.hpss_calls)
453
454
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
455
class TestLossyPush(per_branch.TestCaseWithBranch):
4347.3.2 by Jelmer Vernooij
Add some basic tests for lossy_push.
456
457
    def setUp(self):
458
        self.hook_calls = []
5010.2.18 by Vincent Ladeuil
Fix imports in per_branch/test_push.py.
459
        super(TestLossyPush, self).setUp()
4347.3.2 by Jelmer Vernooij
Add some basic tests for lossy_push.
460
461
    def test_lossy_push_raises_same_vcs(self):
462
        target = self.make_branch('target')
463
        source = self.make_branch('source')
464
        self.assertRaises(errors.LossyPushToSameVCS, source.lossy_push, target)