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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
17
from StringIO import StringIO
20
import bzrlib.errors as errors
21
from bzrlib.errors import BzrBadParameterNotString, NoSuchFile, ReadOnlyError
24
22
from bzrlib.lockable_files import LockableFiles, TransportLock
25
from bzrlib.tests import (
29
from bzrlib.tests.test_smart import TestCaseWithSmartMedium
23
from bzrlib.lockdir import LockDir
24
from bzrlib.tests import TestCaseInTempDir
30
25
from bzrlib.tests.test_transactions import DummyWeave
31
26
from bzrlib.transactions import (PassThroughTransaction,
32
27
ReadOnlyTransaction,
30
from bzrlib.transport import get_transport
37
33
# these tests are applied in each parameterized suite for LockableFiles
39
# they use an old style of parameterization, but we want to remove this class
40
# so won't modernize them now. - mbp 20080430
41
34
class _TestLockableFiles_mixin(object):
36
def test_read_write(self):
37
self.assertRaises(NoSuchFile, self.lockable.get, 'foo')
38
self.assertRaises(NoSuchFile, self.lockable.get_utf8, 'foo')
39
self.lockable.lock_write()
41
unicode_string = u'bar\u1234'
42
self.assertEqual(4, len(unicode_string))
43
byte_string = unicode_string.encode('utf-8')
44
self.assertEqual(6, len(byte_string))
45
self.assertRaises(UnicodeEncodeError, self.lockable.put, 'foo',
46
StringIO(unicode_string))
47
self.lockable.put('foo', StringIO(byte_string))
48
self.assertEqual(byte_string,
49
self.lockable.get('foo').read())
50
self.assertEqual(unicode_string,
51
self.lockable.get_utf8('foo').read())
52
self.assertRaises(BzrBadParameterNotString,
53
self.lockable.put_utf8,
55
StringIO(unicode_string)
57
self.lockable.put_utf8('bar', unicode_string)
58
self.assertEqual(unicode_string,
59
self.lockable.get_utf8('bar').read())
60
self.assertEqual(byte_string,
61
self.lockable.get('bar').read())
62
self.lockable.put_bytes('raw', 'raw\xffbytes')
63
self.assertEqual('raw\xffbytes',
64
self.lockable.get('raw').read())
66
self.lockable.unlock()
69
self.lockable.lock_read()
71
self.assertRaises(ReadOnlyError, self.lockable.put, 'foo',
72
StringIO('bar\u1234'))
74
self.lockable.unlock()
43
76
def test_transactions(self):
44
77
self.assertIs(self.lockable.get_transaction().__class__,
45
78
PassThroughTransaction)
93
127
def test_lock_write_returns_None_refuses_token(self):
94
128
token = self.lockable.lock_write()
95
self.addCleanup(self.lockable.unlock)
97
# This test does not apply, because this lockable supports
99
raise TestNotApplicable("%r uses tokens" % (self.lockable,))
100
self.assertRaises(errors.TokenLockingNotSupported,
101
self.lockable.lock_write, token='token')
130
if token is not None:
131
# This test does not apply, because this lockable supports
134
self.assertRaises(errors.TokenLockingNotSupported,
135
self.lockable.lock_write, token='token')
137
self.lockable.unlock()
103
139
def test_lock_write_returns_token_when_given_token(self):
104
140
token = self.lockable.lock_write()
105
self.addCleanup(self.lockable.unlock)
107
# This test does not apply, because this lockable refuses
110
new_lockable = self.get_lockable()
111
token_from_new_lockable = new_lockable.lock_write(token=token)
112
self.addCleanup(new_lockable.unlock)
113
self.assertEqual(token, token_from_new_lockable)
143
# This test does not apply, because this lockable refuses
146
new_lockable = self.get_lockable()
147
token_from_new_lockable = new_lockable.lock_write(token=token)
149
self.assertEqual(token, token_from_new_lockable)
151
new_lockable.unlock()
153
self.lockable.unlock()
115
155
def test_lock_write_raises_on_token_mismatch(self):
116
156
token = self.lockable.lock_write()
117
self.addCleanup(self.lockable.unlock)
119
# This test does not apply, because this lockable refuses
122
different_token = token + 'xxx'
123
# Re-using the same lockable instance with a different token will
124
# raise TokenMismatch.
125
self.assertRaises(errors.TokenMismatch,
126
self.lockable.lock_write, token=different_token)
127
# A separate instance for the same lockable will also raise
129
# This detects the case where a caller claims to have a lock (via
130
# the token) for an external resource, but doesn't (the token is
131
# different). Clients need a separate lock object to make sure the
132
# external resource is probed, whereas the existing lock object
134
new_lockable = self.get_lockable()
135
self.assertRaises(errors.TokenMismatch,
136
new_lockable.lock_write, token=different_token)
159
# This test does not apply, because this lockable refuses
162
different_token = token + 'xxx'
163
# Re-using the same lockable instance with a different token will
164
# raise TokenMismatch.
165
self.assertRaises(errors.TokenMismatch,
166
self.lockable.lock_write, token=different_token)
167
# A seperate instance for the same lockable will also raise
169
# This detects the case where a caller claims to have a lock (via
170
# the token) for an external resource, but doesn't (the token is
171
# different). Clients need a seperate lock object to make sure the
172
# external resource is probed, whereas the existing lock object
174
new_lockable = self.get_lockable()
175
self.assertRaises(errors.TokenMismatch,
176
new_lockable.lock_write, token=different_token)
178
self.lockable.unlock()
138
180
def test_lock_write_with_matching_token(self):
139
181
# If the token matches, so no exception is raised by lock_write.
140
182
token = self.lockable.lock_write()
141
self.addCleanup(self.lockable.unlock)
143
# This test does not apply, because this lockable refuses
146
# The same instance will accept a second lock_write if the specified
148
self.lockable.lock_write(token=token)
149
self.lockable.unlock()
150
# Calling lock_write on a new instance for the same lockable will
152
new_lockable = self.get_lockable()
153
new_lockable.lock_write(token=token)
154
new_lockable.unlock()
185
# This test does not apply, because this lockable refuses
188
# The same instance will accept a second lock_write if the specified
190
self.lockable.lock_write(token=token)
191
self.lockable.unlock()
192
# Calling lock_write on a new instance for the same lockable will
194
new_lockable = self.get_lockable()
195
new_lockable.lock_write(token=token)
196
new_lockable.unlock()
198
self.lockable.unlock()
156
200
def test_unlock_after_lock_write_with_token(self):
157
201
# If lock_write did not physically acquire the lock (because it was
158
202
# passed a token), then unlock should not physically release it.
159
203
token = self.lockable.lock_write()
160
self.addCleanup(self.lockable.unlock)
162
# This test does not apply, because this lockable refuses
165
new_lockable = self.get_lockable()
166
new_lockable.lock_write(token=token)
167
new_lockable.unlock()
168
self.assertTrue(self.lockable.get_physical_lock_status())
206
# This test does not apply, because this lockable refuses
209
new_lockable = self.get_lockable()
210
new_lockable.lock_write(token=token)
211
new_lockable.unlock()
212
self.assertTrue(self.lockable.get_physical_lock_status())
214
self.lockable.unlock()
170
216
def test_lock_write_with_token_fails_when_unlocked(self):
171
217
# Lock and unlock to get a superficially valid token. This mimics a
265
306
third_lockable.unlock()
268
# This method of adapting tests to parameters is different to
269
# the TestProviderAdapters used elsewhere, but seems simpler for this
309
# This method of adapting tests to parameters is different to
310
# the TestProviderAdapters used elsewhere, but seems simpler for this
271
312
class TestLockableFiles_TransportLock(TestCaseInTempDir,
272
313
_TestLockableFiles_mixin):
275
316
TestCaseInTempDir.setUp(self)
276
t = transport.get_transport('.')
278
self.sub_transport = t.clone('.bzr')
317
transport = get_transport('.')
318
transport.mkdir('.bzr')
319
self.sub_transport = transport.clone('.bzr')
279
320
self.lockable = self.get_lockable()
280
321
self.lockable.create_lock()
282
def stop_server(self):
283
super(TestLockableFiles_TransportLock, self).stop_server()
324
super(TestLockableFiles_TransportLock, self).tearDown()
284
325
# free the subtransport so that we do not get a 5 second
285
326
# timeout due to the SFTP connection cache.
287
del self.sub_transport
288
except AttributeError:
327
del self.sub_transport
291
329
def get_lockable(self):
292
330
return LockableFiles(self.sub_transport, 'my-lock', TransportLock)
295
333
class TestLockableFiles_LockDir(TestCaseInTempDir,
296
334
_TestLockableFiles_mixin):
300
338
TestCaseInTempDir.setUp(self)
301
self.transport = transport.get_transport('.')
339
self.transport = get_transport('.')
302
340
self.lockable = self.get_lockable()
303
# the lock creation here sets mode - test_permissions on branch
304
# tests that implicitly, but it might be a good idea to factor
341
# the lock creation here sets mode - test_permissions on branch
342
# tests that implicitly, but it might be a good idea to factor
305
343
# out the mode checking logic and have it applied to loackable files
306
344
# directly. RBC 20060418
307
345
self.lockable.create_lock()
309
347
def get_lockable(self):
310
return LockableFiles(self.transport, 'my-lock', lockdir.LockDir)
348
return LockableFiles(self.transport, 'my-lock', LockDir)
312
350
def test_lock_created(self):
313
351
self.assertTrue(self.transport.has('my-lock'))
317
355
self.assertFalse(self.transport.has('my-lock/held/info'))
318
356
self.assertTrue(self.transport.has('my-lock'))
320
def test__file_modes(self):
321
self.transport.mkdir('readonly')
322
osutils.make_readonly('readonly')
323
lockable = LockableFiles(self.transport.clone('readonly'), 'test-lock',
325
# The directory mode should be read-write-execute for the current user
326
self.assertEqual(00700, lockable._dir_mode & 00700)
327
# Files should be read-write for the current user
328
self.assertEqual(00600, lockable._file_mode & 00700)
331
class TestLockableFiles_RemoteLockDir(TestCaseWithSmartMedium,
332
_TestLockableFiles_mixin):
333
"""LockableFile tests run with RemoteLockDir on a branch."""
336
TestCaseWithSmartMedium.setUp(self)
337
# can only get a RemoteLockDir with some RemoteObject...
338
# use a branch as thats what we want. These mixin tests test the end
339
# to end behaviour, so stubbing out the backend and simulating would
340
# defeat the purpose. We test the protocol implementation separately
341
# in test_remote and test_smart as usual.
342
b = self.make_branch('foo')
343
self.addCleanup(b.bzrdir.transport.disconnect)
344
self.transport = transport.get_transport('.')
345
self.lockable = self.get_lockable()
347
def get_lockable(self):
348
# getting a new lockable involves opening a new instance of the branch
349
branch = bzrlib.branch.Branch.open(self.get_url('foo'))
350
self.addCleanup(branch.bzrdir.transport.disconnect)
351
return branch.control_files
359
# TODO: Test the lockdir inherits the right file and directory permissions
360
# from the LockableFiles.