/brz/remove-bazaar

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

« back to all changes in this revision

Viewing changes to breezy/tests/test_commit.py

Merge test-run support.

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/test_commit.py
16
16
 
17
17
 
18
18
import os
19
 
from io import BytesIO
20
19
 
21
20
import breezy
22
21
from .. import (
23
22
    config,
24
23
    controldir,
25
24
    errors,
26
 
    trace,
27
25
    )
28
26
from ..branch import Branch
29
27
from ..bzr.bzrdir import BzrDirMetaFormat1
38
36
    BzrError,
39
37
    LockContention,
40
38
    )
41
 
from ..tree import TreeChange
42
39
from . import (
43
40
    TestCase,
44
41
    TestCaseWithTransport,
89
86
        """Commit and check two versions of a single file."""
90
87
        wt = self.make_branch_and_tree('.')
91
88
        b = wt.branch
92
 
        with open('hello', 'w') as f:
93
 
            f.write('hello world')
 
89
        with open('hello', 'w') as f: f.write('hello world')
94
90
        wt.add('hello')
95
91
        rev1 = wt.commit(message='add hello')
96
92
 
97
 
        with open('hello', 'w') as f:
98
 
            f.write('version 2')
 
93
        with open('hello', 'w') as f: f.write('version 2')
99
94
        rev2 = wt.commit(message='commit 2')
100
95
 
101
96
        eq = self.assertEqual
119
114
        """Attempt a lossy commit to a native branch."""
120
115
        wt = self.make_branch_and_tree('.')
121
116
        b = wt.branch
122
 
        with open('hello', 'w') as f:
123
 
            f.write('hello world')
 
117
        with open('hello', 'w') as f: f.write('hello world')
124
118
        wt.add('hello')
125
119
        revid = wt.commit(message='add hello', rev_id=b'revid', lossy=True)
126
 
        self.assertEqual(b'revid', revid)
 
120
        self.assertEqual('revid', revid)
127
121
 
128
122
    def test_commit_lossy_foreign(self):
129
123
        """Attempt a lossy commit to a foreign branch."""
130
124
        test_foreign.register_dummy_foreign_for_test(self)
131
125
        wt = self.make_branch_and_tree('.',
132
 
                                       format=test_foreign.DummyForeignVcsDirFormat())
 
126
            format=test_foreign.DummyForeignVcsDirFormat())
133
127
        b = wt.branch
134
 
        with open('hello', 'w') as f:
135
 
            f.write('hello world')
 
128
        with open('hello', 'w') as f: f.write('hello world')
136
129
        wt.add('hello')
137
130
        revid = wt.commit(message='add hello', lossy=True,
138
 
                          timestamp=1302659388, timezone=0)
139
 
        self.assertEqual(b'dummy-v1:1302659388-0-UNKNOWN', revid)
 
131
            timestamp=1302659388, timezone=0)
 
132
        self.assertEqual('dummy-v1:1302659388.0-0-UNKNOWN', revid)
140
133
 
141
134
    def test_commit_bound_lossy_foreign(self):
142
135
        """Attempt a lossy commit to a bzr branch bound to a foreign branch."""
143
136
        test_foreign.register_dummy_foreign_for_test(self)
144
137
        foreign_branch = self.make_branch('foreign',
145
 
                                          format=test_foreign.DummyForeignVcsDirFormat())
 
138
            format=test_foreign.DummyForeignVcsDirFormat())
146
139
        wt = foreign_branch.create_checkout("local")
147
140
        b = wt.branch
148
 
        with open('local/hello', 'w') as f:
149
 
            f.write('hello world')
 
141
        with open('local/hello', 'w') as f: f.write('hello world')
150
142
        wt.add('hello')
151
143
        revid = wt.commit(message='add hello', lossy=True,
152
 
                          timestamp=1302659388, timezone=0)
153
 
        self.assertEqual(b'dummy-v1:1302659388-0-0', revid)
154
 
        self.assertEqual(b'dummy-v1:1302659388-0-0',
155
 
                         foreign_branch.last_revision())
156
 
        self.assertEqual(b'dummy-v1:1302659388-0-0',
157
 
                         wt.branch.last_revision())
 
144
            timestamp=1302659388, timezone=0)
 
