/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/tests/blackbox/test_merge.py

  • Committer: Jelmer Vernooij
  • Date: 2018-02-18 21:42:57 UTC
  • mto: This revision was merged to the branch mainline in revision 6859.
  • Revision ID: jelmer@jelmer.uk-20180218214257-jpevutp1wa30tz3v
Update TODO to reference Breezy, not Bazaar.

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/blackbox/test_merge.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006-2012, 2016 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
16
16
#
17
17
# Author: Aaron Bentley <aaron.bentley@utoronto.ca>
18
18
 
19
 
"""Black-box tests for bzr merge.
 
19
"""Black-box tests for brz merge.
20
20
"""
21
21
 
 
22
import doctest
22
23
import os
23
24
 
24
 
from bzrlib import (
 
25
from testtools import matchers
 
26
 
 
27
from breezy import (
25
28
    branch,
26
 
    bzrdir,
27
29
    conflicts,
28
 
    errors,
 
30
    controldir,
29
31
    merge_directive,
30
32
    osutils,
31
33
    tests,
32
34
    urlutils,
33
35
    workingtree,
34
36
    )
 
37
from breezy.tests import (
 
38
    scenarios,
 
39
    script,
 
40
    )
 
41
 
 
42
 
 
43
load_tests = scenarios.load_tests_apply_scenarios
35
44
 
36
45
 
37
46
class TestMerge(tests.TestCaseWithTransport):
54
63
            conflict.
55
64
        """
56
65
        builder = self.make_branch_builder('branch')
57
 
        builder.build_snapshot('rev1', None,
 
66
        builder.build_snapshot(None,
58
67
            [('add', ('', 'root-id', 'directory', None)),
59
 
             ('add', ('fname', 'f-id', 'file', 'a\nb\nc\n'))])
60
 
        builder.build_snapshot('rev2other', ['rev1'],
61
 
            [('modify', ('f-id', 'a\nB\nD\n'))])
62
 
        other = builder.get_branch().bzrdir.sprout('other').open_branch()
63
 
        builder.build_snapshot('rev2this', ['rev1'],
64
 
            [('modify', ('f-id', 'a\nB\nC\n'))])
 
68
             ('add', ('fname', 'f-id', 'file', 'a\nb\nc\n'))],
 
69
            revision_id='rev1')
 
70
        builder.build_snapshot(['rev1'],
 
71
            [('modify', ('f-id', 'a\nB\nD\n'))],
 
72
            revision_id='rev2other')
 
73
        other = builder.get_branch().controldir.sprout('other').open_branch()
 
74
        builder.build_snapshot(['rev1'],
 
75
            [('modify', ('f-id', 'a\nB\nC\n'))], revision_id='rev2this')
65
76
        tree = builder.get_branch().create_checkout('tree', lightweight=True)
66
77
        return tree, other
67
78
 
68
79
    def test_merge_reprocess(self):
69
 
        d = bzrdir.BzrDir.create_standalone_workingtree('.')
 
80
        d = controldir.ControlDir.create_standalone_workingtree('.')
70
81
        d.commit('h')
71
82
        self.run_bzr('merge . --reprocess --merge-type weave')
72
83
 
73
84
    def test_merge(self):
74
85
        a_tree = self.example_branch('a')
75
86
        ancestor = a_tree.branch.revno()
76
 
        b_tree = a_tree.bzrdir.sprout('b').open_workingtree()
 
87
        b_tree = a_tree.controldir.sprout('b').open_workingtree()
77
88
        self.build_tree_contents([('b/goodbye', 'quux')])
78
89
        b_tree.commit(message="more u's are always good")
79
90
 
80
91
        self.build_tree_contents([('a/hello', 'quuux')])
81
92
        # We can't merge when there are in-tree changes
82
 
        os.chdir('a')
83
 
        self.run_bzr('merge ../b', retcode=3)
84
 
        a = workingtree.WorkingTree.open('.')
 
93
        self.run_bzr('merge ../b', retcode=3, working_dir='a')
 
94
        a = workingtree.WorkingTree.open('a')
85
95
        a_tip = a.commit("Like an epidemic of u's")
86
 
        self.run_bzr('merge ../b -r last:1..last:1 --merge-type blooof',
87
 
                    retcode=3)
88
 
        self.run_bzr('merge ../b -r last:1..last:1 --merge-type merge3')
89
 
        a_tree.revert(backups=False)
90
 
        self.run_bzr('merge ../b -r last:1..last:1 --merge-type weave')
91
 
        a_tree.revert(backups=False)
92
 
        self.run_bzr('merge ../b -r last:1..last:1 --merge-type lca')
93
 
        a_tree.revert(backups=False)
 
96
 
 
97
        def run_merge_then_revert(args, retcode=None, working_dir='a'):
 
