/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
4797.32.2 by John Arbash Meinel
merge 2.1, resolving NEWS conflict.
1
# Copyright (C) 2006-2010 Canonical Ltd
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
2
#
1553.5.12 by Martin Pool
New LockDir locking mechanism
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.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
7
#
1553.5.12 by Martin Pool
New LockDir locking mechanism
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.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
12
#
1553.5.12 by Martin Pool
New LockDir locking mechanism
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
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1553.5.12 by Martin Pool
New LockDir locking mechanism
16
17
"""Tests for LockDir"""
18
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
19
from cStringIO import StringIO
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
20
import os
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
21
from threading import Thread, Lock
1553.5.12 by Martin Pool
New LockDir locking mechanism
22
import time
23
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
24
import bzrlib
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
25
from bzrlib import (
2055.2.1 by John Arbash Meinel
Make LockDir less sensitive to invalid configuration of email
26
    config,
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
27
    errors,
3331.3.13 by Martin Pool
Fix up imports
28
    lock,
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
29
    osutils,
1551.10.4 by Aaron Bentley
Update to skip on win32
30
    tests,
2555.3.9 by Martin Pool
Add test and fix for locking robustly when rename of directories doesn't act as a mutex (thank John)
31
    transport,
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
32
    )
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
33
from bzrlib.errors import (
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
34
    LockBreakMismatch,
35
    LockBroken,
36
    LockContention,
37
    LockError,
38
    LockFailed,
39
    LockNotHeld,
40
    )
2381.1.4 by Robert Collins
Unbreak lockdir tests after adding fast lockdir timeouts to the test suite default environment.
41
from bzrlib.lockdir import LockDir
4797.70.1 by Vincent Ladeuil
Skip chmodbits dependent tests when running as root
42
from bzrlib.tests import (features, TestCaseWithTransport)
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
43
from bzrlib.trace import note
1553.5.12 by Martin Pool
New LockDir locking mechanism
44
45
# These tests sometimes use threads to test the behaviour of lock files with
46
# concurrent actors.  This is not a typical (or necessarily supported) use;
47
# they're really meant for guarding between processes.
48
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
49
# These tests are run on the default transport provided by the test framework
50
# (typically a local disk transport).  That can be changed by the --transport
51
# option to bzr selftest.  The required properties of the transport
52
# implementation are tested separately.  (The main requirement is just that
53
# they don't allow overwriting nonempty directories.)
54
1553.5.12 by Martin Pool
New LockDir locking mechanism
55
class TestLockDir(TestCaseWithTransport):
56
    """Test LockDir operations"""
57
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
58
    def logging_report_function(self, fmt, *args):
59
        self._logged_reports.append((fmt, args))
60
61
    def setup_log_reporter(self, lock_dir):
62
        self._logged_reports = []
63
        lock_dir._report_function = self.logging_report_function
64
1553.5.12 by Martin Pool
New LockDir locking mechanism
65
    def test_00_lock_creation(self):
66
        """Creation of lock file on a transport"""
67
        t = self.get_transport()
68
        lf = LockDir(t, 'test_lock')
1553.5.13 by Martin Pool
New Transport.rename that mustn't overwrite
69
        self.assertFalse(lf.is_held)
1553.5.12 by Martin Pool
New LockDir locking mechanism
70
71
    def test_01_lock_repr(self):
72
        """Lock string representation"""
73
        lf = LockDir(self.get_transport(), 'test_lock')
74
        r = repr(lf)
75
        self.assertContainsRe(r, r'^LockDir\(.*/test_lock\)$')
76
77
    def test_02_unlocked_peek(self):
78
        lf = LockDir(self.get_transport(), 'test_lock')
79
        self.assertEqual(lf.peek(), None)
80
1687.1.3 by Robert Collins
Make LockDir.unlock check the lock is still intact.
81
    def get_lock(self):
82
        return LockDir(self.get_transport(), 'test_lock')
83
84
    def test_unlock_after_break_raises(self):
85
        ld = self.get_lock()
86
        ld2 = self.get_lock()
87
        ld.create()
88
        ld.attempt_lock()
89
        ld2.force_break(ld2.peek())
90
        self.assertRaises(LockBroken, ld.unlock)
91
1553.5.12 by Martin Pool
New LockDir locking mechanism
92
    def test_03_readonly_peek(self):
93
        lf = LockDir(self.get_readonly_transport(), 'test_lock')
94
        self.assertEqual(lf.peek(), None)
95
96
    def test_10_lock_uncontested(self):
97
        """Acquire and release a lock"""
98
        t = self.get_transport()
99
        lf = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
100
        lf.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
101
        lf.attempt_lock()
102
        try:
1553.5.13 by Martin Pool
New Transport.rename that mustn't overwrite
103
            self.assertTrue(lf.is_held)
1553.5.12 by Martin Pool
New LockDir locking mechanism
104
        finally:
105
            lf.unlock()
1553.5.13 by Martin Pool
New Transport.rename that mustn't overwrite
106
            self.assertFalse(lf.is_held)
1553.5.12 by Martin Pool
New LockDir locking mechanism
107
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
108
    def test_11_create_readonly_transport(self):
109
        """Fail to create lock on readonly transport"""
110
        t = self.get_readonly_transport()
111
        lf = LockDir(t, 'test_lock')
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
112
        self.assertRaises(LockFailed, lf.create)
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
113
114
    def test_12_lock_readonly_transport(self):
1553.5.12 by Martin Pool
New LockDir locking mechanism
115
        """Fail to lock on readonly transport"""
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
116
        lf = LockDir(self.get_transport(), 'test_lock')
