/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:
48
48
    def example_branch(self, path='.'):
49
49
        tree = self.make_branch_and_tree(path)
50
50
        self.build_tree_contents([
51
 
            (osutils.pathjoin(path, 'hello'), b'foo'),
52
 
            (osutils.pathjoin(path, 'goodbye'), b'baz')])
 
51
            (osutils.pathjoin(path, 'hello'), 'foo'),
 
52
            (osutils.pathjoin(path, 'goodbye'), 'baz')])
53
53
        tree.add('hello')
54
54
        tree.commit(message='setup')
55
55
        tree.add('goodbye')
64
64
        """
65
65
        builder = self.make_branch_builder('branch')
66
66
        builder.build_snapshot(None,
67
 
                               [('add', ('', b'root-id', 'directory', None)),
68
 
                                ('add', ('fname', b'f-id', 'file', b'a\nb\nc\n'))],
69
 
                               revision_id=b'rev1')
70
 
        builder.build_snapshot([b'rev1'],
71
 
                               [('modify', ('fname', b'a\nB\nD\n'))],
72
 
                               revision_id=b'rev2other')
 
67
            [('add', ('', 'root-id', 'directory', None)),
 
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
73
        other = builder.get_branch().controldir.sprout('other').open_branch()
74
 
        builder.build_snapshot([b'rev1'],
75
 
                               [('modify', ('fname', b'a\nB\nC\n'))], revision_id=b'rev2this')
 
74
        builder.build_snapshot(['rev1'],
 
75
            [('modify', ('f-id', 'a\nB\nC\n'))], revision_id='rev2this')
76
76
        tree = builder.get_branch().create_checkout('tree', lightweight=True)
77
77
        return tree, other
78
78
 
85
85
        a_tree = self.example_branch('a')
86
86
        ancestor = a_tree.branch.revno()
87
87
        b_tree = a_tree.controldir.sprout('b').open_workingtree()
88
 
        self.build_tree_contents([('b/goodbye', b'quux')])
 
88
        self.build_tree_contents([('b/goodbye', 'quux')])
89
89
        b_tree.commit(message="more u's are always good")
90
90
 
91
 
        self.build_tree_contents([('a/hello', b'quuux')])
 
91
        self.build_tree_contents([('a/hello', 'quuux')])
92
92
        # We can't merge when there are in-tree changes
93
93
        self.run_bzr('merge ../b', retcode=3, working_dir='a')
94
94
        a = workingtree.WorkingTree.open('a')
112
112
                     working_dir='a')
113
113
        a_tree.revert(backups=False)
114
114
        self.run_bzr('merge ../b -r last:1', working_dir='a')
115
 
        self.check_file_contents('a/goodbye', b'quux')
 
115
        self.check_file_contents('a/goodbye', 'quux')
116
116
        # Merging a branch pulls its revision into the tree
117
117
        b = branch.Branch.open('b')
118
118
        b_tip = b.last_revision()
123
123
                                working_dir='a')
124
124
        self.assertTrue("Not a branch" in err)
125
125
        self.run_bzr('merge -r revno:%d:./..revno:%d:../b'
126
 
                     % (ancestor, b.revno()), working_dir='a')
 
126
                    %(ancestor, b.revno()), working_dir='a')
127
127
        self.assertEqual(a.get_parent_ids(),
128
 
                         [a.branch.last_revision(), b.last_revision()])
129
 
        self.check_file_contents('a/goodbye', b'quux')
 
128
                          [a.branch.last_revision(), b.last_revision()])
 
129
        self.check_file_contents('a/goodbye', 'quux')
130
130
        a_tree.revert(backups=False)
131
 
        self.run_bzr('merge -r revno:%d:../b' % b.revno(), working_dir='a')
 
131
        self.run_bzr('merge -r revno:%d:../b'%b.revno(), working_dir='a')
132
132
        self.assertEqual(a.get_parent_ids(),
133
 
                         [a.branch.last_revision(), b.last_revision()])
 
133
                          [a.branch.last_revision(), b.last_revision()])
134
134
        a_tip = a.commit('merged')
135
135
        self.run_bzr('merge ../b -r last:1', working_dir='a')
136
136
        self.assertEqual([a_tip], a.get_parent_ids())
141
141
        # 'show-base' is not set
142
142
        self.run_bzr('merge ../other', working_dir='tree',
143
143
                     retcode=1)
144
 
        self.assertEqualDiff(b'a\n'
145
 
                             b'B\n'
146
 
                             b'<<<<<<< TREE\n'
147
 
                             b'C\n'
148
 
                             b'=======\n'
149
 
                             b'D\n'
150
 
                             b'>>>>>>> MERGE-SOURCE\n',
151
 
                             tree.get_file_text('fname'))
 
144
        self.assertEqualDiff('a\n'
 
145
                             'B\n'
 
146
                             '<<<<<<< TREE\n'
 
147
                             'C\n'
 
148
                             '=======\n'
 
149
                             'D\n'
 
150
                             '>>>>>>> MERGE-SOURCE\n',
 
151
                             tree.get_file_text(tree.id2path('f-id')))
152
152
 
153
153
    def test_merge_explicit_reprocess_show_base(self):
154
154
        tree, other = self.create_conflicting_branches()
162
162
        # Explicitly disable reprocess
163
163
        self.run_bzr('merge ../other --no-reprocess', working_dir='tree',
164
164
                     retcode=1)
165
 
        self.assertEqualDiff(b'a\n'
166
 
                             b'<<<<<<< TREE\n'
167
 
                             b'B\n'
168
 
                             b'C\n'
169
 
                             b'=======\n'
170
 
                             b'B\n'
171
 
                             b'D\n'
172
 
                             b'>>>>>>> MERGE-SOURCE\n',
173
 
                             tree.get_file_text('fname'))
 
165
        self.assertEqualDiff('a\n'
 
166
                             '<<<<<<< TREE\n'
 
167
                             'B\n'
 
168
                             'C\n'
 
169
                             '=======\n'
 
170
                             'B\n'
 
171
                             'D\n'
 
172
                             '>>>>>>> MERGE-SOURCE\n',
 
173
                             tree.get_file_text(tree.id2path('f-id')))
174
174
 
175
175
    def test_merge_override_show_base(self):
176
176
        tree, other = self.create_conflicting_branches()
177
177
        # Setting '--show-base' will auto-disable '--reprocess'
178
178
        self.run_bzr('merge ../other --show-base', working_dir='tree',
179
179
                     retcode=1)
180
 
        self.assertEqualDiff(b'a\n'
181
 
                             b'<<<<<<< TREE\n'
182
 
                             b'B\n'
183
 
                             b'C\n'
184
 
                             b'||||||| BASE-REVISION\n'
185
 
                             b'b\n'
186
 
                             b'c\n'
187
 
                             b'=======\n'
188
 
                             b'B\n'
189
 
                             b'D\n'
190
 
                             b'>>>>>>> MERGE-SOURCE\n',
191
 
                             tree.get_file_text('fname'))
 
180
        self.assertEqualDiff('a\n'
 
181
                             '<<<<<<< TREE\n'
 
182
                             'B\n'
 
183
                             'C\n'
 
184
                             '||||||| BASE-REVISION\n'
 
185
                             'b\n'
 
186
                             'c\n'
 
187
                             '=======\n'
 
188
                             'B\n'
 
189
                             'D\n'
 
190
                             '>>>>>>> MERGE-SOURCE\n',
 
191
                             tree.get_file_text(tree.id2path('f-id')))
192
192
 
193
193
    def test_merge_with_missing_file(self):
194
194
        """Merge handles missing file conflicts"""
195
195
        self.build_tree_contents([
196
196
            ('a/',),
197
197
            ('a/sub/',),
198
 
            ('a/sub/a.txt', b'hello\n'),
199
 
            ('a/b.txt', b'hello\n'),
200
 
            ('a/sub/c.txt', b'hello\n')])
 
198
            ('a/sub/a.txt', 'hello\n'),
 
199
            ('a/b.txt', 'hello\n'),
 
200
            ('a/sub/c.txt', 'hello\n')])
201
201
        a_tree = self.make_branch_and_tree('a')
202
202
        a_tree.add(['sub', 'b.txt', 'sub/c.txt', 'sub/a.txt'])
203
203
        a_tree.commit(message='added a')
204
204
        b_tree = a_tree.controldir.sprout('b').open_workingtree()
205
205
        self.build_tree_contents([
206
 
            ('a/sub/a.txt', b'hello\nthere\n'),
207
 
            ('a/b.txt', b'hello\nthere\n'),
208
 
            ('a/sub/c.txt', b'hello\nthere\n')])
 
206
            ('a/sub/a.txt', 'hello\nthere\n'),
 
207
            ('a/b.txt', 'hello\nthere\n'),
 
208
            ('a/sub/c.txt', 'hello\nthere\n')])
209
209
        a_tree.commit(message='Added there')
210
210
        os.remove('a/sub/a.txt')
211
211
        os.remove('a/sub/c.txt')
213
213
        os.remove('a/b.txt')
214
214
        a_tree.commit(message='Removed a.txt')
215
215
        self.build_tree_contents([
216
 
            ('b/sub/a.txt', b'hello\nsomething\n'),
217
 
            ('b/b.txt', b'hello\nsomething\n'),
218
 
            ('b/sub/c.txt', b'hello\nsomething\n')])
 
216
            ('b/sub/a.txt', 'hello\nsomething\n'),
 
217
            ('b/b.txt', 'hello\nsomething\n'),
 
218
            ('b/sub/c.txt', 'hello\nsomething\n')])
219
219
        b_tree.commit(message='Modified a.txt')
220
220
 
221
221
        self.run_bzr('merge ../a/', retcode=1, working_dir='b')
230
230
        tree, other = self.create_conflicting_branches()
231
231
        self.run_bzr('merge ../other', working_dir='tree',
232
232
                     retcode=1)
233
 
        self.assertFileEqual(b'a\nb\nc\n', 'tree/fname.BASE')
234
 
        self.assertFileEqual(b'a\nB\nD\n', 'tree/fname.OTHER')
235
 
        self.assertFileEqual(b'a\nB\nC\n', 'tree/fname.THIS')
 
233
        self.assertFileEqual('a\nb\nc\n', 'tree/fname.BASE')
 
234
        self.assertFileEqual('a\nB\nD\n', 'tree/fname.OTHER')
 
235
        self.assertFileEqual('a\nB\nC\n', 'tree/fname.THIS')
236
236
 
237
237
    def test_weave_conflict_leaves_base_this_other_files(self):
238
238
        tree, other = self.create_conflicting_branches()
239
239
        self.run_bzr('merge ../other --weave', working_dir='tree',
240
240
                     retcode=1)
241
 
        self.assertFileEqual(b'a\nb\nc\n', 'tree/fname.BASE')
242
 
        self.assertFileEqual(b'a\nB\nD\n', 'tree/fname.OTHER')
243
 
        self.assertFileEqual(b'a\nB\nC\n', 'tree/fname.THIS')
 
241
        self.assertFileEqual('a\nb\nc\n', 'tree/fname.BASE')
 
242
        self.assertFileEqual('a\nB\nD\n', 'tree/fname.OTHER')
 
243
        self.assertFileEqual('a\nB\nC\n', 'tree/fname.THIS')
244
244
 
245
245
    def test_merge_remember(self):
246
246
        """Merge changes from one branch to another, test submit location."""
266
266
        # test merge for failure without parent set
267
267
        out = self.run_bzr('merge', retcode=3, working_dir='branch_b')
268
268
        self.assertEqual(out,
269
 
                         ('', 'brz: ERROR: No location specified or remembered\n'))
 
269
                ('', 'brz: ERROR: No location specified or remembered\n'))
270
270
 
271
271
        # test uncommitted changes
272
272
        self.build_tree(['branch_b/d'])
283
283
        # re-open branch as external run_brz modified it
284
284
        branch_b = branch_b.controldir.open_branch()
285
285
        self.assertEqual(osutils.abspath(branch_b.get_submit_branch()),
286
 
                         osutils.abspath(parent))
 
286
                          osutils.abspath(parent))
287
287
        # test implicit --remember when committing new file
288
288
        self.build_tree(['branch_b/e'])
289
289
        tree_b.add('e')
302
302
        # re-open branch as external run_brz modified it
303
303
        branch_b = branch_b.controldir.open_branch()
304
304
        self.assertEqual(osutils.abspath(branch_b.get_submit_branch()),
305
 
                         osutils.abspath(branch_c.controldir.root_transport.base))
 
305
                          osutils.abspath(branch_c.controldir.root_transport.base))
306
306
        # re-open tree as external run_brz modified it
307
307
        tree_b = branch_b.controldir.open_workingtree()
308
308
        tree_b.commit('merge branch_c')
309
309
 
310
310
    def test_merge_bundle(self):
311
 
        from breezy.bzr.testament import Testament
 
311
        from breezy.testament import Testament
312
312
        tree_a = self.make_branch_and_tree('branch_a')
313
 
        self.build_tree_contents([('branch_a/a', b'hello')])
 
313
        self.build_tree_contents([('branch_a/a', 'hello')])
314
314
        tree_a.add('a')
315
315
        tree_a.commit('message')
316
316
 
317
317
        tree_b = tree_a.controldir.sprout('branch_b').open_workingtree()
318
 
        self.build_tree_contents([('branch_a/a', b'hey there')])
 
318
        self.build_tree_contents([('branch_a/a', 'hey there')])
319
319
        tree_a.commit('message')
320
320
 
321
 
        self.build_tree_contents([('branch_b/a', b'goodbye')])
 
321
        self.build_tree_contents([('branch_b/a', 'goodbye')])
322
322
        tree_b.commit('message')
323
323
        self.run_bzr('bundle ../branch_a -o ../bundle', working_dir='branch_b')
324
324
        self.run_bzr('merge ../bundle', retcode=1, working_dir='branch_a')
327
327
        testament_b = Testament.from_revision(tree_b.branch.repository,
328
328
                                              tree_b.get_parent_ids()[0])
329
329
        self.assertEqualDiff(testament_a.as_text(),
330
 
                             testament_b.as_text())
 
330
                         testament_b.as_text())
331
331
        tree_a.set_conflicts(conflicts.ConflictList())
332
332
        tree_a.commit('message')
333
333
        # it is legal to attempt to merge an already-merged bundle
334
334
        err = self.run_bzr('merge ../bundle', working_dir='branch_a')[1]
335
335
        # but it does nothing
336
 
        self.assertFalse(tree_a.changes_from(
337
 
            tree_a.basis_tree()).has_changed())
 
336
        self.assertFalse(tree_a.changes_from(tree_a.basis_tree()).has_changed())
338
337
        self.assertEqual('Nothing to do.\n', err)
339
338
 
340
339
    def test_merge_uncommitted(self):
371
370
        """It should not be possible to merge changes from a file which