98
            self.run_bzr(['merge', '../b', '-r', 'last:1..last:1'] + args,
 
99
                         retcode=retcode, working_dir=working_dir)
 
100
            if retcode != 3:
 
101
                a_tree.revert(backups=False)
 
102
 
 
103
        run_merge_then_revert(['--merge-type', 'bloof'], retcode=3)
 
104
        run_merge_then_revert(['--merge-type', 'merge3'])
 
105
        run_merge_then_revert(['--merge-type', 'weave'])
 
106
        run_merge_then_revert(['--merge-type', 'lca'])
94
107
        self.run_bzr_error(['Show-base is not supported for this merge type'],
95
108
                           'merge ../b -r last:1..last:1 --merge-type weave'
96
 
                           ' --show-base')
97
 
        a_tree.revert(backups=False)
98
 
        self.run_bzr('merge ../b -r last:1..last:1 --reprocess')
99
 
        a_tree.revert(backups=False)
100
 
        self.run_bzr('merge ../b -r last:1')
101
 
        self.check_file_contents('goodbye', 'quux')
 
109
                           ' --show-base', working_dir='a')
 
110
        a_tree.revert(backups=False)
 
111
        self.run_bzr('merge ../b -r last:1..last:1 --reprocess',
 
112
                     working_dir='a')
 
113
        a_tree.revert(backups=False)
 
114
        self.run_bzr('merge ../b -r last:1', working_dir='a')
 
115
        self.check_file_contents('a/goodbye', 'quux')
102
116
        # Merging a branch pulls its revision into the tree
103
 
        b = branch.Branch.open('../b')
 
117
        b = branch.Branch.open('b')
104
118
        b_tip = b.last_revision()
105
 
        self.failUnless(a.branch.repository.has_revision(b_tip))
 
119
        self.assertTrue(a.branch.repository.has_revision(b_tip))
106
120
        self.assertEqual([a_tip, b_tip], a.get_parent_ids())
107
121
        a_tree.revert(backups=False)
108
 
        out, err = self.run_bzr('merge -r revno:1:./hello', retcode=3)
 
122
        out, err = self.run_bzr('merge -r revno:1:./hello', retcode=3,
 
123
                                working_dir='a')
109
124
        self.assertTrue("Not a branch" in err)
110
125
        self.run_bzr('merge -r revno:%d:./..revno:%d:../b'
111
 
                    %(ancestor,b.revno()))
112
 
        self.assertEquals(a.get_parent_ids(),
 
126
                    %(ancestor, b.revno()), working_dir='a')
 
127
        self.assertEqual(a.get_parent_ids(),
113
128
                          [a.branch.last_revision(), b.last_revision()])
114
 
        self.check_file_contents('goodbye', 'quux')
 
129
        self.check_file_contents('a/goodbye', 'quux')
115
130
        a_tree.revert(backups=False)
116
 
        self.run_bzr('merge -r revno:%d:../b'%b.revno())
117
 
        self.assertEquals(a.get_parent_ids(),
 
131
        self.run_bzr('merge -r revno:%d:../b'%b.revno(), working_dir='a')
 
132
        self.assertEqual(a.get_parent_ids(),
118
133
                          [a.branch.last_revision(), b.last_revision()])
119
134
        a_tip = a.commit('merged')
120
 
        self.run_bzr('merge ../b -r last:1')
 
135
        self.run_bzr('merge ../b -r last:1', working_dir='a')
121
136
        self.assertEqual([a_tip], a.get_parent_ids())
122
137
 
123
138
    def test_merge_defaults_to_reprocess(self):
133
148
                             '=======\n'
134
149
                             'D\n'
135
150
                             '>>>>>>> MERGE-SOURCE\n',
136
 
                             tree.get_file_text('f-id'))
 
151
                             tree.get_file_text(tree.id2path('f-id')))
137
152
 
138
153
    def test_merge_explicit_reprocess_show_base(self):
139
154
        tree, other = self.create_conflicting_branches()
155
170
                             'B\n'
156
171
                             'D\n'
157
172
                             '>>>>>>> MERGE-SOURCE\n',
158
 
                             tree.get_file_text('f-id'))
 
173
                             tree.get_file_text(tree.id2path('f-id')))
159
174
 
160
175
    def test_merge_override_show_base(self):
161
176
        tree, other = self.create_conflicting_branches()
173
188
                             'B\n'
174
189
                             'D\n'
175
190
                             '>>>>>>> MERGE-SOURCE\n',
176
 
                             tree.get_file_text('f-id'))
 
191
                             tree.get_file_text(tree.id2path('f-id')))
177
192
 
178
193
    def test_merge_with_missing_file(self):
179
194
        """Merge handles missing file conflicts"""
186
201
        a_tree = self.make_branch_and_tree('a')