145
        self.assertEqual('dummy-v1:1302659388.0-0-0', revid)
 
146
        self.assertEqual('dummy-v1:1302659388.0-0-0',
 
147
            foreign_branch.last_revision())
 
148
        self.assertEqual('dummy-v1:1302659388.0-0-0',
 
149
            wt.branch.last_revision())
158
150
 
159
151
    def test_missing_commit(self):
160
152
        """Test a commit with a missing file"""
161
153
        wt = self.make_branch_and_tree('.')
162
154
        b = wt.branch
163
 
        with open('hello', 'w') as f:
164
 
            f.write('hello world')
 
155
        with open('hello', 'w') as f: f.write('hello world')
165
156
        wt.add(['hello'], [b'hello-id'])
166
157
        wt.commit(message='add hello')
167
158
 
173
164
            reporter.calls)
174
165
 
175
166
        tree = b.repository.revision_tree(b'rev2')
176
 
        self.assertFalse(tree.has_filename('hello'))
 
167
        self.assertFalse(tree.has_id(b'hello-id'))
177
168
 
178
169
    def test_partial_commit_move(self):
179
170
        """Test a partial commit where a file was renamed but not committed.
188
179
        b = wt.branch
189
180
        self.build_tree(['annotate/', 'annotate/foo.py',
190
181
                         'olive/', 'olive/dialog.py'
191
 
                         ])
 
182
                        ])
192
183
        wt.add(['annotate', 'olive', 'annotate/foo.py', 'olive/dialog.py'])
193
184
        wt.commit(message='add files')
194
185
        wt.rename_one("olive/dialog.py", "aaa")
199
190
        """Commit refuses unless there are changes or it's forced."""
200
191
        wt = self.make_branch_and_tree('.')
201
192
        b = wt.branch
202
 
        with open('hello', 'w') as f:
203
 
            f.write('hello')
 
193
        with open('hello', 'w') as f: f.write('hello')
204
194
        wt.add(['hello'])
205
195
        wt.commit(message='add hello')
206
196
        self.assertEqual(b.revno(), 1)
226
216
        """Selective commit in tree with deletions"""
227
217
        wt = self.make_branch_and_tree('.')
228
218
        b = wt.branch
229
 
        with open('hello', 'w') as f:
230
 
            f.write('hello')
231
 
        with open('buongia', 'w') as f:
232
 
            f.write('buongia')
 
219
        with open('hello', 'w') as f: f.write('hello')
 
220
        with open('buongia', 'w') as f: f.write('buongia')
233
221
        wt.add(['hello', 'buongia'],
234
 
               [b'hello-id', b'buongia-id'])
 
222
              [b'hello-id', b'buongia-id'])
235
223
        wt.commit(message='add files',
236
 
                  rev_id=b'test@rev-1')
 
224
                 rev_id=b'test@rev-1')
237
225
 
238
226
        os.remove('hello')
239
 
        with open('buongia', 'w') as f:
240
 
            f.write('new text')
 
227
        with open('buongia', 'w') as f: f.write('new text')
241
228
        wt.commit(message='update text',
242
 
                  specific_files=['buongia'],
243
 
                  allow_pointless=False,
244
 
                  rev_id=b'test@rev-2')
 
229
                 specific_files=['buongia'],
 
230
                 allow_pointless=False,
 
231
                 rev_id=b'test@rev-2')
245
232
 
246
233
        wt.commit(message='remove hello',
247
 
                  specific_files=['hello'],
248
 
                  allow_pointless=False,
249
 
                  rev_id=b'test@rev-3')
 
234
                 specific_files=['hello'],
 
235
                 allow_pointless=False,
 
236
                 rev_id=b'test@rev-3')
250
237
 
251
238
        eq = self.assertEqual
252
239
        eq(b.revno(), 3)
273
260
        tree.commit(message='one', rev_id=b'test@rev-1', allow_pointless=False)
274
261
 
275
262
        tree.rename_one('hello', 'fruity')
276
 
        tree.commit(message='renamed', rev_id=b'test@rev-2',
277
 
                    allow_pointless=False)
 
263
        tree.commit(message='renamed', rev_id=b'test@rev-2', allow_pointless=False)
278
264
 
279
265
        eq = self.assertEqual
280
266
        tree1 = b.repository.revision_tree(b'test@rev-1')
