~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_conflicts.py

  • Committer: Andrew Bennetts
  • Date: 2010-03-11 04:33:41 UTC
  • mfrom: (4797.33.4 2.1)
  • mto: This revision was merged to the branch mainline in revision 5082.
  • Revision ID: andrew.bennetts@canonical.com-20100311043341-rzdik83fnactjsxs
Merge lp:bzr/2.1, including fixes for #496813, #526211, #526353.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
18
18
import os
19
19
 
20
20
from bzrlib import (
 
21
    branchbuilder,
21
22
    bzrdir,
22
23
    conflicts,
23
24
    errors,
24
25
    option,
25
 
    osutils,
26
26
    tests,
27
 
    )
28
 
from bzrlib.tests import (
29
 
    script,
30
 
    scenarios,
31
 
    )
32
 
 
33
 
 
34
 
load_tests = scenarios.load_tests_apply_scenarios
 
27
    workingtree,
 
28
    )
 
29
from bzrlib.tests import script
 
30
 
 
31
 
 
32
def load_tests(standard_tests, module, loader):
 
33
    result = loader.suiteClass()
 
34
 
 
35
    sp_tests, remaining_tests = tests.split_suite_by_condition(
 
36
        standard_tests, tests.condition_isinstance((
 
37
                TestResolveContentConflicts,
 
38
                )))
 
39
    tests.multiply_tests(sp_tests, content_conflict_scenarios(), result)
 
40
 
 
41
    # No parametrization for the remaining tests
 
42
    result.addTests(remaining_tests)
 
43
 
 
44
    return result
35
45
 
36
46
 
37
47
# TODO: Test commit with some added, and added-but-missing files
62
72
])
63
73
 
64
74
 
65
 
def vary_by_conflicts():
66
 
    for conflict in example_conflicts:
67
 
        yield (conflict.__class__.__name__, {"conflict": conflict})
68
 
 
69
 
 
70
75
class TestConflicts(tests.TestCaseWithTransport):
71
76
 
 
77
    def test_conflicts(self):
 
78
        """Conflicts are detected properly"""
 
79
        # Use BzrDirFormat6 so we can fake conflicts
 
80
        tree = self.make_branch_and_tree('.', format=bzrdir.BzrDirFormat6())
 
81
        self.build_tree_contents([('hello', 'hello world4'),
 
82
                                  ('hello.THIS', 'hello world2'),
 
83
                                  ('hello.BASE', 'hello world1'),
 
84
                                  ('hello.OTHER', 'hello world3'),
 
85
                                  ('hello.sploo.BASE', 'yellowworld'),
 
86
                                  ('hello.sploo.OTHER', 'yellowworld2'),
 
87
                                  ])
 
88
        tree.lock_read()
 
89
        self.assertLength(6, list(tree.list_files()))
 
90
        tree.unlock()
 
91
        tree_conflicts = tree.conflicts()
 
92
        self.assertLength(2, tree_conflicts)
 
93
        self.assertTrue('hello' in tree_conflicts[0].path)
 
94
        self.assertTrue('hello.sploo' in tree_conflicts[1].path)
 
95
        conflicts.restore('hello')
 
96
        conflicts.restore('hello.sploo')
 
97
        self.assertLength(0, tree.conflicts())
 
98
        self.assertFileEqual('hello world2', 'hello')
 
99
        self.assertFalse(os.path.lexists('hello.sploo'))
 
100
        self.assertRaises(errors.NotConflicted, conflicts.restore, 'hello')
 
101
        self.assertRaises(errors.NotConflicted,
 
102
                          conflicts.restore, 'hello.sploo')
 
103
 
72
104
    def test_resolve_conflict_dir(self):
73
105
        tree = self.make_branch_and_tree('.')
