← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/blackbox/test_commit.py

  • Committer: John Arbash Meinel
  • Date: 2009-12-02 17:59:15 UTC
  • mto: This revision was merged to the branch mainline in revision 4854.
  • Revision ID: john@arbash-meinel.com-20091202175915-me81n1ovkuml4ake
Explicitly call stream.close() in the index code.

This isn't in a finally block, but it is 'good enough'. At least .close is
being called rather than waiting for the garbage collector to run.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2009 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
107
107
                              'modified hello\.txt\n'
108
108
                              'Committed revision 2\.\n$')
109
109
 
110
 
    def test_warn_about_forgotten_commit_message(self):
111
 
        """Test that the lack of -m parameter is caught"""
112
 
        wt = self.make_branch_and_tree('.')
113
 
        self.build_tree(['one', 'two'])
114
 
        wt.add(['two'])
115
 
        out, err = self.run_bzr('commit -m one two')
116
 
        self.assertContainsRe(err, "The commit message is a file name")
117
 
 
118
110
    def test_verbose_commit_renamed(self):
119
111
        # Verbose commit of renamed file should say so
120
112
        wt = self.prepare_simple_history()
343
335
        trunk = self.make_branch_and_tree('trunk')
344
336
 
345
337
        u1 = trunk.branch.create_checkout('u1')
346
 
        self.build_tree_contents([('u1/hosts', 'initial contents\n')])
 
338
        self.build_tree_contents([('u1/hosts', 'initial contents')])
347
339
        u1.add('hosts')
348
340
        self.run_bzr('commit -m add-hosts u1')
349
341
 
350
342
        u2 = trunk.branch.create_checkout('u2')
351
 
        self.build_tree_contents([('u2/hosts', 'altered in u2\n')])
 
343
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
352
344
        self.run_bzr('commit -m checkin-from-u2 u2')
353
345
 
354
346
        # make an offline commits
355
 
        self.build_tree_contents([('u1/hosts', 'first offline change in u1\n')])
 
347
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
356
348
        self.run_bzr('commit -m checkin-offline --local u1')
357
349
 
358
350
        # now try to pull in online work from u2, and then commit our offline
359
351
        # work as a merge
360
352
        # retcode 1 as we expect a text conflict
361
353
        self.run_bzr('update u1', retcode=1)
362
 
        self.assertFileEqual('''\
363
 
<<<<<<< TREE
364
 
first offline change in u1
365
 
=======
366
 
altered in u2
367
 
>>>>>>> MERGE-SOURCE
368
 
''',
369
 
                             'u1/hosts')
370
 
 
371
354
        self.run_bzr('resolved u1/hosts')
372
355
        # add a text change here to represent resolving the merge conflicts in
373
356
        # favour of a new version of the file not identical to either the u1
675
658
        self.assertContainsRe(err,
676
659
            r'^bzr: ERROR: Cannot lock.*readonly transport')
677
660
 
678
 
    def setup_editor(self):
 
661
    def test_commit_hook_template(self):
679
662
        # Test that commit template hooks work
 
663
        def restoreDefaults():
 
664
            msgeditor.hooks['commit_message_template'] = []
 
665
            osutils.set_or_unset_env('BZR_EDITOR', default_editor)
680
666
        if sys.platform == "win32":
681
667
            f = file('fed.bat', 'w')
682
668
            f.write('@rem dummy fed')
683
669
            f.close()
684
 
            osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
 
670
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "fed.bat")
685
671
        else:
686
672
            f = file('fed.sh', 'wb')
687
673
            f.write('#!/bin/sh\n')
688
674
            f.close()
689
675
            os.chmod('fed.sh', 0755)
690
 
            osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
691
 
 
692
 
    def setup_commit_with_template(self):
693
 
        self.setup_editor()
 
676
            default_editor = osutils.set_or_unset_env('BZR_EDITOR', "./fed.sh")
 
677
        self.addCleanup(restoreDefaults)
694
678
        msgeditor.hooks.install_named_hook("commit_message_template",
695
679
                lambda commit_obj, msg: "save me some typing\n", None)
696
680
        tree = self.make_branch_and_tree('tree')
697
681
        self.build_tree(['tree/hello.txt'])
698
682
        tree.add('hello.txt')
699
 
        return tree
700
 
 
701
 
    def test_commit_hook_template_accepted(self):
702
 
        tree = self.setup_commit_with_template()
703
 
        out, err = self.run_bzr("commit tree/hello.txt", stdin="y\n")
 
683
        out, err = self.run_bzr("commit tree/hello.txt")
704
684
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
705
685
        self.assertEqual('save me some typing\n', last_rev.message)
706
 
 
707
 
    def test_commit_hook_template_rejected(self):
708
 
        tree = self.setup_commit_with_template()
709
 
        expected = tree.last_revision()
710
 
        out, err = self.run_bzr_error(["empty commit message"],
711
 
            "commit tree/hello.txt", stdin="n\n")
712
 
        self.assertEqual(expected, tree.last_revision())