~bzr-pqm/bzr/bzr.dev

2255.2.217 by Martin Pool
docs
1
# Copyright (C) 2005, 2006, 2007 Canonical Ltd
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
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
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
1185.1.41 by Robert Collins
massive patch from Alexander Belchenko - many PEP8 fixes, removes unused function uuid
17
import os
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
18
from StringIO import StringIO
1185.1.41 by Robert Collins
massive patch from Alexander Belchenko - many PEP8 fixes, removes unused function uuid
19
2221.4.16 by Aaron Bentley
Add tests for get_merge_type_registry
20
from bzrlib import (
21
    conflicts,
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
22
    errors,
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
23
    knit,
2221.4.16 by Aaron Bentley
Add tests for get_merge_type_registry
24
    merge as _mod_merge,
25
    option,
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
26
    progress,
3008.1.21 by Aaron Bentley
Make compute_transform private, test make_preview_transform
27
    transform,
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
28
    versionedfile,
2221.4.16 by Aaron Bentley
Add tests for get_merge_type_registry
29
    )
974.1.56 by aaron.bentley at utoronto
Added merge test
30
from bzrlib.branch import Branch
1551.7.10 by Aaron Bentley
Remerge doesn't clear unrelated conflicts
31
from bzrlib.conflicts import ConflictList, TextConflict
1185.24.3 by Aaron Bentley
Integrated reprocessing into the rest of the merge stuff
32
from bzrlib.errors import UnrelatedBranches, NoCommits, BzrCommandError
3062.1.13 by Aaron Bentley
Make _PlanMerge an implementation detail of _PlanMergeVersionedFile
33
from bzrlib.merge import transform_tree, merge_inner, _PlanMerge
1959.4.6 by Aaron Bentley
Ensure merge works across kind changes
34
from bzrlib.osutils import pathjoin, file_kind
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
35
from bzrlib.tests import TestCaseWithTransport, TestCaseWithMemoryTransport
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
36
from bzrlib.trace import (enable_test_log, disable_test_log)
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
37
from bzrlib.workingtree import WorkingTree
38
39
40
class TestMerge(TestCaseWithTransport):
974.1.56 by aaron.bentley at utoronto
Added merge test
41
    """Test appending more than one revision"""
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
42
974.1.56 by aaron.bentley at utoronto
Added merge test
43
    def test_pending(self):
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
44
        wt = self.make_branch_and_tree('.')
1908.6.11 by Robert Collins
Remove usage of tree.pending_merges().
45
        rev_a = wt.commit("lala!")
46
        self.assertEqual([rev_a], wt.get_parent_ids())
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
47
        self.assertRaises(errors.PointlessMerge, wt.merge_from_branch,
48
                          wt.branch)
1908.6.11 by Robert Collins
Remove usage of tree.pending_merges().
49
        self.assertEqual([rev_a], wt.get_parent_ids())
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
50
        return wt
974.1.80 by Aaron Bentley
Improved merge error handling and testing
51
1558.4.11 by Aaron Bentley
Allow merge against self, make fetching self a noop
52
    def test_undo(self):
53
        wt = self.make_branch_and_tree('.')
54
        wt.commit("lala!")
55
        wt.commit("haha!")
56
        wt.commit("blabla!")
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
57
        wt.merge_from_branch(wt.branch, wt.branch.get_rev_id(2),
58
                             wt.branch.get_rev_id(1))
1558.4.11 by Aaron Bentley
Allow merge against self, make fetching self a noop
59
974.1.80 by Aaron Bentley
Improved merge error handling and testing
60
    def test_nocommits(self):
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
61
        wt = self.test_pending()
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
62
        wt2 = self.make_branch_and_tree('branch2')
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
63
        self.assertRaises(NoCommits, wt.merge_from_branch, wt2.branch)
64
        return wt, wt2
974.1.80 by Aaron Bentley
Improved merge error handling and testing
65
66
    def test_unrelated(self):
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
67
        wt, wt2 = self.test_nocommits()
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
68
        wt2.commit("blah")
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
69
        self.assertRaises(UnrelatedBranches, wt.merge_from_branch, wt2.branch)
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
70
        return wt2
974.1.80 by Aaron Bentley
Improved merge error handling and testing
71
1908.6.11 by Robert Collins
Remove usage of tree.pending_merges().
72
    def test_merge_one_file(self):
73
        """Do a partial merge of a tree which should not affect tree parents."""
1645.1.1 by Aaron Bentley
Implement single-file merge
74
        wt1 = self.make_branch_and_tree('branch1')
1908.6.11 by Robert Collins
Remove usage of tree.pending_merges().
75
        tip = wt1.commit('empty commit')
1645.1.1 by Aaron Bentley
Implement single-file merge
76
        wt2 = self.make_branch_and_tree('branch2')
77
        wt2.pull(wt1.branch)
78
        file('branch1/foo', 'wb').write('foo')
79
        file('branch1/bar', 'wb').write('bar')
80
        wt1.add('foo')
81
        wt1.add('bar')
82
        wt1.commit('add foobar')
83
        os.chdir('branch2')
2530.3.1 by Martin Pool
Cleanup old variations on run_bzr in the test suite
84
        self.run_bzr('merge ../branch1/baz', retcode=3)
85
        self.run_bzr('merge ../branch1/foo')
1645.1.1 by Aaron Bentley
Implement single-file merge
86
        self.failUnlessExists('foo')
87
        self.failIfExists('bar')
1908.6.11 by Robert Collins
Remove usage of tree.pending_merges().
88
        wt2 = WorkingTree.open('.') # opens branch2
89
        self.assertEqual([tip], wt2.get_parent_ids())
1908.6.9 by Robert Collins
Fix status to not use pending_merges.
90
        
974.1.88 by Aaron Bentley
Set a pending_merge if the merge base is forced to revno 0
91
    def test_pending_with_null(self):
1551.8.25 by Aaron Bentley
Fix deprecated use of pending_merges
92
        """When base is forced to revno 0, parent_ids are set"""
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
93
        wt2 = self.test_unrelated()
1508.1.19 by Robert Collins
Give format3 working trees their own last-revision marker.
94
        wt1 = WorkingTree.open('.')
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
95
        br1 = wt1.branch
1534.1.31 by Robert Collins
Deprecated fetch.fetch and fetch.greedy_fetch for branch.fetch, and move the Repository.fetch internals to InterRepo and InterWeaveRepo.
96
        br1.fetch(wt2.branch)
1390 by Robert Collins
pair programming worx... merge integration and weave
97
        # merge all of branch 2 into branch 1 even though they 
98
        # are not related.
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
99
        wt1.merge_from_branch(wt2.branch, wt2.last_revision(), 'null:')
1908.6.11 by Robert Collins
Remove usage of tree.pending_merges().
100
        self.assertEqual([br1.last_revision(), wt2.branch.last_revision()],
101
            wt1.get_parent_ids())
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
102
        return (wt1, wt2.branch)
974.1.89 by Aaron Bentley
Fixed merging with multiple roots, by using null as graph root.
103
104
    def test_two_roots(self):
105
        """Merge base is sane when two unrelated branches are merged"""
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
106
        wt1, br2 = self.test_pending_with_null()
107
        wt1.commit("blah")
3228.4.15 by John Arbash Meinel
Stop using common_ancestor
108
        wt1.lock_read()
109
        try:
110
            last = wt1.branch.last_revision()
111
            last2 = br2.last_revision()
112
            graph = wt1.branch.repository.get_graph()
113
            self.assertEqual(last2, graph.find_unique_lca(last, last2))
114
        finally:
115
            wt1.unlock()
1185.46.1 by Aaron Bentley
Test case when file to be renamed is also deleted
116
117
    def test_create_rename(self):
118
        """Rename an inventory entry while creating the file"""
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
119
        tree =self.make_branch_and_tree('.')
1185.46.1 by Aaron Bentley
Test case when file to be renamed is also deleted
120
        file('name1', 'wb').write('Hello')
121
        tree.add('name1')
122
        tree.commit(message="hello")
123
        tree.rename_one('name1', 'name2')
124
        os.unlink('name2')
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
125
        transform_tree(tree, tree.branch.basis_tree())