117
        lf.create()
118
        lf = LockDir(self.get_readonly_transport(), 'test_lock')
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
119
        self.assertRaises(LockFailed, lf.attempt_lock)
1553.5.12 by Martin Pool
New LockDir locking mechanism
120
121
    def test_20_lock_contested(self):
122
        """Contention to get a lock"""
123
        t = self.get_transport()
124
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
125
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
126
        lf1.attempt_lock()
127
        lf2 = LockDir(t, 'test_lock')
128
        try:
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
129
            # locking is between LockDir instances; aliases within
1553.5.12 by Martin Pool
New LockDir locking mechanism
130
            # a single process are not detected
131
            lf2.attempt_lock()
132
            self.fail('Failed to detect lock collision')
133
        except LockContention, e:
134
            self.assertEqual(e.lock, lf2)
135
            self.assertContainsRe(str(e),
136
                    r'^Could not acquire.*test_lock.*$')
137
        lf1.unlock()
138
139
    def test_20_lock_peek(self):
140
        """Peek at the state of a lock"""
141
        t = self.get_transport()
142
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
143
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
144
        lf1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
145
        self.addCleanup(lf1.unlock)
1553.5.12 by Martin Pool
New LockDir locking mechanism
146
        # lock is held, should get some info on it
147
        info1 = lf1.peek()
148
        self.assertEqual(set(info1.keys()),
149
                         set(['user', 'nonce', 'hostname', 'pid', 'start_time']))
150
        # should get the same info if we look at it through a different
151
        # instance
152
        info2 = LockDir(t, 'test_lock').peek()
153
        self.assertEqual(info1, info2)
154
        # locks which are never used should be not-held
155
        self.assertEqual(LockDir(t, 'other_lock').peek(), None)
156
157
    def test_21_peek_readonly(self):
158
        """Peek over a readonly transport"""
159
        t = self.get_transport()
160
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
161
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
162
        lf2 = LockDir(self.get_readonly_transport(), 'test_lock')
163
        self.assertEqual(lf2.peek(), None)
164
        lf1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
165
        self.addCleanup(lf1.unlock)
1553.5.12 by Martin Pool
New LockDir locking mechanism
166
        info2 = lf2.peek()
167
        self.assertTrue(info2)
168
        self.assertEqual(info2['nonce'], lf1.nonce)
169
170
    def test_30_lock_wait_fail(self):
171
        """Wait on a lock, then fail
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
172
1553.5.12 by Martin Pool
New LockDir locking mechanism
173
        We ask to wait up to 400ms; this should fail within at most one
174
        second.  (Longer times are more realistic but we don't want the test
175
        suite to take too long, and this should do for now.)
176
        """
177
        t = self.get_transport()
178
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
179
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
180
        lf2 = LockDir(t, 'test_lock')
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
181
        self.setup_log_reporter(lf2)
1553.5.12 by Martin Pool
New LockDir locking mechanism
182
        lf1.attempt_lock()
183
        try:
184
            before = time.time()
185
            self.assertRaises(LockContention, lf2.wait_lock,
186
                              timeout=0.4, poll=0.1)
187
            after = time.time()
1704.2.1 by Martin Pool
Fix time-dependency in LockDir tests -- allow more margin for error in time to detect lock contention
188
            # it should only take about 0.4 seconds, but we allow more time in
189
            # case the machine is heavily loaded
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
190
            self.assertTrue(after - before <= 8.0,
1704.2.1 by Martin Pool
Fix time-dependency in LockDir tests -- allow more margin for error in time to detect lock contention
191
                    "took %f seconds to detect lock contention" % (after - before))
1553.5.12 by Martin Pool
New LockDir locking mechanism
192
        finally:
193
            lf1.unlock()
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
194
        lock_base = lf2.transport.abspath(lf2.path)
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
195
        self.assertEqual(1, len(self._logged_reports))
3441.4.2 by Martin Albisetti
Fixed failing tests
196
        lock_url = lf2.transport.abspath(lf2.path)
1957.1.9 by John Arbash Meinel
Change default timeouts, and report differently the first failure
197
        self.assertEqual('%s %s\n'
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
198
                         '%s\n%s\n'
3441.4.2 by Martin Albisetti
Fixed failing tests
199
                         'Will continue to try until %s, unless '
4960.2.2 by Martin Pool
Tweak Lockdir contention message to suggest reading help break-lock.
200
                         'you press Ctrl-C.\n'
201
                         'See "bzr help break-lock" for more.',
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
202
                         self._logged_reports[0][0])
203
        args = self._logged_reports[0][1]
1957.1.9 by John Arbash Meinel
Change default timeouts, and report differently the first failure
204
        self.assertEqual('Unable to obtain', args[0])
205
        self.assertEqual('lock %s' % (lock_base,), args[1])
206
        self.assertStartsWith(args[2], 'held by ')
207
        self.assertStartsWith(args[3], 'locked ')
208
        self.assertEndsWith(args[3], ' ago')
1957.1.13 by John Arbash Meinel
Change to reporting the time when we will stop trying to grab the lock
209
        self.assertContainsRe(args[4], r'\d\d:\d\d:\d\d')
1553.5.12 by Martin Pool
New LockDir locking mechanism
210
211
    def test_31_lock_wait_easy(self):
212
        """Succeed when waiting on a lock with no contention.
213
        """
214
        t = self.get_transport()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
215
        lf1 = LockDir(t, 'test_lock')
216
        lf1.create()
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
217
        self.setup_log_reporter(lf1)
