/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/commands/test_commit.py

  • Committer: Martin Pool
  • Date: 2007-10-03 08:06:44 UTC
  • mto: This revision was merged to the branch mainline in revision 2901.
  • Revision ID: mbp@sourcefrog.net-20071003080644-oivy0gkg98sex0ed
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).

Add new LockFailed, which doesn't imply that we failed to get it because of
contention.  Raise this if we fail to create the pending or lock directories
because of Transport errors.

UnlockableTransport is not an internal error.

ReadOnlyLockError has a message which didn't match its name or usage; it's now
deprecated and callers are updated to use LockFailed which is more appropriate.

Add zero_ninetytwo deprecation symbol.

Unify assertMatchesRe with TestCase.assertContainsRe.

When the constructor is deprecated, just say that the class is deprecated, not
the __init__ method - this works better with applyDeprecated in tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/commands/test_commit.py
1
 
# Copyright (C) 2007-2010 Canonical Ltd
2
 
#
3
 
# This program is free software; you can redistribute it and/or modify
4
 
# it under the terms of the GNU General Public License as published by
5
 
# the Free Software Foundation; either version 2 of the License, or
6
 
# (at your option) any later version.
7
 
#
8
 
# This program is distributed in the hope that it will be useful,
9
 
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
 
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
 
# GNU General Public License for more details.
12
 
#
13
 
# You should have received a copy of the GNU General Public License
14
 
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
 
 
17
 
import os
18
 
from bzrlib import (
19
 
    branch,
20
 
    builtins,
21
 
    errors,
22
 
    )
23
 
from bzrlib.tests import transport_util
24
 
 
25
 
 
26
 
class TestCommitWithBoundBranch(
27
 
    transport_util.TestCaseWithConnectionHookedTransport):
28
 
 
29
 
    def setUp(self):
30
 
        super(TestCommitWithBoundBranch, self).setUp()
31
 
        self.master_wt = self.make_branch_and_tree('master')
32
 
        self.local_wt = self.make_branch_and_tree('local')
33
 
 
34
 
        master_branch = branch.Branch.open(self.get_url('master'))
35
 
        self.local_wt.branch.bind(master_branch)
36
 
 
37
 
    def test_commit_mine_modified(self):
38
 
 
39
 
        self.start_logging_connections()
40
 
 
41
 
        commit = builtins.cmd_commit()
42
 
        # commit do not provide a directory parameter, we have to change dir
43
 
        # manually
44
 
        os.chdir('local')
45
 
        commit.run(message=u'empty commit', unchanged=True)
46
 
        self.assertEquals(1, len(self.connections))
47
 
 
48
 
    def test_commit_both_modified(self):
49
 
        self.master_wt.commit('empty commit on master')
50
 
        self.start_logging_connections()
51
 
 
52
 
        commit = builtins.cmd_commit()
53
 
        # commit do not provide a directory parameter, we have to change dir
54
 
        # manually
55
 
        os.chdir('local')
56
 
        self.assertRaises(errors.BoundBranchOutOfDate, commit.run,
57
 
                          message=u'empty commit', unchanged=True)
58
 
        self.assertEquals(1, len(self.connections))
59
 
 
60
 
    def test_commit_local(self):
61
 
        """Commits with --local should not connect to the master!"""
62
 
        self.start_logging_connections()
63
 
 
64
 
        commit = builtins.cmd_commit()
65
 
        # commit do not provide a directory parameter, we have to change dir
66
 
        # manually
67
 
        os.chdir('local')
68
 
        commit.run(message=u'empty commit', unchanged=True, local=True)
69
 
 
70
 
        #it shouldn't open any connections
71
 
        self.assertEquals(0, len(self.connections))