1185.46.2 by Aaron Bentley
Added test for renaming both parent and child
126
127
    def test_layered_rename(self):
128
        """Rename both child and parent at same time"""
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
129
        tree =self.make_branch_and_tree('.')
1185.46.2 by Aaron Bentley
Added test for renaming both parent and child
130
        os.mkdir('dirname1')
131
        tree.add('dirname1')
1185.31.32 by John Arbash Meinel
Updated the bzr sourcecode to use bzrlib.osutils.pathjoin rather than os.path.join to enforce internal use of / instead of \
132
        filename = pathjoin('dirname1', 'name1')
1185.46.2 by Aaron Bentley
Added test for renaming both parent and child
133
        file(filename, 'wb').write('Hello')
134
        tree.add(filename)
135
        tree.commit(message="hello")
1185.31.32 by John Arbash Meinel
Updated the bzr sourcecode to use bzrlib.osutils.pathjoin rather than os.path.join to enforce internal use of / instead of \
136
        filename2 = pathjoin('dirname1', 'name2')
1185.46.2 by Aaron Bentley
Added test for renaming both parent and child
137
        tree.rename_one(filename, filename2)
138
        tree.rename_one('dirname1', 'dirname2')
1534.4.26 by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create.
139
        transform_tree(tree, tree.branch.basis_tree())
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
140
141
    def test_ignore_zero_merge_inner(self):
1907.4.1 by Matthieu Moy
Fixed merge to work nicely with -r revno:N:path
142
        # Test that merge_inner's ignore zero parameter is effective
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
143
        tree_a =self.make_branch_and_tree('a')
144
        tree_a.commit(message="hello")
145
        dir_b = tree_a.bzrdir.sprout('b')
146
        tree_b = dir_b.open_workingtree()
3146.4.4 by Aaron Bentley
Add write lock, so merge_inner works properly
147
        tree_b.lock_write()
148
        self.addCleanup(tree_b.unlock)
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
149
        tree_a.commit(message="hello again")
150
        log = StringIO()
151
        merge_inner(tree_b.branch, tree_a, tree_b.basis_tree(), 
152
                    this_tree=tree_b, ignore_zero=True)
1927.3.1 by Carl Friedrich Bolz
Throw away on-disk logfile when possible.
153
        log = self._get_log(keep_log_file=True)
1551.4.1 by Aaron Bentley
Workaround for silly _get_log behaviour in test
154
        self.failUnless('All changes applied successfully.\n' not in log)
2796.1.3 by Aaron Bentley
update new test case
155
        tree_b.revert()
1551.2.23 by Aaron Bentley
Got merge_inner's ignore_zero parameter working
156
        merge_inner(tree_b.branch, tree_a, tree_b.basis_tree(), 
157
                    this_tree=tree_b, ignore_zero=False)
1927.3.1 by Carl Friedrich Bolz
Throw away on-disk logfile when possible.
158
        log = self._get_log(keep_log_file=True)
1551.4.1 by Aaron Bentley
Workaround for silly _get_log behaviour in test
159
        self.failUnless('All changes applied successfully.\n' in log)
1551.7.10 by Aaron Bentley
Remerge doesn't clear unrelated conflicts
160
161
    def test_merge_inner_conflicts(self):
162
        tree_a = self.make_branch_and_tree('a')
163
        tree_a.set_conflicts(ConflictList([TextConflict('patha')]))
1551.7.11 by Aaron Bentley
Add WorkingTree.add_conflicts
164
        merge_inner(tree_a.branch, tree_a, tree_a, this_tree=tree_a)
1551.7.13 by Aaron Bentley
Switched from actual, expected to expected, actual, for John.
165
        self.assertEqual(1, len(tree_a.conflicts()))
1551.8.22 by Aaron Bentley
Improve message when OTHER deletes an in-use tree
166
167
    def test_rmdir_conflict(self):
168
        tree_a = self.make_branch_and_tree('a')
169
        self.build_tree(['a/b/'])
170
        tree_a.add('b', 'b-id')
171
        tree_a.commit('added b')
2255.7.12 by John Arbash Meinel
Some comments on merge code, fix merge tests that
172
        # basis_tree() is only guaranteed to be valid as long as it is actually
173
        # the basis tree. This mutates the tree after grabbing basis, so go to
174
        # the repository.
175
        base_tree = tree_a.branch.repository.revision_tree(tree_a.last_revision())
1551.8.22 by Aaron Bentley
Improve message when OTHER deletes an in-use tree
176
        tree_z = tree_a.bzrdir.sprout('z').open_workingtree()
177
        self.build_tree(['a/b/c'])
178
        tree_a.add('b/c')
179
        tree_a.commit('added c')
180
        os.rmdir('z/b')
181
        tree_z.commit('removed b')
182
        merge_inner(tree_z.branch, tree_a, base_tree, this_tree=tree_z)
183
        self.assertEqual([
184
            conflicts.MissingParent('Created directory', 'b', 'b-id'),
185
            conflicts.UnversionedParent('Versioned directory', 'b', 'b-id')],
186
            tree_z.conflicts())
2255.2.216 by Robert Collins
simplify merge_nested tests.
187
        merge_inner(tree_a.branch, tree_z.basis_tree(), base_tree,
1551.8.22 by Aaron Bentley
Improve message when OTHER deletes an in-use tree
188
                    this_tree=tree_a)
189
        self.assertEqual([
190
            conflicts.DeletingParent('Not deleting', 'b', 'b-id'),
191
            conflicts.UnversionedParent('Versioned directory', 'b', 'b-id')],
192
            tree_a.conflicts())
1551.10.2 by Aaron Bentley
Handle merge with dangling inventory entries
193
2100.3.29 by Aaron Bentley
Get merge working initially
194
    def test_nested_merge(self):
2255.2.194 by Robert Collins
[BROKEN] Many updates to stop using experimental formats in tests.
195
        tree = self.make_branch_and_tree('tree',
196
            format='dirstate-with-subtree')
2100.3.29 by Aaron Bentley
Get merge working initially
197
        sub_tree = self.make_branch_and_tree('tree/sub-tree',
2255.2.194 by Robert Collins
[BROKEN] Many updates to stop using experimental formats in tests.
198
            format='dirstate-with-subtree')
2100.3.29 by Aaron Bentley
Get merge working initially
199
        sub_tree.set_root_id('sub-tree-root')
200
        self.build_tree_contents([('tree/sub-tree/file', 'text1')])
201
        sub_tree.add('file')
202
        sub_tree.commit('foo')
203
        tree.add_reference(sub_tree)
204
        tree.commit('set text to 1')
205
        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()
2255.2.217 by Martin Pool
docs
206
        # modify the file in the subtree
2100.3.29 by Aaron Bentley
Get merge working initially
207
        self.build_tree_contents([('tree2/sub-tree/file', 'text2')])
2255.2.217 by Martin Pool
docs
208
        # and merge the changes from the diverged subtree into the containing
209
        # tree
2255.2.216 by Robert Collins
simplify merge_nested tests.
210
        tree2.commit('changed file text')
2100.3.29 by Aaron Bentley
Get merge working initially
211
        tree.merge_from_branch(tree2.branch)
212
        self.assertFileEqual('text2', 'tree/sub-tree/file')
213
1551.10.2 by Aaron Bentley
Handle merge with dangling inventory entries
214
    def test_merge_with_missing(self):
215
        tree_a = self.make_branch_and_tree('tree_a')
216
        self.build_tree_contents([('tree_a/file', 'content_1')])
217
        tree_a.add('file')
218
        tree_a.commit('commit base')
2255.7.12 by John Arbash Meinel
Some comments on merge code, fix merge tests that
219
        # basis_tree() is only guaranteed to be valid as long as it is actually
220
        # the basis tree. This mutates the tree after grabbing basis, so go to
221
        # the repository.
222
        base_tree = tree_a.branch.repository.revision_tree(tree_a.last_revision())
1551.10.2 by Aaron Bentley
Handle merge with dangling inventory entries
223
        tree_b = tree_a.bzrdir.sprout('tree_b').open_workingtree()
224
        self.build_tree_contents([('tree_a/file', 'content_2')])