1553.5.12 by Martin Pool
New LockDir locking mechanism
218
        try:
219
            before = time.time()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
220
            lf1.wait_lock(timeout=0.4, poll=0.1)
1553.5.12 by Martin Pool
New LockDir locking mechanism
221
            after = time.time()
222
            self.assertTrue(after - before <= 1.0)
223
        finally:
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
224
            lf1.unlock()
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
225
        self.assertEqual([], self._logged_reports)
1553.5.12 by Martin Pool
New LockDir locking mechanism
226
1551.15.18 by Aaron Bentley
Skip itermittently-failing test instead of deleting it
227
    def test_32_lock_wait_succeed(self):
228
        """Succeed when trying to acquire a lock that gets released
229
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
230
        One thread holds on a lock and then releases it; another
1551.15.18 by Aaron Bentley
Skip itermittently-failing test instead of deleting it
231
        tries to lock it.
232
        """
1551.15.22 by Aaron Bentley
Redo test skip
233
        # This test sometimes fails like this:
234
        # Traceback (most recent call last):
235
236
        #   File "/home/pqm/bzr-pqm-workdir/home/+trunk/bzrlib/tests/
237
        # test_lockdir.py", line 247, in test_32_lock_wait_succeed
238
        #     self.assertEqual(1, len(self._logged_reports))
239
        # AssertionError: not equal:
240
        # a = 1
241
        # b = 0
242
        raise tests.TestSkipped("Test fails intermittently")
1551.15.18 by Aaron Bentley
Skip itermittently-failing test instead of deleting it
243
        t = self.get_transport()
244
        lf1 = LockDir(t, 'test_lock')
245
        lf1.create()
246
        lf1.attempt_lock()
247
248
        def wait_and_unlock():
249
            time.sleep(0.1)
250
            lf1.unlock()
251
        unlocker = Thread(target=wait_and_unlock)
252
        unlocker.start()
253
        try:
254
            lf2 = LockDir(t, 'test_lock')
255
            self.setup_log_reporter(lf2)
256
            before = time.time()
257
            # wait and then lock
258
            lf2.wait_lock(timeout=0.4, poll=0.1)
259
            after = time.time()
260
            self.assertTrue(after - before <= 1.0)
261
        finally:
262
            unlocker.join()
263
264
        # There should be only 1 report, even though it should have to
265
        # wait for a while
266
        lock_base = lf2.transport.abspath(lf2.path)
267
        self.assertEqual(1, len(self._logged_reports))
268
        self.assertEqual('%s %s\n'
269
                         '%s\n%s\n'
270
                         'Will continue to try until %s\n',
271
                         self._logged_reports[0][0])
272
        args = self._logged_reports[0][1]
273
        self.assertEqual('Unable to obtain', args[0])
274
        self.assertEqual('lock %s' % (lock_base,), args[1])
275
        self.assertStartsWith(args[2], 'held by ')
276
        self.assertStartsWith(args[3], 'locked ')
277
        self.assertEndsWith(args[3], ' ago')
278
        self.assertContainsRe(args[4], r'\d\d:\d\d:\d\d')
279
1957.1.2 by John Arbash Meinel
Switch the default from instantly aborting, to waiting as long as 1 minute (down from 5 minutes)
280
    def test_34_lock_write_waits(self):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
281
        """LockDir.lock_write() will wait for the lock."""
2381.1.4 by Robert Collins
Unbreak lockdir tests after adding fast lockdir timeouts to the test suite default environment.
282
        # the test suite sets the default to 0 to make deadlocks fail fast.
283
        # change it for this test, as we want to try a manual deadlock.
2631.1.4 by Aaron Bentley
Fix test skip
284
        raise tests.TestSkipped('Timing-sensitive test')
2381.1.4 by Robert Collins
Unbreak lockdir tests after adding fast lockdir timeouts to the test suite default environment.
285
        bzrlib.lockdir._DEFAULT_TIMEOUT_SECONDS = 300
1957.1.2 by John Arbash Meinel
Switch the default from instantly aborting, to waiting as long as 1 minute (down from 5 minutes)
286
        t = self.get_transport()
287
        lf1 = LockDir(t, 'test_lock')
288
        lf1.create()
289
        lf1.attempt_lock()
290
291
        def wait_and_unlock():
292
            time.sleep(0.1)
293
            lf1.unlock()
294
        unlocker = Thread(target=wait_and_unlock)
295
        unlocker.start()
296
        try:
297
            lf2 = LockDir(t, 'test_lock')
298
            self.setup_log_reporter(lf2)
299
            before = time.time()
300
            # wait and then lock
301
            lf2.lock_write()
302
            after = time.time()
303
        finally:
304
            unlocker.join()
305
306
        # There should be only 1 report, even though it should have to
307
        # wait for a while
308
        lock_base = lf2.transport.abspath(lf2.path)
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
309
        self.assertEqual(1, len(self._logged_reports))
1957.1.9 by John Arbash Meinel
Change default timeouts, and report differently the first failure
310
        self.assertEqual('%s %s\n'
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
311
                         '%s\n%s\n'
1957.1.13 by John Arbash Meinel
Change to reporting the time when we will stop trying to grab the lock
312
                         'Will continue to try until %s\n',
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
313
                         self._logged_reports[0][0])
314
        args = self._logged_reports[0][1]
1957.1.9 by John Arbash Meinel
Change default timeouts, and report differently the first failure
315
        self.assertEqual('Unable to obtain', args[0])
316
        self.assertEqual('lock %s' % (lock_base,), args[1])
317
        self.assertStartsWith(args[2], 'held by ')
