/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 bzrlib/tests/test_merge.py

  • Committer: Vincent Ladeuil
  • Date: 2011-07-07 09:34:06 UTC
  • mto: (6011.2.1 trunk)
  • mto: This revision was merged to the branch mainline in revision 6012.
  • Revision ID: v.ladeuil+lp@free.fr-20110707093406-aqne3jdrydt80t5r
Add news entry and further tweaks.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_merge.py
18
18
from StringIO import StringIO
19
19
 
20
20
from bzrlib import (
 
21
    branch as _mod_branch,
 
22
    cleanup,
21
23
    conflicts,
22
24
    errors,
 
25
    inventory,
23
26
    knit,
24
27
    memorytree,
25
28
    merge as _mod_merge,
26
29
    option,
27
 
    progress,
 
30
    revision as _mod_revision,
28
31
    tests,
29
32
    transform,
30
33
    versionedfile,
32
35
from bzrlib.conflicts import ConflictList, TextConflict
33
36
from bzrlib.errors import UnrelatedBranches, NoCommits
34
37
from bzrlib.merge import transform_tree, merge_inner, _PlanMerge
35
 
from bzrlib.osutils import pathjoin, file_kind
 
38
from bzrlib.osutils import basename, pathjoin, file_kind
36
39
from bzrlib.tests import (
37
40
    TestCaseWithMemoryTransport,
38
41
    TestCaseWithTransport,
87
90
        os.chdir('branch2')
88
91
        self.run_bzr('merge ../branch1/baz', retcode=3)
89
92
        self.run_bzr('merge ../branch1/foo')
90
 
        self.failUnlessExists('foo')
91
 
        self.failIfExists('bar')
 
93
        self.assertPathExists('foo')
 
94
        self.assertPathDoesNotExist('bar')
92
95
        wt2 = WorkingTree.open('.') # opens branch2
93
96
        self.assertEqual([tip], wt2.get_parent_ids())
94
97
 
118
121
        finally:
119
122
            wt1.unlock()
120
123
 
 
124
    def test_merge_into_null_tree(self):
 
125
        wt = self.make_branch_and_tree('tree')
 
126
        null_tree = wt.basis_tree()
 
127
        self.build_tree(['tree/file'])
 
128
        wt.add('file')
 
129
        wt.commit('tree with root')
 
130
        merger = _mod_merge.Merge3Merger(null_tree, null_tree, null_tree, wt,
 
131
                                         this_branch=wt.branch,
 
132
                                         do_merge=False)
 
133
        with merger.make_preview_transform() as tt:
 
134
            self.assertEqual([], tt.find_conflicts())
 
135
            preview = tt.get_preview_tree()
 
136
            self.assertEqual(wt.get_root_id(), preview.get_root_id())
 
137
 
 
138
    def test_merge_preview_unrelated_retains_root(self):
 
139
        wt = self.make_branch_and_tree('tree')
 
140
        other_tree = self.make_branch_and_tree('other')
 
141
        other_tree.commit('add root in other')
 
142
        merger = _mod_merge.Merge3Merger(wt, wt, wt.basis_tree(), other_tree,
 
143
                                         this_branch=wt.branch,
 
144
                                         do_merge=False)
 
145
        with merger.make_preview_transform() as tt:
 
146
            preview = tt.get_preview_tree()
 
147
            self.assertEqual(wt.get_root_id(), preview.get_root_id())
 
148
 
 
149
    def test_merge_unrelated_retains_root(self):
 
150
        wt = self.make_branch_and_tree('tree')
 
151
        root_id_before_merge = wt.get_root_id()
 
152
        other_tree = self.make_branch_and_tree('other')
 
153
        # Do a commit so there is something to merge
 
154
        other_tree.commit('add root to other')
 
155
        self.assertNotEquals(root_id_before_merge, other_tree.get_root_id())
 
156
        wt.merge_from_branch(other_tree.branch,
 
157
                             from_revision=_mod_revision.NULL_REVISION)
 
158
        self.assertEqual(root_id_before_merge, wt.get_root_id())
 
159
 
121
160
    def test_create_rename(self):
122
161
        """Rename an inventory entry while creating the file"""
123
162
        tree =self.make_branch_and_tree('.')
154
193
        log = StringIO()
155
194
        merge_inner(tree_b.branch, tree_a, tree_b.basis_tree(),
156
195
                    this_tree=tree_b, ignore_zero=True)
157
 
        self.failUnless('All changes applied successfully.\n' not in
 
196
        self.assertTrue('All changes applied successfully.\n' not in
158
197
            self.get_log())
159
198
        tree_b.revert()
160
199
        merge_inner(tree_b.branch, tree_a, tree_b.basis_tree(),
161
200
                    this_tree=tree_b, ignore_zero=False)
162
 
        self.failUnless('All changes applied successfully.\n' in self.get_log())
 
201
        self.assertTrue('All changes applied successfully.\n' in self.get_log())
163
202
 
164
203
    def test_merge_inner_conflicts(self):
165
204
        tree_a = self.make_branch_and_tree('a')
384
423
                             '>>>>>>> MERGE-SOURCE\n',
385
424
                             'this/file')
386
425
 
 
426
    def test_merge_reverse_revision_range(self):
 
427
        tree = self.make_branch_and_tree(".")
 
428
        tree.lock_write()
 
429
        self.addCleanup(tree.unlock)
 
430
        self.build_tree(['a'])
 
431
        tree.add('a')
 
432
        tree.commit("added a")
 
433
        first_rev = tree.branch.revision_history()[0]
 
434
        merger = _mod_merge.Merger.from_revision_ids(None, tree,
 
435
                                          _mod_revision.NULL_REVISION,
 
436
                                          first_rev)
 
437
        merger.merge_type = _mod_merge.Merge3Merger
 
438
        merger.interesting_files = 'a'
 
439
        conflict_count = merger.do_merge()
 
440
        self.assertEqual(0, conflict_count)
 
441
 
 
442
        self.assertPathDoesNotExist("a")
 
443
        tree.revert()
 
444
        self.assertPathExists("a")
 
445
 
387
446
    def test_make_merger(self):
388
447
        this_tree = self.make_branch_and_tree('this')
389
448
        this_tree.commit('rev1', rev_id='rev1')
453
512
        finally:
454
513
            tree_file.close()
455
514
 
 
515
    def test_merge_require_tree_root(self):
 
516
        tree = self.make_branch_and_tree(".")
 
517
        tree.lock_write()
 
518
        self.addCleanup(tree.unlock)
 
519
        self.build_tree(['a'])
 
520
        tree.add('a')
 
521
        tree.commit("added a")
 
522
        old_root_id = tree.get_root_id()
 
523
        first_rev = tree.branch.revision_history()[0]
 
524
        merger = _mod_merge.Merger.from_revision_ids(None, tree,
 
525
                                          _mod_revision.NULL_REVISION,
 
526
                                          first_rev)
 
527
        merger.merge_type = _mod_merge.Merge3Merger
 
528
        conflict_count = merger.do_merge()
 
529
        self.assertEqual(0, conflict_count)
 
530
        self.assertEquals(set([old_root_id]), tree.all_file_ids())
 
531
        tree.set_parent_ids([])
 
532
 
456
533
    def test_merge_add_into_deleted_root(self):
457
534
        # Yes, people actually do this.  And report bugs if it breaks.
458
535
        source = self.make_branch_and_tree('source', format='rich-root-pack')
1267
1344
        self.assertEqual(['B-id', 'C-id', 'F-id'],
1268
1345
                         [t.get_revision_id() for t in merger._lca_trees])
1269
1346
 
 
1347
    def test_find_base_new_root_criss_cross(self):
 
1348
        # A   B
 
1349
        # |\ /|
 
1350
        # | X |
 
1351
        # |/ \|
 
1352
        # C   D
 
1353
        
 
1354
        builder = self.get_builder()
 
1355
        builder.build_snapshot('A-id', None,
 
1356
            [('add', ('', None, 'directory', None))])
 
1357
        builder.build_snapshot('B-id', [],
 
1358
            [('add', ('', None, 'directory', None))])
 
1359
        builder.build_snapshot('D-id', ['A-id', 'B-id'], [])
 
1360
        builder.build_snapshot('C-id', ['A-id', 'B-id'], [])
 
1361
        merger = self.make_Merger(builder, 'D-id')
 
1362
        self.assertEqual('A-id', merger.base_rev_id)
 
1363
        self.assertTrue(merger._is_criss_cross)
 
1364
        self.assertEqual(['A-id', 'B-id'], [t.get_revision_id()
 
1365
                                            for t in merger._lca_trees])
 
1366
 
1270
1367
    def test_no_criss_cross_passed_to_merge_type(self):
1271
1368
        class LCATreesMerger(LoggingMerger):
1272
1369
            supports_lca_trees = True
2840
2937
 
2841
2938
    def get_merger_factory(self):
2842
2939
        # Allows  the inner methods to access the test attributes
2843
 
        test = self
 
2940
        calls = self.calls
2844
2941
 
2845
2942
        class FooMerger(_mod_merge.ConfigurableFileMerger):
2846
2943
            name_prefix = "foo"
2847
2944
            default_files = ['bar']
2848
2945
 
2849
2946
            def merge_text(self, params):
2850
 
                test.calls.append('merge_text')
 
2947
                calls.append('merge_text')
2851
2948
                return ('not_applicable', None)
2852
2949
 
2853
2950
        def factory(merger):
2917
3014
        conflicts = builder.merge()
2918
3015
        # The hook should not call the merge_text() method
2919
3016
        self.assertEqual([], self.calls)
 
3017
 
 
3018
 
 
3019
class TestMergeIntoBase(tests.TestCaseWithTransport):
 
3020
 
 
3021
    def setup_simple_branch(self, relpath, shape=None, root_id=None):
 
3022
        """One commit, containing tree specified by optional shape.
 
3023
        
 
3024
        Default is empty tree (just root entry).
 
3025
        """
 
3026
        if root_id is None:
 
3027
            root_id = '%s-root-id' % (relpath,)
 
3028
        wt = self.make_branch_and_tree(relpath)
 
3029
        wt.set_root_id(root_id)
 
3030
        if shape is not None:
 
3031
            adjusted_shape = [relpath + '/' + elem for elem in shape]
 
3032
            self.build_tree(adjusted_shape)
 
3033
            ids = ['%s-%s-id' % (relpath, basename(elem.rstrip('/')))
 
3034
                   for elem in shape]
 
3035
            wt.add(shape, ids=ids)
 
3036
        rev_id = 'r1-%s' % (relpath,)
 
3037
        wt.commit("Initial commit of %s" % (relpath,), rev_id=rev_id)
 
3038
        self.assertEqual(root_id, wt.path2id(''))
 
3039
        return wt
 
3040
 
 
3041
    def setup_two_branches(self, custom_root_ids=True):
 
3042
        """Setup 2 branches, one will be a library, the other a project."""
 
3043
        if custom_root_ids:
 
3044
            root_id = None
 
3045
        else:
 
3046
            root_id = inventory.ROOT_ID
 
3047
        project_wt = self.setup_simple_branch(
 
3048
            'project', ['README', 'dir/', 'dir/file.c'],
 
3049
            root_id)
 
3050
        lib_wt = self.setup_simple_branch(
 
3051
            'lib1', ['README', 'Makefile', 'foo.c'], root_id)
 
3052
 
 
3053
        return project_wt, lib_wt
 
3054
 
 
3055
    def do_merge_into(self, location, merge_as):
 
3056
        """Helper for using MergeIntoMerger.
 
3057
        
 
3058
        :param location: location of directory to merge from, either the
 
3059
            location of a branch or of a path inside a branch.
 
3060
        :param merge_as: the path in a tree to add the new directory as.
 
3061
        :returns: the conflicts from 'do_merge'.
 
3062
        """
 
3063
        operation = cleanup.OperationWithCleanups(self._merge_into)
 
3064
        return operation.run(location, merge_as)
 
3065
 
 
3066
    def _merge_into(self, op, location, merge_as):
 
3067
        # Open and lock the various tree and branch objects
 
3068
        wt, subdir_relpath = WorkingTree.open_containing(merge_as)
 
3069
        op.add_cleanup(wt.lock_write().unlock)
 
3070
        branch_to_merge, subdir_to_merge = _mod_branch.Branch.open_containing(
 
3071
            location)
 
3072
        op.add_cleanup(branch_to_merge.lock_read().unlock)
 
3073
        other_tree = branch_to_merge.basis_tree()
 
3074
        op.add_cleanup(other_tree.lock_read().unlock)
 
3075
        # Perform the merge
 
3076
        merger = _mod_merge.MergeIntoMerger(this_tree=wt, other_tree=other_tree,
 
3077
            other_branch=branch_to_merge, target_subdir=subdir_relpath,
 
3078
            source_subpath=subdir_to_merge)
 
3079
        merger.set_base_revision(_mod_revision.NULL_REVISION, branch_to_merge)
 
3080
        conflicts = merger.do_merge()
 
3081
        merger.set_pending()
 
3082
        return conflicts
 
3083
 
 
3084
    def assertTreeEntriesEqual(self, expected_entries, tree):
 
3085
        """Assert that 'tree' contains the expected inventory entries.
 
3086
 
 
3087
        :param expected_entries: sequence of (path, file-id) pairs.
 
3088
        """
 
3089
        files = [(path, ie.file_id) for path, ie in tree.iter_entries_by_dir()]
 
3090
        self.assertEqual(expected_entries, files)
 
3091
 
 
3092
 
 
3093
class TestMergeInto(TestMergeIntoBase):
 
3094
 
 
3095
    def test_newdir_with_unique_roots(self):
 
3096
        """Merge a branch with a unique root into a new directory."""
 
3097
        project_wt, lib_wt = self.setup_two_branches()
 
3098
        self.do_merge_into('lib1', 'project/lib1')
 
3099
        project_wt.lock_read()
 
3100
        self.addCleanup(project_wt.unlock)
 
3101
        # The r1-lib1 revision should be merged into this one
 
3102
        self.assertEqual(['r1-project', 'r1-lib1'], project_wt.get_parent_ids())
 
3103
        self.assertTreeEntriesEqual(
 
3104
            [('', 'project-root-id'),
 
3105
             ('README', 'project-README-id'),
 
3106
             ('dir', 'project-dir-id'),
 
3107
             ('lib1', 'lib1-root-id'),
 
3108
             ('dir/file.c', 'project-file.c-id'),
 
3109
             ('lib1/Makefile', 'lib1-Makefile-id'),
 
3110
             ('lib1/README', 'lib1-README-id'),
 
3111
             ('lib1/foo.c', 'lib1-foo.c-id'),
 
3112
            ], project_wt)
 
3113
 
 
3114
    def test_subdir(self):
 
3115
        """Merge a branch into a subdirectory of an existing directory."""
 
3116
        project_wt, lib_wt = self.setup_two_branches()
 
3117
        self.do_merge_into('lib1', 'project/dir/lib1')
 
3118
        project_wt.lock_read()
 
3119
        self.addCleanup(project_wt.unlock)
 
3120
        # The r1-lib1 revision should be merged into this one
 
3121
        self.assertEqual(['r1-project', 'r1-lib1'], project_wt.get_parent_ids())
 
3122
        self.assertTreeEntriesEqual(
 
3123
            [('', 'project-root-id'),
 
3124
             ('README', 'project-README-id'),
 
3125
             ('dir', 'project-dir-id'),
 
3126
             ('dir/file.c', 'project-file.c-id'),
 
3127
             ('dir/lib1', 'lib1-root-id'),
 
3128
             ('dir/lib1/Makefile', 'lib1-Makefile-id'),
 
3129
             ('dir/lib1/README', 'lib1-README-id'),
 
3130
             ('dir/lib1/foo.c', 'lib1-foo.c-id'),
 
3131
            ], project_wt)
 
3132
 
 
3133
    def test_newdir_with_repeat_roots(self):
 
3134
        """If the file-id of the dir to be merged already exists a new ID will
 
3135
        be allocated to let the merge happen.
 
3136
        """
 
3137
        project_wt, lib_wt = self.setup_two_branches(custom_root_ids=False)
 
3138
        root_id = project_wt.path2id('')
 
3139
        self.do_merge_into('lib1', 'project/lib1')
 
3140
        project_wt.lock_read()
 
3141
        self.addCleanup(project_wt.unlock)
 
3142
        # The r1-lib1 revision should be merged into this one
 
3143
        self.assertEqual(['r1-project', 'r1-lib1'], project_wt.get_parent_ids())
 
3144
        new_lib1_id = project_wt.path2id('lib1')
 
3145
        self.assertNotEqual(None, new_lib1_id)
 
3146
        self.assertTreeEntriesEqual(
 
3147
            [('', root_id),
 
3148
             ('README', 'project-README-id'),
 
3149
             ('dir', 'project-dir-id'),
 
3150
             ('lib1', new_lib1_id),
 
3151
             ('dir/file.c', 'project-file.c-id'),
 
3152
             ('lib1/Makefile', 'lib1-Makefile-id'),
 
3153
             ('lib1/README', 'lib1-README-id'),
 
3154
             ('lib1/foo.c', 'lib1-foo.c-id'),
 
3155
            ], project_wt)
 
3156
 
 
3157
    def test_name_conflict(self):
 
3158
        """When the target directory name already exists a conflict is
 
3159
        generated and the original directory is renamed to foo.moved.
 
3160
        """
 
3161
        dest_wt = self.setup_simple_branch('dest', ['dir/', 'dir/file.txt'])
 
3162
        src_wt = self.setup_simple_branch('src', ['README'])
 
3163
        conflicts = self.do_merge_into('src', 'dest/dir')
 
3164
        self.assertEqual(1, conflicts)
 
3165
        dest_wt.lock_read()
 
3166
        self.addCleanup(dest_wt.unlock)
 
3167
        # The r1-lib1 revision should be merged into this one
 
3168
        self.assertEqual(['r1-dest', 'r1-src'], dest_wt.get_parent_ids())
 
3169
        self.assertTreeEntriesEqual(
 
3170
            [('', 'dest-root-id'),
 
3171
             ('dir', 'src-root-id'),
 
3172
             ('dir.moved', 'dest-dir-id'),
 
3173
             ('dir/README', 'src-README-id'),
 
3174
             ('dir.moved/file.txt', 'dest-file.txt-id'),
 
3175
            ], dest_wt)
 
3176
 
 
3177
    def test_file_id_conflict(self):
 
3178
        """A conflict is generated if the merge-into adds a file (or other
 
3179
        inventory entry) with a file-id that already exists in the target tree.
 
3180
        """
 
3181
        dest_wt = self.setup_simple_branch('dest', ['file.txt'])
 
3182
        # Make a second tree with a file-id that will clash with file.txt in
 
3183
        # dest.
 
3184
        src_wt = self.make_branch_and_tree('src')
 
3185
        self.build_tree(['src/README'])
 
3186
        src_wt.add(['README'], ids=['dest-file.txt-id'])
 
3187
        src_wt.commit("Rev 1 of src.", rev_id='r1-src')
 
3188
        conflicts = self.do_merge_into('src', 'dest/dir')
 
3189
        # This is an edge case that shouldn't happen to users very often.  So
 
3190
        # we don't care really about the exact presentation of the conflict,
 
3191
        # just that there is one.
 
3192
        self.assertEqual(1, conflicts)
 
3193
 
 
3194
    def test_only_subdir(self):
 
3195
        """When the location points to just part of a tree, merge just that
 
3196
        subtree.
 
3197
        """
 
3198
        dest_wt = self.setup_simple_branch('dest')
 
3199
        src_wt = self.setup_simple_branch(
 
3200
            'src', ['hello.txt', 'dir/', 'dir/foo.c'])
 
3201
        conflicts = self.do_merge_into('src/dir', 'dest/dir')
 
3202
        dest_wt.lock_read()
 
3203
        self.addCleanup(dest_wt.unlock)
 
3204
        # The r1-lib1 revision should NOT be merged into this one (this is a
 
3205
        # partial merge).
 
3206
        self.assertEqual(['r1-dest'], dest_wt.get_parent_ids())
 
3207
        self.assertTreeEntriesEqual(
 
3208
            [('', 'dest-root-id'),
 
3209
             ('dir', 'src-dir-id'),
 
3210
             ('dir/foo.c', 'src-foo.c-id'),
 
3211
            ], dest_wt)
 
3212
 
 
3213
    def test_only_file(self):
 
3214
        """An edge case: merge just one file, not a whole dir."""
 
3215
        dest_wt = self.setup_simple_branch('dest')
 
3216
        two_file_wt = self.setup_simple_branch(
 
3217
            'two-file', ['file1.txt', 'file2.txt'])
 
3218
        conflicts = self.do_merge_into('two-file/file1.txt', 'dest/file1.txt')
 
3219
        dest_wt.lock_read()
 
3220
        self.addCleanup(dest_wt.unlock)
 
3221
        # The r1-lib1 revision should NOT be merged into this one
 
3222
        self.assertEqual(['r1-dest'], dest_wt.get_parent_ids())
 
3223
        self.assertTreeEntriesEqual(
 
3224
            [('', 'dest-root-id'), ('file1.txt', 'two-file-file1.txt-id')],
 
3225
            dest_wt)
 
3226
 
 
3227
    def test_no_such_source_path(self):
 
3228
        """PathNotInTree is raised if the specified path in the source tree
 
3229
        does not exist.
 
3230
        """
 
3231
        dest_wt = self.setup_simple_branch('dest')
 
3232
        two_file_wt = self.setup_simple_branch('src', ['dir/'])
 
3233
        self.assertRaises(_mod_merge.PathNotInTree, self.do_merge_into,
 
3234
            'src/no-such-dir', 'dest/foo')
 
3235
        dest_wt.lock_read()
 
3236
        self.addCleanup(dest_wt.unlock)
 
3237
        # The dest tree is unmodified.
 
3238
        self.assertEqual(['r1-dest'], dest_wt.get_parent_ids())
 
3239
        self.assertTreeEntriesEqual([('', 'dest-root-id')], dest_wt)
 
3240
 
 
3241
    def test_no_such_target_path(self):
 
3242
        """PathNotInTree is also raised if the specified path in the target
 
3243
        tree does not exist.
 
3244
        """
 
3245
        dest_wt = self.setup_simple_branch('dest')
 
3246
        two_file_wt = self.setup_simple_branch('src', ['file.txt'])
 
3247
        self.assertRaises(_mod_merge.PathNotInTree, self.do_merge_into,
 
3248
            'src', 'dest/no-such-dir/foo')
 
3249
        dest_wt.lock_read()
 
3250
        self.addCleanup(dest_wt.unlock)
 
3251
        # The dest tree is unmodified.
 
3252
        self.assertEqual(['r1-dest'], dest_wt.get_parent_ids())
 
3253
        self.assertTreeEntriesEqual([('', 'dest-root-id')], dest_wt)