225
        tree_a.commit('commit other')
226
        other_tree = tree_a.basis_tree()
227
        os.unlink('tree_b/file')
228
        merge_inner(tree_b.branch, other_tree, base_tree, this_tree=tree_b)
1959.4.6 by Aaron Bentley
Ensure merge works across kind changes
229
230
    def test_merge_kind_change(self):
231
        tree_a = self.make_branch_and_tree('tree_a')
232
        self.build_tree_contents([('tree_a/file', 'content_1')])
233
        tree_a.add('file', 'file-id')
234
        tree_a.commit('added file')
235
        tree_b = tree_a.bzrdir.sprout('tree_b').open_workingtree()
236
        os.unlink('tree_a/file')
237
        self.build_tree(['tree_a/file/'])
238
        tree_a.commit('changed file to directory')
239
        tree_b.merge_from_branch(tree_a.branch)
240
        self.assertEqual('directory', file_kind('tree_b/file'))
2748.3.2 by Aaron Bentley
Fix revert, remove-tree, and various tests to use None for 'no files specified'
241
        tree_b.revert()
1959.4.6 by Aaron Bentley
Ensure merge works across kind changes
242
        self.assertEqual('file', file_kind('tree_b/file'))
243
        self.build_tree_contents([('tree_b/file', 'content_2')])
244
        tree_b.commit('content change')
245
        tree_b.merge_from_branch(tree_a.branch)
246
        self.assertEqual(tree_b.conflicts(),
247
                         [conflicts.ContentsConflict('file',
248
                          file_id='file-id')])
2221.4.16 by Aaron Bentley
Add tests for get_merge_type_registry
249
    
250
    def test_merge_type_registry(self):
251
        merge_type_option = option.Option.OPTIONS['merge-type']
252
        self.assertFalse('merge4' in [x[0] for x in 
253
                        merge_type_option.iter_switches()])
254
        registry = _mod_merge.get_merge_type_registry()
255
        registry.register_lazy('merge4', 'bzrlib.merge', 'Merge4Merger',
256
                               'time-travelling merge')
257
        self.assertTrue('merge4' in [x[0] for x in 
258
                        merge_type_option.iter_switches()])
259
        registry.remove('merge4')
260
        self.assertFalse('merge4' in [x[0] for x in 
261
                        merge_type_option.iter_switches()])
1551.16.2 by Aaron Bentley
Don't crash on merging renamed deleted files (#110279)
262
1551.16.3 by Aaron Bentley
Rename test case
263
    def test_merge_other_moves_we_deleted(self):
1551.16.2 by Aaron Bentley
Don't crash on merging renamed deleted files (#110279)
264
        tree_a = self.make_branch_and_tree('A')
265
        tree_a.lock_write()
266
        self.addCleanup(tree_a.unlock)
267
        self.build_tree(['A/a'])
268
        tree_a.add('a')
269
        tree_a.commit('1', rev_id='rev-1')
270
        tree_a.flush()
271
        tree_a.rename_one('a', 'b')
272
        tree_a.commit('2')
273
        bzrdir_b = tree_a.bzrdir.sprout('B', revision_id='rev-1')
274
        tree_b = bzrdir_b.open_workingtree()
275
        tree_b.lock_write()
276
        self.addCleanup(tree_b.unlock)
277
        os.unlink('B/a')
278
        tree_b.commit('3')
279
        try:
280
            tree_b.merge_from_branch(tree_a.branch)
281
        except AttributeError:
282
            self.fail('tried to join a path when name was None')
2644.1.1 by Wouter van Heyst
Fix bug #127115 by checking for self.other_rev_id being None in Merger.set_pending()
283
2644.1.2 by Wouter van Heyst
As Aaron explained #127115 is more general, failing whenever other's basis is an ancestor of this' basis.
284
    def test_merge_uncommitted_otherbasis_ancestor_of_thisbasis(self):
2644.1.1 by Wouter van Heyst
Fix bug #127115 by checking for self.other_rev_id being None in Merger.set_pending()
285
        tree_a = self.make_branch_and_tree('a')
2644.1.2 by Wouter van Heyst
As Aaron explained #127115 is more general, failing whenever other's basis is an ancestor of this' basis.
286
        self.build_tree(['a/file_1', 'a/file_2'])
2644.1.1 by Wouter van Heyst
Fix bug #127115 by checking for self.other_rev_id being None in Merger.set_pending()
287
        tree_a.add(['file_1'])
288
        tree_a.commit('commit 1')
2644.1.2 by Wouter van Heyst
As Aaron explained #127115 is more general, failing whenever other's basis is an ancestor of this' basis.
289
        tree_a.add(['file_2'])
290
        tree_a.commit('commit 2')
2644.1.1 by Wouter van Heyst
Fix bug #127115 by checking for self.other_rev_id being None in Merger.set_pending()
291
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
2644.1.2 by Wouter van Heyst
As Aaron explained #127115 is more general, failing whenever other's basis is an ancestor of this' basis.
292
        tree_b.rename_one('file_1', 'renamed')
1551.15.72 by Aaron Bentley
remove builtins._merge_helper
293
        merger = _mod_merge.Merger.from_uncommitted(tree_a, tree_b,
294
                                                    progress.DummyProgress())
295
        merger.merge_type = _mod_merge.Merge3Merger
296
        merger.do_merge()
2644.1.2 by Wouter van Heyst
As Aaron explained #127115 is more general, failing whenever other's basis is an ancestor of this' basis.
297
        self.assertEqual(tree_a.get_parent_ids(), [tree_b.last_revision()])
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
298
3146.5.1 by Aaron Bentley
Make merge --uncommitted work with merge-type weave
299
    def test_merge_uncommitted_otherbasis_ancestor_of_thisbasis_weave(self):
300
        tree_a = self.make_branch_and_tree('a')
301
        self.build_tree(['a/file_1', 'a/file_2'])
302
        tree_a.add(['file_1'])
303
        tree_a.commit('commit 1')
304
        tree_a.add(['file_2'])
305
        tree_a.commit('commit 2')
306
        tree_b = tree_a.bzrdir.sprout('b').open_workingtree()
307
        tree_b.rename_one('file_1', 'renamed')
308
        merger = _mod_merge.Merger.from_uncommitted(tree_a, tree_b,
309
                                                    progress.DummyProgress())
310
        merger.merge_type = _mod_merge.WeaveMerger
311
        merger.do_merge()
312
        self.assertEqual(tree_a.get_parent_ids(), [tree_b.last_revision()])
313
3062.2.7 by Aaron Bentley
Prevent reverse cherry-picking with weave
314
    def prepare_cherrypick(self):
3062.2.13 by Aaron Bentley
Update prepare_cherrypick docstring
315
        """Prepare a pair of trees for cherrypicking tests.
316
317
        Both trees have a file, 'file'.
318
        rev1 sets content to 'a'.
319
        rev2b adds 'b'.
320
        rev3b adds 'c'.
321
        A full merge of rev2b and rev3b into this_tree would add both 'b' and
322
        'c'.  A successful cherrypick of rev2b-rev3b into this_tree will add
323
        'c', but not 'b'.
324
        """
3062.2.6 by Aaron Bentley
Get cherrypick-on-weave working
325
        this_tree = self.make_branch_and_tree('this')
326
        self.build_tree_contents([('this/file', "a\n")])
327
        this_tree.add('file')
328
        this_tree.commit('rev1')
329
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
330
        self.build_tree_contents([('other/file', "a\nb\n")])
331
        other_tree.commit('rev2b', rev_id='rev2b')
332
        self.build_tree_contents([('other/file', "c\na\nb\n")])
333
        other_tree.commit('rev3b', rev_id='rev3b')
334
        this_tree.lock_write()
335
        self.addCleanup(this_tree.unlock)
3062.2.7 by Aaron Bentley
Prevent reverse cherry-picking with weave
336
        return this_tree, other_tree
337
338
    def test_weave_cherrypick(self):
339
        this_tree, other_tree = self.prepare_cherrypick()
3062.2.6 by Aaron Bentley
Get cherrypick-on-weave working
340
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
341
            this_tree, 'rev3b', 'rev2b', other_tree.branch)