318
        self.assertStartsWith(args[3], 'locked ')
319
        self.assertEndsWith(args[3], ' ago')
1957.1.13 by John Arbash Meinel
Change to reporting the time when we will stop trying to grab the lock
320
        self.assertContainsRe(args[4], r'\d\d:\d\d:\d\d')
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
321
322
    def test_35_wait_lock_changing(self):
323
        """LockDir.wait_lock() will report if the lock changes underneath.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
324
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
325
        This is the stages we want to happen:
326
327
        0) Synchronization locks are created and locked.
328
        1) Lock1 obtains the lockdir, and releases the 'check' lock.
329
        2) Lock2 grabs the 'check' lock, and checks the lockdir.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
330
           It sees the lockdir is already acquired, reports the fact,
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
331
           and unsets the 'checked' lock.
332
        3) Thread1 blocks on acquiring the 'checked' lock, and then tells
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
333
           Lock1 to release and acquire the lockdir. This resets the 'check'
334
           lock.
335
        4) Lock2 acquires the 'check' lock, and checks again. It notices
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
336
           that the holder of the lock has changed, and so reports a new
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
337
           lock holder.
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
338
        5) Thread1 blocks on the 'checked' lock, this time, it completely
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
339
           unlocks the lockdir, allowing Lock2 to acquire the lock.
340
        """
341
3497.1.1 by Martin Pool
xfail test_35_wait_lock_changing
342
        raise tests.KnownFailure(
343
            "timing dependency in lock tests (#213182)")
344
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
345
        wait_to_check_lock = Lock()
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
346
        wait_until_checked_lock = Lock()
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
347
348
        wait_to_check_lock.acquire()
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
349
        wait_until_checked_lock.acquire()
350
        note('locked check and checked locks')
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
351
352
        class LockDir1(LockDir):
353
            """Use the synchronization points for the first lock."""
354
355
            def attempt_lock(self):
356
                # Once we have acquired the lock, it is okay for
357
                # the other lock to check it
358
                try:
359
                    return super(LockDir1, self).attempt_lock()
360
                finally:
361
                    note('lock1: releasing check lock')
362
                    wait_to_check_lock.release()
363
364
        class LockDir2(LockDir):
365
            """Use the synchronization points for the second lock."""
366
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
367
            def attempt_lock(self):
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
368
                note('lock2: waiting for check lock')
369
                wait_to_check_lock.acquire()
370
                note('lock2: acquired check lock')
371
                try:
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
372
                    return super(LockDir2, self).attempt_lock()
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
373
                finally:
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
374
                    note('lock2: releasing checked lock')
375
                    wait_until_checked_lock.release()
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
376
377
        t = self.get_transport()
378
        lf1 = LockDir1(t, 'test_lock')
379
        lf1.create()
380
381
        lf2 = LockDir2(t, 'test_lock')
382
        self.setup_log_reporter(lf2)
383
384
        def wait_and_switch():
385
            lf1.attempt_lock()
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
386
            # Block until lock2 has had a chance to check
387
            note('lock1: waiting 1 for checked lock')
388
            wait_until_checked_lock.acquire()
389
            note('lock1: acquired for checked lock')
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
390
            note('lock1: released lockdir')
391
            lf1.unlock()
392
            note('lock1: acquiring lockdir')
393
            # Create a new nonce, so the lock looks different.
394
            lf1.nonce = osutils.rand_chars(20)
395
            lf1.lock_write()
396
            note('lock1: acquired lockdir')
397
398
            # Block until lock2 has peeked again
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
399
            note('lock1: waiting 2 for checked lock')
400
            wait_until_checked_lock.acquire()
401
            note('lock1: acquired for checked lock')
402
            # Now unlock, and let lock 2 grab the lock
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
403
            lf1.unlock()
404
            wait_to_check_lock.release()
405
406
        unlocker = Thread(target=wait_and_switch)
407
        unlocker.start()
408
        try:
1957.1.11 by John Arbash Meinel
Switch from locking the peek() to locking attempt_lock(), which is much more stable
409
            # Wait and play against the other thread
2709.1.1 by Martin Pool
Make TestLockDir.test_35_wait_lock_changing less timing-sensitive
410
            lf2.wait_lock(timeout=20.0, poll=0.01)
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
411
        finally:
412
            unlocker.join()
413
        lf2.unlock()
414
415
        # There should be 2 reports, because the lock changed
416
        lock_base = lf2.transport.abspath(lf2.path)
417
        self.assertEqual(2, len(self._logged_reports))
3441.4.2 by Martin Albisetti
Fixed failing tests
418
        lock_url = lf2.transport.abspath(lf2.path)
1957.1.15 by John Arbash Meinel
Review feedback from Robert
419
        self.assertEqual('%s %s\n'
420
                         '%s\n%s\n'
3441.4.2 by Martin Albisetti
Fixed failing tests
421
                         'Will continue to try until %s, unless '
4960.2.2 by Martin Pool
Tweak Lockdir contention message to suggest reading help break-lock.
422
                         'you press Ctrl-C.\n'
423
                         'See "bzr help break-lock" for more.',
1957.1.15 by John Arbash Meinel
Review feedback from Robert
424
                         self._logged_reports[0][0])
425
        args = self._logged_reports[0][1]
426
        self.assertEqual('Unable to obtain', args[0])
427
        self.assertEqual('lock %s' % (lock_base,), args[1])
428
        self.assertStartsWith(args[2], 'held by ')
429
        self.assertStartsWith(args[3], 'locked ')
430
        self.assertEndsWith(args[3], ' ago')