187
202
        a_tree.add(['sub', 'b.txt', 'sub/c.txt', 'sub/a.txt'])
188
203
        a_tree.commit(message='added a')
189
 
        b_tree = a_tree.bzrdir.sprout('b').open_workingtree()
 
204
        b_tree = a_tree.controldir.sprout('b').open_workingtree()
190
205
        self.build_tree_contents([
191
206
            ('a/sub/a.txt', 'hello\nthere\n'),
192
207
            ('a/b.txt', 'hello\nthere\n'),
202
217
            ('b/b.txt', 'hello\nsomething\n'),
203
218
            ('b/sub/c.txt', 'hello\nsomething\n')])
204
219
        b_tree.commit(message='Modified a.txt')
205
 
        os.chdir('b')
206
 
        self.run_bzr('merge ../a/', retcode=1)
207
 
        self.failUnlessExists('sub/a.txt.THIS')
208
 
        self.failUnlessExists('sub/a.txt.BASE')
209
 
        os.chdir('../a')
210
 
        self.run_bzr('merge ../b/', retcode=1)
211
 
        self.failUnlessExists('sub/a.txt.OTHER')
212
 
        self.failUnlessExists('sub/a.txt.BASE')
 
220
 
 
221
        self.run_bzr('merge ../a/', retcode=1, working_dir='b')
 
222
        self.assertPathExists('b/sub/a.txt.THIS')
 
223
        self.assertPathExists('b/sub/a.txt.BASE')
 
224
 
 
225
        self.run_bzr('merge ../b/', retcode=1, working_dir='a')
 
226
        self.assertPathExists('a/sub/a.txt.OTHER')
 
227
        self.assertPathExists('a/sub/a.txt.BASE')
213
228
 
214
229
    def test_conflict_leaves_base_this_other_files(self):
215
230
        tree, other = self.create_conflicting_branches()
234
249
        self.build_tree(['branch_a/a'])
235
250
        tree_a.add('a')
236
251
        tree_a.commit('commit a')
237
 
        branch_b = branch_a.bzrdir.sprout('branch_b').open_branch()
238
 
        tree_b = branch_b.bzrdir.open_workingtree()
239
 
        branch_c = branch_a.bzrdir.sprout('branch_c').open_branch()
240
 
        tree_c = branch_c.bzrdir.open_workingtree()
 
252
        branch_b = branch_a.controldir.sprout('branch_b').open_branch()
 
253
        tree_b = branch_b.controldir.open_workingtree()
 
254
        branch_c = branch_a.controldir.sprout('branch_c').open_branch()
 
255
        tree_c = branch_c.controldir.open_workingtree()
241
256
        self.build_tree(['branch_a/b'])
242
257
        tree_a.add('b')
243
258
        tree_a.commit('commit b')
249
264
        branch_b.set_parent(None)
250
265
        self.assertEqual(None, branch_b.get_parent())
251
266
        # test merge for failure without parent set
252
 
        os.chdir('branch_b')
253
 
        out = self.run_bzr('merge', retcode=3)
254
 
        self.assertEquals(out,
255
 
                ('','bzr: ERROR: No location specified or remembered\n'))
 
267
        out = self.run_bzr('merge', retcode=3, working_dir='branch_b')
 
268
        self.assertEqual(out,
 
269
                ('', 'brz: ERROR: No location specified or remembered\n'))
256
270
 
257
271
        # test uncommitted changes
258
 
        self.build_tree(['d'])
 
272
        self.build_tree(['branch_b/d'])
259
273
        tree_b.add('d')
260
274
        self.run_bzr_error(['Working tree ".*" has uncommitted changes'],
261
 
                           'merge')
 
275
                           'merge', working_dir='branch_b')
262
276
 
263
277
        # merge should now pass and implicitly remember merge location
264
278
        tree_b.commit('commit d')
265
 
        out, err = self.run_bzr('merge ../branch_a')
 
279
        out, err = self.run_bzr('merge ../branch_a', working_dir='branch_b')
266
280
 
267
281
        base = urlutils.local_path_from_url(branch_a.base)
268
282
        self.assertEndsWith(err, '+N  b\nAll changes applied successfully.\n')