342
        merger.merge_type = _mod_merge.WeaveMerger
343
        merger.do_merge()
344
        self.assertFileEqual('c\na\n', 'this/file')
345
3062.2.7 by Aaron Bentley
Prevent reverse cherry-picking with weave
346
    def test_weave_cannot_reverse_cherrypick(self):
347
        this_tree, other_tree = self.prepare_cherrypick()
348
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
349
            this_tree, 'rev2b', 'rev3b', other_tree.branch)
350
        merger.merge_type = _mod_merge.WeaveMerger
351
        self.assertRaises(errors.CannotReverseCherrypick, merger.do_merge)
352
353
    def test_merge3_can_reverse_cherrypick(self):
354
        this_tree, other_tree = self.prepare_cherrypick()
355
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
356
            this_tree, 'rev2b', 'rev3b', other_tree.branch)
357
        merger.merge_type = _mod_merge.Merge3Merger
358
        merger.do_merge()
359
3249.3.1 by John Arbash Meinel
Implement cherrypick support for Merge3
360
    def test_merge3_will_detect_cherrypick(self):
361
        this_tree = self.make_branch_and_tree('this')
362
        self.build_tree_contents([('this/file', "a\n")])
363
        this_tree.add('file')
364
        this_tree.commit('rev1')
365
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
366
        self.build_tree_contents([('other/file', "a\nb\n")])
367
        other_tree.commit('rev2b', rev_id='rev2b')
368
        self.build_tree_contents([('other/file', "a\nb\nc\n")])
369
        other_tree.commit('rev3b', rev_id='rev3b')
370
        this_tree.lock_write()
371
        self.addCleanup(this_tree.unlock)
372
373
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
374
            this_tree, 'rev3b', 'rev2b', other_tree.branch)
375
        merger.merge_type = _mod_merge.Merge3Merger
376
        merger.do_merge()
377
        self.assertFileEqual('a\n'
378
                             '<<<<<<< TREE\n'
379
                             '=======\n'
380
                             'c\n'
381
                             '>>>>>>> MERGE-SOURCE\n',
382
                             'this/file')
383
3008.1.21 by Aaron Bentley
Make compute_transform private, test make_preview_transform
384
    def test_make_merger(self):
385
        this_tree = self.make_branch_and_tree('this')
386
        this_tree.commit('rev1', rev_id='rev1')
387
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
388
        this_tree.commit('rev2', rev_id='rev2a')
389
        other_tree.commit('rev2', rev_id='rev2b')
390
        this_tree.lock_write()
391
        self.addCleanup(this_tree.unlock)
392
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress,
393
            this_tree, 'rev2b', other_branch=other_tree.branch)
394
        merger.merge_type = _mod_merge.Merge3Merger
395
        tree_merger = merger.make_merger()
396
        self.assertIs(_mod_merge.Merge3Merger, tree_merger.__class__)
397
        self.assertEqual('rev2b', tree_merger.other_tree.get_revision_id())
398
        self.assertEqual('rev1', tree_merger.base_tree.get_revision_id())
399
400
    def test_make_preview_transform(self):
401
        this_tree = self.make_branch_and_tree('this')
402
        self.build_tree_contents([('this/file', '1\n')])
403
        this_tree.add('file', 'file-id')
404
        this_tree.commit('rev1', rev_id='rev1')
405
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
406
        self.build_tree_contents([('this/file', '1\n2a\n')])
407
        this_tree.commit('rev2', rev_id='rev2a')
408
        self.build_tree_contents([('other/file', '2b\n1\n')])
409
        other_tree.commit('rev2', rev_id='rev2b')
410
        this_tree.lock_write()
411
        self.addCleanup(this_tree.unlock)
412
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
413
            this_tree, 'rev2b', other_branch=other_tree.branch)
414
        merger.merge_type = _mod_merge.Merge3Merger
415
        tree_merger = merger.make_merger()
416
        tt = tree_merger.make_preview_transform()
3199.1.5 by Vincent Ladeuil
Fix two more leaking tmp dirs, by reworking TransformPreview lock handling.
417
        self.addCleanup(tt.finalize)
3008.1.21 by Aaron Bentley
Make compute_transform private, test make_preview_transform
418
        preview_tree = tt.get_preview_tree()
419
        tree_file = this_tree.get_file('file-id')
420
        try:
421
            self.assertEqual('1\n2a\n', tree_file.read())
422
        finally:
423
            tree_file.close()
424
        preview_file = preview_tree.get_file('file-id')
425
        try:
426
            self.assertEqual('2b\n1\n2a\n', preview_file.read())
427
        finally:
428
            preview_file.close()
429
3008.1.22 by Aaron Bentley
Get do_merge under test
430
    def test_do_merge(self):
431
        this_tree = self.make_branch_and_tree('this')
432
        self.build_tree_contents([('this/file', '1\n')])
433
        this_tree.add('file', 'file-id')
434
        this_tree.commit('rev1', rev_id='rev1')
435
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
436
        self.build_tree_contents([('this/file', '1\n2a\n')])
437
        this_tree.commit('rev2', rev_id='rev2a')
438
        self.build_tree_contents([('other/file', '2b\n1\n')])
439
        other_tree.commit('rev2', rev_id='rev2b')
440
        this_tree.lock_write()
441
        self.addCleanup(this_tree.unlock)
442
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
443
            this_tree, 'rev2b', other_branch=other_tree.branch)
444
        merger.merge_type = _mod_merge.Merge3Merger
445
        tree_merger = merger.make_merger()
446
        tt = tree_merger.do_merge()
447
        tree_file = this_tree.get_file('file-id')
448
        try:
449
            self.assertEqual('2b\n1\n2a\n', tree_file.read())
450
        finally:
451
            tree_file.close()
452
1551.19.32 by Aaron Bentley
Don't traceback when adding files to a deleted root (abentley, #210092)
453
    def test_merge_add_into_deleted_root(self):
454
        # Yes, people actually do this.  And report bugs if it breaks.
455
        source = self.make_branch_and_tree('source', format='rich-root-pack')
456
        self.build_tree(['source/foo/'])
457
        source.add('foo', 'foo-id')
458
        source.commit('Add foo')
459
        target = source.bzrdir.sprout('target').open_workingtree()
460
        subtree = target.extract('foo-id')
461
        subtree.commit('Delete root')
462
        self.build_tree(['source/bar'])
463
        source.add('bar', 'bar-id')
464
        source.commit('Add bar')
465
        subtree.merge_from_branch(source.branch)
466
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
467
468
class TestPlanMerge(TestCaseWithMemoryTransport):
469
470
    def setUp(self):
471
        TestCaseWithMemoryTransport.setUp(self)
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
472
        mapper = versionedfile.PrefixMapper()
473
        factory = knit.make_file_factory(True, mapper)
474
        self.vf = factory(self.get_transport())
475
        self.plan_merge_vf = versionedfile._PlanMergeVersionedFile('root')
476
        self.plan_merge_vf.fallback_versionedfiles.append(self.vf)
477
478
    def add_version(self, key, parents, text):
479
        self.vf.add_lines(key, parents, [c+'\n' for c in text])
480
3514.2.10 by John Arbash Meinel
Handle more edge cases.
481
    def add_rev(self, prefix, revision_id, parents, text):
482
        self.add_version((prefix, revision_id), [(prefix, p) for p in parents],
483
                         text)
484
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
485
    def add_uncommitted_version(self, key, parents, text):
486
        self.plan_merge_vf.add_lines(key, parents,
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
487
                                     [c+'\n' for c in text])
488
489
    def setup_plan_merge(self):
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
490
        self.add_rev('root', 'A', [], 'abc')
491
        self.add_rev('root', 'B', ['A'], 'acehg')
492
        self.add_rev('root', 'C', ['A'], 'fabg')
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
493
        return _PlanMerge('B', 'C', self.plan_merge_vf, ('root',))
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
494
495
    def setup_plan_merge_uncommitted(self):
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
496
        self.add_version(('root', 'A'), [], 'abc')
497
        self.add_uncommitted_version(('root', 'B:'), [('root', 'A')], 'acehg')