431
        self.assertContainsRe(args[4], r'\d\d:\d\d:\d\d')
1957.1.13 by John Arbash Meinel
Change to reporting the time when we will stop trying to grab the lock
432
1957.1.15 by John Arbash Meinel
Review feedback from Robert
433
        self.assertEqual('%s %s\n'
434
                         '%s\n%s\n'
3441.4.2 by Martin Albisetti
Fixed failing tests
435
                         'Will continue to try until %s, unless '
4960.2.2 by Martin Pool
Tweak Lockdir contention message to suggest reading help break-lock.
436
                         'you press Ctrl-C.\n'
437
                         'See "bzr help break-lock" for more.',
1957.1.15 by John Arbash Meinel
Review feedback from Robert
438
                         self._logged_reports[1][0])
439
        args = self._logged_reports[1][1]
440
        self.assertEqual('Lock owner changed for', args[0])
441
        self.assertEqual('lock %s' % (lock_base,), args[1])
442
        self.assertStartsWith(args[2], 'held by ')
443
        self.assertStartsWith(args[3], 'locked ')
444
        self.assertEndsWith(args[3], ' ago')
445
        self.assertContainsRe(args[4], r'\d\d:\d\d:\d\d')
1957.1.2 by John Arbash Meinel
Switch the default from instantly aborting, to waiting as long as 1 minute (down from 5 minutes)
446
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
447
    def test_40_confirm_easy(self):
448
        """Confirm a lock that's already held"""
449
        t = self.get_transport()
450
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
451
        lf1.create()
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
452
        lf1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
453
        self.addCleanup(lf1.unlock)
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
454
        lf1.confirm()
455
456
    def test_41_confirm_not_held(self):
457
        """Confirm a lock that's already held"""
458
        t = self.get_transport()
459
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
460
        lf1.create()
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
461
        self.assertRaises(LockNotHeld, lf1.confirm)
1553.5.23 by Martin Pool
Start LockDir.confirm method and LockBroken exception
462
463
    def test_42_confirm_broken_manually(self):
464
        """Confirm a lock broken by hand"""
465
        t = self.get_transport()
466
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
467
        lf1.create()
1553.5.23 by Martin Pool
Start LockDir.confirm method and LockBroken exception
468
        lf1.attempt_lock()
469
        t.move('test_lock', 'lock_gone_now')
470
        self.assertRaises(LockBroken, lf1.confirm)
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
471
        # Clean up
472
        t.move('lock_gone_now', 'test_lock')
473
        lf1.unlock()
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
474
475
    def test_43_break(self):
476
        """Break a lock whose caller has forgotten it"""
477
        t = self.get_transport()
478
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
479
        lf1.create()
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
480
        lf1.attempt_lock()
481
        # we incorrectly discard the lock object without unlocking it
482
        del lf1
483
        # someone else sees it's still locked
484
        lf2 = LockDir(t, 'test_lock')
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
485
        holder_info = lf2.peek()
486
        self.assertTrue(holder_info)
487
        lf2.force_break(holder_info)
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
488
        # now we should be able to take it
489
        lf2.attempt_lock()
4327.1.4 by Vincent Ladeuil
Fix lock test failures by taking lock breaking into account.
490
        self.addCleanup(lf2.unlock)
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
491
        lf2.confirm()
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
492
493
    def test_44_break_already_released(self):
494
        """Lock break races with regular release"""
495
        t = self.get_transport()
496
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
497
        lf1.create()
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
498
        lf1.attempt_lock()
499
        # someone else sees it's still locked
500
        lf2 = LockDir(t, 'test_lock')
501
        holder_info = lf2.peek()
502
        # in the interim the lock is released
503
        lf1.unlock()
504
        # break should succeed
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
505
        lf2.force_break(holder_info)
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
506
        # now we should be able to take it
507
        lf2.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
508
        self.addCleanup(lf2.unlock)
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
509
        lf2.confirm()
510
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
511
    def test_45_break_mismatch(self):
512
        """Lock break races with someone else acquiring it"""
513
        t = self.get_transport()
514
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
515
        lf1.create()
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
516
        lf1.attempt_lock()
517
        # someone else sees it's still locked
518
        lf2 = LockDir(t, 'test_lock')
519
        holder_info = lf2.peek()
520
        # in the interim the lock is released
521
        lf1.unlock()
522
        lf3 = LockDir(t, 'test_lock')
523
        lf3.attempt_lock()
524
        # break should now *fail*
525
        self.assertRaises(LockBreakMismatch, lf2.force_break,
526
                          holder_info)
527
        lf3.unlock()
1553.5.54 by Martin Pool
Add LockDir.read_lock fake method
528
529
    def test_46_fake_read_lock(self):
530
        t = self.get_transport()
531
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
532
        lf1.create()
1553.5.54 by Martin Pool
Add LockDir.read_lock fake method
533
        lf1.lock_read()
534
        lf1.unlock()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
535
536
    def test_50_lockdir_representation(self):
537
        """Check the on-disk representation of LockDirs is as expected.
538
539
        There should always be a top-level directory named by the lock.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
540
        When the lock is held, there should be a lockname/held directory
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
541
        containing an info file.
542
        """
543
        t = self.get_transport()
544
        lf1 = LockDir(t, 'test_lock')
545
        lf1.create()
546
        self.assertTrue(t.has('test_lock'))
547
        lf1.lock_write()
548
        self.assertTrue(t.has('test_lock/held/info'))
549
        lf1.unlock()
550
        self.assertFalse(t.has('test_lock/held/info'))
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
551
552
    def test_break_lock(self):