372
371
        does not exist."""
373
372
        tree_a = self.make_branch_and_tree('tree_a')
374
 
        self.build_tree_contents([('tree_a/file', b'bar\n')])
 
373
        self.build_tree_contents([('tree_a/file', 'bar\n')])
375
374
        tree_a.add(['file'])
376
375
        tree_a.commit('commit 1')
377
 
        self.run_bzr_error(('Path\\(s\\) do not exist: non/existing',),
 
376
        self.run_bzr_error(('Path\(s\) do not exist: non/existing',),
378
377
                           ['merge', 'non/existing'], working_dir='tree_a')
379
378
 
380
379
    def pullable_branch(self):
381
380
        tree_a = self.make_branch_and_tree('a')
382
 
        self.build_tree_contents([('a/file', b'bar\n')])
 
381
        self.build_tree_contents([('a/file', 'bar\n')])
383
382
        tree_a.add(['file'])
384
383
        self.id1 = tree_a.commit('commit 1')
385
384
 
386
385
        tree_b = self.make_branch_and_tree('b')
387
386
        tree_b.pull(tree_a.branch)
388
 
        self.build_tree_contents([('b/file', b'foo\n')])
 
387
        self.build_tree_contents([('b/file', 'foo\n')])
389
388
        self.id2 = tree_b.commit('commit 2')
390
389
 
391
390
    def test_merge_pull(self):
399
398
        self.pullable_branch()
400
399
        (out, err) = self.run_bzr('merge --pull --preview -d a b')
401
400
        self.assertThat(out, matchers.DocTestMatches(
402
 
            """=== modified file 'file'
 