281
267
        tree1.lock_read()
282
268
        self.addCleanup(tree1.unlock)
283
 
        eq(tree1.id2path(b'hello-id'), 'hello')
 
269
        eq(tree1.id2path(b'hello-id'), b'hello')
284
270
        eq(tree1.get_file_text('hello'), b'contents of hello\n')
285
271
        self.assertFalse(tree1.has_filename('fruity'))
286
272
        self.check_tree_shape(tree1, ['hello'])
329
315
        wt.lock_read()
330
316
        try:
331
317
            self.check_tree_shape(wt,
332
 
                                  ['a/', 'a/hello', 'a/b/'])
 
318
                                       ['a/', 'a/hello', 'a/b/'])
333
319
            self.check_tree_shape(b.repository.revision_tree(r3),
334
 
                                  ['a/', 'a/hello', 'a/b/'])
 
320
                                       ['a/', 'a/hello', 'a/b/'])
335
321
        finally:
336
322
            wt.unlock()
337
323
 
353
339
        """Commit with a removed file"""
354
340
        wt = self.make_branch_and_tree('.')
355
341
        b = wt.branch
356
 
        with open('hello', 'w') as f:
357
 
            f.write('hello world')
 
342
        with open('hello', 'w') as f: f.write('hello world')
358
343
        wt.add(['hello'], [b'hello-id'])
359
344
        wt.commit(message='add hello')
360
345
        wt.remove('hello')
361
346
        wt.commit('removed hello', rev_id=b'rev2')
362
347
 
363
348
        tree = b.repository.revision_tree(b'rev2')
364
 
        self.assertFalse(tree.has_filename('hello'))
 
349
        self.assertFalse(tree.has_id(b'hello-id'))
365
350
 
366
351
    def test_committed_ancestry(self):
367
352
        """Test commit appends revisions to ancestry."""
369
354
        b = wt.branch
370
355
        rev_ids = []
371
356
        for i in range(4):
372
 
            with open('hello', 'w') as f:
373
 
                f.write((str(i) * 4) + '\n')
 
357
            with open('hello', 'w') as f: f.write((str(i) * 4) + '\n')
374
358
            if i == 0:
375
359
                wt.add(['hello'], [b'hello-id'])
376
 
            rev_id = b'test@rev-%d' % (i + 1)
 
360
            rev_id = 'test@rev-%d' % (i+1)
377
361
            rev_ids.append(rev_id)
378
 
            wt.commit(message='rev %d' % (i + 1),
379
 
                      rev_id=rev_id)
 
362
            wt.commit(message='rev %d' % (i+1),
 
363
                     rev_id=rev_id)
380
364
        for i in range(4):
381
 
            self.assertThat(rev_ids[:i + 1],
382
 
                            MatchesAncestry(b.repository, rev_ids[i]))
 
365
            self.assertThat(rev_ids[:i+1],
 
366
                MatchesAncestry(b.repository, rev_ids[i]))
383
367
 
384
368
    def test_commit_new_subdir_child_selective(self):
385
369
        wt = self.make_branch_and_tree('.')
386
370
        b = wt.branch
387
371
        self.build_tree(['dir/', 'dir/file1', 'dir/file2'])
388
372
        wt.add(['dir', 'dir/file1', 'dir/file2'],
389
 
               [b'dirid', b'file1id', b'file2id'])
 
373
              [b'dirid', b'file1id', b'file2id'])
390
374
        wt.commit('dir/file1', specific_files=['dir/file1'], rev_id=b'1')
391
375
        inv = b.repository.get_inventory(b'1')
392
376
        self.assertEqual(b'1', inv.get_entry(b'dirid').revision)
399
383
        from ..errors import StrictCommitFailed
400
384
        wt = self.make_branch_and_tree('.')
401
385
        b = wt.branch
402
 
        with open('hello', 'w') as f:
403
 
            f.write('hello world')
 
386
        with open('hello', 'w') as f: f.write('hello world')
404
387
        wt.add('hello')
405
 
        with open('goodbye', 'w') as f:
406
 
            f.write('goodbye cruel world!')
 
388
        with open('goodbye', 'w') as f: f.write('goodbye cruel world!')