498
        self.add_uncommitted_version(('root', 'C:'), [('root', 'A')], 'fabg')
499
        return _PlanMerge('B:', 'C:', self.plan_merge_vf, ('root',))
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
500
501
    def test_unique_lines(self):
502
        plan = self.setup_plan_merge()
503
        self.assertEqual(plan._unique_lines(
504
            plan._get_matching_blocks('B', 'C')),
505
            ([1, 2, 3], [0, 2]))
506
3514.2.6 by John Arbash Meinel
Write a (failing) test for complex ancestry.
507
    def test_plan_merge(self):
508
        self.setup_plan_merge()
509
        plan = self.plan_merge_vf.plan_merge('B', 'C')
510
        self.assertEqual([
511
                          ('new-b', 'f\n'),
512
                          ('unchanged', 'a\n'),
513
                          ('killed-a', 'b\n'),
514
                          ('killed-b', 'c\n'),
515
                          ('new-a', 'e\n'),
516
                          ('new-a', 'h\n'),
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
517
                          ('new-a', 'g\n'),
518
                          ('new-b', 'g\n')],
3514.2.6 by John Arbash Meinel
Write a (failing) test for complex ancestry.
519
                         list(plan))
520
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
521
    def test_plan_merge_cherrypick(self):
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
522
        self.add_rev('root', 'A', [], 'abc')
523
        self.add_rev('root', 'B', ['A'], 'abcde')
524
        self.add_rev('root', 'C', ['A'], 'abcefg')
525
        self.add_rev('root', 'D', ['A', 'B', 'C'], 'abcdegh')
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
526
        my_plan = _PlanMerge('B', 'D', self.plan_merge_vf, ('root',))
3514.2.11 by John Arbash Meinel
Shortcut the case when one revision is in the ancestry of the other.
527
        # We shortcut when one text supersedes the other in the per-file graph.
528
        # We don't actually need to compare the texts at this point.
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
529
        self.assertEqual([
3514.2.11 by John Arbash Meinel
Shortcut the case when one revision is in the ancestry of the other.
530
                          ('new-b', 'a\n'),
531
                          ('new-b', 'b\n'),
532
                          ('new-b', 'c\n'),
533
                          ('new-b', 'd\n'),
534
                          ('new-b', 'e\n'),
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
535
                          ('new-b', 'g\n'),
536
                          ('new-b', 'h\n')],
537
                          list(my_plan.plan_merge()))
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
538
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
539
    def test_plan_merge_no_common_ancestor(self):
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
540
        self.add_rev('root', 'A', [], 'abc')
541
        self.add_rev('root', 'B', [], 'xyz')
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
542
        my_plan = _PlanMerge('A', 'B', self.plan_merge_vf, ('root',))
543
        self.assertEqual([
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
544
                          ('new-a', 'a\n'),
545
                          ('new-a', 'b\n'),
546
                          ('new-a', 'c\n'),
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
547
                          ('new-b', 'x\n'),
548
                          ('new-b', 'y\n'),
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
549
                          ('new-b', 'z\n')],
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
550
                          list(my_plan.plan_merge()))
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
551
3514.2.10 by John Arbash Meinel
Handle more edge cases.
552
    def test_plan_merge_tail_ancestors(self):
553
        # The graph looks like this:
554
        #       A       # Common to all ancestors
555
        #      / \
556
        #     B   C     # Ancestors of E, only common to one side
557
        #     |\ /|
558
        #     D E F     # D, F are unique to G, H respectively
559
        #     |/ \|     # E is the LCA for G & H, and the unique LCA for
560
        #     G   H     # I, J
561
        #     |\ /|
562
        #     | X |
563
        #     |/ \|
564
        #     I   J     # criss-cross merge of G, H
565
        #
566
        # In this situation, a simple pruning of ancestors of E will leave D &
567
        # F "dangling", which looks like they introduce lines different from
568
        # the ones in E, but in actuality C&B introduced the lines, and they
569
        # are already present in E
570
571
        # Introduce the base text
572
        self.add_rev('root', 'A', [], 'abc')
573
        # Introduces a new line B
574
        self.add_rev('root', 'B', ['A'], 'aBbc')
575
        # Introduces a new line C
576
        self.add_rev('root', 'C', ['A'], 'abCc')
577
        # Introduce new line D
578
        self.add_rev('root', 'D', ['B'], 'DaBbc')
579
        # Merges B and C by just incorporating both
580
        self.add_rev('root', 'E', ['B', 'C'], 'aBbCc')
581
        # Introduce new line F
582
        self.add_rev('root', 'F', ['C'], 'abCcF')
583
        # Merge D & E by just combining the texts
584
        self.add_rev('root', 'G', ['D', 'E'], 'DaBbCc')
585
        # Merge F & E by just combining the texts
586
        self.add_rev('root', 'H', ['F', 'E'], 'aBbCcF')
587
        # Merge G & H by just combining texts
588
        self.add_rev('root', 'I', ['G', 'H'], 'DaBbCcF')
589
        # Merge G & H but supersede an old line in B
590
        self.add_rev('root', 'J', ['H', 'G'], 'DaJbCcF')
591
        plan = self.plan_merge_vf.plan_merge('I', 'J')
592
        self.assertEqual([
593
                          ('unchanged', 'D\n'),
594
                          ('unchanged', 'a\n'),
595
                          ('killed-b', 'B\n'),
596
                          ('new-b', 'J\n'),
597
                          ('unchanged', 'b\n'),
598
                          ('unchanged', 'C\n'),
599
                          ('unchanged', 'c\n'),
600
                          ('unchanged', 'F\n')],
601
                         list(plan))
602
603
    def test_plan_merge_tail_triple_ancestors(self):
604
        # The graph looks like this:
605
        #       A       # Common to all ancestors
606
        #      / \
607
        #     B   C     # Ancestors of E, only common to one side
608
        #     |\ /|
609
        #     D E F     # D, F are unique to G, H respectively
610
        #     |/|\|     # E is the LCA for G & H, and the unique LCA for
611
        #     G Q H     # I, J
612
        #     |\ /|     # Q is just an extra node which is merged into both
613
        #     | X |     # I and J
614
        #     |/ \|
615
        #     I   J     # criss-cross merge of G, H
616
        #
617
        # This is the same as the test_plan_merge_tail_ancestors, except we add
618
        # a third LCA that doesn't add new lines, but will trigger our more
619
        # involved ancestry logic
620
621
        self.add_rev('root', 'A', [], 'abc')
622
        self.add_rev('root', 'B', ['A'], 'aBbc')
623
        self.add_rev('root', 'C', ['A'], 'abCc')
624
        self.add_rev('root', 'D', ['B'], 'DaBbc')
625
        self.add_rev('root', 'E', ['B', 'C'], 'aBbCc')
626
        self.add_rev('root', 'F', ['C'], 'abCcF')
627
        self.add_rev('root', 'G', ['D', 'E'], 'DaBbCc')
628
        self.add_rev('root', 'H', ['F', 'E'], 'aBbCcF')
629
        self.add_rev('root', 'Q', ['E'], 'aBbCc')
630
        self.add_rev('root', 'I', ['G', 'Q', 'H'], 'DaBbCcF')
631
        # Merge G & H but supersede an old line in B
632
        self.add_rev('root', 'J', ['H', 'Q', 'G'], 'DaJbCcF')
633
        plan = self.plan_merge_vf.plan_merge('I', 'J')
634
        self.assertEqual([
635
                          ('unchanged', 'D\n'),
636
                          ('unchanged', 'a\n'),
637
                          ('killed-b', 'B\n'),
638
                          ('new-b', 'J\n'),
639
                          ('unchanged', 'b\n'),
640
                          ('unchanged', 'C\n'),
641
                          ('unchanged', 'c\n'),
642
                          ('unchanged', 'F\n')],
643
                         list(plan))
644
3514.2.14 by John Arbash Meinel
Bring in the code to collapse linear portions of the graph.
645
    def test_plan_merge_2_tail_triple_ancestors(self):
646
        # The graph looks like this:
647
        #     A   B     # 2 tails going back to NULL
648
        #     |\ /|
649
        #     D E F     # D, is unique to G, F to H