401
"""=== modified file 'file'
403
402
--- file\t...
404
403
+++ file\t...
405
404
@@ -1,1 +1,1 @@
412
411
 
413
412
    def test_merge_kind_change(self):
414
413
        tree_a = self.make_branch_and_tree('tree_a')
415
 
        self.build_tree_contents([('tree_a/file', b'content_1')])
416
 
        tree_a.add('file', b'file-id')
 
414
        self.build_tree_contents([('tree_a/file', 'content_1')])
 
415
        tree_a.add('file', 'file-id')
417
416
        tree_a.commit('added file')
418
417
        tree_b = tree_a.controldir.sprout('tree_b').open_workingtree()
419
418
        os.unlink('tree_a/file')
423
422
        self.assertEqual('directory', osutils.file_kind('tree_b/file'))
424
423
        tree_b.revert()
425
424
        self.assertEqual('file', osutils.file_kind('tree_b/file'))
426
 
        self.build_tree_contents([('tree_b/file', b'content_2')])
 
425
        self.build_tree_contents([('tree_b/file', 'content_2')])
427
426
        tree_b.commit('content change')
428
427
        self.run_bzr('merge ../tree_a', retcode=1, working_dir='tree_b')
429
428
        self.assertEqual(tree_b.conflicts(),
440
439
        target = source.controldir.sprout('target').open_workingtree()
441
440
        self.build_tree(['source/a'])
442
441
        source.add('a')
443
 
        source.commit('Added a', rev_id=b'rev1')
 
442
        source.commit('Added a', rev_id='rev1')
444
443
        self.build_tree(['source/b'])
445
444
        source.add('b')
446
 
        source.commit('Added b', rev_id=b'rev2')
 
445
        source.commit('Added b', rev_id='rev2')
447
446
        target.commit('empty commit')
448
 
        self.write_directive('directive', source.branch, 'target', b'rev2',
449
 
                             b'rev1')
 
447
        self.write_directive('directive', source.branch, 'target', 'rev2',
 
448
                             'rev1')
450
449
        out, err = self.run_bzr('merge -d target directive')
451
450
        self.assertPathDoesNotExist('target/a')
452
451
        self.assertPathExists('target/b')
458
457
            source.repository, revision_id, 0, 0, target,
459
458
            base_revision_id=base_revision_id)
460
459
        if mangle_patch:
461
 
            md.patch = b'asdf\n'
462
 
        self.build_tree_contents([(filename, b''.join(md.to_lines()))])
 
460
            md.patch = 'asdf\n'
 
461
        self.build_tree_contents([(filename, ''.join(md.to_lines()))])
463
462
 
464
463
    def test_directive_verify_warning(self):
465
464
        source = self.make_branch_and_tree('source')
466
465
        self.build_tree(['source/a'])
467
466
        source.add('a')
468
 
        source.commit('Added a', rev_id=b'rev1')
 
467
        source.commit('Added a', rev_id='rev1')
469
468
        target = self.make_branch_and_tree('target')
470
469
        target.commit('empty commit')
471
 
        self.write_directive('directive', source.branch, 'target', b'rev1')
 
470
        self.write_directive('directive', source.branch, 'target', 'rev1')
472
471
        err = self.run_bzr('merge -d target directive')[1]
473
472
        self.assertNotContainsRe(err, 'Preview patch does not match changes')
474
473
        target.revert()
475
 
        self.write_directive('directive', source.branch, 'target', b'rev1',
 
474
        self.write_directive('directive', source.branch, 'target', 'rev1',
476
475
                             mangle_patch=True)
477
476
        err = self.run_bzr('merge -d target directive')[1]
478
477
        self.assertContainsRe(err, 'Preview patch does not match changes')
484
483
        branch_a = target.controldir.sprout('branch_a').open_workingtree()
485
484
        self.build_tree(['branch_a/file1'])
486
485
        branch_a.add('file1')
487
 
        branch_a.commit('added file1', rev_id=b'rev2a')
 
486
        branch_a.commit('added file1', rev_id='rev2a')
488
487
        branch_b = target.controldir.sprout('branch_b').open_workingtree()
489
488
        self.build_tree(['branch_b/file2'])
490
489
        branch_b.add('file2')
491
 
        branch_b.commit('added file2', rev_id=b'rev2b')
 
490
        branch_b.commit('added file2', rev_id='rev2b')
492
491
        branch_b.merge_from_branch(branch_a.branch)
493
492
        self.assertPathExists('branch_b/file1')
494
 
        branch_b.commit('merged branch_a', rev_id=b'rev3b')
 
493
        branch_b.commit('merged branch_a', rev_id='rev3b')
495
494
 
496
495
        # It works if the revid has an interger revno
497
496
        self.run_bzr('merge -d target -r revid:rev2a branch_a')
522
521
        # make source branch
523
522
        source = self.make_branch_and_tree('source')
524
523
        for f in ('a', 'b', 'c', 'd'):
525
 
            self.build_tree(['source/' + f])
 
524
            self.build_tree(['source/'+f])
526
525
            source.add(f)
527
 
            source.commit('added ' + f, rev_id=b'rev_' + f.encode('ascii'))
 
526
            source.commit('added '+f, rev_id='rev_'+f)
528
527
        # target branch
529
 
        target = source.controldir.sprout(
530
 
            'target', b'rev_a').open_workingtree()
 
528
        target = source.controldir.sprout('target', 'rev_a').open_workingtree()
531
529
        self.assertDirectoryContent('target', ['.bzr', 'a'])
532
530
        # pick 1 revision
533
531
        self.run_bzr('merge -d target -r revid:rev_b..revid:rev_c source')
543
541
 
544
542
    def test_merge_criss_cross(self):
545
543
        tree_a = self.make_branch_and_tree('a')
546
 
        tree_a.commit('', rev_id=b'rev1')
 
544
        tree_a.commit('', rev_id='rev1')
547
545
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
548
 
        tree_a.commit('', rev_id=b'rev2a')
549
 
        tree_b.commit('', rev_id=b'rev2b')
 
546
        tree_a.commit('', rev_id='rev2a')
 
547
        tree_b.commit('', rev_id='rev2b')
550
548
        tree_a.merge_from_branch(tree_b.branch)
551
549
        tree_b.merge_from_branch(tree_a.branch)
552
 
        tree_a.commit('', rev_id=b'rev3a')
553
 
        tree_b.commit('', rev_id=b'rev3b')
 
550
        tree_a.commit('', rev_id='rev3a')
 
551
        tree_b.commit('', rev_id='rev3b')
554
552
        graph = tree_a.branch.repository.get_graph(tree_b.branch.repository)
555
553
        out, err = self.run_bzr(['merge', '-d', 'a', 'b'])
556
554
        self.assertContainsRe(err, 'Warning: criss-cross merge encountered.')
562
560
        tree_c = tree_a.controldir.sprout('c').open_workingtree()
563
561
        out, err = self.run_bzr(['merge', '-d', 'c'])
564
562
        self.assertContainsRe(err,
565
 
                              'Merging from remembered parent location .*a\\/')
566
 
        with tree_c.branch.lock_write():
567
 
            tree_c.branch.set_submit_branch(
568
 
                tree_b.controldir.root_transport.base)
 
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
569
        out, err = self.run_bzr(['merge', '-d', 'c'])
570
570
        self.assertContainsRe(err,
571
 
                              'Merging from remembered submit location .*b\\/')
 
571
                              'Merging from remembered submit location .*b\/')
572
572
 
573
573
    def test_remember_sets_submit(self):
574
574
        tree_a = self.make_branch_and_tree('a')
589
589
 
590
590
    def test_no_remember_dont_set_submit(self):
591
591
        tree_a = self.make_branch_and_tree('a')
592
 
        self.build_tree_contents([('a/file', b"a\n")])
 
592
        self.build_tree_contents([('a/file', "a\n")])
593
593
        tree_a.add('file')
594
594
        tree_a.commit('rev1')
595
595
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
606
606
 
607
607
    def test_weave_cherrypick(self):
608
608
        this_tree = self.make_branch_and_tree('this')
609
 
        self.build_tree_contents([('this/file', b"a\n")])
 
609
        self.build_tree_contents([('this/file', "a\n")])
610
610
        this_tree.add('file')
611
611
        this_tree.commit('rev1')
612
612
        other_tree = this_tree.controldir.sprout('other').open_workingtree()
613
 
        self.build_tree_contents([('other/file', b"a\nb\n")])
 
613
        self.build_tree_contents([('other/file', "a\nb\n")])
614
614
        other_tree.commit('rev2b')
615
 
        self.build_tree_contents([('other/file', b"c\na\nb\n")])
 
615
        self.build_tree_contents([('other/file', "c\na\nb\n")])
616
616
        other_tree.commit('rev3b')
617
617
        self.run_bzr('merge --weave -d this other -r -2..-1')
618
 
        self.assertFileEqual(b'c\na\n', 'this/file')
 
618
        self.assertFileEqual('c\na\n', 'this/file')
619
619
 
620
620
    def test_lca_merge_criss_cross(self):
621
621
        tree_a = self.make_branch_and_tree('a')
622
 
        self.build_tree_contents([('a/file', b'base-contents\n')])
 
622
        self.build_tree_contents([('a/file', 'base-contents\n')])
623
623
        tree_a.add('file')
624
 
        tree_a.commit('', rev_id=b'rev1')
 
624
        tree_a.commit('', rev_id='rev1')
625
625
        tree_b = tree_a.controldir.sprout('b').open_workingtree()
626
626
        self.build_tree_contents([('a/file',
627
 
                                   b'base-contents\nthis-contents\n')])
628
 
        tree_a.commit('', rev_id=b'rev2a')
 
627
                                   'base-contents\nthis-contents\n')])
 
628
        tree_a.commit('', rev_id='rev2a')
629
629
        self.build_tree_contents([('b/file',
630
 
                                   b'base-contents\nother-contents\n')])
631
 
        tree_b.commit('', rev_id=b'rev2b')
 
630
                                   'base-contents\nother-contents\n')])
 
631
        tree_b.commit('', rev_id='rev2b')
632
632
        tree_a.merge_from_branch(tree_b.branch)
633
633
        self.build_tree_contents([('a/file',
634
 
                                   b'base-contents\nthis-contents\n')])
 
634
                                   'base-contents\nthis-contents\n')])
635
635
        tree_a.set_conflicts(conflicts.ConflictList())
636
636
        tree_b.merge_from_branch(tree_a.branch)
637
637
        self.build_tree_contents([('b/file',
638
 
                                   b'base-contents\nother-contents\n')])
 
638
                                   'base-contents\nother-contents\n')])
639
639
        tree_b.set_conflicts(conflicts.ConflictList())
640
 
        tree_a.commit('', rev_id=b'rev3a')
641
 
        tree_b.commit('', rev_id=b'rev3b')
 
640
        tree_a.commit('', rev_id='rev3a')
 
641
        tree_b.commit('', rev_id='rev3b')
642
642
        out, err = self.run_bzr(['merge', '-d', 'a', 'b', '--lca'], retcode=1)
643
 
        self.assertFileEqual(b'base-contents\n<<<<<<< TREE\nthis-contents\n'
644
 
                             b'=======\nother-contents\n>>>>>>> MERGE-SOURCE\n',
 
643
        self.assertFileEqual('base-contents\n<<<<<<< TREE\nthis-contents\n'
 
644
                             '=======\nother-contents\n>>>>>>> MERGE-SOURCE\n',
645
645
                             'a/file')
646
646
 
647
647
    def test_merge_preview(self):
648
648
        this_tree = self.make_branch_and_tree('this')
649
649
        this_tree.commit('rev1')
650
650
        other_tree = this_tree.controldir.sprout('other').open_workingtree()
651
 
        self.build_tree_contents([('other/file', b'new line')])
 
651
        self.build_tree_contents([('other/file', 'new line')])
652
652
        other_tree.add('file')
653
653
        other_tree.commit('rev2a')
654
654
        this_tree.commit('rev2b')
655
655
        out, err = self.run_bzr(['merge', '-d', 'this', 'other', '--preview'])
656
 
        self.assertContainsRe(out, '\\+new line')
657
 
        self.assertNotContainsRe(err, '\\+N  file\n')
 
656
        self.assertContainsRe(out, '\+new line')
 
657
        self.assertNotContainsRe(err, '\+N  file\n')
658
658
        this_tree.lock_read()
659
659
        self.addCleanup(this_tree.unlock)
660
660
        self.assertEqual([],
686
686
        """