553
        # the ui based break_lock routine should Just Work (tm)
554
        ld1 = self.get_lock()
555
        ld2 = self.get_lock()
556
        ld1.create()
557
        ld1.lock_write()
1687.1.6 by Robert Collins
Extend LockableFiles to support break_lock() calls.
558
        # do this without IO redirection to ensure it doesn't prompt.
559
        self.assertRaises(AssertionError, ld1.break_lock)
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
560
        orig_factory = bzrlib.ui.ui_factory
4449.3.27 by Martin Pool
More test updates to use CannedInputUIFactory
561
        bzrlib.ui.ui_factory = bzrlib.ui.CannedInputUIFactory([True])
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
562
        try:
563
            ld2.break_lock()
564
            self.assertRaises(LockBroken, ld1.unlock)
565
        finally:
566
            bzrlib.ui.ui_factory = orig_factory
1955.1.1 by John Arbash Meinel
LockDir can create the root directory if it fails to create a pending directory due to NoSuchFile.
567
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
568
    def test_break_lock_corrupt_info(self):
569
        """break_lock works even if the info file is corrupt (and tells the UI
570
        that it is corrupt).
571
        """
572
        ld = self.get_lock()
573
        ld2 = self.get_lock()
574
        ld.create()
575
        ld.lock_write()
576
        ld.transport.put_bytes_non_atomic('test_lock/held/info', '\0')
577
        class LoggingUIFactory(bzrlib.ui.SilentUIFactory):
578
            def __init__(self):
579
                self.prompts = []
580
            def get_boolean(self, prompt):
581
                self.prompts.append(('boolean', prompt))
582
                return True
583
        ui = LoggingUIFactory()
584
        orig_factory = bzrlib.ui.ui_factory
585
        bzrlib.ui.ui_factory = ui
586
        try:
587
            ld2.break_lock()
588
            self.assertLength(1, ui.prompts)
589
            self.assertEqual('boolean', ui.prompts[0][0])
590
            self.assertStartsWith(ui.prompts[0][1], 'Break (corrupt LockDir')
591
            self.assertRaises(LockBroken, ld.unlock)
592
        finally:
593
            bzrlib.ui.ui_factory = orig_factory
594
595
    def test_break_lock_missing_info(self):
596
        """break_lock works even if the info file is missing (and tells the UI
597
        that it is corrupt).
598
        """
599
        ld = self.get_lock()
600
        ld2 = self.get_lock()
601
        ld.create()
602
        ld.lock_write()
603
        ld.transport.delete('test_lock/held/info')
604
        class LoggingUIFactory(bzrlib.ui.SilentUIFactory):
605
            def __init__(self):
606
                self.prompts = []
607
            def get_boolean(self, prompt):
608
                self.prompts.append(('boolean', prompt))
609
                return True
610
        ui = LoggingUIFactory()
611
        orig_factory = bzrlib.ui.ui_factory
612
        bzrlib.ui.ui_factory = ui
613
        try:
614
            ld2.break_lock()
615
            self.assertRaises(LockBroken, ld.unlock)
616
            self.assertLength(0, ui.prompts)
617
        finally:
618
            bzrlib.ui.ui_factory = orig_factory
619
        # Suppress warnings due to ld not being unlocked
620
        # XXX: if lock_broken hook was invoked in this case, this hack would
621
        # not be necessary.  - Andrew Bennetts, 2010-09-06.
622
        del self._lock_actions[:]
623
1955.1.1 by John Arbash Meinel
LockDir can create the root directory if it fails to create a pending directory due to NoSuchFile.
624
    def test_create_missing_base_directory(self):
625
        """If LockDir.path doesn't exist, it can be created
626
627
        Some people manually remove the entire lock/ directory trying
628
        to unlock a stuck repository/branch/etc. Rather than failing
629
        after that, just create the lock directory when needed.
630
        """
631
        t = self.get_transport()
632
        lf1 = LockDir(t, 'test_lock')
633
634
        lf1.create()
635
        self.failUnless(t.has('test_lock'))
636
637
        t.rmdir('test_lock')
638
        self.failIf(t.has('test_lock'))
639
640
        # This will create 'test_lock' if it needs to
641
        lf1.lock_write()
642
        self.failUnless(t.has('test_lock'))
643
        self.failUnless(t.has('test_lock/held/info'))
644
645
        lf1.unlock()
646
        self.failIf(t.has('test_lock/held/info'))
1957.1.6 by John Arbash Meinel
[merge] bzr.dev 2009
647
1957.1.5 by John Arbash Meinel
Create a helper function for formatting lock information
648
    def test__format_lock_info(self):
649
        ld1 = self.get_lock()
650
        ld1.create()
651
        ld1.lock_write()
652
        try:
653
            info_list = ld1._format_lock_info(ld1.peek())
654
        finally:
655
            ld1.unlock()
656
        self.assertEqual('lock %s' % (ld1.transport.abspath(ld1.path),),
657
                         info_list[0])
658
        self.assertContainsRe(info_list[1],
659
                              r'^held by .* on host .* \[process #\d*\]$')
660
        self.assertContainsRe(info_list[2], r'locked \d+ seconds? ago$')
2055.2.1 by John Arbash Meinel
Make LockDir less sensitive to invalid configuration of email
661
662
    def test_lock_without_email(self):
663
        global_config = config.GlobalConfig()
664
        # Intentionally has no email address
665
        global_config.set_user_option('email', 'User Identity')
666
        ld1 = self.get_lock()
667
        ld1.create()
668
        ld1.lock_write()