650
        #     |/|\|     # E is the LCA for G & H, and the unique LCA for
651
        #     G Q H     # I, J
652
        #     |\ /|     # Q is just an extra node which is merged into both
653
        #     | X |     # I and J
654
        #     |/ \|
655
        #     I   J     # criss-cross merge of G, H (and Q)
656
        #
657
658
        # This is meant to test after hitting a 3-way LCA, and multiple tail
659
        # ancestors (only have NULL_REVISION in common)
660
661
        self.add_rev('root', 'A', [], 'abc')
662
        self.add_rev('root', 'B', [], 'def')
663
        self.add_rev('root', 'D', ['A'], 'Dabc')
664
        self.add_rev('root', 'E', ['A', 'B'], 'abcdef')
665
        self.add_rev('root', 'F', ['B'], 'defF')
666
        self.add_rev('root', 'G', ['D', 'E'], 'Dabcdef')
667
        self.add_rev('root', 'H', ['F', 'E'], 'abcdefF')
668
        self.add_rev('root', 'Q', ['E'], 'abcdef')
669
        self.add_rev('root', 'I', ['G', 'Q', 'H'], 'DabcdefF')
670
        # Merge G & H but supersede an old line in B
671
        self.add_rev('root', 'J', ['H', 'Q', 'G'], 'DabcdJfF')
672
        plan = self.plan_merge_vf.plan_merge('I', 'J')
673
        self.assertEqual([
674
                          ('unchanged', 'D\n'),
675
                          ('unchanged', 'a\n'),
676
                          ('unchanged', 'b\n'),
677
                          ('unchanged', 'c\n'),
678
                          ('unchanged', 'd\n'),
679
                          ('killed-b', 'e\n'),
680
                          ('new-b', 'J\n'),
681
                          ('unchanged', 'f\n'),
682
                          ('unchanged', 'F\n')],
683
                         list(plan))
684
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
685
    def test_plan_merge_uncommitted_files(self):
3062.1.13 by Aaron Bentley
Make _PlanMerge an implementation detail of _PlanMergeVersionedFile
686
        self.setup_plan_merge_uncommitted()
687
        plan = self.plan_merge_vf.plan_merge('B:', 'C:')
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
688
        self.assertEqual([
689
                          ('new-b', 'f\n'),
690
                          ('unchanged', 'a\n'),
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
691
                          ('killed-a', 'b\n'),
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
692
                          ('killed-b', 'c\n'),
693
                          ('new-a', 'e\n'),
694
                          ('new-a', 'h\n'),
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
695
                          ('new-a', 'g\n'),
696
                          ('new-b', 'g\n')],
697
                         list(plan))
698
699
    def test_plan_merge_insert_order(self):
700
        """Weave merges are sensitive to the order of insertion.
701
        
702
        Specifically for overlapping regions, it effects which region gets put
703
        'first'. And when a user resolves an overlapping merge, if they use the
704
        same ordering, then the lines match the parents, if they don't only
705
        *some* of the lines match.
706
        """
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
707
        self.add_rev('root', 'A', [], 'abcdef')
708
        self.add_rev('root', 'B', ['A'], 'abwxcdef')
709
        self.add_rev('root', 'C', ['A'], 'abyzcdef')
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
710
        # Merge, and resolve the conflict by adding *both* sets of lines
711
        # If we get the ordering wrong, these will look like new lines in D,
712
        # rather than carried over from B, C
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
713
        self.add_rev('root', 'D', ['B', 'C'],
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
714
                         'abwxyzcdef')
715
        # Supersede the lines in B and delete the lines in C, which will
716
        # conflict if they are treated as being in D
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
717
        self.add_rev('root', 'E', ['C', 'B'],
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
718
                         'abnocdef')
719
        # Same thing for the lines in C
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
720
        self.add_rev('root', 'F', ['C'], 'abpqcdef')
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
721
        plan = self.plan_merge_vf.plan_merge('D', 'E')
722
        self.assertEqual([
723
                          ('unchanged', 'a\n'),
724
                          ('unchanged', 'b\n'),
725
                          ('killed-b', 'w\n'),
726
                          ('killed-b', 'x\n'),
727
                          ('killed-b', 'y\n'),
728
                          ('killed-b', 'z\n'),
729
                          ('new-b', 'n\n'),
730
                          ('new-b', 'o\n'),
731
                          ('unchanged', 'c\n'),
732
                          ('unchanged', 'd\n'),
733
                          ('unchanged', 'e\n'),
734
                          ('unchanged', 'f\n')],
735
                         list(plan))
736
        plan = self.plan_merge_vf.plan_merge('E', 'D')
737
        # Going in the opposite direction shows the effect of the opposite plan
738
        self.assertEqual([
739
                          ('unchanged', 'a\n'),
740
                          ('unchanged', 'b\n'),
741
                          ('new-b', 'w\n'),
742
                          ('new-b', 'x\n'),
743
                          ('killed-a', 'y\n'),
744
                          ('killed-a', 'z\n'),
745
                          ('killed-both', 'w\n'),
746
                          ('killed-both', 'x\n'),
747
                          ('new-a', 'n\n'),
748
                          ('new-a', 'o\n'),
749
                          ('unchanged', 'c\n'),
750
                          ('unchanged', 'd\n'),
751
                          ('unchanged', 'e\n'),
752
                          ('unchanged', 'f\n')],
3062.1.9 by Aaron Bentley
Move PlanMerge into merge and _PlanMergeVersionedFile into versionedfile
753
                         list(plan))
3062.2.1 by Aaron Bentley
Add support for plan-merge with a base
754
3514.2.6 by John Arbash Meinel
Write a (failing) test for complex ancestry.
755
    def test_plan_merge_criss_cross(self):
756
        # This is specificly trying to trigger problems when using limited
757
        # ancestry and weaves. The ancestry graph looks like:
3514.2.8 by John Arbash Meinel
The insertion ordering into the weave has an impact on conflicts.
758
        #       XX      unused ancestor, should not show up in the weave
759
        #       |
3514.2.6 by John Arbash Meinel
Write a (failing) test for complex ancestry.
760
        #       A       Unique LCA
761
        #       |\
762
        #       B \     Introduces a line 'foo'
763
        #      / \ \
764
        #     C   D E   C & D both have 'foo', E has different changes
765
        #     |\ /| |
766
        #     | X | |
767
        #     |/ \|/
768
        #     F   G      All of C, D, E are merged into F and G, so they are
769
        #                all common ancestors.
770
        #
771
        # The specific issue with weaves:
772
        #   B introduced a text ('foo') that is present in both C and D.
773
        #   If we do not include B (because it isn't an ancestor of E), then
774
        #   the A=>C and A=>D look like both sides independently introduce the
775
        #   text ('foo'). If F does not modify the text, it would still appear
776
        #   to have deleted on of the versions from C or D. If G then modifies
777
        #   'foo', it should appear as superseding the value in F (since it
778
        #   came from B), rather than conflict because of the resolution during
779
        #   C & D.
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
780
        self.add_rev('root', 'XX', [], 'qrs')
781
        self.add_rev('root', 'A', ['XX'], 'abcdef')
782
        self.add_rev('root', 'B', ['A'], 'axcdef')
783
        self.add_rev('root', 'C', ['B'], 'axcdefg')
784
        self.add_rev('root', 'D', ['B'], 'haxcdef')
785
        self.add_rev('root', 'E', ['A'], 'abcdyf')
786
        # Simple combining of all texts
787
        self.add_rev('root', 'F', ['C', 'D', 'E'], 'haxcdyfg')
788
        # combine and supersede 'x'
789
        self.add_rev('root', 'G', ['C', 'D', 'E'], 'hazcdyfg')
3514.2.6 by John Arbash Meinel
Write a (failing) test for complex ancestry.
790
        plan = self.plan_merge_vf.plan_merge('F', 'G')
