~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/per_lock/test_lock.py

  • Committer: Robert Collins
  • Date: 2007-04-23 02:29:35 UTC
  • mfrom: (2441 +trunk)
  • mto: This revision was merged to the branch mainline in revision 2442.
  • Revision ID: robertc@robertcollins.net-20070423022935-9hhongamvk6bfdso
Resolve conflicts with bzr.dev.

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2007 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., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Tests for OS level locks."""
 
18
 
 
19
from bzrlib import (
 
20
    errors,
 
21
    osutils,
 
22
    )
 
23
 
 
24
from bzrlib.tests.per_lock import TestCaseWithLock
 
25
 
 
26
 
 
27
class TestLock(TestCaseWithLock):
 
28
 
 
29
    def setUp(self):
 
30
        super(TestLock, self).setUp()
 
31
        self.build_tree(['a-file'])
 
32
 
 
33
    def test_read_lock(self):
 
34
        """Smoke test for read locks."""
 
35
        a_lock = self.read_lock('a-file')
 
36
        self.addCleanup(a_lock.unlock)
 
37
        # The lock file should be opened for reading
 
38
        txt = a_lock.f.read()
 
39
        self.assertEqual('contents of a-file\n', txt)
 
40
 
 
41
    def test_create_if_needed_read(self):
 
42
        """We will create the file if it doesn't exist yet."""
 
43
        a_lock = self.read_lock('other-file')
 
44
        self.addCleanup(a_lock.unlock)
 
45
        txt = a_lock.f.read()
 
46
        self.assertEqual('', txt)
 
47
 
 
48
    def test_create_if_needed_write(self):
 
49
        """We will create the file if it doesn't exist yet."""
 
50
        a_lock = self.write_lock('other-file')
 
51
        self.addCleanup(a_lock.unlock)
 
52
        txt = a_lock.f.read()
 
53
        self.assertEqual('', txt)
 
54
        a_lock.f.seek(0)
 
55
        a_lock.f.write('foo\n')
 
56
        a_lock.f.seek(0)
 
57
        txt = a_lock.f.read()
 
58
        self.assertEqual('foo\n', txt)
 
59
 
 
60
    def test_readonly_file(self):
 
61
        """If the file is readonly, we can take a read lock.
 
62
 
 
63
        But we shouldn't be able to take a write lock.
 
64
        """
 
65
        osutils.make_readonly('a-file')
 
66
        # Make sure the file is read-only (on all platforms)
 
67
        self.assertRaises(IOError, open, 'a-file', 'rb+')
 
68
        a_lock = self.read_lock('a-file')
 
69
        a_lock.unlock()
 
70
 
 
71
        self.assertRaises(errors.ReadOnlyLockError, self.write_lock, 'a-file')
 
72
 
 
73
    def test_write_lock(self):
 
74
        """Smoke test for write locks."""
 
75
        a_lock = self.write_lock('a-file')
 
76
        self.addCleanup(a_lock.unlock)
 
77
        # You should be able to read and write to the lock file.
 
78
        txt = a_lock.f.read()
 
79
        self.assertEqual('contents of a-file\n', txt)
 
80
        # Win32 requires that you call seek() when switching between a read
 
81
        # operation and a write operation.
 
82
        a_lock.f.seek(0, 2)
 
83
        a_lock.f.write('more content\n')
 
84
        a_lock.f.seek(0)
 
85
        txt = a_lock.f.read()
 
86
        self.assertEqual('contents of a-file\nmore content\n', txt)
 
87
 
 
88
    def test_multiple_read_locks(self):
 
89
        """You can take out more than one read lock on the same file."""
 
90
        a_lock = self.read_lock('a-file')
 
91
        self.addCleanup(a_lock.unlock)
 
92
        b_lock = self.read_lock('a-file')
 
93
        self.addCleanup(b_lock.unlock)
 
94
 
 
95
    def test_multiple_write_locks_exclude(self):
 
96
        """Taking out more than one write lock should fail."""
 
97
        a_lock = self.write_lock('a-file')
 
98
        self.addCleanup(a_lock.unlock)
 
99
        # Taking out a lock on a locked file should raise LockContention
 
100
        self.assertRaises(errors.LockContention, self.write_lock, 'a-file')
 
101
 
 
102
    def _disabled_test_read_then_write_excludes(self):
 
103
        """If a file is read-locked, taking out a write lock should fail."""
 
104
        a_lock = self.read_lock('a-file')
 
105
        self.addCleanup(a_lock.unlock)
 
106
        # Taking out a lock on a locked file should raise LockContention
 
107
        self.assertRaises(errors.LockContention, self.write_lock, 'a-file')
 
108
 
 
109
    def test_read_unlock_write(self):
 
110
        """Make sure that unlocking allows us to lock write"""
 
111
        a_lock = self.read_lock('a-file')
 
112
        a_lock.unlock()
 
113
        a_lock = self.write_lock('a-file')
 
114
        a_lock.unlock()
 
115
 
 
116
    # TODO: jam 20070319 fcntl read locks are not currently fully
 
117
    #       mutually exclusive with write locks. This will be fixed
 
118
    #       in the next release.
 
119
    def _disabled_test_write_then_read_excludes(self):
 
120
        """If a file is write-locked, taking out a read lock should fail.
 
121
 
 
122
        The file is exclusively owned by the write lock, so we shouldn't be
 
123
        able to take out a shared read lock.
 
124
        """
 
125
        a_lock = self.write_lock('a-file')
 
126
        self.addCleanup(a_lock.unlock)
 
127
        # Taking out a lock on a locked file should raise LockContention
 
128
        self.assertRaises(errors.LockContention, self.read_lock, 'a-file')
 
129
 
 
130
    # TODO: jam 20070319 fcntl write locks are not currently fully
 
131
    #       mutually exclusive with read locks. This will be fixed
 
132
    #       in the next release.
 
133
    def _disabled_test_write_unlock_read(self):
 
134
        """If we have removed the write lock, we can grab a read lock."""
 
135
        a_lock = self.write_lock('a-file')
 
136
        a_lock.unlock()
 
137
        a_lock = self.read_lock('a-file')
 
138
        a_lock.unlock()
 
139
 
 
140
    def _disabled_test_multiple_read_unlock_write(self):
 
141
        """We can only grab a write lock if all read locks are done."""
 
142
        a_lock = b_lock = c_lock = None
 
143
        try:
 
144
            a_lock = self.read_lock('a-file')
 
145
            b_lock = self.read_lock('a-file')
 
146
            self.assertRaises(errors.LockContention, self.write_lock, 'a-file')
 
147
            a_lock.unlock()
 
148
            a_lock = None
 
149
            self.assertRaises(errors.LockContention, self.write_lock, 'a-file')
 
150
            b_lock.unlock()
 
151
            b_lock = None
 
152
            c_lock = self.write_lock('a-file')
 
153
            c_lock.unlock()
 
154
            c_lock = None
 
155
        finally:
 
156
            # Cleanup as needed
 
157
            if a_lock is not None:
 
158
                a_lock.unlock()
 
159
            if b_lock is not None:
 
160
                b_lock.unlock()
 
161
            if c_lock is not None:
 
162
                c_lock.unlock()