669
        ld1.unlock()
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
670
671
    def test_lock_permission(self):
4797.70.1 by Vincent Ladeuil
Skip chmodbits dependent tests when running as root
672
        self.requireFeature(features.not_running_as_root)
1551.10.4 by Aaron Bentley
Update to skip on win32
673
        if not osutils.supports_posix_readonly():
3107.2.2 by John Arbash Meinel
feedback from Martin.
674
            raise tests.TestSkipped('Cannot induce a permission failure')
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
675
        ld1 = self.get_lock()
676
        lock_path = ld1.transport.local_abspath('test_lock')
677
        os.mkdir(lock_path)
678
        osutils.make_readonly(lock_path)
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
679
        self.assertRaises(errors.LockFailed, ld1.attempt_lock)
2555.3.5 by Martin Pool
Return token directly from LockDir.acquire to avoid unnecessary peek()
680
681
    def test_lock_by_token(self):
682
        ld1 = self.get_lock()
683
        token = ld1.lock_write()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
684
        self.addCleanup(ld1.unlock)
2555.3.5 by Martin Pool
Return token directly from LockDir.acquire to avoid unnecessary peek()
685
        self.assertNotEqual(None, token)
686
        ld2 = self.get_lock()
687
        t2 = ld2.lock_write(token)
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
688
        self.addCleanup(ld2.unlock)
2555.3.5 by Martin Pool
Return token directly from LockDir.acquire to avoid unnecessary peek()
689
        self.assertEqual(token, t2)
2555.3.9 by Martin Pool
Add test and fix for locking robustly when rename of directories doesn't act as a mutex (thank John)
690
691
    def test_lock_with_buggy_rename(self):
692
        # test that lock acquisition handles servers which pretend they
693
        # renamed correctly but that actually fail
694
        t = transport.get_transport('brokenrename+' + self.get_url())
695
        ld1 = LockDir(t, 'test_lock')
696
        ld1.create()
697
        ld1.attempt_lock()
698
        ld2 = LockDir(t, 'test_lock')
2555.3.14 by Martin Pool
Better handling in LockDir of rename that moves one directory within another
699
        # we should fail to lock
700
        e = self.assertRaises(errors.LockContention, ld2.attempt_lock)
701
        # now the original caller should succeed in unlocking
702
        ld1.unlock()
703
        # and there should be nothing left over
704
        self.assertEquals([], t.list_dir('test_lock'))
2555.3.12 by Martin Pool
Add test for https://bugs.launchpad.net/bzr/+bug/109169 -- test_failed_lock_leaves_no_trash
705
706
    def test_failed_lock_leaves_no_trash(self):
707
        # if we fail to acquire the lock, we don't leave pending directories
708
        # behind -- https://bugs.launchpad.net/bzr/+bug/109169
709
        ld1 = self.get_lock()
710
        ld2 = self.get_lock()
711
        # should be nothing before we start
712
        ld1.create()
713
        t = self.get_transport().clone('test_lock')
714
        def check_dir(a):
715
            self.assertEquals(a, t.list_dir('.'))
716
        check_dir([])
717
        # when held, that's all we see
718
        ld1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
719
        self.addCleanup(ld1.unlock)
2555.3.12 by Martin Pool
Add test for https://bugs.launchpad.net/bzr/+bug/109169 -- test_failed_lock_leaves_no_trash
720
        check_dir(['held'])
721
        # second guy should fail
722
        self.assertRaises(errors.LockContention, ld2.attempt_lock)
723
        # no kibble
724
        check_dir(['held'])
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
725
4634.138.1 by Martin Pool
Add failing test for \#185103
726
    def test_no_lockdir_info(self):
727
        """We can cope with empty info files."""
728
        # This seems like a fairly common failure case - see
4634.165.4 by Vincent Ladeuil
Fix all comments where bugs.edge.launchpad.net was used.
729
        # <https://bugs.launchpad.net/bzr/+bug/185103> and all its dupes.
4634.138.1 by Martin Pool
Add failing test for \#185103
730
        # Processes are often interrupted after opening the file
731
        # before the actual contents are committed.
732
        t = self.get_transport()
733
        t.mkdir('test_lock')
734
        t.mkdir('test_lock/held')
735
        t.put_bytes('test_lock/held/info', '')
736
        lf = LockDir(t, 'test_lock')
4634.138.2 by Martin Pool
Copy with lock info file being empty
737
        info = lf.peek()
738
        formatted_info = lf._format_lock_info(info)
739
        self.assertEquals(
740
            ['lock %s' % t.abspath('test_lock'),
4634.138.3 by Martin Pool
Say <unknown> not None when lock info couldn't be determined (thanks vila)
741
             'held by <unknown> on host <unknown> [process #<unknown>]',
4634.138.2 by Martin Pool
Copy with lock info file being empty
742
             'locked (unknown)'],
743
            formatted_info)
4634.138.1 by Martin Pool
Add failing test for \#185103
744
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
745
    def test_corrupt_lockdir_info(self):
746
        """We can cope with corrupt (and thus unparseable) info files."""
747
        # This seems like a fairly common failure case too - see
4634.165.4 by Vincent Ladeuil
Fix all comments where bugs.edge.launchpad.net was used.
748
        # <https://bugs.launchpad.net/bzr/+bug/619872> for instance.
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
749
        # In particular some systems tend to fill recently created files with
750
        # nul bytes after recovering from a system crash.
751
        t = self.get_transport()
752
        t.mkdir('test_lock')
753
        t.mkdir('test_lock/held')
754
        t.put_bytes('test_lock/held/info', '\0')