791
        self.assertEqual([
792
                          ('unchanged', 'h\n'),
793
                          ('unchanged', 'a\n'),
3514.2.7 by John Arbash Meinel
Fix the failing test by implementing the fallback logic.
794
                          ('killed-base', 'b\n'),
3514.2.6 by John Arbash Meinel
Write a (failing) test for complex ancestry.
795
                          ('killed-b', 'x\n'),
796
                          ('new-b', 'z\n'),
797
                          ('unchanged', 'c\n'),
798
                          ('unchanged', 'd\n'),
799
                          ('killed-base', 'e\n'),
800
                          ('unchanged', 'y\n'),
801
                          ('unchanged', 'f\n'),
802
                          ('unchanged', 'g\n')],
803
                         list(plan))
804
3514.2.12 by John Arbash Meinel
Start refactoring into helper functions
805
    def assertRemoveExternalReferences(self, filtered_parent_map,
806
                                       child_map, tails, parent_map):
807
        """Assert results for _PlanMerge._remove_external_references."""
808
        (act_filtered_parent_map, act_child_map,
809
         act_tails) = _PlanMerge._remove_external_references(parent_map)
810
811
        # The parent map *should* preserve ordering, but the ordering of
812
        # children is not strictly defined
813
        # child_map = dict((k, sorted(children))
814
        #                  for k, children in child_map.iteritems())
815
        # act_child_map = dict(k, sorted(children)
816
        #                      for k, children in act_child_map.iteritems())
817
        self.assertEqual(filtered_parent_map, act_filtered_parent_map)
818
        self.assertEqual(child_map, act_child_map)
819
        self.assertEqual(sorted(tails), sorted(act_tails))
820
821
    def test__remove_external_references(self):
822
        # First, nothing to remove
823
        self.assertRemoveExternalReferences({3: [2], 2: [1], 1: []},
824
            {1: [2], 2: [3], 3: []}, [1], {3: [2], 2: [1], 1: []})
825
        # The reverse direction
826
        self.assertRemoveExternalReferences({1: [2], 2: [3], 3: []},
827
            {3: [2], 2: [1], 1: []}, [3], {1: [2], 2: [3], 3: []})
828
        # Extra references
829
        self.assertRemoveExternalReferences({3: [2], 2: [1], 1: []},
830
            {1: [2], 2: [3], 3: []}, [1], {3: [2, 4], 2: [1, 5], 1: [6]})
831
        # Multiple tails
832
        self.assertRemoveExternalReferences(
833
            {4: [2, 3], 3: [], 2: [1], 1: []},
834
            {1: [2], 2: [4], 3: [4], 4: []},
835
            [1, 3],
836
            {4: [2, 3], 3: [5], 2: [1], 1: [6]})
837
        # Multiple children
838
        self.assertRemoveExternalReferences(
839
            {1: [3], 2: [3, 4], 3: [], 4: []},
840
            {1: [], 2: [], 3: [1, 2], 4: [2]},
841
            [3, 4],
842
            {1: [3], 2: [3, 4], 3: [5], 4: []})
843
3514.2.13 by John Arbash Meinel
Add the ability to prune extra tails from the parent_map.
844
    def assertPruneTails(self, pruned_map, tails, parent_map):
845
        child_map = {}
846
        for key, parent_keys in parent_map.iteritems():
847
            child_map.setdefault(key, [])
848
            for pkey in parent_keys:
849
                child_map.setdefault(pkey, []).append(key)
850
        _PlanMerge._prune_tails(parent_map, child_map, tails)
851
        self.assertEqual(pruned_map, parent_map)
852
853
    def test__prune_tails(self):
854
        # Nothing requested to prune
855
        self.assertPruneTails({1: [], 2: [], 3: []}, [],
856
                              {1: [], 2: [], 3: []})
857
        # Prune a single entry
858
        self.assertPruneTails({1: [], 3: []}, [2],
859
                              {1: [], 2: [], 3: []})
860
        # Prune a chain
861
        self.assertPruneTails({1: []}, [3],
862
                              {1: [], 2: [3], 3: []})
863
        # Prune a chain with a diamond
864
        self.assertPruneTails({1: []}, [5],
865
                              {1: [], 2: [3, 4], 3: [5], 4: [5], 5: []})
866
        # Prune a partial chain
867
        self.assertPruneTails({1: [6], 6:[]}, [5],
868
                              {1: [2, 6], 2: [3, 4], 3: [5], 4: [5], 5: [],
869
                               6: []})
870
        # Prune a chain with multiple tips, that pulls out intermediates
871
        self.assertPruneTails({1:[3], 3:[]}, [4, 5],
872
                              {1: [2, 3], 2: [4, 5], 3: [], 4:[], 5:[]})
873
        self.assertPruneTails({1:[3], 3:[]}, [5, 4],
874
                              {1: [2, 3], 2: [4, 5], 3: [], 4:[], 5:[]})
875
3062.2.1 by Aaron Bentley
Add support for plan-merge with a base
876
    def test_subtract_plans(self):
877
        old_plan = [
878
        ('unchanged', 'a\n'),
879
        ('new-a', 'b\n'),
880
        ('killed-a', 'c\n'),
881
        ('new-b', 'd\n'),
882
        ('new-b', 'e\n'),
883
        ('killed-b', 'f\n'),
884
        ('killed-b', 'g\n'),
885
        ]
886
        new_plan = [
887
        ('unchanged', 'a\n'),
888
        ('new-a', 'b\n'),
889
        ('killed-a', 'c\n'),
890
        ('new-b', 'd\n'),
891
        ('new-b', 'h\n'),
892
        ('killed-b', 'f\n'),
893
        ('killed-b', 'i\n'),
894
        ]
895
        subtracted_plan = [
896
        ('unchanged', 'a\n'),
897
        ('new-a', 'b\n'),
898
        ('killed-a', 'c\n'),
899
        ('new-b', 'h\n'),
900
        ('unchanged', 'f\n'),
901
        ('killed-b', 'i\n'),
902
        ]
903
        self.assertEqual(subtracted_plan,
3062.2.3 by Aaron Bentley
Sync up with bzr.dev API changes
904
            list(_PlanMerge._subtract_plans(old_plan, new_plan)))
3062.2.1 by Aaron Bentley
Add support for plan-merge with a base
905
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
906
    def setup_merge_with_base(self):
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
907
        self.add_rev('root', 'COMMON', [], 'abc')
908
        self.add_rev('root', 'THIS', ['COMMON'], 'abcd')
909
        self.add_rev('root', 'BASE', ['COMMON'], 'eabc')
910
        self.add_rev('root', 'OTHER', ['BASE'], 'eafb')
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
911
912
    def test_plan_merge_with_base(self):
913
        self.setup_merge_with_base()
3062.2.3 by Aaron Bentley
Sync up with bzr.dev API changes
914
        plan = self.plan_merge_vf.plan_merge('THIS', 'OTHER', 'BASE')
3062.2.1 by Aaron Bentley
Add support for plan-merge with a base
915
        self.assertEqual([('unchanged', 'a\n'),
916
                          ('new-b', 'f\n'),
917
                          ('unchanged', 'b\n'),
918
                          ('killed-b', 'c\n'),
919
                          ('new-a', 'd\n')
920
                         ], list(plan))
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
921
922
    def test_plan_lca_merge(self):
923
        self.setup_plan_merge()
924
        plan = self.plan_merge_vf.plan_lca_merge('B', 'C')
925
        self.assertEqual([
926
                          ('new-b', 'f\n'),
927
                          ('unchanged', 'a\n'),
928
                          ('killed-b', 'c\n'),
929
                          ('new-a', 'e\n'),
930
                          ('new-a', 'h\n'),
931
                          ('killed-a', 'b\n'),
932
                          ('unchanged', 'g\n')],
933
                         list(plan))
934
935
    def test_plan_lca_merge_uncommitted_files(self):
936
        self.setup_plan_merge_uncommitted()
937
        plan = self.plan_merge_vf.plan_lca_merge('B:', 'C:')
938
        self.assertEqual([
939
                          ('new-b', 'f\n'),
940
                          ('unchanged', 'a\n'),
941
                          ('killed-b', 'c\n'),
942
                          ('new-a', 'e\n'),
943
                          ('new-a', 'h\n'),
944
                          ('killed-a', 'b\n'),
945
                          ('unchanged', 'g\n')],
946
                         list(plan))
947
948
    def test_plan_lca_merge_with_base(self):
949
        self.setup_merge_with_base()