407
389
        self.assertRaises(StrictCommitFailed, wt.commit,
408
 
                          message='add hello but not goodbye', strict=True)
 
390
            message='add hello but not goodbye', strict=True)
409
391
 
410
392
    def test_strict_commit_without_unknowns(self):
411
393
        """Try and commit with no unknown files and strict = True,
412
394
        should work."""
413
395
        wt = self.make_branch_and_tree('.')
414
396
        b = wt.branch
415
 
        with open('hello', 'w') as f:
416
 
            f.write('hello world')
 
397
        with open('hello', 'w') as f: f.write('hello world')
417
398
        wt.add('hello')
418
399
        wt.commit(message='add hello', strict=True)
419
400
 
421
402
        """Try and commit with unknown files and strict = False, should work."""
422
403
        wt = self.make_branch_and_tree('.')
423
404
        b = wt.branch
424
 
        with open('hello', 'w') as f:
425
 
            f.write('hello world')
 
405
        with open('hello', 'w') as f: f.write('hello world')
426
406
        wt.add('hello')
427
 
        with open('goodbye', 'w') as f:
428
 
            f.write('goodbye cruel world!')
 
407
        with open('goodbye', 'w') as f: f.write('goodbye cruel world!')
429
408
        wt.commit(message='add hello but not goodbye', strict=False)
430
409
 
431
410
    def test_nonstrict_commit_without_unknowns(self):