755
        lf = LockDir(t, 'test_lock')
756
        self.assertRaises(errors.LockCorrupt, lf.peek)
757
        # Currently attempt_lock gives LockContention, but LockCorrupt would be
758
        # a reasonable result too.
759
        self.assertRaises(
760
            (errors.LockCorrupt, errors.LockContention), lf.attempt_lock)
761
        self.assertRaises(errors.LockCorrupt, lf.validate_token, 'fake token')
762
763
    def test_missing_lockdir_info(self):
764
        """We can cope with absent info files."""
765
        t = self.get_transport()
766
        t.mkdir('test_lock')
767
        t.mkdir('test_lock/held')
768
        lf = LockDir(t, 'test_lock')
769
        # In this case we expect the 'not held' result from peek, because peek
770
        # cannot be expected to notice that there is a 'held' directory with no
771
        # 'info' file.
772
        self.assertEqual(None, lf.peek())
773
        # And lock/unlock may work or give LockContention (but not any other
774
        # error).
775
        try:
776
            lf.attempt_lock()
777
        except LockContention:
778
            # LockContention is ok, and expected on Windows
779
            pass
780
        else:
781
            # no error is ok, and expected on POSIX (because POSIX allows
782
            # os.rename over an empty directory).
783
            lf.unlock()
784
        # Currently raises TokenMismatch, but LockCorrupt would be reasonable
785
        # too.
786
        self.assertRaises(
787
            (errors.TokenMismatch, errors.LockCorrupt),
788
            lf.validate_token, 'fake token')
789
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
790
791
class TestLockDirHooks(TestCaseWithTransport):
792
793
    def setUp(self):
794
        super(TestLockDirHooks, self).setUp()
795
        self._calls = []
796
797
    def get_lock(self):
798
        return LockDir(self.get_transport(), 'test_lock')
799
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
800
    def record_hook(self, result):
801
        self._calls.append(result)
802
3331.3.12 by Martin Pool
Remove PhysicalLock class
803
    def test_LockDir_acquired_success(self):
804
        # the LockDir.lock_acquired hook fires when a lock is acquired.
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
805
        LockDir.hooks.install_named_hook('lock_acquired',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
806
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
807
        ld = self.get_lock()
808
        ld.create()
809
        self.assertEqual([], self._calls)
810
        result = ld.attempt_lock()
3331.3.2 by Robert Collins
Polish on lock hooks to be easier to use.
811
        lock_path = ld.transport.abspath(ld.path)
812
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
813
        ld.unlock()
3331.3.2 by Robert Collins
Polish on lock hooks to be easier to use.
814
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
815
3331.3.12 by Martin Pool
Remove PhysicalLock class
816
    def test_LockDir_acquired_fail(self):
817
        # the LockDir.lock_acquired hook does not fire on failure.
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
818
        ld = self.get_lock()
819
        ld.create()
820
        ld2 = self.get_lock()
821
        ld2.attempt_lock()
822
        # install a lock hook now, when the disk lock is locked
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
823
        LockDir.hooks.install_named_hook('lock_acquired',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
824
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
825
        self.assertRaises(errors.LockContention, ld.attempt_lock)
826
        self.assertEqual([], self._calls)
827
        ld2.unlock()
828
        self.assertEqual([], self._calls)
829
3331.3.12 by Martin Pool
Remove PhysicalLock class
830
    def test_LockDir_released_success(self):
831
        # the LockDir.lock_released hook fires when a lock is acquired.
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
832
        LockDir.hooks.install_named_hook('lock_released',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
833
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
834
        ld = self.get_lock()
835
        ld.create()
836
        self.assertEqual([], self._calls)
837
        result = ld.attempt_lock()
838
        self.assertEqual([], self._calls)
839
        ld.unlock()
3331.3.2 by Robert Collins
Polish on lock hooks to be easier to use.
840
        lock_path = ld.transport.abspath(ld.path)
841
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
842
3331.3.12 by Martin Pool
Remove PhysicalLock class
843
    def test_LockDir_released_fail(self):
844
        # the LockDir.lock_released hook does not fire on failure.
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
845
        ld = self.get_lock()
846
        ld.create()
847
        ld2 = self.get_lock()
848
        ld.attempt_lock()
849
        ld2.force_break(ld2.peek())
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
850
        LockDir.hooks.install_named_hook('lock_released',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
851
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
852
        self.assertRaises(LockBroken, ld.unlock)
853
        self.assertEqual([], self._calls)
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
854
855
    def test_LockDir_broken_success(self):
856
        # the LockDir.lock_broken hook fires when a lock is broken.
857
        ld = self.get_lock()
858
        ld.create()
859
        ld2 = self.get_lock()
860
        result = ld.attempt_lock()
861
        LockDir.hooks.install_named_hook('lock_broken',
862
                                         self.record_hook, 'record_hook')
863
        ld2.force_break(ld2.peek())
864
        lock_path = ld.transport.abspath(ld.path)
865
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
866
867
    def test_LockDir_broken_failure(self):
868
        # the LockDir.lock_broken hook does not fires when a lock is already
869
        # released.
870
        ld = self.get_lock()
871
        ld.create()
872
        ld2 = self.get_lock()
873
        result = ld.attempt_lock()
874
        holder_info = ld2.peek()
875
        ld.unlock()
876
        LockDir.hooks.install_named_hook('lock_broken',
877
                                         self.record_hook, 'record_hook')
878
        ld2.force_break(holder_info)
879
        lock_path = ld.transport.abspath(ld.path)
880
        self.assertEqual([], self._calls)