74
106
        self.build_tree_contents([('hello', 'hello world4'),
125
157
        self.assertEqual(conflicts.ConflictList([]), tree.conflicts())
126
158
 
127
159
 
128
 
class TestPerConflict(tests.TestCase):
129
 
 
130
 
    scenarios = scenarios.multiply_scenarios(vary_by_conflicts())
131
 
 
132
 
    def test_stringification(self):
133
 
        text = unicode(self.conflict)
134
 
        self.assertContainsString(text, self.conflict.path)
135
 
        self.assertContainsString(text.lower(), "conflict")
136
 
        self.assertContainsString(repr(self.conflict),
137
 
            self.conflict.__class__.__name__)
 
160
class TestConflictStanzas(tests.TestCase):
138
161
 
139
162
    def test_stanza_roundtrip(self):
140
 
        p = self.conflict
141
 
        o = conflicts.Conflict.factory(**p.as_stanza().as_dict())
142
 
        self.assertEqual(o, p)
143
 
 
144
 
        self.assertIsInstance(o.path, unicode)
145
 
 
146
 
        if o.file_id is not None:
147
 
            self.assertIsInstance(o.file_id, str)
148
 
 
149
 
        conflict_path = getattr(o, 'conflict_path', None)
150
 
        if conflict_path is not None:
151
 
            self.assertIsInstance(conflict_path, unicode)
152
 
 
153
 
        conflict_file_id = getattr(o, 'conflict_file_id', None)
154
 
        if conflict_file_id is not None:
155
 
            self.assertIsInstance(conflict_file_id, str)
 
163
        # write and read our example stanza.
 
164
        stanza_iter = example_conflicts.to_stanzas()
 
165
        processed = conflicts.ConflictList.from_stanzas(stanza_iter)
 
166
        for o, p in zip(processed, example_conflicts):
 
167
            self.assertEqual(o, p)
 
168
 
 
169
            self.assertIsInstance(o.path, unicode)
 
170
 
 
171
            if o.file_id is not None:
 
172
                self.assertIsInstance(o.file_id, str)
 
173
 
 
174
            conflict_path = getattr(o, 'conflict_path', None)
 
175
            if conflict_path is not None:
 
176
                self.assertIsInstance(conflict_path, unicode)
 
177
 
 
178
            conflict_file_id = getattr(o, 'conflict_file_id', None)
 
179
            if conflict_file_id is not None:
 
180
                self.assertIsInstance(conflict_file_id, str)
156
181
 
157
182
    def test_stanzification(self):
158
 
        stanza = self.conflict.as_stanza()
159
 
        if 'file_id' in stanza:
160
 
            # In Stanza form, the file_id has to be unicode.
161
 
            self.assertStartsWith(stanza['file_id'], u'\xeed')
162
 
        self.assertStartsWith(stanza['path'], u'p\xe5th')
163
 
        if 'conflict_path' in stanza:
164
 
            self.assertStartsWith(stanza['conflict_path'], u'p\xe5th')
165
 
        if 'conflict_file_id' in stanza:
166
 
            self.assertStartsWith(stanza['conflict_file_id'], u'\xeed')
167
 
 
168
 
 
169
 
class TestConflictList(tests.TestCase):
170
 
 
171
 
    def test_stanzas_roundtrip(self):
172
 
        stanzas_iter = example_conflicts.to_stanzas()
173
 
        processed = conflicts.ConflictList.from_stanzas(stanzas_iter)
174
 
        self.assertEqual(example_conflicts, processed)
175
 
 
176
 
    def test_stringification(self):
177
 
        for text, o in zip(example_conflicts.to_strings(), example_conflicts):
178
 
            self.assertEqual(text, unicode(o))
 
183
        for stanza in example_conflicts.to_stanzas():
 
184
            if 'file_id' in stanza:
 
185
                # In Stanza form, the file_id has to be unicode.
 
186
                self.assertStartsWith(stanza['file_id'], u'\xeed')
 
187
            self.assertStartsWith(stanza['path'], u'p\xe5th')
 
188
            if 'conflict_path' in stanza:
 
189
                self.assertStartsWith(stanza['conflict_path'], u'p\xe5th')
 
190
            if 'conflict_file_id' in stanza:
 
191
                self.assertStartsWith(stanza['conflict_file_id'], u'\xeed')
179
192
 
180
193
 
181
194
# FIXME: The shell-like tests should be converted to real whitebox tests... or
182
195
# moved to a blackbox module -- vila 20100205
183
196
 
184
 
# FIXME: test missing for multiple conflicts
185
 
 
186
197
# FIXME: Tests missing for DuplicateID conflict type
187
198
class TestResolveConflicts(script.TestCaseWithTransportAndScript):
188
199
 
193
204
        self.run_script(self.preamble)
194
205
 
195
206
 
196
 
def mirror_scenarios(base_scenarios):
197
 
    """Return a list of mirrored scenarios.
198
 
 
199
 
    Each scenario in base_scenarios is duplicated switching the roles of 'this'
200
 
    and 'other'
201
 
    """
202
 
    scenarios = []
203
 
    for common, (lname, ldict), (rname, rdict) in base_scenarios:
204
 
        a = tests.multiply_scenarios([(lname, dict(_this=ldict))],
205
 
                                     [(rname, dict(_other=rdict))])
206
 
        b = tests.multiply_scenarios([(rname, dict(_this=rdict))],
207
 
                                     [(lname, dict(_other=ldict))])
208
 
        # Inject the common parameters in all scenarios
209
 
        for name, d in a + b:
210
 
            d.update(common)
211
 
        scenarios.extend(a + b)
212
 
    return scenarios
213
 
 
214
 
 
215
 
# FIXME: Get rid of parametrized (in the class name) once we delete
216
 
# TestResolveConflicts -- vila 20100308
217
 
class TestParametrizedResolveConflicts(tests.TestCaseWithTransport):
218
 
    """This class provides a base to test single conflict resolution.
219
 
 
220
 
    Since all conflict objects are created with specific semantics for their
221
 
    attributes, each class should implement the necessary functions and
222
 
    attributes described below.
223
 
 
224
 
    Each class should define the scenarios that create the expected (single)
225
 
    conflict.
226
 
 
227
 
    Each scenario describes:
228
 
    * how to create 'base' tree (and revision)
229
 
    * how to create 'left' tree (and revision, parent rev 'base')
230
 
    * how to create 'right' tree (and revision, parent rev 'base')
231
 
    * how to check that changes in 'base'->'left' have been taken
232
 
    * how to check that changes in 'base'->'right' have been taken
233
 
 
234
 
    From each base scenario, we generate two concrete scenarios where:
235
 
    * this=left, other=right
236
 
    * this=right, other=left
237
 
 
238
 
    Then the test case verifies each concrete scenario by:
239
 
    * creating a branch containing the 'base', 'this' and 'other' revisions
240
 
    * creating a working tree for the 'this' revision
241
 
    * performing the merge of 'other' into 'this'
242
 
    * verifying the expected conflict was generated
243
 
    * resolving with --take-this or --take-other, and running the corresponding
244
 
      checks (for either 'base'->'this', or 'base'->'other')
245
 
 
246
 
    :cvar _conflict_type: The expected class of the generated conflict.
247
 
 
248
 
    :cvar _assert_conflict: A method receiving the working tree and the
249
 
        conflict object and checking its attributes.
250
 
 
251
 
    :cvar _base_actions: The branchbuilder actions to create the 'base'
252
 
        revision.
253
 
 
254
 
    :cvar _this: The dict related to 'base' -> 'this'. It contains at least:
255
 
      * 'actions': The branchbuilder actions to create the 'this'
256
 
          revision.
257
 
      * 'check': how to check the changes after resolution with --take-this.
258
 
 
259
 
    :cvar _other: The dict related to 'base' -> 'other'. It contains at least:
260
 
      * 'actions': The branchbuilder actions to create the 'other'
261
 
          revision.
262
 
      * 'check': how to check the changes after resolution with --take-other.
263
 
    """
264
 
 
265
 
    # Set by daughter classes
266
 
    _conflict_type = None
267
 
    _assert_conflict = None
 
207
class TestResolveTextConflicts(TestResolveConflicts):
 
208
    # TBC
 
209
    pass
 
210
 
 
211
 
 
212
def content_conflict_scenarios():
 
213
    return [('file,None', dict(_this_actions='modify_file',
 
214
                               _check_this='file_has_more_content',
 
215
                               _other_actions='delete_file',
 
216
                               _check_other='file_doesnt_exist',
 
217
                               )),
 
218
            ('None,file', dict(_this_actions='delete_file',
 
219
                               _check_this='file_doesnt_exist',
 
220
                               _other_actions='modify_file',
 
221
                               _check_other='file_has_more_content',
 
222
                               )),
 
223
            ]
 
224
 
 
225
 
 
226
class TestResolveContentConflicts(tests.TestCaseWithTransport):
268
227
 
269
228
    # Set by load_tests
270
 
    _base_actions = None
271
 
    _this = None
272
 
    _other = None
273
 
 
274
 
    scenarios = []
275
 
    """The scenario list for the conflict type defined by the class.
276
 
 
277
 
    Each scenario is of the form:
278
 
    (common, (left_name, left_dict), (right_name, right_dict))
279
 
 
280
 
    * common is a dict
281
 
 
282
 
    * left_name and right_name are the scenario names that will be combined
283
 
 
284
 
    * left_dict and right_dict are the attributes specific to each half of
285
 
      the scenario. They should include at least 'actions' and 'check' and
286
 
      will be available as '_this' and '_other' test instance attributes.
287
 
 
288
 
    Daughters classes are free to add their specific attributes as they see
289
 
    fit in any of the three dicts.
290
 
 
291
 
    This is a class method so that load_tests can find it.
292
 
 
293
 
    '_base_actions' in the common dict, 'actions' and 'check' in the left
294
 
    and right dicts use names that map to methods in the test classes. Some
295
 
    prefixes are added to these names to get the correspong methods (see
296
 
    _get_actions() and _get_check()). The motivation here is to avoid
297
 
    collisions in the class namespace.
298
 
    """
 
229
    this_actions = None
 
230
    other_actions = None
299
231
 
300
232
    def setUp(self):
301
 
        super(TestParametrizedResolveConflicts, self).setUp()
 
233
        super(TestResolveContentConflicts, self).setUp()
302
234
        builder = self.make_branch_builder('trunk')
303
235
        builder.start_series()
304
 
 
305
236
        # Create an empty trunk
306
237
        builder.build_snapshot('start', None, [
307
238
                ('add', ('', 'root-id', 'directory', ''))])
308
239
        # Add a minimal base content
309
 
        base_actions = self._get_actions(self._base_actions)()
310
 
        builder.build_snapshot('base', ['start'], base_actions)
 
240
        builder.build_snapshot('base', ['start'], [
 
241
                ('add', ('file', 'file-id', 'file', 'trunk content\n'))])
311
242
        # Modify the base content in branch
312
 
        actions_other = self._get_actions(self._other['actions'])()
313
 
        builder.build_snapshot('other', ['base'], actions_other)
 
243
        other_actions = self._get_actions(self._other_actions)
 
244
        builder.build_snapshot('other', ['base'], other_actions())
314
245
        # Modify the base content in trunk
315
 
        actions_this = self._get_actions(self._this['actions'])()
316
 
        builder.build_snapshot('this', ['base'], actions_this)
317
 
        # builder.get_branch() tip is now 'this'
318
 
 
 
246
        this_actions = self._get_actions(self._this_actions)
 
247
        builder.build_snapshot('this', ['base'], this_actions())
319
248
        builder.finish_series()
320
249
        self.builder = builder
321
250
 
325
254
    def _get_check(self, name):
326
255
        return getattr(self, 'check_%s' % name)
327
256
 
 
257
    def do_modify_file(self):
 
258
        return [('modify', ('file-id', 'trunk content\nmore content\n'))]
 
259
 
 
260
    def check_file_has_more_content(self):
 
261
        self.assertFileEqual('trunk content\nmore content\n', 'branch/file')
 
262
 
 
263
    def do_delete_file(self):
 
264
        return [('unversion', 'file-id')]
 
265
 
 
266
    def check_file_doesnt_exist(self):
 
267
        self.failIfExists('branch/file')
 
268
 
328
269
    def _merge_other_into_this(self):
329
270
        b = self.builder.get_branch()
330
271
        wt = b.bzrdir.sprout('branch').open_workingtree()
331
272
        wt.merge_from_branch(b, 'other')
332
273
        return wt
333
274
 
334
 
    def assertConflict(self, wt):
 
275
    def assertConflict(self, wt, ctype, **kwargs):
335
276
        confs = wt.conflicts()
336
277
        self.assertLength(1, confs)
337
278
        c = confs[0]
338
 
        self.assertIsInstance(c, self._conflict_type)
339
 
        self._assert_conflict(wt, c)
340
 
 
341
 
    def _get_resolve_path_arg(self, wt, action):
342
 
        raise NotImplementedError(self._get_resolve_path_arg)
343
 
 
344
 
    def check_resolved(self, wt, action):
345
 
        path = self._get_resolve_path_arg(wt, action)
346
 
        conflicts.resolve(wt, [path], action=action)
 
279
        self.assertIsInstance(c, ctype)
 
280
        sentinel = object() # An impossible value
 
281
        for k, v in kwargs.iteritems():
 
282
            self.assertEqual(v, getattr(c, k, sentinel))
 
283
 
 
284
    def check_resolved(self, wt, item, action):
 
285
        conflicts.resolve(wt, [item], action=action)
347
286
        # Check that we don't have any conflicts nor unknown left
348
287
        self.assertLength(0, wt.conflicts())
349
288
        self.assertLength(0, list(wt.unknowns()))
350
289
 
351
290
    def test_resolve_taking_this(self):
352
291
        wt = self._merge_other_into_this()
353
 
        self.assertConflict(wt)
354
 
        self.check_resolved(wt, 'take_this')
355
 
        check_this = self._get_check(self._this['check'])
 
292
        self.assertConflict(wt, conflicts.ContentsConflict,
 
293
                            path='file', file_id='file-id',)
 
294
        self.check_resolved(wt, 'file', 'take_this')
 
295
        check_this = self._get_check(self._check_this)
356
296
        check_this()
357
297
 
358
298
    def test_resolve_taking_other(self):
359
299
        wt = self._merge_other_into_this()
360
 
        self.assertConflict(wt)
361
 
        self.check_resolved(wt, 'take_other')
362
 
        check_other = self._get_check(self._other['check'])
 
300
        self.assertConflict(wt, conflicts.ContentsConflict,
 
301
                            path='file', file_id='file-id',)
 
302
        self.check_resolved(wt, 'file', 'take_other')
 
303
        check_other = self._get_check(self._check_other)
363
304
        check_other()
364
305
 
365
306
 
366
 
class TestResolveTextConflicts(TestParametrizedResolveConflicts):
367
 
 
368
 
    _conflict_type = conflicts.TextConflict
369
 
 
370
 
    # Set by the scenarios
371
 
    # path and file-id for the file involved in the conflict
372
 
    _path = None
373
 
    _file_id = None
374
 
 
375
 
    scenarios = mirror_scenarios(
376
 
        [
377
 
            # File modified on both sides
378
 
            (dict(_base_actions='create_file',
379
 
                  _path='file', _file_id='file-id'),
380
 
             ('filed_modified_A',
381
 
              dict(actions='modify_file_A', check='file_has_content_A')),
382
 
             ('file_modified_B',
383
 
              dict(actions='modify_file_B', check='file_has_content_B')),),
384
 
            # File modified on both sides in dir
385
 
            (dict(_base_actions='create_file_in_dir',
386
 
                  _path='dir/file', _file_id='file-id'),
387
 
             ('filed_modified_A_in_dir',
388
 
              dict(actions='modify_file_A',
389
 
                   check='file_in_dir_has_content_A')),
390
 
             ('file_modified_B',
391
 
              dict(actions='modify_file_B',
392
 
                   check='file_in_dir_has_content_B')),),
393
 
            ])
394
 
 
395
 
    def do_create_file(self, path='file'):
396
 
        return [('add', (path, 'file-id', 'file', 'trunk content\n'))]
397
 
 
398
 
    def do_modify_file_A(self):
399
 
        return [('modify', ('file-id', 'trunk content\nfeature A\n'))]
400
 
 
401
 
    def do_modify_file_B(self):
402
 
        return [('modify', ('file-id', 'trunk content\nfeature B\n'))]
403
 
 
404
 
    def check_file_has_content_A(self, path='file'):
405
 
        self.assertFileEqual('trunk content\nfeature A\n',
406
 
                             osutils.pathjoin('branch', path))
407
 
 
408
 
    def check_file_has_content_B(self, path='file'):
409
 
        self.assertFileEqual('trunk content\nfeature B\n',
410
 
                             osutils.pathjoin('branch', path))
411
 
 
412
 
    def do_create_file_in_dir(self):
413
 
        return [('add', ('dir', 'dir-id', 'directory', '')),
414
 
            ] + self.do_create_file('dir/file')
415
 
 
416
 
    def check_file_in_dir_has_content_A(self):
417
 
        self.check_file_has_content_A('dir/file')
418
 
 
419
 
    def check_file_in_dir_has_content_B(self):
420
 
        self.check_file_has_content_B('dir/file')
421
 
 
422
 
    def _get_resolve_path_arg(self, wt, action):
423
 
        return self._path
424
 
 
425
 
    def assertTextConflict(self, wt, c):
426
 
        self.assertEqual(self._file_id, c.file_id)
427
 
        self.assertEqual(self._path, c.path)
428
 
    _assert_conflict = assertTextConflict
429
 
 
430
 
 
431
 
class TestResolveContentsConflict(TestParametrizedResolveConflicts):
432
 
 
433
 
    _conflict_type = conflicts.ContentsConflict
434
 
 
435
 
    # Set by the scenarios
436
 
    # path and file-id for the file involved in the conflict
437
 
    _path = None
438
 
    _file_id = None
439
 
 
440
 
    scenarios = mirror_scenarios(
441
 
        [
442
 
            # File modified/deleted
443
 
            (dict(_base_actions='create_file',
444
 
                  _path='file', _file_id='file-id'),
445
 
             ('file_modified',
446
 
              dict(actions='modify_file', check='file_has_more_content')),
447
 
             ('file_deleted',
448
 
              dict(actions='delete_file', check='file_doesnt_exist')),),
449
 
            # File renamed-modified/deleted
450
 
            (dict(_base_actions='create_file',
451
 
                  _path='new-file', _file_id='file-id'),
452
 
             ('file_renamed_and_modified',
453
 
              dict(actions='modify_and_rename_file',
454
 
                   check='file_renamed_and_more_content')),
455
 
             ('file_deleted',
456
 
              dict(actions='delete_file', check='file_doesnt_exist')),),
457
 
            # File modified/deleted in dir
458
 
            (dict(_base_actions='create_file_in_dir',
459
 
                  _path='dir/file', _file_id='file-id'),
460
 
             ('file_modified_in_dir',
461
 
              dict(actions='modify_file_in_dir',
462
 
                   check='file_in_dir_has_more_content')),
463
 
             ('file_deleted_in_dir',
464
 
              dict(actions='delete_file',
465
 
                   check='file_in_dir_doesnt_exist')),),
466
 
            ])
467
 
 
468
 
    def do_create_file(self):
469
 
        return [('add', ('file', 'file-id', 'file', 'trunk content\n'))]
470
 
 
471
 
    def do_modify_file(self):
472
 
        return [('modify', ('file-id', 'trunk content\nmore content\n'))]
473
 
 
474
 
    def do_modify_and_rename_file(self):
475
 
        return [('modify', ('file-id', 'trunk content\nmore content\n')),
476
 
                ('rename', ('file', 'new-file'))]
477
 
 
478
 
    def check_file_has_more_content(self):
479
 
        self.assertFileEqual('trunk content\nmore content\n', 'branch/file')
480
 
 
481
 
    def check_file_renamed_and_more_content(self):
482
 
        self.assertFileEqual('trunk content\nmore content\n', 'branch/new-file')
483
 
 
484
 
    def do_delete_file(self):
485
 
        return [('unversion', 'file-id')]
486
 
 
487
 
    def check_file_doesnt_exist(self):
488
 
        self.assertPathDoesNotExist('branch/file')
489
 
 
490
 
    def do_create_file_in_dir(self):
491
 
        return [('add', ('dir', 'dir-id', 'directory', '')),
492
 
                ('add', ('dir/file', 'file-id', 'file', 'trunk content\n'))]
493
 
 
494
 
    def do_modify_file_in_dir(self):
495
 
        return [('modify', ('file-id', 'trunk content\nmore content\n'))]
496
 
 
497
 
    def check_file_in_dir_has_more_content(self):
498
 
        self.assertFileEqual('trunk content\nmore content\n', 'branch/dir/file')
499
 
 
500
 
    def check_file_in_dir_doesnt_exist(self):
501
 
        self.assertPathDoesNotExist('branch/dir/file')
502
 
 
503
 
    def _get_resolve_path_arg(self, wt, action):
504
 
        return self._path
505
 
 
506
 
    def assertContentsConflict(self, wt, c):
507
 
        self.assertEqual(self._file_id, c.file_id)
508
 
        self.assertEqual(self._path, c.path)
509
 
    _assert_conflict = assertContentsConflict
510
 
 
511
 
 
512
 
class TestResolvePathConflict(TestParametrizedResolveConflicts):
513
 
 
514
 
    _conflict_type = conflicts.PathConflict
515
 
 
516
 
    def do_nothing(self):
517
 
        return []
518
 
 
519
 
    # Each side dict additionally defines:
520
 
    # - path path involved (can be '<deleted>')
521
 
    # - file-id involved
522
 
    scenarios = mirror_scenarios(
523
 
        [
524
 
            # File renamed/deleted
525
 
            (dict(_base_actions='create_file'),
526
 
             ('file_renamed',
527
 
              dict(actions='rename_file', check='file_renamed',
528
 
                   path='new-file', file_id='file-id')),
529
 
             ('file_deleted',
530
 
              dict(actions='delete_file', check='file_doesnt_exist',
531
 
                   # PathConflicts deletion handling requires a special
532
 
                   # hard-coded value
533
 
                   path='<deleted>', file_id='file-id')),),
534
 
            # File renamed/deleted in dir
535
 
            (dict(_base_actions='create_file_in_dir'),
536
 
             ('file_renamed_in_dir',
537
 
              dict(actions='rename_file_in_dir', check='file_in_dir_renamed',
538
 
                   path='dir/new-file', file_id='file-id')),
539
 
             ('file_deleted',
540
 
              dict(actions='delete_file', check='file_in_dir_doesnt_exist',
541
 
                   # PathConflicts deletion handling requires a special
542
 
                   # hard-coded value
543
 
                   path='<deleted>', file_id='file-id')),),
544
 
            # File renamed/renamed differently
545
 
            (dict(_base_actions='create_file'),
546
 
             ('file_renamed',
547
 
              dict(actions='rename_file', check='file_renamed',
548
 
                   path='new-file', file_id='file-id')),
549
 
             ('file_renamed2',
550
 
              dict(actions='rename_file2', check='file_renamed2',
551
 
                   path='new-file2', file_id='file-id')),),
552
 
            # Dir renamed/deleted
553
 
            (dict(_base_actions='create_dir'),
554
 
             ('dir_renamed',
555
 
              dict(actions='rename_dir', check='dir_renamed',
556
 
                   path='new-dir', file_id='dir-id')),
557
 
             ('dir_deleted',
558
 
              dict(actions='delete_dir', check='dir_doesnt_exist',
559
 
                   # PathConflicts deletion handling requires a special
560
 
                   # hard-coded value
561
 
                   path='<deleted>', file_id='dir-id')),),
562
 
            # Dir renamed/renamed differently
563
 
            (dict(_base_actions='create_dir'),
564
 
             ('dir_renamed',
565
 
              dict(actions='rename_dir', check='dir_renamed',
566
 
                   path='new-dir', file_id='dir-id')),
567
 
             ('dir_renamed2',
568
 
              dict(actions='rename_dir2', check='dir_renamed2',
569
 
                   path='new-dir2', file_id='dir-id')),),
570
 
            ])
571
 
 
572
 
    def do_create_file(self):
573
 
        return [('add', ('file', 'file-id', 'file', 'trunk content\n'))]
574
 
 
575
 
    def do_create_dir(self):
576
 
        return [('add', ('dir', 'dir-id', 'directory', ''))]
577
 
 
578
 
    def do_rename_file(self):
579
 
        return [('rename', ('file', 'new-file'))]
580
 
 
581
 
    def check_file_renamed(self):
582
 
        self.assertPathDoesNotExist('branch/file')
583
 
        self.assertPathExists('branch/new-file')
584
 
 
585
 
    def do_rename_file2(self):
586
 
        return [('rename', ('file', 'new-file2'))]
587
 
 
588
 
    def check_file_renamed2(self):
589
 
        self.assertPathDoesNotExist('branch/file')
590
 
        self.assertPathExists('branch/new-file2')
591
 
 
592
 
    def do_rename_dir(self):
593
 
        return [('rename', ('dir', 'new-dir'))]
594
 
 
595
 
    def check_dir_renamed(self):
596
 
        self.assertPathDoesNotExist('branch/dir')
597
 
        self.assertPathExists('branch/new-dir')
598
 
 
599
 
    def do_rename_dir2(self):
600
 
        return [('rename', ('dir', 'new-dir2'))]
601
 
 
602
 
    def check_dir_renamed2(self):
603
 
        self.assertPathDoesNotExist('branch/dir')
604
 
        self.assertPathExists('branch/new-dir2')
605
 
 
606
 
    def do_delete_file(self):
607
 
        return [('unversion', 'file-id')]
608
 
 
609
 
    def check_file_doesnt_exist(self):
610
 
        self.assertPathDoesNotExist('branch/file')
611
 
 
612
 
    def do_delete_dir(self):
613
 
        return [('unversion', 'dir-id')]
614
 
 
615
 
    def check_dir_doesnt_exist(self):
616
 
        self.assertPathDoesNotExist('branch/dir')
617
 
 
618
 
    def do_create_file_in_dir(self):
619
 
        return [('add', ('dir', 'dir-id', 'directory', '')),
620
 
                ('add', ('dir/file', 'file-id', 'file', 'trunk content\n'))]
621
 
 
622
 
    def do_rename_file_in_dir(self):
623
 
        return [('rename', ('dir/file', 'dir/new-file'))]
624
 
 
625
 
    def check_file_in_dir_renamed(self):
626
 
        self.assertPathDoesNotExist('branch/dir/file')
627
 
        self.assertPathExists('branch/dir/new-file')
628
 
 
629
 
    def check_file_in_dir_doesnt_exist(self):
630
 
        self.assertPathDoesNotExist('branch/dir/file')
631
 
 
632
 
    def _get_resolve_path_arg(self, wt, action):
633
 
        tpath = self._this['path']
634
 
        opath = self._other['path']
635
 
        if tpath == '<deleted>':
636
 
            path = opath
637
 
        else:
638
 
            path = tpath
639
 
        return path
640
 
 
641
 
    def assertPathConflict(self, wt, c):
642
 
        tpath = self._this['path']
643
 
        tfile_id = self._this['file_id']
644
 
        opath = self._other['path']
645
 
        ofile_id = self._other['file_id']
646
 
        self.assertEqual(tfile_id, ofile_id) # Sanity check
647
 
        self.assertEqual(tfile_id, c.file_id)
648
 
        self.assertEqual(tpath, c.path)
649
 
        self.assertEqual(opath, c.conflict_path)
650
 
    _assert_conflict = assertPathConflict
651
 
 
652
 
 
653
 
class TestResolvePathConflictBefore531967(TestResolvePathConflict):
654
 
    """Same as TestResolvePathConflict but a specific conflict object.
655
 
    """
656
 
 
657
 
    def assertPathConflict(self, c):
658
 
        # We create a conflict object as it was created before the fix and
659
 
        # inject it into the working tree, the test will exercise the
660
 
        # compatibility code.
661
 
        old_c = conflicts.PathConflict('<deleted>', self._item_path,
662
 
                                       file_id=None)
663
 
        wt.set_conflicts(conflicts.ConflictList([old_c]))
664
 
 
665
 
 
666
 
class TestResolveDuplicateEntry(TestParametrizedResolveConflicts):
667
 
 
668
 
    _conflict_type = conflicts.DuplicateEntry
669
 
 
670
 
    scenarios = mirror_scenarios(
671
 
        [
672
 
            # File created with different file-ids
673
 
            (dict(_base_actions='nothing'),
674
 
             ('filea_created',
675
 
              dict(actions='create_file_a', check='file_content_a',
676
 
                   path='file', file_id='file-a-id')),
677
 
             ('fileb_created',
678
 
              dict(actions='create_file_b', check='file_content_b',
679
 
                   path='file', file_id='file-b-id')),),
680
 
            ])
681
 
 
682
 
    def do_nothing(self):
683
 
        return []
684
 
 
685
 
    def do_create_file_a(self):
686
 
        return [('add', ('file', 'file-a-id', 'file', 'file a content\n'))]
687
 
 
688
 
    def check_file_content_a(self):
689
 
        self.assertFileEqual('file a content\n', 'branch/file')
690
 
 
691
 
    def do_create_file_b(self):
692
 
        return [('add', ('file', 'file-b-id', 'file', 'file b content\n'))]
693
 
 
694
 
    def check_file_content_b(self):
695
 
        self.assertFileEqual('file b content\n', 'branch/file')
696
 
 
697
 
    def _get_resolve_path_arg(self, wt, action):
698
 
        return self._this['path']
699
 
 
700
 
    def assertDuplicateEntry(self, wt, c):
701
 
        tpath = self._this['path']
702
 
        tfile_id = self._this['file_id']
703
 
        opath = self._other['path']
704
 
        ofile_id = self._other['file_id']
705
 
        self.assertEqual(tpath, opath) # Sanity check
706
 
        self.assertEqual(tfile_id, c.file_id)
707
 
        self.assertEqual(tpath + '.moved', c.path)
708
 
        self.assertEqual(tpath, c.conflict_path)
709
 
    _assert_conflict = assertDuplicateEntry
 
307
class TestResolveDuplicateEntry(TestResolveConflicts):
 
308
 
 
309
    preamble = """
 
310
$ bzr init trunk
 
311
$ cd trunk
 
312
$ echo 'trunk content' >file
 
313
$ bzr add file
 
314
$ bzr commit -m 'Create trunk'
 
315
 
 
316
$ echo 'trunk content too' >file2
 
317
$ bzr add file2
 
318
$ bzr commit -m 'Add file2 in trunk'
 
319
 
 
320
$ bzr branch . -r 1 ../branch
 
321
$ cd ../branch
 
322
$ echo 'branch content' >file2
 
323
$ bzr add file2
 
324
$ bzr commit -m 'Add file2 in branch'
 
325
 
 
326
$ bzr merge ../trunk
 
327
2>+N  file2
 
328
2>R   file2 => file2.moved
 
329
2>Conflict adding file file2.  Moved existing file to file2.moved.
 
330
2>1 conflicts encountered.
 
331
"""
 
332
 
 
333
    def test_keep_this(self):
 
334
        self.run_script("""
 
335
$ bzr rm file2  --force
 
336
$ bzr mv file2.moved file2
 
337
$ bzr resolve file2
 
338
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
339
""")
 
340
 
 
341
    def test_keep_other(self):
 
342
        self.failIfExists('branch/file2.moved')
 
343
        self.run_script("""
 
344
$ bzr rm file2.moved --force
 
345
$ bzr resolve file2
 
346
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
347
""")
 
348
        self.failIfExists('branch/file2.moved')
 
349
 
 
350
    def test_resolve_taking_this(self):
 
351
        self.run_script("""
 
352
$ bzr resolve --take-this file2
 
353
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
354
""")
 
355
 
 
356
    def test_resolve_taking_other(self):
 
357
        self.run_script("""
 
358
$ bzr resolve --take-other file2
 
359
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
360
""")
710
361
 
711
362
 
712
363
class TestResolveUnversionedParent(TestResolveConflicts):
717
368
    # tests MissingParent resolution :-/
718
369
    preamble = """
719
370
$ bzr init trunk
720
 
...
721
371
$ cd trunk
722
372
$ mkdir dir
723
 
$ bzr add -q dir
724
 
$ bzr commit -m 'Create trunk' -q
 
373
$ bzr add dir
 
374
$ bzr commit -m 'Create trunk'
 
375
 
725
376
$ echo 'trunk content' >dir/file
726
 
$ bzr add -q dir/file
727
 
$ bzr commit -q -m 'Add dir/file in trunk'
728
 
$ bzr branch -q . -r 1 ../branch
 
377
$ bzr add dir/file
 
378
$ bzr commit -m 'Add dir/file in trunk'
 
379
 
 
380
$ bzr branch . -r 1 ../branch
729
381
$ cd ../branch
730
 
$ bzr rm dir -q
731
 
$ bzr commit -q -m 'Remove dir in branch'
 
382
$ bzr rm dir
 
383
$ bzr commit -m 'Remove dir in branch'
 
384
 
732
385
$ bzr merge ../trunk
733
386
2>+N  dir/
734
387
2>+N  dir/file
739
392
 
740
393
    def test_take_this(self):
741
394
        self.run_script("""
742
 
$ bzr rm -q dir  --force
 
395
$ bzr rm dir  --force
743
396
$ bzr resolve dir
744
 
2>2 conflict(s) resolved, 0 remaining
745
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
397
$ bzr commit --strict -m 'No more conflicts nor unknown files'
746
398
""")
747
399
 
748
400
    def test_take_other(self):
749
401
        self.run_script("""
750
402
$ bzr resolve dir
751
 
2>2 conflict(s) resolved, 0 remaining
752
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
403
$ bzr commit --strict -m 'No more conflicts nor unknown files'
753
404
""")
754
405
 
755
406
 
757
408
 
758
409
    preamble = """
759
410
$ bzr init trunk
760
 
...
761
411
$ cd trunk
762
412
$ mkdir dir
763
413
$ echo 'trunk content' >dir/file
764
 
$ bzr add -q
765
 
$ bzr commit -m 'Create trunk' -q
 
414
$ bzr add
 
415
$ bzr commit -m 'Create trunk'
 
416
 
766
417
$ echo 'trunk content' >dir/file2
767
 
$ bzr add -q dir/file2
768
 
$ bzr commit -q -m 'Add dir/file2 in branch'
769
 
$ bzr branch -q . -r 1 ../branch
 
418
$ bzr add dir/file2
 
419
$ bzr commit -m 'Add dir/file2 in branch'
 
420
 
 
421
$ bzr branch . -r 1 ../branch
770
422
$ cd ../branch
771
 
$ bzr rm -q dir/file --force
772
 
$ bzr rm -q dir
773
 
$ bzr commit -q -m 'Remove dir/file'
 
423
$ bzr rm dir/file --force
 
424
$ bzr rm dir
 
425
$ bzr commit -m 'Remove dir/file'
 
426
 
774
427
$ bzr merge ../trunk
775
428
2>+N  dir/
776
429
2>+N  dir/file2
782
435
    def test_keep_them_all(self):
783
436
        self.run_script("""
784
437
$ bzr resolve dir
785
 
2>2 conflict(s) resolved, 0 remaining
786
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
438
$ bzr commit --strict -m 'No more conflicts nor unknown files'
787
439
""")
788
440
 
789
441
    def test_adopt_child(self):
790
442
        self.run_script("""
791
 
$ bzr mv -q dir/file2 file2
792
 
$ bzr rm -q dir --force
 
443
$ bzr mv dir/file2 file2
 
444
$ bzr rm dir --force
793
445
$ bzr resolve dir
794
 
2>2 conflict(s) resolved, 0 remaining
795
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
446
$ bzr commit --strict -m 'No more conflicts nor unknown files'
796
447
""")
797
448
 
798
449
    def test_kill_them_all(self):
799
450
        self.run_script("""
800
 
$ bzr rm -q dir --force
 
451
$ bzr rm dir --force
801
452
$ bzr resolve dir
802
 
2>2 conflict(s) resolved, 0 remaining
803
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
453
$ bzr commit --strict -m 'No more conflicts nor unknown files'
804
454
""")
805
455
 
806
456
    def test_resolve_taking_this(self):
807
457
        self.run_script("""
808
458
$ bzr resolve --take-this dir
809
 
2>...
810
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
459
$ bzr commit --strict -m 'No more conflicts nor unknown files'
811
460
""")
812
461
 
813
462
    def test_resolve_taking_other(self):
814
463
        self.run_script("""
815
464
$ bzr resolve --take-other dir
816
 
2>...
817
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
465
$ bzr commit --strict -m 'No more conflicts nor unknown files'
818
466
""")
819
467
 
820
468
 
822
470
 
823
471
    preamble = """
824
472
$ bzr init trunk
825
 
...
826
473
$ cd trunk
827
474
$ mkdir dir
828
475
$ echo 'trunk content' >dir/file
829
 
$ bzr add -q
830
 
$ bzr commit -m 'Create trunk' -q
831
 
$ bzr rm -q dir/file --force
832
 
$ bzr rm -q dir --force
833
 
$ bzr commit -q -m 'Remove dir/file'
834
 
$ bzr branch -q . -r 1 ../branch
 
476
$ bzr add
 
477
$ bzr commit -m 'Create trunk'
 
478
 
 
479
$ bzr rm dir/file --force
 
480
$ bzr rm dir --force
 
481
$ bzr commit -m 'Remove dir/file'
 
482
 
 
483
$ bzr branch . -r 1 ../branch
835
484
$ cd ../branch
836
485
$ echo 'branch content' >dir/file2
837
 
$ bzr add -q dir/file2
838
 
$ bzr commit -q -m 'Add dir/file2 in branch'
 
486
$ bzr add dir/file2
 
487
$ bzr commit -m 'Add dir/file2 in branch'
 
488
 
839
489
$ bzr merge ../trunk
840
490
2>-D  dir/file
841
491
2>Conflict: can't delete dir because it is not empty.  Not deleting.
846
496
    def test_keep_them_all(self):
847
497
        self.run_script("""
848
498
$ bzr resolve dir
849
 
2>2 conflict(s) resolved, 0 remaining
850
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
499
$ bzr commit --strict -m 'No more conflicts nor unknown files'
851
500
""")
852
501
 
853
502
    def test_adopt_child(self):
854
503
        self.run_script("""
855
 
$ bzr mv -q dir/file2 file2
856
 
$ bzr rm -q dir --force
 
504
$ bzr mv dir/file2 file2
 
505
$ bzr rm dir --force
857
506
$ bzr resolve dir
858
 
2>2 conflict(s) resolved, 0 remaining
859
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
507
$ bzr commit --strict -m 'No more conflicts nor unknown files'
860
508
""")
861
509
 
862
510
    def test_kill_them_all(self):
863
511
        self.run_script("""
864
 
$ bzr rm -q dir --force
 
512
$ bzr rm dir --force
865
513
$ bzr resolve dir
866
 
2>2 conflict(s) resolved, 0 remaining
867
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
514
$ bzr commit --strict -m 'No more conflicts nor unknown files'
868
515
""")
869
516
 
870
517
    def test_resolve_taking_this(self):
871
518
        self.run_script("""
872
519
$ bzr resolve --take-this dir
873
 
2>2 conflict(s) resolved, 0 remaining
874
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
520
$ bzr commit --strict -m 'No more conflicts nor unknown files'
875
521
""")
876
522
 
877
523
    def test_resolve_taking_other(self):
878
524
        self.run_script("""
879
525
$ bzr resolve --take-other dir
880
 
2>deleted dir/file2
881
 
2>deleted dir
882
 
2>2 conflict(s) resolved, 0 remaining
883
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
884
 
""")
885
 
 
886
 
 
887
 
class TestResolveParentLoop(TestParametrizedResolveConflicts):
888
 
 
889
 
    _conflict_type = conflicts.ParentLoop
890
 
 
891
 
    _this_args = None
892
 
    _other_args = None
893
 
 
894
 
    # Each side dict additionally defines:
895
 
    # - dir_id: the directory being moved
896
 
    # - target_id: The target directory
897
 
    # - xfail: whether the test is expected to fail if the action is
898
 
    #   involved as 'other'
899
 
    scenarios = mirror_scenarios(
900
 
        [
901
 
            # Dirs moved into each other
902
 
            (dict(_base_actions='create_dir1_dir2'),
903
 
             ('dir1_into_dir2',
904
 
              dict(actions='move_dir1_into_dir2', check='dir1_moved',
905
 
                   dir_id='dir1-id', target_id='dir2-id', xfail=False)),
906
 
             ('dir2_into_dir1',
907
 
              dict(actions='move_dir2_into_dir1', check='dir2_moved',
908
 
                   dir_id='dir2-id', target_id='dir1-id', xfail=False))),
909
 
            # Subdirs moved into each other
910
 
            (dict(_base_actions='create_dir1_4'),
911
 
             ('dir1_into_dir4',
912
 
              dict(actions='move_dir1_into_dir4', check='dir1_2_moved',
913
 
                   dir_id='dir1-id', target_id='dir4-id', xfail=True)),
914
 
             ('dir3_into_dir2',
915
 
              dict(actions='move_dir3_into_dir2', check='dir3_4_moved',
916
 
                   dir_id='dir3-id', target_id='dir2-id', xfail=True))),
917
 
            ])
918
 
 
919
 
    def do_create_dir1_dir2(self):
920
 
        return [('add', ('dir1', 'dir1-id', 'directory', '')),
921
 
                ('add', ('dir2', 'dir2-id', 'directory', '')),]
922
 
 
923
 
    def do_move_dir1_into_dir2(self):
924
 
        return [('rename', ('dir1', 'dir2/dir1'))]
925
 
 
926
 
    def check_dir1_moved(self):
927
 
        self.assertPathDoesNotExist('branch/dir1')
928
 
        self.assertPathExists('branch/dir2/dir1')
929
 
 
930
 
    def do_move_dir2_into_dir1(self):
931
 
        return [('rename', ('dir2', 'dir1/dir2'))]
932
 
 
933
 
    def check_dir2_moved(self):
934
 
        self.assertPathDoesNotExist('branch/dir2')
935
 
        self.assertPathExists('branch/dir1/dir2')
936
 
 
937
 
    def do_create_dir1_4(self):
938
 
        return [('add', ('dir1', 'dir1-id', 'directory', '')),
939
 
                ('add', ('dir1/dir2', 'dir2-id', 'directory', '')),
940
 
                ('add', ('dir3', 'dir3-id', 'directory', '')),
941
 
                ('add', ('dir3/dir4', 'dir4-id', 'directory', '')),]
942
 
 
943
 
    def do_move_dir1_into_dir4(self):
944
 
        return [('rename', ('dir1', 'dir3/dir4/dir1'))]
945
 
 
946
 
    def check_dir1_2_moved(self):
947
 
        self.assertPathDoesNotExist('branch/dir1')
948
 
        self.assertPathExists('branch/dir3/dir4/dir1')
949
 
        self.assertPathExists('branch/dir3/dir4/dir1/dir2')
950
 
 
951
 
    def do_move_dir3_into_dir2(self):
952
 
        return [('rename', ('dir3', 'dir1/dir2/dir3'))]
953
 
 
954
 
    def check_dir3_4_moved(self):
955
 
        self.assertPathDoesNotExist('branch/dir3')
956
 
        self.assertPathExists('branch/dir1/dir2/dir3')
957
 
        self.assertPathExists('branch/dir1/dir2/dir3/dir4')
958
 
 
959
 
    def _get_resolve_path_arg(self, wt, action):
960
 
        # ParentLoop says: moving <conflict_path> into <path>. Cancelled move.
961
 
        # But since <path> doesn't exist in the working tree, we need to use
962
 
        # <conflict_path> instead, and that, in turn, is given by dir_id. Pfew.
963
 
        return wt.id2path(self._other['dir_id'])
964
 
 
965
 
    def assertParentLoop(self, wt, c):
966
 
        self.assertEqual(self._other['dir_id'], c.file_id)
967
 
        self.assertEqual(self._other['target_id'], c.conflict_file_id)
968
 
        # The conflict paths are irrelevant (they are deterministic but not
969
 
        # worth checking since they don't provide the needed information
970
 
        # anyway)
971
 
        if self._other['xfail']:
972
 
            # It's a bit hackish to raise from here relying on being called for
973
 
            # both tests but this avoid overriding test_resolve_taking_other
974
 
            raise tests.KnownFailure(
975
 
                "ParentLoop doesn't carry enough info to resolve --take-other")
976
 
    _assert_conflict = assertParentLoop
 
526
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
527
""")
 
528
 
 
529
 
 
530
class TestResolvePathConflict(TestResolveConflicts):
 
531
 
 
532
    preamble = """
 
533
$ bzr init trunk
 
534
$ cd trunk
 
535
$ echo 'Boo!' >file
 
536
$ bzr add
 
537
$ bzr commit -m 'Create trunk'
 
538
 
 
539
$ bzr mv file file-in-trunk
 
540
$ bzr commit -m 'Renamed to file-in-trunk'
 
541
 
 
542
$ bzr branch . -r 1 ../branch
 
543
$ cd ../branch
 
544
$ bzr mv file file-in-branch
 
545
$ bzr commit -m 'Renamed to file-in-branch'
 
546
 
 
547
$ bzr merge ../trunk
 
548
2>R   file-in-branch => file-in-trunk
 
549
2>Path conflict: file-in-branch / file-in-trunk
 
550
2>1 conflicts encountered.
 
551
"""
 
552
 
 
553
    def test_keep_source(self):
 
554
        self.run_script("""
 
555
$ bzr resolve file-in-trunk
 
556
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
557
""")
 
558
 
 
559
    def test_keep_target(self):
 
560
        self.run_script("""
 
561
$ bzr mv file-in-trunk file-in-branch
 
562
$ bzr resolve file-in-branch
 
563
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
564
""")
 
565
 
 
566
    def test_resolve_taking_this(self):
 
567
        self.run_script("""
 
568
$ bzr resolve --take-this file-in-branch
 
569
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
570
""")
 
571
 
 
572
    def test_resolve_taking_other(self):
 
573
        self.run_script("""
 
574
$ bzr resolve --take-other file-in-branch
 
575
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
576
""")
 
577
 
 
578
 
 
579
class TestResolveParentLoop(TestResolveConflicts):
 
580
 
 
581
    preamble = """
 
582
$ bzr init trunk
 
583
$ cd trunk
 
584
$ bzr mkdir dir1
 
585
$ bzr mkdir dir2
 
586
$ bzr commit -m 'Create trunk'
 
587
 
 
588
$ bzr mv dir2 dir1
 
589
$ bzr commit -m 'Moved dir2 into dir1'
 
590
 
 
591
$ bzr branch . -r 1 ../branch
 
592
$ cd ../branch
 
593
$ bzr mv dir1 dir2
 
594
$ bzr commit -m 'Moved dir1 into dir2'
 
595
 
 
596
$ bzr merge ../trunk
 
597
2>Conflict moving dir2/dir1 into dir2.  Cancelled move.
 
598
2>1 conflicts encountered.
 
599
"""
 
600
 
 
601
    def test_take_this(self):
 
602
        self.run_script("""
 
603
$ bzr resolve dir2
 
604
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
605
""")
 
606
 
 
607
    def test_take_other(self):
 
608
        self.run_script("""
 
609
$ bzr mv dir2/dir1 dir1
 
610
$ bzr mv dir2 dir1
 
611
$ bzr resolve dir2
 
612
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
613
""")
 
614
 
 
615
    def test_resolve_taking_this(self):
 
616
        self.run_script("""
 
617
$ bzr resolve --take-this dir2
 
618
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
619
""")
 
620
        self.failUnlessExists('dir2')
 
621
 
 
622
    def test_resolve_taking_other(self):
 
623
        self.run_script("""
 
624
$ bzr resolve --take-other dir2
 
625
$ bzr commit --strict -m 'No more conflicts nor unknown files'
 
626
""")
 
627
        self.failUnlessExists('dir1')
977
628
 
978
629
 
979
630
class TestResolveNonDirectoryParent(TestResolveConflicts):
980
631
 
981
632
    preamble = """
982
633
$ bzr init trunk
983
 
...
984
634
$ cd trunk
985
635
$ bzr mkdir foo
986
 
...
987
 
$ bzr commit -m 'Create trunk' -q
 
636
$ bzr commit -m 'Create trunk'
988
637
$ echo "Boing" >foo/bar
989
 
$ bzr add -q foo/bar
990
 
$ bzr commit -q -m 'Add foo/bar'
991
 
$ bzr branch -q . -r 1 ../branch
 
638
$ bzr add foo/bar
 
639
$ bzr commit -m 'Add foo/bar'
 
640
 
 
641
$ bzr branch . -r 1 ../branch
992
642
$ cd ../branch
993
643
$ rm -r foo
994
644
$ echo "Boo!" >foo
995
 
$ bzr commit -q -m 'foo is now a file'
 
645
$ bzr commit -m 'foo is now a file'
 
646
 
996
647
$ bzr merge ../trunk
997
648
2>+N  foo.new/bar
998
649
2>RK  foo => foo.new/
1004
655
 
1005
656
    def test_take_this(self):
1006
657
        self.run_script("""
1007
 
$ bzr rm -q foo.new --force
 
658
$ bzr rm foo.new --force
1008
659
# FIXME: Isn't it weird that foo is now unkown even if foo.new has been put
1009
660
# aside ? -- vila 090916
1010
 
$ bzr add -q foo
 
661
$ bzr add foo
1011
662
$ bzr resolve foo.new
1012
 
2>1 conflict(s) resolved, 0 remaining
1013
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
663
$ bzr commit --strict -m 'No more conflicts nor unknown files'
1014
664
""")
1015
665
 
1016
666
    def test_take_other(self):
1017
667
        self.run_script("""
1018
 
$ bzr rm -q foo --force
1019
 
$ bzr mv -q foo.new foo
 
668
$ bzr rm foo --force
 
669
$ bzr mv foo.new foo
1020
670
$ bzr resolve foo
1021
 
2>1 conflict(s) resolved, 0 remaining
1022
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
671
$ bzr commit --strict -m 'No more conflicts nor unknown files'
1023
672
""")
1024
673
 
1025
674
    def test_resolve_taking_this(self):
1026
675
        self.run_script("""
1027
676
$ bzr resolve --take-this foo.new
1028
 
2>...
1029
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
677
$ bzr commit --strict -m 'No more conflicts nor unknown files'
1030
678
""")
1031
679
 
1032
680
    def test_resolve_taking_other(self):
1033
681
        self.run_script("""
1034
682
$ bzr resolve --take-other foo.new
1035
 
2>...
1036
 
$ bzr commit -q --strict -m 'No more conflicts nor unknown files'
 
683
$ bzr commit --strict -m 'No more conflicts nor unknown files'
1037
684
""")
1038
685
 
1039
686
 
1045
692
        # conflict.
1046
693
        self.run_script("""
1047
694
$ bzr init trunk
1048
 
...
1049
695
$ cd trunk
1050
696
$ bzr mkdir foo
1051
 
...
1052
 
$ bzr commit -m 'Create trunk' -q
 
697
$ bzr commit -m 'Create trunk'
1053
698
$ rm -r foo
1054
699
$ echo "Boo!" >foo
1055
 
$ bzr commit -m 'foo is now a file' -q
1056
 
$ bzr branch -q . -r 1 ../branch -q
 
700
$ bzr commit -m 'foo is now a file'
 
701
 
 
702
$ bzr branch . -r 1 ../branch
1057
703
$ cd ../branch
1058
704
$ echo "Boing" >foo/bar
1059
 
$ bzr add -q foo/bar -q
1060
 
$ bzr commit -m 'Add foo/bar' -q
 
705
$ bzr add foo/bar
 
706
$ bzr commit -m 'Add foo/bar'
 
707
 
1061
708
$ bzr merge ../trunk
1062
709
2>bzr: ERROR: Tree transform is malformed [('unversioned executability', 'new-1')]
1063
710
""")