433
412
        should work."""
434
413
        wt = self.make_branch_and_tree('.')
435
414
        b = wt.branch
436
 
        with open('hello', 'w') as f:
437
 
            f.write('hello world')
 
415
        with open('hello', 'w') as f: f.write('hello world')
438
416
        wt.add('hello')
439
417
        wt.commit(message='add hello', strict=False)
440
418
 
445
423
        wt = self.make_branch_and_tree('.')
446
424
        branch = wt.branch
447
425
        wt.commit("base", allow_pointless=True, rev_id=b'A')
448
 
        self.assertFalse(branch.repository.has_signature_for_revision_id(b'A'))
 
426
        self.assertFalse(branch.repository.has_signature_for_revision_id('A'))
449
427
        try:
450
 
            from ..bzr.testament import Testament
 
428
            from ..testament import Testament
451
429
            # monkey patch gpg signing mechanism
452
430
            breezy.gpg.GPGStrategy = breezy.gpg.LoopbackGPGStrategy
453
431
            conf = config.MemoryStack(b'''
456
434
            commit.Commit(config_stack=conf).commit(
457
435
                message="base", allow_pointless=True, rev_id=b'B',
458
436
                working_tree=wt)
459
 
 
460
437
            def sign(text):
461
438
                return breezy.gpg.LoopbackGPGStrategy(None).sign(
462
 
                    text, breezy.gpg.MODE_CLEAR)
 
439
                        text, breezy.gpg.MODE_CLEAR)
463
440
            self.assertEqual(sign(Testament.from_revision(branch.repository,
464
 
                                                          b'B').as_short_text()),
465
 
                             branch.repository.get_signature_text(b'B'))
 
441
                                                          'B').as_short_text()),
 
442
                             branch.repository.get_signature_text('B'))
466
443
        finally:
467
444
            breezy.gpg.GPGStrategy = oldstrategy
468
445
 
497
474
        wt = self.make_branch_and_tree('.')
498
475
        branch = wt.branch
499
476
        calls = []
500
 
 
501
477
        def called(branch, rev_id):
502
478
            calls.append('called')
503
479
        breezy.ahook = called
504
480
        try:
505
481
            conf = config.MemoryStack(b'post_commit=breezy.ahook breezy.ahook')
506
482
            commit.Commit(config_stack=conf).commit(
507
 
                message="base", allow_pointless=True, rev_id=b'A',
508
 
                working_tree=wt)
 
483
                message = "base", allow_pointless=True, rev_id=b'A',
 
484
                working_tree = wt)
509
485
            self.assertEqual(['called', 'called'], calls)
510
486
        finally:
511
487
            del breezy.ahook
518
494
        self.assertEqual(wt.branch.revno(), 1)
519
495
        self.assertEqual({},
520
496
                         wt.branch.repository.get_revision(
521
 
            wt.branch.last_revision()).properties)
 
497
                            wt.branch.last_revision()).properties)
522
498
 
523
499
    def test_safe_master_lock(self):
524
500
        os.mkdir('master')
542
518
        bound_tree = self.make_branch_and_tree('bound')
543
519
        bound_tree.branch.bind(master_branch)
544
520
 
545
 
        self.build_tree_contents(
546
 
            [('bound/content_file', b'initial contents\n')])
 
521
        self.build_tree_contents([('bound/content_file', b'initial contents\n')])
547
522
        bound_tree.add(['content_file'])
548
523
        bound_tree.commit(message='woo!')
549
524
 
553
528
        # do a commit to the other branch changing the content file so
554
529
        # that our commit after merging will have a merged revision in the
555
530
        # content file history.
556
 
        self.build_tree_contents(
557
 
            [('other/content_file', b'change in other\n')])
 
531
        self.build_tree_contents([('other/content_file', b'change in other\n')])
558
532
        other_tree.commit('change in other')
559
533
 
560
534
        # do a merge into the bound branch from other, and then change the
561
535
        # content file locally to force a new revision (rather than using the
562
536
        # revision from other). This forces extra processing in commit.
563
537
        bound_tree.merge_from_branch(other_tree.branch)
564
 
        self.build_tree_contents(
565
 
            [('bound/content_file', b'change in bound\n')])
 
538
        self.build_tree_contents([('bound/content_file', b'change in bound\n')])
566
539
 
567
540
        # before #34959 was fixed, this failed with 'revision not present in
568
541
        # weave' when trying to implicitly push from the bound branch to the master
604
577
            other_tree.rename_one('dirtorename', 'renameddir')
605
578
            other_tree.rename_one('dirtoreparent', 'renameddir/reparenteddir')
606
579
            other_tree.rename_one('filetorename', 'renamedfile')
607
 
            other_tree.rename_one(
608
 
                'filetoreparent', 'renameddir/reparentedfile')
 
580
            other_tree.rename_one('filetoreparent', 'renameddir/reparentedfile')
609
581
            other_tree.remove(['dirtoremove', 'filetoremove'])
610
582
            self.build_tree_contents([
611
583
                ('other/newdir/', ),
679
651
        finally:
680
652
            basis.unlock()
681
653
 
682
 
    def test_unsupported_symlink_commit(self):
683
 
        self.requireFeature(SymlinkFeature)
684
 
        tree = self.make_branch_and_tree('.')
685
 
        self.build_tree(['hello'])
686
 
        tree.add('hello')
687
 
        tree.commit('added hello', rev_id=b'hello_id')
688
 
        os.symlink('hello', 'foo')
689
 
        tree.add('foo')
690
 
        tree.commit('added foo', rev_id=b'foo_id')
691
 
        log = BytesIO()
692
 
        trace.push_log_file(log)
693
 
        os_symlink = getattr(os, 'symlink', None)
694
 
        os.symlink = None
695
 
        try:
696
 
            # At this point as bzr thinks symlinks are not supported
697
 
            # we should get a warning about symlink foo and bzr should
698
 
            # not think its removed.
699
 
            os.unlink('foo')
700
 
            self.build_tree(['world'])
701
 
            tree.add('world')
702
 
            tree.commit('added world', rev_id=b'world_id')
703
 
        finally:
704
 
            if os_symlink:
705
 
                os.symlink = os_symlink
706
 
        self.assertContainsRe(
707
 
            log.getvalue(),
708
 
            b'Ignoring "foo" as symlinks are not '
709
 
            b'supported on this filesystem\\.')
710
 
 
711
654
    def test_commit_kind_changes(self):
712
655
        self.requireFeature(SymlinkFeature)
713
656
        tree = self.make_branch_and_tree('.')
803
746
        cb = self.Callback(u'commit 2', self)
804
747
        repository = tree.branch.repository
805
748
        # simulate network failure
806
 
 
807
749
        def raise_(self, arg, arg2, arg3=None, arg4=None):
808
750
            raise errors.NoSuchFile('foo')
809
751
        repository.add_inventory = raise_
820
762
        self.build_tree(['foo/bar', 'foo/baz'])
821
763
        tree.add(['bar', 'baz'])
822
764
        err = self.assertRaises(CannotCommitSelectedFileMerge,
823
 
                                tree.commit, 'commit 2', specific_files=['bar', 'baz'])
 
765
            tree.commit, 'commit 2', specific_files=['bar', 'baz'])
824
766
        self.assertEqual(['bar', 'baz'], err.files)
825
767
        self.assertEqual('Selected-file commit of merges is not supported'
826
768
                         ' yet: files bar, baz', str(err))
870
812
    def test_multiple_authors(self):
871
813
        tree = self.make_branch_and_tree('foo')
872
814
        rev_id = tree.commit('commit 1',
873
 
                             authors=['John Doe <jdoe@example.com>',
874
 
                                      'Jane Rey <jrey@example.com>'])
 
815
                authors=['John Doe <jdoe@example.com>',
 
816
                         'Jane Rey <jrey@example.com>'])
875
817
        rev = tree.branch.repository.get_revision(rev_id)
876
818
        self.assertEqual('John Doe <jdoe@example.com>\n'
877
 
                         'Jane Rey <jrey@example.com>', rev.properties['authors'])
 
819
                'Jane Rey <jrey@example.com>', rev.properties['authors'])
878
820
        self.assertFalse('author' in rev.properties)
879
821
 
880
822
    def test_author_with_newline_rejected(self):
881
823
        tree = self.make_branch_and_tree('foo')
882
824
        self.assertRaises(AssertionError, tree.commit, 'commit 1',
883
 
                          authors=['John\nDoe <jdoe@example.com>'])
 
825
                authors=['John\nDoe <jdoe@example.com>'])
884
826
 
885
827
    def test_commit_with_checkout_and_branch_sharing_repo(self):
886
828
        repo = self.make_repository('repo', shared=True)
895
837
 
896
838
    def test_add_file_not_excluded(self):
897
839
        changes = [
898
 
            TreeChange(
899
 
                'fid', (None, 'newpath'),
900
 
                0, (False, False), ('pid', 'pid'), ('newpath', 'newpath'),
901
 
                ('file', 'file'), (True, True))]
902
 
        self.assertEqual(changes, list(
903
 
            filter_excluded(changes, ['otherpath'])))
 
840
            ('fid', (None, 'newpath'),
 
841
             0, (False, False), ('pid', 'pid'), ('newpath', 'newpath'),
 
842
             ('file', 'file'), (True, True))]
 
843
        self.assertEqual(changes, list(filter_excluded(changes, ['otherpath'])))
904
844
 
905
845
    def test_add_file_excluded(self):
906
846
        changes = [
907
 
            TreeChange(
908
 
                'fid', (None, 'newpath'),
909
 
                0, (False, False), ('pid', 'pid'), ('newpath', 'newpath'),
910
 
                ('file', 'file'), (True, True))]
 
847
            ('fid', (None, 'newpath'),
 
848
             0, (False, False), ('pid', 'pid'), ('newpath', 'newpath'),
 
849
             ('file', 'file'), (True, True))]
911
850
        self.assertEqual([], list(filter_excluded(changes, ['newpath'])))
912
851
 
913
852
    def test_delete_file_excluded(self):
914
853
        changes = [
915
 
            TreeChange(
916
 
                'fid', ('somepath', None),
917
 
                0, (False, None), ('pid', None), ('newpath', None),
918
 
                ('file', None), (True, None))]
 
854
            ('fid', ('somepath', None),
 
855
             0, (False, None), ('pid', None), ('newpath', None),
 
856
             ('file', None), (True, None))]
919
857
        self.assertEqual([], list(filter_excluded(changes, ['somepath'])))
920
858
 
921
859
    def test_move_from_or_to_excluded(self):
922
860
        changes = [
923
 
            TreeChange(
924
 
                'fid', ('oldpath', 'newpath'),
925
 
                0, (False, False), ('pid', 'pid'), ('oldpath', 'newpath'),
926
 
                ('file', 'file'), (True, True))]
 
861
            ('fid', ('oldpath', 'newpath'),
 
862
             0, (False, False), ('pid', 'pid'), ('oldpath', 'newpath'),
 
863
             ('file', 'file'), (True, True))]
927
864
        self.assertEqual([], list(filter_excluded(changes, ['oldpath'])))
928
865
        self.assertEqual([], list(filter_excluded(changes, ['newpath'])))