269
 
        self.assertEquals(osutils.abspath(branch_b.get_submit_branch()),
 
283
        # re-open branch as external run_brz modified it
 
284
        branch_b = branch_b.controldir.open_branch()
 
285
        self.assertEqual(osutils.abspath(branch_b.get_submit_branch()),
270
286
                          osutils.abspath(parent))
271
287
        # test implicit --remember when committing new file
272
 
        self.build_tree(['e'])
 
288
        self.build_tree(['branch_b/e'])
273
289
        tree_b.add('e')
274
290
        tree_b.commit('commit e')
275
 
        out, err = self.run_bzr('merge')
276
 
        self.assertStartsWith(err,
277
 
                          'Merging from remembered submit location %s\n' % (base,))
278
 
        # re-open tree as external run_bzr modified it
279
 
        tree_b = branch_b.bzrdir.open_workingtree()
 
291
        out, err = self.run_bzr('merge', working_dir='branch_b')
 
292
        self.assertStartsWith(
 
293
            err, 'Merging from remembered submit location %s\n' % (base,))
 
294
        # re-open tree as external run_brz modified it
 
295
        tree_b = branch_b.controldir.open_workingtree()
280
296
        tree_b.commit('merge branch_a')
281
297
        # test explicit --remember
282
 
        out, err = self.run_bzr('merge ../branch_c --remember')
283
 
        self.assertEquals(out, '')
284
 
        self.assertEquals(err, '+N  c\nAll changes applied successfully.\n')
285
 
        self.assertEquals(osutils.abspath(branch_b.get_submit_branch()),
286
 
                          osutils.abspath(branch_c.bzrdir.root_transport.base))
287
 
        # re-open tree as external run_bzr modified it
288
 
        tree_b = branch_b.bzrdir.open_workingtree()
 
298
        out, err = self.run_bzr('merge ../branch_c --remember',
 
299
                                working_dir='branch_b')
 
300
        self.assertEqual(out, '')
 
301
        self.assertEqual(err, '+N  c\nAll changes applied successfully.\n')
 
302
        # re-open branch as external run_brz modified it
 
303
        branch_b = branch_b.controldir.open_branch()
 
304
        self.assertEqual(osutils.abspath(branch_b.get_submit_branch()),
 
305
                          osutils.abspath(branch_c.controldir.root_transport.base))
 
306
        # re-open tree as external run_brz modified it
 
307
        tree_b = branch_b.controldir.open_workingtree()
289
308
        tree_b.commit('merge branch_c')
290
309
 
291
310
    def test_merge_bundle(self):
292
 
        from bzrlib.testament import Testament
 
311
        from breezy.testament import Testament
293
312
        tree_a = self.make_branch_and_tree('branch_a')
294
313
        self.build_tree_contents([('branch_a/a', 'hello')])
295
314
        tree_a.add('a')
296
315
        tree_a.commit('message')
297
316
 
298
 
        tree_b = tree_a.bzrdir.sprout('branch_b').open_workingtree()
 
317
        tree_b = tree_a.controldir.sprout('branch_b').open_workingtree()
299
318
        self.build_tree_contents([('branch_a/a', 'hey there')])
300
319
        tree_a.commit('message')
301
320
 
302
321
        self.build_tree_contents([('branch_b/a', 'goodbye')])
303
322
        tree_b.commit('message')
304
 
        os.chdir('branch_b')
305
 
        self.run_bzr('bundle ../branch_a -o ../bundle')
306
 
        os.chdir('../branch_a')
307
 
        self.run_bzr('merge ../bundle', retcode=1)
 
323
        self.run_bzr('bundle ../branch_a -o ../bundle', working_dir='branch_b')
 
324
        self.run_bzr('merge ../bundle', retcode=1, working_dir='branch_a')
308
325
        testament_a = Testament.from_revision(tree_a.branch.repository,
309
326
                                              tree_b.get_parent_ids()[0])
310
327
        testament_b = Testament.from_revision(tree_b.branch.repository,
314
331
        tree_a.set_conflicts(conflicts.ConflictList())
315
332
        tree_a.commit('message')
316
333
        # it is legal to attempt to merge an already-merged bundle
317
 
        output = self.run_bzr('merge ../bundle')[1]
 
334
        err = self.run_bzr('merge ../bundle', working_dir='branch_a')[1]
318
335
        # but it does nothing
319
336
        self.assertFalse(tree_a.changes_from(tree_a.basis_tree()).has_changed())
320
 
        self.assertEqual('Nothing to do.\n', output)
 
337
        self.assertEqual('Nothing to do.\n', err)
321
338
 
322
339
    def test_merge_uncommitted(self):
323
340
        """Check that merge --uncommitted behaves properly"""
325
342
        self.build_tree(['a/file_1', 'a/file_2'])
326
343
        tree_a.add(['file_1', 'file_2'])
327
344
        tree_a.commit('commit 1')
328
 
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
329
 
        self.failUnlessExists('b/file_1')
 
345
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
 
346
        self.assertPathExists('b/file_1')
330
347
        tree_a.rename_one('file_1', 'file_i')
331
348
        tree_a.commit('commit 2')
332
349
        tree_a.rename_one('file_2', 'file_ii')
333
 
        ## os.chdir('b')
334
350
        self.run_bzr('merge a --uncommitted -d b')
335
 
        self.failUnlessExists('b/file_1')
336
 
        self.failUnlessExists('b/file_ii')
 
351
        self.assertPathExists('b/file_1')
 
352
        self.assertPathExists('b/file_ii')
337
353
        tree_b.revert()
338
354
        self.run_bzr_error(('Cannot use --uncommitted and --revision',),
339
355
                           'merge /a --uncommitted -r1 -d b')
342
358
        """It should be possible to merge changes from a single file."""
343
359
        tree_a = self.make_branch_and_tree('tree_a')
344
360
        tree_a.commit('initial commit')
345
 
        tree_a.bzrdir.sprout('tree_b')
 
361
        tree_a.controldir.sprout('tree_b')
346
362
        self.build_tree(['tree_a/file1', 'tree_a/file2'])
347
363
        tree_a.add(['file1', 'file2'])
348
 
        os.chdir('tree_b')
349
 
        self.run_bzr(['merge', '--uncommitted', '../tree_a/file1'])
350
 
        self.failUnlessExists('file1')
351
 
        self.failIfExists('file2')
 
364
        self.run_bzr(['merge', '--uncommitted', '../tree_a/file1'],
 
365
                     working_dir='tree_b')
 
366
        self.assertPathExists('tree_b/file1')
 
367
        self.assertPathDoesNotExist('tree_b/file2')
 
368
 
 
369
    def test_merge_nonexistent_file(self):
 
370
        """It should not be possible to merge changes from a file which
 
371
        does not exist."""
 
372
        tree_a = self.make_branch_and_tree('tree_a')
 
373
        self.build_tree_contents([('tree_a/file', 'bar\n')])
 
374
        tree_a.add(['file'])
 
375
        tree_a.commit('commit 1')
 
376
        self.run_bzr_error(('Path\(s\) do not exist: non/existing',),
 
377
                           ['merge', 'non/existing'], working_dir='tree_a')
352
378
 
353
379
    def pullable_branch(self):
354
380
        tree_a = self.make_branch_and_tree('a')
355
 
        self.build_tree(['a/file'])
 
381
        self.build_tree_contents([('a/file', 'bar\n')])
356
382
        tree_a.add(['file'])
357
383
        self.id1 = tree_a.commit('commit 1')
358
384
 
359
385
        tree_b = self.make_branch_and_tree('b')
360
386
        tree_b.pull(tree_a.branch)
361
 
        file('b/file', 'wb').write('foo')
 
387
        self.build_tree_contents([('b/file', 'foo\n')])
362
388
        self.id2 = tree_b.commit('commit 2')
363
389
 
364
390
    def test_merge_pull(self):
365
391
        self.pullable_branch()
366
 
        os.chdir('a')
367
 
        (out, err) = self.run_bzr('merge --pull ../b')
 
392
        (out, err) = self.run_bzr('merge --pull ../b', working_dir='a')
368
393
        self.assertContainsRe(out, 'Now on revision 2\\.')
369
 
        tree_a = workingtree.WorkingTree.open('.')
 
394
        tree_a = workingtree.WorkingTree.open('a')
370
395
        self.assertEqual([self.id2], tree_a.get_parent_ids())
371
396
 
 
397
    def test_merge_pull_preview(self):
 
398
        self.pullable_branch()
 
399
        (out, err) = self.run_bzr('merge --pull --preview -d a b')
 
400
        self.assertThat(out, matchers.DocTestMatches(
 
401
"""=== modified file 'file'
 
402
--- file\t...
 
403
+++ file\t...
 
404
@@ -1,1 +1,1 @@
 
405
-bar
 
406
+foo
 
407
 
 
408
""", doctest.ELLIPSIS | doctest.REPORT_UDIFF))
 
409
        tree_a = workingtree.WorkingTree.open('a')
 
410
        self.assertEqual([self.id1], tree_a.get_parent_ids())
 
411
 
372
412
    def test_merge_kind_change(self):
373
413
        tree_a = self.make_branch_and_tree('tree_a')
374
414
        self.build_tree_contents([('tree_a/file', 'content_1')])
375
415
        tree_a.add('file', 'file-id')
376
416
        tree_a.commit('added file')
377
 
        tree_b = tree_a.bzrdir.sprout('tree_b').open_workingtree()
 
417
        tree_b = tree_a.controldir.sprout('tree_b').open_workingtree()
378
418
        os.unlink('tree_a/file')
379
419
        self.build_tree(['tree_a/file/'])
380
420
        tree_a.commit('changed file to directory')
381
 
        os.chdir('tree_b')
382
 
        self.run_bzr('merge ../tree_a')
383
 
        self.assertEqual('directory', osutils.file_kind('file'))
 
421
        self.run_bzr('merge ../tree_a', working_dir='tree_b')
 
422
        self.assertEqual('directory', osutils.file_kind('tree_b/file'))
384
423
        tree_b.revert()
385
 
        self.assertEqual('file', osutils.file_kind('file'))
386
 
        self.build_tree_contents([('file', 'content_2')])
 
424
        self.assertEqual('file', osutils.file_kind('tree_b/file'))
 
425
        self.build_tree_contents([('tree_b/file', 'content_2')])
387
426
        tree_b.commit('content change')
388
 
        self.run_bzr('merge ../tree_a', retcode=1)
 
427
        self.run_bzr('merge ../tree_a', retcode=1, working_dir='tree_b')
389
428
        self.assertEqual(tree_b.conflicts(),
390
429
                         [conflicts.ContentsConflict('file',
391
430
                                                     file_id='file-id')])
393
432
    def test_directive_cherrypick(self):
394
433
        source = self.make_branch_and_tree('source')
395
434
        source.commit("nothing")
396
 
        # see https://bugs.edge.launchpad.net/bzr/+bug/409688 - trying to
 
435
        # see https://bugs.launchpad.net/bzr/+bug/409688 - trying to
397
436
        # cherrypick from one branch into another unrelated branch with a
398
437
        # different root id will give shape conflicts.  as a workaround we
399
438
        # make sure they share the same root id.
400
 
        target = source.bzrdir.sprout('target').open_workingtree()
 
439
        target = source.controldir.sprout('target').open_workingtree()
401
440
        self.build_tree(['source/a'])
402
441
        source.add('a')
403
442
        source.commit('Added a', rev_id='rev1')
408
447
        self.write_directive('directive', source.branch, 'target', 'rev2',
409
448
                             'rev1')
410
449
        out, err = self.run_bzr('merge -d target directive')
411
 
        self.failIfExists('target/a')
412
 
        self.failUnlessExists('target/b')
 
450
        self.assertPathDoesNotExist('target/a')
 
451
        self.assertPathExists('target/b')
413
452
        self.assertContainsRe(err, 'Performing cherrypick')
414
453
 
415
454
    def write_directive(self, filename, source, target, revision_id,
441
480
        target = self.make_branch_and_tree('target')
442
481
        target.commit('empty')
443
482
        # We need a revision that has no integer revno
444
 
        branch_a = target.bzrdir.sprout('branch_a').open_workingtree()
 
483
        branch_a = target.controldir.sprout('branch_a').open_workingtree()
445
484
        self.build_tree(['branch_a/file1'])
446
485
        branch_a.add('file1')
447
486
        branch_a.commit('added file1', rev_id='rev2a')
448
 
        branch_b = target.bzrdir.sprout('branch_b').open_workingtree()
 
487
        branch_b = target.controldir.sprout('branch_b').open_workingtree()
449
488
        self.build_tree(['branch_b/file2'])
450
489
        branch_b.add('file2')
451
490
        branch_b.commit('added file2', rev_id='rev2b')
452
491
        branch_b.merge_from_branch(branch_a.branch)
453
 
        self.failUnlessExists('branch_b/file1')
 
492
        self.assertPathExists('branch_b/file1')
454
493
        branch_b.commit('merged branch_a', rev_id='rev3b')
455
494
 
456
495
        # It works if the revid has an interger revno
457
496
        self.run_bzr('merge -d target -r revid:rev2a branch_a')
458
 
        self.failUnlessExists('target/file1')
459
 
        self.failIfExists('target/file2')
 
497
        self.assertPathExists('target/file1')
 
498
        self.assertPathDoesNotExist('target/file2')
460
499
        target.revert()
461
500
 
462
501
        # It should work if the revid has no integer revno
463
502
        self.run_bzr('merge -d target -r revid:rev2a branch_b')
464
 
        self.failUnlessExists('target/file1')
465
 
        self.failIfExists('target/file2')
 
503
        self.assertPathExists('target/file1')
 
504
        self.assertPathDoesNotExist('target/file2')
466
505
 
467
506
    def assertDirectoryContent(self, directory, entries, message=''):
468
507
        """Assert whether entries (file or directories) exist in a directory.
486
525
            source.add(f)
487
526
            source.commit('added '+f, rev_id='rev_'+f)
488
527
        # target branch
489
 
        target = source.bzrdir.sprout('target', 'rev_a').open_workingtree()
 
528
        target = source.controldir.sprout('target', 'rev_a').open_workingtree()
490
529
        self.assertDirectoryContent('target', ['.bzr', 'a'])
491
530
        # pick 1 revision
492
531
        self.run_bzr('merge -d target -r revid:rev_b..revid:rev_c source')
503
542
    def test_merge_criss_cross(self):
504
543
        tree_a = self.make_branch_and_tree('a')
505
544
        tree_a.commit('', rev_id='rev1')
506
 
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
 
545
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
507
546
        tree_a.commit('', rev_id='rev2a')
508
547
        tree_b.commit('', rev_id='rev2b')
509
548
        tree_a.merge_from_branch(tree_b.branch)
516
555
 
517
556
    def test_merge_from_submit(self):
518
557
        tree_a = self.make_branch_and_tree('a')
519
 
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
520
 
        tree_c = tree_a.bzrdir.sprout('c').open_workingtree()
521
 
        out, err = self.run_bzr(['merge', '-d', 'c'])
522
 
        self.assertContainsRe(err, 'Merging from remembered parent location .*a\/')
523
 
        tree_c.branch.set_submit_branch(tree_b.bzrdir.root_transport.base)
524
 
        out, err = self.run_bzr(['merge', '-d', 'c'])
525
 
        self.assertContainsRe(err, 'Merging from remembered submit location .*b\/')
 
558
        tree_a.commit('test')
 
559
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
 
560
        tree_c = tree_a.controldir.sprout('c').open_workingtree()
 
561
        out, err = self.run_bzr(['merge', '-d', 'c'])
 
562
        self.assertContainsRe(err,
 
563
                              'Merging from remembered parent location .*a\/')
 
564
        tree_c.branch.lock_write()
 
565
        try:
 
566
            tree_c.branch.set_submit_branch(tree_b.controldir.root_transport.base)
 
567
        finally:
 
568
            tree_c.branch.unlock()
 
569
        out, err = self.run_bzr(['merge', '-d', 'c'])
 
570
        self.assertContainsRe(err,
 
571
                              'Merging from remembered submit location .*b\/')
526
572
 
527
573
    def test_remember_sets_submit(self):
528
574
        tree_a = self.make_branch_and_tree('a')
529
 
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
 
575
        tree_a.commit('rev1')
 
576
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
530
577
        self.assertIs(tree_b.branch.get_submit_branch(), None)
531
578
 
532
579
        # Remember should not happen if using default from parent
533
580
        out, err = self.run_bzr(['merge', '-d', 'b'])
534
 
        self.assertIs(tree_b.branch.get_submit_branch(), None)
 
581
        refreshed = workingtree.WorkingTree.open('b')
 
582
        self.assertIs(refreshed.branch.get_submit_branch(), None)
535
583
 
536
584
        # Remember should happen if user supplies location
537
585
        out, err = self.run_bzr(['merge', '-d', 'b', 'a'])
538
 
        self.assertEqual(tree_b.branch.get_submit_branch(),
539
 
                         tree_a.bzrdir.root_transport.base)
 
586
        refreshed = workingtree.WorkingTree.open('b')
 
587
        self.assertEqual(refreshed.branch.get_submit_branch(),
 
588
                         tree_a.controldir.root_transport.base)
 
589
 
 
590
    def test_no_remember_dont_set_submit(self):
 
591
        tree_a = self.make_branch_and_tree('a')
 
592
        self.build_tree_contents([('a/file', "a\n")])
 
593
        tree_a.add('file')
 
594
        tree_a.commit('rev1')
 
595
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
 
596
        self.assertIs(tree_b.branch.get_submit_branch(), None)
 
597
 
 
598
        # Remember should not happen if using default from parent
 
599
        out, err = self.run_bzr(['merge', '-d', 'b', '--no-remember'])
 
600
        self.assertEqual(None, tree_b.branch.get_submit_branch())
 
601
 
 
602
        # Remember should not happen if user supplies location but ask for not
 
603
        # remembering it
 
604
        out, err = self.run_bzr(['merge', '-d', 'b', '--no-remember', 'a'])
 
605
        self.assertEqual(None, tree_b.branch.get_submit_branch())
540
606
 
541
607
    def test_weave_cherrypick(self):
542
608
        this_tree = self.make_branch_and_tree('this')
543
609
        self.build_tree_contents([('this/file', "a\n")])
544
610
        this_tree.add('file')
545
611
        this_tree.commit('rev1')
546
 
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
 
612
        other_tree = this_tree.controldir.sprout('other').open_workingtree()
547
613
        self.build_tree_contents([('other/file', "a\nb\n")])
548
614
        other_tree.commit('rev2b')
549
615
        self.build_tree_contents([('other/file', "c\na\nb\n")])
556
622
        self.build_tree_contents([('a/file', 'base-contents\n')])
557
623
        tree_a.add('file')
558
624
        tree_a.commit('', rev_id='rev1')
559
 
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
 
625
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
560
626
        self.build_tree_contents([('a/file',
561
627
                                   'base-contents\nthis-contents\n')])
562
628
        tree_a.commit('', rev_id='rev2a')
581
647
    def test_merge_preview(self):
582
648
        this_tree = self.make_branch_and_tree('this')
583
649
        this_tree.commit('rev1')
584
 
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
 
650
        other_tree = this_tree.controldir.sprout('other').open_workingtree()
585
651
        self.build_tree_contents([('other/file', 'new line')])
586
652
        other_tree.add('file')
587
653
        other_tree.commit('rev2a')
603
669
        other.add('other_file')
604
670
        other.commit('rev1b')
605
671
        self.run_bzr('merge -d this other -r0..')
606
 
        self.failUnlessExists('this/other_file')
 
672
        self.assertPathExists('this/other_file')
607
673
 
608
674
    def test_merge_interactive_unlocks_branch(self):
609
675
        this = self.make_branch_and_tree('this')
610
 
        other = self.make_branch_and_tree('other')
611
 
        other.commit('empty commit')
 
676
        this.commit('empty commit')
 
677
        other = this.controldir.sprout('other').open_workingtree()
 
678
        other.commit('empty commit 2')
612
679
        self.run_bzr('merge -i -d this other')
613
680
        this.lock_write()
614
681
        this.unlock()
615
682
 
616
 
    def test_merge_reversed_revision_range(self):
617
 
        tree = self.make_branch_and_tree(".")
 
683
    def test_merge_fetches_tags(self):
 
684
        """Tags are updated by merge, and revisions named in those tags are
 
685
        fetched.
 
686
        """
 
687
        # Make a source, sprout a target off it
 
688
        builder = self.make_branch_builder('source')
 
689
        builder.build_commit(message="Rev 1", rev_id='rev-1')
 
690
        source = builder.get_branch()
 
691
        target_bzrdir = source.controldir.sprout('target')
 
692
        # Add a non-ancestry tag to source
 
693
        builder.build_commit(message="Rev 2a", rev_id='rev-2a')
 
694
        source.tags.set_tag('tag-a', 'rev-2a')
 
695
        source.set_last_revision_info(1, 'rev-1')
 
696
        source.get_config_stack().set('branch.fetch_tags', True)
 
697
        builder.build_commit(message="Rev 2b", rev_id='rev-2b')
 
698
        # Merge from source
 
699
        self.run_bzr('merge -d target source')
 
700
        target = target_bzrdir.open_branch()
 
701
        # The tag is present, and so is its revision.
 
702
        self.assertEqual('rev-2a', target.tags.lookup_tag('tag-a'))
 
703
        target.repository.get_revision('rev-2a')
 
704
 
 
705
 
 
706
class TestMergeRevisionRange(tests.TestCaseWithTransport):
 
707
 
 
708
    scenarios = (('whole-tree', dict(context='.')),
 
709
                 ('file-only', dict(context='a')))
 
710
 
 
711
    def setUp(self):
 
712
        super(TestMergeRevisionRange, self).setUp()
 
713
        self.tree = self.make_branch_and_tree(".")
 
714
        self.tree.commit('initial commit')
618
715
        for f in ("a", "b"):
619
716
            self.build_tree([f])
620
 
            tree.add(f)
621
 
            tree.commit("added "+f)
622
 
        for context in (".", "", "a"):
623
 
            self.run_bzr("merge -r 1..0 " + context)
624
 
            self.failIfExists("a")
625
 
            tree.revert()
626
 
            self.failUnlessExists("a")
627
 
 
 
717
            self.tree.add(f)
 
718
            self.tree.commit("added " + f)
 
719
 
 
720
    def test_merge_reversed_revision_range(self):
 
721
        self.run_bzr("merge -r 2..1 " + self.context)
 
722
        self.assertPathDoesNotExist("a")
 
723
        self.assertPathExists("b")
 
724
 
 
725
 
 
726
class TestMergeScript(script.TestCaseWithTransportAndScript):
 
727
    def test_merge_empty_branch(self):
 
728
        source = self.make_branch_and_tree('source')
 
729
        self.build_tree(['source/a'])
 
730
        source.add('a')
 
731
        source.commit('Added a', rev_id='rev1')
 
732
        target = self.make_branch_and_tree('target')
 
733
        self.run_script("""\
 
734
$ brz merge -d target source
 
735
2>brz: ERROR: Merging into empty branches not currently supported, https://bugs.launchpad.net/bzr/+bug/308562
 
736
""")
628
737
 
629
738
class TestMergeForce(tests.TestCaseWithTransport):
630
739
 
634
743
        self.build_tree(['a/foo'])
635
744
        self.tree_a.add(['foo'])
636
745
        self.tree_a.commit('add file')
637
 
        self.tree_b = self.tree_a.bzrdir.sprout('b').open_workingtree()
 
746
        self.tree_b = self.tree_a.controldir.sprout('b').open_workingtree()
638
747
        self.build_tree_contents([('a/foo', 'change 1')])
639
748
        self.tree_a.commit('change file')
640
749
        self.tree_b.merge_from_branch(self.tree_a.branch)