687
687
        # Make a source, sprout a target off it
688
688
        builder = self.make_branch_builder('source')
689
 
        builder.build_commit(message="Rev 1", rev_id=b'rev-1')
 
689
        builder.build_commit(message="Rev 1", rev_id='rev-1')
690
690
        source = builder.get_branch()
691
691
        target_bzrdir = source.controldir.sprout('target')
692
692
        # Add a non-ancestry tag to source
693
 
        builder.build_commit(message="Rev 2a", rev_id=b'rev-2a')
694
 
        source.tags.set_tag('tag-a', b'rev-2a')
695
 
        source.set_last_revision_info(1, b'rev-1')
 
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
696
        source.get_config_stack().set('branch.fetch_tags', True)
697
 
        builder.build_commit(message="Rev 2b", rev_id=b'rev-2b')
 
697
        builder.build_commit(message="Rev 2b", rev_id='rev-2b')
698
698
        # Merge from source
699
699
        self.run_bzr('merge -d target source')
700
700
        target = target_bzrdir.open_branch()
701
701
        # The tag is present, and so is its revision.
702
 
        self.assertEqual(b'rev-2a', target.tags.lookup_tag('tag-a'))
703
 
        target.repository.get_revision(b'rev-2a')
 
702
        self.assertEqual('rev-2a', target.tags.lookup_tag('tag-a'))
 