950
        plan = self.plan_merge_vf.plan_lca_merge('THIS', 'OTHER', 'BASE')
951
        self.assertEqual([('unchanged', 'a\n'),
952
                          ('new-b', 'f\n'),
953
                          ('unchanged', 'b\n'),
954
                          ('killed-b', 'c\n'),
955
                          ('new-a', 'd\n')
956
                         ], list(plan))
957
958
    def test_plan_lca_merge_with_criss_cross(self):
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
959
        self.add_version(('root', 'ROOT'), [], 'abc')
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
960
        # each side makes a change
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
961
        self.add_version(('root', 'REV1'), [('root', 'ROOT')], 'abcd')
962
        self.add_version(('root', 'REV2'), [('root', 'ROOT')], 'abce')
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
963
        # both sides merge, discarding others' changes
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
964
        self.add_version(('root', 'LCA1'),
965
            [('root', 'REV1'), ('root', 'REV2')], 'abcd')
966
        self.add_version(('root', 'LCA2'),
967
            [('root', 'REV1'), ('root', 'REV2')], 'fabce')
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
968
        plan = self.plan_merge_vf.plan_lca_merge('LCA1', 'LCA2')
3144.3.10 by Aaron Bentley
Use correct index when emitting conflicted-b
969
        self.assertEqual([('new-b', 'f\n'),
970
                          ('unchanged', 'a\n'),
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
971
                          ('unchanged', 'b\n'),
972
                          ('unchanged', 'c\n'),
3144.3.3 by Aaron Bentley
Update test for new conflicted types
973
                          ('conflicted-a', 'd\n'),
974
                          ('conflicted-b', 'e\n'),
3144.3.1 by Aaron Bentley
Implement LCA merge, with problematic conflict markers
975
                         ], list(plan))
3144.5.3 by Aaron Bentley
Test interesting_files for LCA merge
976
3287.17.1 by John Arbash Meinel
Fix bug #235715 by using the empty list as the text for a base of NULL_REVISION.
977
    def test_plan_lca_merge_with_null(self):
3350.6.5 by Robert Collins
Update to bzr.dev.
978
        self.add_version(('root', 'A'), [], 'ab')
979
        self.add_version(('root', 'B'), [], 'bc')
3287.17.1 by John Arbash Meinel
Fix bug #235715 by using the empty list as the text for a base of NULL_REVISION.
980
        plan = self.plan_merge_vf.plan_lca_merge('A', 'B')
981
        self.assertEqual([('new-a', 'a\n'),
982
                          ('unchanged', 'b\n'),
983
                          ('new-b', 'c\n'),
984
                         ], list(plan))
985
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
986
    def test_plan_merge_with_delete_and_change(self):
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
987
        self.add_rev('root', 'C', [], 'a')
988
        self.add_rev('root', 'A', ['C'], 'b')
989
        self.add_rev('root', 'B', ['C'], '')
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
990
        plan = self.plan_merge_vf.plan_merge('A', 'B')
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
991
        self.assertEqual([('killed-both', 'a\n'),
992
                          ('new-a', 'b\n'),
993
                         ], list(plan))
994
995
    def test_plan_merge_with_move_and_change(self):
3514.2.17 by John Arbash Meinel
On Ian's suggestion, change the 'plan_merge' tests to use the clearer 'add_rev' instead of 'add_version'
996
        self.add_rev('root', 'C', [], 'abcd')
997
        self.add_rev('root', 'A', ['C'], 'acbd')
998
        self.add_rev('root', 'B', ['C'], 'aBcd')
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
999
        plan = self.plan_merge_vf.plan_merge('A', 'B')
1000
        self.assertEqual([('unchanged', 'a\n'),
1001
                          ('new-a', 'c\n'),
1002
                          ('killed-b', 'b\n'),
1003
                          ('new-b', 'B\n'),
1004
                          ('killed-a', 'c\n'),
1005
                          ('unchanged', 'd\n'),
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1006
                         ], list(plan))
1007
3144.5.3 by Aaron Bentley
Test interesting_files for LCA merge
1008
1009
class TestMergeImplementation(object):
1010
1011
    def do_merge(self, target_tree, source_tree, **kwargs):
1012
        merger = _mod_merge.Merger.from_revision_ids(progress.DummyProgress(),
1013
            target_tree, source_tree.last_revision(),
1014
            other_branch=source_tree.branch)
1015
        merger.merge_type=self.merge_type
1016
        for name, value in kwargs.items():
1017
            setattr(merger, name, value)
1018
        merger.do_merge()
1019
1020
    def test_merge_specific_file(self):
1021
        this_tree = self.make_branch_and_tree('this')
1022
        this_tree.lock_write()
1023
        self.addCleanup(this_tree.unlock)
1024
        self.build_tree_contents([
1025
            ('this/file1', 'a\nb\n'),
1026
            ('this/file2', 'a\nb\n')
1027
        ])
1028
        this_tree.add(['file1', 'file2'])
1029
        this_tree.commit('Added files')
1030
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
1031
        self.build_tree_contents([
1032
            ('other/file1', 'a\nb\nc\n'),
1033
            ('other/file2', 'a\nb\nc\n')
1034
        ])
1035
        other_tree.commit('modified both')
1036
        self.build_tree_contents([
1037
            ('this/file1', 'd\na\nb\n'),
1038
            ('this/file2', 'd\na\nb\n')
1039
        ])
1040
        this_tree.commit('modified both')
1041
        self.do_merge(this_tree, other_tree, interesting_files=['file1'])
1042
        self.assertFileEqual('d\na\nb\nc\n', 'this/file1')
1043
        self.assertFileEqual('d\na\nb\n', 'this/file2')
1044
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1045
    def test_merge_move_and_change(self):
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1046
        this_tree = self.make_branch_and_tree('this')
1047
        this_tree.lock_write()
1048
        self.addCleanup(this_tree.unlock)
1049
        self.build_tree_contents([
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1050
            ('this/file1', 'line 1\nline 2\nline 3\nline 4\n'),
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1051
        ])
1052
        this_tree.add('file1',)
1053
        this_tree.commit('Added file')
1054
        other_tree = this_tree.bzrdir.sprout('other').open_workingtree()
1055
        self.build_tree_contents([
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1056
            ('other/file1', 'line 1\nline 2 to 2.1\nline 3\nline 4\n'),
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1057
        ])
3514.2.16 by John Arbash Meinel
Review feedback from Ian.
1058
        other_tree.commit('Changed 2 to 2.1')
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1059
        self.build_tree_contents([
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1060
            ('this/file1', 'line 1\nline 3\nline 2\nline 4\n'),
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1061
        ])
3514.2.16 by John Arbash Meinel
Review feedback from Ian.
1062
        this_tree.commit('Swapped 2 & 3')
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1063
        self.do_merge(this_tree, other_tree)
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1064
        self.assertFileEqual('line 1\n'
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1065
            '<<<<<<< TREE\n'
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1066
            'line 3\n'
1067
            'line 2\n'
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1068
            '=======\n'
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1069
            'line 2 to 2.1\n'
1070
            'line 3\n'
1071
            '>>>>>>> MERGE-SOURCE\n'
1072
            'line 4\n', 'this/file1')
3514.2.1 by Aaron Bentley
Test for correct conflicts on delete + change
1073
3144.5.3 by Aaron Bentley
Test interesting_files for LCA merge
1074
1075
class TestMerge3Merge(TestCaseWithTransport, TestMergeImplementation):
1076
1077
    merge_type = _mod_merge.Merge3Merger
1078
1079
1080
class TestWeaveMerge(TestCaseWithTransport, TestMergeImplementation):
1081
1082
    merge_type = _mod_merge.WeaveMerger
1083
1084
1085
class TestLCAMerge(TestCaseWithTransport, TestMergeImplementation):
1086
1087
    merge_type = _mod_merge.LCAMerger
3514.2.2 by John Arbash Meinel
Restore a real weave merge to 'bzr merge --weave'.
1088
1089
    def test_merge_move_and_change(self):
1090
        self.expectFailure("lca merge doesn't conflict for move and change",
1091
            super(TestLCAMerge, self).test_merge_move_and_change)