13
13
# You should have received a copy of the GNU General Public License
14
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
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
17
17
from StringIO import StringIO
20
from bzrlib.branch import Branch
21
import bzrlib.errors as errors
22
25
from bzrlib.errors import BzrBadParameterNotString, NoSuchFile, ReadOnlyError
23
26
from bzrlib.lockable_files import LockableFiles, TransportLock
24
from bzrlib.lockdir import LockDir
25
from bzrlib.tests import TestCaseInTempDir
27
from bzrlib.symbol_versioning import (
30
from bzrlib.tests import (
34
from bzrlib.tests.test_smart import TestCaseWithSmartMedium
26
35
from bzrlib.tests.test_transactions import DummyWeave
27
36
from bzrlib.transactions import (PassThroughTransaction,
28
37
ReadOnlyTransaction,
34
43
# these tests are applied in each parameterized suite for LockableFiles
45
# they use an old style of parameterization, but we want to remove this class
46
# so won't modernize them now. - mbp 20080430
35
47
class _TestLockableFiles_mixin(object):
37
def test_read_write(self):
38
self.assertRaises(NoSuchFile, self.lockable.get, 'foo')
39
self.assertRaises(NoSuchFile, self.lockable.get_utf8, 'foo')
40
self.lockable.lock_write()
42
unicode_string = u'bar\u1234'
43
self.assertEqual(4, len(unicode_string))
44
byte_string = unicode_string.encode('utf-8')
45
self.assertEqual(6, len(byte_string))
46
self.assertRaises(UnicodeEncodeError, self.lockable.put, 'foo',
47
StringIO(unicode_string))
48
self.lockable.put('foo', StringIO(byte_string))
49
self.assertEqual(byte_string,
50
self.lockable.get('foo').read())
51
self.assertEqual(unicode_string,
52
self.lockable.get_utf8('foo').read())
53
self.assertRaises(BzrBadParameterNotString,
54
self.lockable.put_utf8,
56
StringIO(unicode_string)
58
self.lockable.put_utf8('bar', unicode_string)
59
self.assertEqual(unicode_string,
60
self.lockable.get_utf8('bar').read())
61
self.assertEqual(byte_string,
62
self.lockable.get('bar').read())
64
self.lockable.unlock()
67
self.lockable.lock_read()
69
self.assertRaises(ReadOnlyError, self.lockable.put, 'foo',
70
StringIO('bar\u1234'))
72
self.lockable.unlock()
74
49
def test_transactions(self):
75
50
self.assertIs(self.lockable.get_transaction().__class__,
76
51
PassThroughTransaction)
122
96
self.assertRaises(errors.LockBroken, self.lockable.unlock)
123
97
self.assertFalse(self.lockable.is_locked())
126
# This method of adapting tests to parameters is different to
127
# the TestProviderAdapters used elsewhere, but seems simpler for this
99
def test_lock_write_returns_None_refuses_token(self):
100
token = self.lockable.lock_write()
101
self.addCleanup(self.lockable.unlock)
102
if token is not None:
103
# This test does not apply, because this lockable supports
105
raise TestNotApplicable("%r uses tokens" % (self.lockable,))
106
self.assertRaises(errors.TokenLockingNotSupported,
107
self.lockable.lock_write, token='token')
109
def test_lock_write_returns_token_when_given_token(self):
110
token = self.lockable.lock_write()
111
self.addCleanup(self.lockable.unlock)
113
# This test does not apply, because this lockable refuses
116
new_lockable = self.get_lockable()
117
token_from_new_lockable = new_lockable.lock_write(token=token)
118
self.addCleanup(new_lockable.unlock)
119
self.assertEqual(token, token_from_new_lockable)
121
def test_lock_write_raises_on_token_mismatch(self):
122
token = self.lockable.lock_write()
123
self.addCleanup(self.lockable.unlock)
125
# This test does not apply, because this lockable refuses
128
different_token = token + 'xxx'
129
# Re-using the same lockable instance with a different token will
130
# raise TokenMismatch.
131
self.assertRaises(errors.TokenMismatch,
132
self.lockable.lock_write, token=different_token)
133
# A separate instance for the same lockable will also raise
135
# This detects the case where a caller claims to have a lock (via
136
# the token) for an external resource, but doesn't (the token is
137
# different). Clients need a separate lock object to make sure the
138
# external resource is probed, whereas the existing lock object
140
new_lockable = self.get_lockable()
141
self.assertRaises(errors.TokenMismatch,
142
new_lockable.lock_write, token=different_token)
144
def test_lock_write_with_matching_token(self):
145
# If the token matches, so no exception is raised by lock_write.
146
token = self.lockable.lock_write()
147
self.addCleanup(self.lockable.unlock)
149
# This test does not apply, because this lockable refuses
152
# The same instance will accept a second lock_write if the specified
154
self.lockable.lock_write(token=token)
155
self.lockable.unlock()
156
# Calling lock_write on a new instance for the same lockable will
158
new_lockable = self.get_lockable()
159
new_lockable.lock_write(token=token)
160
new_lockable.unlock()
162
def test_unlock_after_lock_write_with_token(self):
163
# If lock_write did not physically acquire the lock (because it was
164
# passed a token), then unlock should not physically release it.
165
token = self.lockable.lock_write()
166
self.addCleanup(self.lockable.unlock)
168
# This test does not apply, because this lockable refuses
171
new_lockable = self.get_lockable()
172
new_lockable.lock_write(token=token)
173
new_lockable.unlock()
174
self.assertTrue(self.lockable.get_physical_lock_status())
176
def test_lock_write_with_token_fails_when_unlocked(self):
177
# Lock and unlock to get a superficially valid token. This mimics a
178
# likely programming error, where a caller accidentally tries to lock
179
# with a token that is no longer valid (because the original lock was
181
token = self.lockable.lock_write()
182
self.lockable.unlock()
184
# This test does not apply, because this lockable refuses
188
self.assertRaises(errors.TokenMismatch,
189
self.lockable.lock_write, token=token)
191
def test_lock_write_reenter_with_token(self):
192
token = self.lockable.lock_write()
195
# This test does not apply, because this lockable refuses
198
# Relock with a token.
199
token_from_reentry = self.lockable.lock_write(token=token)
201
self.assertEqual(token, token_from_reentry)
203
self.lockable.unlock()
205
self.lockable.unlock()
206
# The lock should be unlocked on disk. Verify that with a new lock
208
new_lockable = self.get_lockable()
209
# Calling lock_write now should work, rather than raise LockContention.
210
new_lockable.lock_write()
211
new_lockable.unlock()
213
def test_second_lock_write_returns_same_token(self):
214
first_token = self.lockable.lock_write()
216
if first_token is None:
217
# This test does not apply, because this lockable refuses
220
# Relock the already locked lockable. It should return the same
222
second_token = self.lockable.lock_write()
224
self.assertEqual(first_token, second_token)
226
self.lockable.unlock()
228
self.lockable.unlock()
230
def test_leave_in_place(self):
231
token = self.lockable.lock_write()
234
# This test does not apply, because this lockable refuses
237
self.lockable.leave_in_place()
239
self.lockable.unlock()
240
# At this point, the lock is still in place on disk
241
self.assertRaises(errors.LockContention, self.lockable.lock_write)
242
# But should be relockable with a token.
243
self.lockable.lock_write(token=token)
244
self.lockable.unlock()
245
# Cleanup: we should still be able to get the lock, but we restore the
246
# behavior to clearing the lock when unlocking.
247
self.lockable.lock_write(token=token)
248
self.lockable.dont_leave_in_place()
249
self.lockable.unlock()
251
def test_dont_leave_in_place(self):
252
token = self.lockable.lock_write()
255
# This test does not apply, because this lockable refuses
258
self.lockable.leave_in_place()
260
self.lockable.unlock()
261
# At this point, the lock is still in place on disk.
262
# Acquire the existing lock with the token, and ask that it is removed
263
# when this object unlocks, and unlock to trigger that removal.
264
new_lockable = self.get_lockable()
265
new_lockable.lock_write(token=token)
266
new_lockable.dont_leave_in_place()
267
new_lockable.unlock()
268
# At this point, the lock is no longer on disk, so we can lock it.
269
third_lockable = self.get_lockable()
270
third_lockable.lock_write()
271
third_lockable.unlock()
274
# This method of adapting tests to parameters is different to
275
# the TestProviderAdapters used elsewhere, but seems simpler for this
129
277
class TestLockableFiles_TransportLock(TestCaseInTempDir,
130
278
_TestLockableFiles_mixin):
133
super(TestLockableFiles_TransportLock, self).setUp()
281
TestCaseInTempDir.setUp(self)
134
282
transport = get_transport('.')
135
283
transport.mkdir('.bzr')
136
284
self.sub_transport = transport.clone('.bzr')
141
289
super(TestLockableFiles_TransportLock, self).tearDown()
142
290
# free the subtransport so that we do not get a 5 second
143
291
# timeout due to the SFTP connection cache.
144
del self.sub_transport
293
del self.sub_transport
294
except AttributeError:
146
297
def get_lockable(self):
147
298
return LockableFiles(self.sub_transport, 'my-lock', TransportLock)
150
301
class TestLockableFiles_LockDir(TestCaseInTempDir,
151
302
_TestLockableFiles_mixin):
152
303
"""LockableFile tests run with LockDir underneath"""
155
super(TestLockableFiles_LockDir, self).setUp()
306
TestCaseInTempDir.setUp(self)
156
307
self.transport = get_transport('.')
157
308
self.lockable = self.get_lockable()
158
# the lock creation here sets mode - test_permissions on branch
159
# tests that implicitly, but it might be a good idea to factor
309
# the lock creation here sets mode - test_permissions on branch
310
# tests that implicitly, but it might be a good idea to factor
160
311
# out the mode checking logic and have it applied to loackable files
161
312
# directly. RBC 20060418
162
313
self.lockable.create_lock()
164
315
def get_lockable(self):
165
return LockableFiles(self.transport, 'my-lock', LockDir)
316
return LockableFiles(self.transport, 'my-lock', lockdir.LockDir)
167
318
def test_lock_created(self):
168
319
self.assertTrue(self.transport.has('my-lock'))
172
323
self.assertFalse(self.transport.has('my-lock/held/info'))
173
324
self.assertTrue(self.transport.has('my-lock'))
176
# TODO: Test the lockdir inherits the right file and directory permissions
177
# from the LockableFiles.
326
def test__file_modes(self):
327
self.transport.mkdir('readonly')
328
osutils.make_readonly('readonly')
329
lockable = LockableFiles(self.transport.clone('readonly'), 'test-lock',
331
# The directory mode should be read-write-execute for the current user
332
self.assertEqual(00700, lockable._dir_mode & 00700)
333
# Files should be read-write for the current user
334
self.assertEqual(00600, lockable._file_mode & 00700)
337
class TestLockableFiles_RemoteLockDir(TestCaseWithSmartMedium,
338
_TestLockableFiles_mixin):
339
"""LockableFile tests run with RemoteLockDir on a branch."""
342
TestCaseWithSmartMedium.setUp(self)
343
# can only get a RemoteLockDir with some RemoteObject...
344
# use a branch as thats what we want. These mixin tests test the end
345
# to end behaviour, so stubbing out the backend and simulating would
346
# defeat the purpose. We test the protocol implementation separately
347
# in test_remote and test_smart as usual.
348
b = self.make_branch('foo')
349
self.addCleanup(b.bzrdir.transport.disconnect)
350
self.transport = get_transport('.')
351
self.lockable = self.get_lockable()
353
def get_lockable(self):
354
# getting a new lockable involves opening a new instance of the branch
355
branch = bzrlib.branch.Branch.open(self.get_url('foo'))
356
self.addCleanup(branch.bzrdir.transport.disconnect)
357
return branch.control_files
added
removed
bzrlib/tests/test_lockable_files.py