703
        target.repository.get_revision('rev-2a')
704
704
 
705
705
 
706
706
class TestMergeRevisionRange(tests.TestCaseWithTransport):
728
728
        source = self.make_branch_and_tree('source')
729
729
        self.build_tree(['source/a'])
730
730
        source.add('a')
731
 
        source.commit('Added a', rev_id=b'rev1')
 
731
        source.commit('Added a', rev_id='rev1')
732
732
        target = self.make_branch_and_tree('target')
733
733
        self.run_script("""\
734
734
$ brz merge -d target source
735
735
2>brz: ERROR: Merging into empty branches not currently supported, https://bugs.launchpad.net/bzr/+bug/308562
736
736
""")
737
737
 
738
 
 
739
738
class TestMergeForce(tests.TestCaseWithTransport):
740
739
 
741
740
    def setUp(self):
745
744
        self.tree_a.add(['foo'])
746
745
        self.tree_a.commit('add file')
747
746
        self.tree_b = self.tree_a.controldir.sprout('b').open_workingtree()
748
 
        self.build_tree_contents([('a/foo', b'change 1')])
 
747
        self.build_tree_contents([('a/foo', 'change 1')])
749
748
        self.tree_a.commit('change file')
750
749
        self.tree_b.merge_from_branch(self.tree_a.branch)
751
750
 
754
753
        # Second merge on top of the uncommitted one
755
754
        self.run_bzr(['merge', '../a', '--force'], working_dir='b')
756
755
 
 
756
 
757
757
    def test_merge_with_uncommitted_changes(self):
758
758
        self.run_bzr_error(['Working tree .* has uncommitted changes'],
759
759
                           ['merge', '../a'], working_dir='b')