~bzr-pqm/bzr/bzr.dev

5557.1.15 by John Arbash Meinel
Merge bzr.dev 5597 to resolve NEWS, aka bzr-2.3.txt
1
# Copyright (C) 2006-2011 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
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
19
import os
5425.5.1 by Martin
Fix NameError issues in bt.test_lockdir
20
import sys
1553.5.12 by Martin Pool
New LockDir locking mechanism
21
import time
22
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
23
import bzrlib
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
24
from bzrlib import (
2055.2.1 by John Arbash Meinel
Make LockDir less sensitive to invalid configuration of email
25
    config,
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
26
    errors,
3331.3.13 by Martin Pool
Fix up imports
27
    lock,
5425.4.9 by Martin Pool
Defensively handle 'localhost' in detecting dead locks
28
    lockdir,
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
    LockFailed,
38
    LockNotHeld,
39
    )
5425.4.6 by Martin Pool
Move preparation of new locks into LockHeldInfo
40
from bzrlib.lockdir import (
41
    LockDir,
42
    LockHeldInfo,
43
    )
5579.3.1 by Jelmer Vernooij
Remove unused imports.
44
from bzrlib.tests import (
45
    features,
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
46
    TestCase,
5579.3.1 by Jelmer Vernooij
Remove unused imports.
47
    TestCaseWithTransport,
48
    )
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
49
from bzrlib.trace import note
1553.5.12 by Martin Pool
New LockDir locking mechanism
50
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
51
# These tests are run on the default transport provided by the test framework
52
# (typically a local disk transport).  That can be changed by the --transport
53
# option to bzr selftest.  The required properties of the transport
54
# implementation are tested separately.  (The main requirement is just that
55
# they don't allow overwriting nonempty directories.)
56
5425.4.27 by Martin Pool
pep8 cleanups
57
1553.5.12 by Martin Pool
New LockDir locking mechanism
58
class TestLockDir(TestCaseWithTransport):
59
    """Test LockDir operations"""
60
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
61
    def logging_report_function(self, fmt, *args):
62
        self._logged_reports.append((fmt, args))
63
64
    def setup_log_reporter(self, lock_dir):
65
        self._logged_reports = []
66
        lock_dir._report_function = self.logging_report_function
67
1553.5.12 by Martin Pool
New LockDir locking mechanism
68
    def test_00_lock_creation(self):
69
        """Creation of lock file on a transport"""
70
        t = self.get_transport()
71
        lf = LockDir(t, 'test_lock')
1553.5.13 by Martin Pool
New Transport.rename that mustn't overwrite
72
        self.assertFalse(lf.is_held)
1553.5.12 by Martin Pool
New LockDir locking mechanism
73
74
    def test_01_lock_repr(self):
75
        """Lock string representation"""
76
        lf = LockDir(self.get_transport(), 'test_lock')
77
        r = repr(lf)
78
        self.assertContainsRe(r, r'^LockDir\(.*/test_lock\)$')
79
80
    def test_02_unlocked_peek(self):
81
        lf = LockDir(self.get_transport(), 'test_lock')
82
        self.assertEqual(lf.peek(), None)
83
1687.1.3 by Robert Collins
Make LockDir.unlock check the lock is still intact.
84
    def get_lock(self):
85
        return LockDir(self.get_transport(), 'test_lock')
86
87
    def test_unlock_after_break_raises(self):
88
        ld = self.get_lock()
89
        ld2 = self.get_lock()
90
        ld.create()
91
        ld.attempt_lock()
92
        ld2.force_break(ld2.peek())
93
        self.assertRaises(LockBroken, ld.unlock)
94
1553.5.12 by Martin Pool
New LockDir locking mechanism
95
    def test_03_readonly_peek(self):
96
        lf = LockDir(self.get_readonly_transport(), 'test_lock')
97
        self.assertEqual(lf.peek(), None)
98
99
    def test_10_lock_uncontested(self):
100
        """Acquire and release a lock"""
101
        t = self.get_transport()
102
        lf = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
103
        lf.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
104
        lf.attempt_lock()
105
        try:
1553.5.13 by Martin Pool
New Transport.rename that mustn't overwrite
106
            self.assertTrue(lf.is_held)
1553.5.12 by Martin Pool
New LockDir locking mechanism
107
        finally:
108
            lf.unlock()
1553.5.13 by Martin Pool
New Transport.rename that mustn't overwrite
109
            self.assertFalse(lf.is_held)
1553.5.12 by Martin Pool
New LockDir locking mechanism
110
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
111
    def test_11_create_readonly_transport(self):
112
        """Fail to create lock on readonly transport"""
113
        t = self.get_readonly_transport()
114
        lf = LockDir(t, 'test_lock')
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
115
        self.assertRaises(LockFailed, lf.create)
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
116
117
    def test_12_lock_readonly_transport(self):
1553.5.12 by Martin Pool
New LockDir locking mechanism
118
        """Fail to lock on readonly transport"""
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
119
        lf = LockDir(self.get_transport(), 'test_lock')
120
        lf.create()
121
        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).
122
        self.assertRaises(LockFailed, lf.attempt_lock)
1553.5.12 by Martin Pool
New LockDir locking mechanism
123
124
    def test_20_lock_contested(self):
125
        """Contention to get a lock"""
126
        t = self.get_transport()
127
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
128
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
129
        lf1.attempt_lock()
130
        lf2 = LockDir(t, 'test_lock')
131
        try:
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
132
            # locking is between LockDir instances; aliases within
1553.5.12 by Martin Pool
New LockDir locking mechanism
133
            # a single process are not detected
134
            lf2.attempt_lock()
135
            self.fail('Failed to detect lock collision')
136
        except LockContention, e:
137
            self.assertEqual(e.lock, lf2)
138
            self.assertContainsRe(str(e),
139
                    r'^Could not acquire.*test_lock.*$')
140
        lf1.unlock()
141
142
    def test_20_lock_peek(self):
143
        """Peek at the state of a lock"""
144
        t = self.get_transport()
145
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
146
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
147
        lf1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
148
        self.addCleanup(lf1.unlock)
1553.5.12 by Martin Pool
New LockDir locking mechanism
149
        # lock is held, should get some info on it
150
        info1 = lf1.peek()
5425.4.3 by Martin Pool
Represent lock held info as an object, not just a dict
151
        self.assertEqual(set(info1.info_dict.keys()),
5425.4.27 by Martin Pool
pep8 cleanups
152
            set(['user', 'nonce', 'hostname', 'pid', 'start_time']))
1553.5.12 by Martin Pool
New LockDir locking mechanism
153
        # should get the same info if we look at it through a different
154
        # instance
155
        info2 = LockDir(t, 'test_lock').peek()
156
        self.assertEqual(info1, info2)
157
        # locks which are never used should be not-held
158
        self.assertEqual(LockDir(t, 'other_lock').peek(), None)
159
160
    def test_21_peek_readonly(self):
161
        """Peek over a readonly transport"""
162
        t = self.get_transport()
163
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
164
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
165
        lf2 = LockDir(self.get_readonly_transport(), 'test_lock')
166
        self.assertEqual(lf2.peek(), None)
167
        lf1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
168
        self.addCleanup(lf1.unlock)
1553.5.12 by Martin Pool
New LockDir locking mechanism
169
        info2 = lf2.peek()
170
        self.assertTrue(info2)
5425.4.3 by Martin Pool
Represent lock held info as an object, not just a dict
171
        self.assertEqual(info2.get('nonce'), lf1.nonce)
1553.5.12 by Martin Pool
New LockDir locking mechanism
172
173
    def test_30_lock_wait_fail(self):
174
        """Wait on a lock, then fail
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
175
1553.5.12 by Martin Pool
New LockDir locking mechanism
176
        We ask to wait up to 400ms; this should fail within at most one
177
        second.  (Longer times are more realistic but we don't want the test
178
        suite to take too long, and this should do for now.)
179
        """
180
        t = self.get_transport()
181
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
182
        lf1.create()
1553.5.12 by Martin Pool
New LockDir locking mechanism
183
        lf2 = LockDir(t, 'test_lock')
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
184
        self.setup_log_reporter(lf2)
1553.5.12 by Martin Pool
New LockDir locking mechanism
185
        lf1.attempt_lock()
186
        try:
187
            before = time.time()
188
            self.assertRaises(LockContention, lf2.wait_lock,
189
                              timeout=0.4, poll=0.1)
190
            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
191
            # it should only take about 0.4 seconds, but we allow more time in
192
            # case the machine is heavily loaded
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
193
            self.assertTrue(after - before <= 8.0,
5425.4.27 by Martin Pool
pep8 cleanups
194
                "took %f seconds to detect lock contention" % (after - before))
1553.5.12 by Martin Pool
New LockDir locking mechanism
195
        finally:
196
            lf1.unlock()
1957.1.7 by John Arbash Meinel
Add the ability to report if the lock changes from underneath you
197
        self.assertEqual(1, len(self._logged_reports))
5425.4.18 by Martin Pool
Cleaner and consistent user-orientend representation of LockHeldInfo
198
        self.assertContainsRe(self._logged_reports[0][0],
199
            r'Unable to obtain lock .* held by jrandom@example\.com on .*'
200
            r' \(process #\d+\), acquired .* ago\.\n'
201
            r'Will continue to try until \d{2}:\d{2}:\d{2}, unless '
202
            r'you press Ctrl-C.\n'
203
            r'See "bzr help break-lock" for more.')
1553.5.12 by Martin Pool
New LockDir locking mechanism
204
205
    def test_31_lock_wait_easy(self):
206
        """Succeed when waiting on a lock with no contention.
207
        """
208
        t = self.get_transport()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
209
        lf1 = LockDir(t, 'test_lock')
210
        lf1.create()
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
211
        self.setup_log_reporter(lf1)
1553.5.12 by Martin Pool
New LockDir locking mechanism
212
        try:
213
            before = time.time()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
214
            lf1.wait_lock(timeout=0.4, poll=0.1)
1553.5.12 by Martin Pool
New LockDir locking mechanism
215
            after = time.time()
216
            self.assertTrue(after - before <= 1.0)
217
        finally:
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
218
            lf1.unlock()
1957.1.1 by John Arbash Meinel
Report to the user when we are spinning on a lock
219
        self.assertEqual([], self._logged_reports)
1553.5.12 by Martin Pool
New LockDir locking mechanism
220
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
221
    def test_40_confirm_easy(self):
222
        """Confirm a lock that's already held"""
223
        t = self.get_transport()
224
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
225
        lf1.create()
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
226
        lf1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
227
        self.addCleanup(lf1.unlock)
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
228
        lf1.confirm()
229
230
    def test_41_confirm_not_held(self):
231
        """Confirm a lock that's already held"""
232
        t = self.get_transport()
233
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
234
        lf1.create()
1553.5.20 by Martin Pool
Start adding LockDir.confirm() method
235
        self.assertRaises(LockNotHeld, lf1.confirm)
1553.5.23 by Martin Pool
Start LockDir.confirm method and LockBroken exception
236
237
    def test_42_confirm_broken_manually(self):
238
        """Confirm a lock broken by hand"""
239
        t = self.get_transport()
240
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
241
        lf1.create()
1553.5.23 by Martin Pool
Start LockDir.confirm method and LockBroken exception
242
        lf1.attempt_lock()
243
        t.move('test_lock', 'lock_gone_now')
244
        self.assertRaises(LockBroken, lf1.confirm)
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
245
        # Clean up
246
        t.move('lock_gone_now', 'test_lock')
247
        lf1.unlock()
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
248
249
    def test_43_break(self):
250
        """Break a lock whose caller has forgotten it"""
251
        t = self.get_transport()
252
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
253
        lf1.create()
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
254
        lf1.attempt_lock()
255
        # we incorrectly discard the lock object without unlocking it
256
        del lf1
257
        # someone else sees it's still locked
258
        lf2 = LockDir(t, 'test_lock')
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
259
        holder_info = lf2.peek()
260
        self.assertTrue(holder_info)
261
        lf2.force_break(holder_info)
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
262
        # now we should be able to take it
263
        lf2.attempt_lock()
4327.1.4 by Vincent Ladeuil
Fix lock test failures by taking lock breaking into account.
264
        self.addCleanup(lf2.unlock)
1553.5.25 by Martin Pool
New LockDir.force_break and simple test case
265
        lf2.confirm()
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
266
267
    def test_44_break_already_released(self):
268
        """Lock break races with regular release"""
269
        t = self.get_transport()
270
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
271
        lf1.create()
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
272
        lf1.attempt_lock()
273
        # someone else sees it's still locked
274
        lf2 = LockDir(t, 'test_lock')
275
        holder_info = lf2.peek()
276
        # in the interim the lock is released
277
        lf1.unlock()
278
        # break should succeed
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
279
        lf2.force_break(holder_info)
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
280
        # now we should be able to take it
281
        lf2.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
282
        self.addCleanup(lf2.unlock)
1553.5.26 by Martin Pool
Breaking an already-released lock should just succeed
283
        lf2.confirm()
284
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
285
    def test_45_break_mismatch(self):
286
        """Lock break races with someone else acquiring it"""
287
        t = self.get_transport()
288
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
289
        lf1.create()
1553.5.27 by Martin Pool
Confirm that only the intended holder of a lock was broken.
290
        lf1.attempt_lock()
291
        # someone else sees it's still locked
292
        lf2 = LockDir(t, 'test_lock')
293
        holder_info = lf2.peek()
294
        # in the interim the lock is released
295
        lf1.unlock()
296
        lf3 = LockDir(t, 'test_lock')
297
        lf3.attempt_lock()
298
        # break should now *fail*
299
        self.assertRaises(LockBreakMismatch, lf2.force_break,
300
                          holder_info)
301
        lf3.unlock()
1553.5.54 by Martin Pool
Add LockDir.read_lock fake method
302
303
    def test_46_fake_read_lock(self):
304
        t = self.get_transport()
305
        lf1 = LockDir(t, 'test_lock')
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
306
        lf1.create()
1553.5.54 by Martin Pool
Add LockDir.read_lock fake method
307
        lf1.lock_read()
308
        lf1.unlock()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
309
310
    def test_50_lockdir_representation(self):
311
        """Check the on-disk representation of LockDirs is as expected.
312
313
        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
314
        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"
315
        containing an info file.
316
        """
317
        t = self.get_transport()
318
        lf1 = LockDir(t, 'test_lock')
319
        lf1.create()
320
        self.assertTrue(t.has('test_lock'))
321
        lf1.lock_write()
322
        self.assertTrue(t.has('test_lock/held/info'))
323
        lf1.unlock()
324
        self.assertFalse(t.has('test_lock/held/info'))
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
325
326
    def test_break_lock(self):
327
        # the ui based break_lock routine should Just Work (tm)
328
        ld1 = self.get_lock()
329
        ld2 = self.get_lock()
330
        ld1.create()
331
        ld1.lock_write()
1687.1.6 by Robert Collins
Extend LockableFiles to support break_lock() calls.
332
        # do this without IO redirection to ensure it doesn't prompt.
333
        self.assertRaises(AssertionError, ld1.break_lock)
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
334
        orig_factory = bzrlib.ui.ui_factory
4449.3.27 by Martin Pool
More test updates to use CannedInputUIFactory
335
        bzrlib.ui.ui_factory = bzrlib.ui.CannedInputUIFactory([True])
1687.1.5 by Robert Collins
Add break_lock utility function to LockDir.
336
        try:
337
            ld2.break_lock()
338
            self.assertRaises(LockBroken, ld1.unlock)
339
        finally:
340
            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.
341
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
342
    def test_break_lock_corrupt_info(self):
343
        """break_lock works even if the info file is corrupt (and tells the UI
344
        that it is corrupt).
345
        """
346
        ld = self.get_lock()
347
        ld2 = self.get_lock()
348
        ld.create()
349
        ld.lock_write()
350
        ld.transport.put_bytes_non_atomic('test_lock/held/info', '\0')
5425.4.27 by Martin Pool
pep8 cleanups
351
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
352
        class LoggingUIFactory(bzrlib.ui.SilentUIFactory):
353
            def __init__(self):
354
                self.prompts = []
5425.4.27 by Martin Pool
pep8 cleanups
355
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
356
            def get_boolean(self, prompt):
357
                self.prompts.append(('boolean', prompt))
358
                return True
5425.4.27 by Martin Pool
pep8 cleanups
359
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
360
        ui = LoggingUIFactory()
5599.2.1 by John Arbash Meinel
Fix breaking corrupt lock files on Windows.
361
        self.overrideAttr(bzrlib.ui, 'ui_factory', ui)
362
        ld2.break_lock()
363
        self.assertLength(1, ui.prompts)
364
        self.assertEqual('boolean', ui.prompts[0][0])
365
        self.assertStartsWith(ui.prompts[0][1], 'Break (corrupt LockDir')
366
        self.assertRaises(LockBroken, ld.unlock)
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
367
368
    def test_break_lock_missing_info(self):
369
        """break_lock works even if the info file is missing (and tells the UI
370
        that it is corrupt).
371
        """
372
        ld = self.get_lock()
373
        ld2 = self.get_lock()
374
        ld.create()
375
        ld.lock_write()
376
        ld.transport.delete('test_lock/held/info')
5425.4.27 by Martin Pool
pep8 cleanups
377
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
378
        class LoggingUIFactory(bzrlib.ui.SilentUIFactory):
379
            def __init__(self):
380
                self.prompts = []
5425.4.27 by Martin Pool
pep8 cleanups
381
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
382
            def get_boolean(self, prompt):
383
                self.prompts.append(('boolean', prompt))
384
                return True
5425.4.27 by Martin Pool
pep8 cleanups
385
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
386
        ui = LoggingUIFactory()
387
        orig_factory = bzrlib.ui.ui_factory
388
        bzrlib.ui.ui_factory = ui
389
        try:
390
            ld2.break_lock()
391
            self.assertRaises(LockBroken, ld.unlock)
392
            self.assertLength(0, ui.prompts)
393
        finally:
394
            bzrlib.ui.ui_factory = orig_factory
395
        # Suppress warnings due to ld not being unlocked
396
        # XXX: if lock_broken hook was invoked in this case, this hack would
397
        # not be necessary.  - Andrew Bennetts, 2010-09-06.
398
        del self._lock_actions[:]
399
1955.1.1 by John Arbash Meinel
LockDir can create the root directory if it fails to create a pending directory due to NoSuchFile.
400
    def test_create_missing_base_directory(self):
401
        """If LockDir.path doesn't exist, it can be created
402
403
        Some people manually remove the entire lock/ directory trying
404
        to unlock a stuck repository/branch/etc. Rather than failing
405
        after that, just create the lock directory when needed.
406
        """
407
        t = self.get_transport()
408
        lf1 = LockDir(t, 'test_lock')
409
410
        lf1.create()
5784.1.1 by Martin Pool
Stop using failIf, failUnless, etc
411
        self.assertTrue(t.has('test_lock'))
1955.1.1 by John Arbash Meinel
LockDir can create the root directory if it fails to create a pending directory due to NoSuchFile.
412
413
        t.rmdir('test_lock')
5784.1.1 by Martin Pool
Stop using failIf, failUnless, etc
414
        self.assertFalse(t.has('test_lock'))
1955.1.1 by John Arbash Meinel
LockDir can create the root directory if it fails to create a pending directory due to NoSuchFile.
415
416
        # This will create 'test_lock' if it needs to
417
        lf1.lock_write()
5784.1.1 by Martin Pool
Stop using failIf, failUnless, etc
418
        self.assertTrue(t.has('test_lock'))
419
        self.assertTrue(t.has('test_lock/held/info'))
1955.1.1 by John Arbash Meinel
LockDir can create the root directory if it fails to create a pending directory due to NoSuchFile.
420
421
        lf1.unlock()
5784.1.1 by Martin Pool
Stop using failIf, failUnless, etc
422
        self.assertFalse(t.has('test_lock/held/info'))
1957.1.6 by John Arbash Meinel
[merge] bzr.dev 2009
423
5425.4.3 by Martin Pool
Represent lock held info as an object, not just a dict
424
    def test_display_form(self):
1957.1.5 by John Arbash Meinel
Create a helper function for formatting lock information
425
        ld1 = self.get_lock()
426
        ld1.create()
427
        ld1.lock_write()
428
        try:
5425.4.18 by Martin Pool
Cleaner and consistent user-orientend representation of LockHeldInfo
429
            info_list = ld1.peek().to_readable_dict()
1957.1.5 by John Arbash Meinel
Create a helper function for formatting lock information
430
        finally:
431
            ld1.unlock()
5425.4.18 by Martin Pool
Cleaner and consistent user-orientend representation of LockHeldInfo
432
        self.assertEqual(info_list['user'], u'jrandom@example.com')
5425.4.27 by Martin Pool
pep8 cleanups
433
        self.assertContainsRe(info_list['pid'], '^\d+$')
434
        self.assertContainsRe(info_list['time_ago'], r'^\d+ seconds? ago$')
2055.2.1 by John Arbash Meinel
Make LockDir less sensitive to invalid configuration of email
435
436
    def test_lock_without_email(self):
437
        global_config = config.GlobalConfig()
438
        # Intentionally has no email address
439
        global_config.set_user_option('email', 'User Identity')
440
        ld1 = self.get_lock()
441
        ld1.create()
442
        ld1.lock_write()
443
        ld1.unlock()
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
444
445
    def test_lock_permission(self):
4797.70.1 by Vincent Ladeuil
Skip chmodbits dependent tests when running as root
446
        self.requireFeature(features.not_running_as_root)
1551.10.4 by Aaron Bentley
Update to skip on win32
447
        if not osutils.supports_posix_readonly():
3107.2.2 by John Arbash Meinel
feedback from Martin.
448
            raise tests.TestSkipped('Cannot induce a permission failure')
1551.10.3 by Aaron Bentley
Lock attempts don't treat permission problems as lock contention
449
        ld1 = self.get_lock()
450
        lock_path = ld1.transport.local_abspath('test_lock')
451
        os.mkdir(lock_path)
452
        osutils.make_readonly(lock_path)
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
453
        self.assertRaises(errors.LockFailed, ld1.attempt_lock)
2555.3.5 by Martin Pool
Return token directly from LockDir.acquire to avoid unnecessary peek()
454
455
    def test_lock_by_token(self):
456
        ld1 = self.get_lock()
457
        token = ld1.lock_write()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
458
        self.addCleanup(ld1.unlock)
2555.3.5 by Martin Pool
Return token directly from LockDir.acquire to avoid unnecessary peek()
459
        self.assertNotEqual(None, token)
460
        ld2 = self.get_lock()
461
        t2 = ld2.lock_write(token)
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
462
        self.addCleanup(ld2.unlock)
2555.3.5 by Martin Pool
Return token directly from LockDir.acquire to avoid unnecessary peek()
463
        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)
464
465
    def test_lock_with_buggy_rename(self):
466
        # test that lock acquisition handles servers which pretend they
467
        # renamed correctly but that actually fail
468
        t = transport.get_transport('brokenrename+' + self.get_url())
469
        ld1 = LockDir(t, 'test_lock')
470
        ld1.create()
471
        ld1.attempt_lock()
472
        ld2 = LockDir(t, 'test_lock')
2555.3.14 by Martin Pool
Better handling in LockDir of rename that moves one directory within another
473
        # we should fail to lock
474
        e = self.assertRaises(errors.LockContention, ld2.attempt_lock)
475
        # now the original caller should succeed in unlocking
476
        ld1.unlock()
477
        # and there should be nothing left over
478
        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
479
480
    def test_failed_lock_leaves_no_trash(self):
481
        # if we fail to acquire the lock, we don't leave pending directories
482
        # behind -- https://bugs.launchpad.net/bzr/+bug/109169
483
        ld1 = self.get_lock()
484
        ld2 = self.get_lock()
485
        # should be nothing before we start
486
        ld1.create()
487
        t = self.get_transport().clone('test_lock')
5425.4.27 by Martin Pool
pep8 cleanups
488
2555.3.12 by Martin Pool
Add test for https://bugs.launchpad.net/bzr/+bug/109169 -- test_failed_lock_leaves_no_trash
489
        def check_dir(a):
490
            self.assertEquals(a, t.list_dir('.'))
5425.4.27 by Martin Pool
pep8 cleanups
491
2555.3.12 by Martin Pool
Add test for https://bugs.launchpad.net/bzr/+bug/109169 -- test_failed_lock_leaves_no_trash
492
        check_dir([])
493
        # when held, that's all we see
494
        ld1.attempt_lock()
4327.1.1 by Vincent Ladeuil
Start addressing test failing when run with -Dlock.
495
        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
496
        check_dir(['held'])
497
        # second guy should fail
498
        self.assertRaises(errors.LockContention, ld2.attempt_lock)
499
        # no kibble
500
        check_dir(['held'])
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
501
4634.138.1 by Martin Pool
Add failing test for \#185103
502
    def test_no_lockdir_info(self):
503
        """We can cope with empty info files."""
504
        # This seems like a fairly common failure case - see
5243.1.2 by Martin
Point launchpad links in comments at production server rather than edge
505
        # <https://bugs.launchpad.net/bzr/+bug/185103> and all its dupes.
4634.138.1 by Martin Pool
Add failing test for \#185103
506
        # Processes are often interrupted after opening the file
507
        # before the actual contents are committed.
508
        t = self.get_transport()
509
        t.mkdir('test_lock')
510
        t.mkdir('test_lock/held')
511
        t.put_bytes('test_lock/held/info', '')
512
        lf = LockDir(t, 'test_lock')
4634.138.2 by Martin Pool
Copy with lock info file being empty
513
        info = lf.peek()
5425.4.18 by Martin Pool
Cleaner and consistent user-orientend representation of LockHeldInfo
514
        formatted_info = info.to_readable_dict()
4634.138.2 by Martin Pool
Copy with lock info file being empty
515
        self.assertEquals(
5425.4.18 by Martin Pool
Cleaner and consistent user-orientend representation of LockHeldInfo
516
            dict(user='<unknown>', hostname='<unknown>', pid='<unknown>',
517
                time_ago='(unknown)'),
4634.138.2 by Martin Pool
Copy with lock info file being empty
518
            formatted_info)
4634.138.1 by Martin Pool
Add failing test for \#185103
519
4634.161.1 by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files.
520
    def test_corrupt_lockdir_info(self):
521
        """We can cope with corrupt (and thus unparseable) info files."""
522
        # 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.
523
        # <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.
524
        # In particular some systems tend to fill recently created files with
525
        # nul bytes after recovering from a system crash.
526
        t = self.get_transport()
527
        t.mkdir('test_lock')
528
        t.mkdir('test_lock/held')
529
        t.put_bytes('test_lock/held/info', '\0')
530
        lf = LockDir(t, 'test_lock')
531
        self.assertRaises(errors.LockCorrupt, lf.peek)
532
        # Currently attempt_lock gives LockContention, but LockCorrupt would be
533
        # a reasonable result too.
534
        self.assertRaises(
535
            (errors.LockCorrupt, errors.LockContention), lf.attempt_lock)
536
        self.assertRaises(errors.LockCorrupt, lf.validate_token, 'fake token')
537
538
    def test_missing_lockdir_info(self):
539
        """We can cope with absent info files."""
540
        t = self.get_transport()
541
        t.mkdir('test_lock')
542
        t.mkdir('test_lock/held')
543
        lf = LockDir(t, 'test_lock')
544
        # In this case we expect the 'not held' result from peek, because peek
545
        # cannot be expected to notice that there is a 'held' directory with no
546
        # 'info' file.
547
        self.assertEqual(None, lf.peek())
548
        # And lock/unlock may work or give LockContention (but not any other
549
        # error).
550
        try:
551
            lf.attempt_lock()
552
        except LockContention:
553
            # LockContention is ok, and expected on Windows
554
            pass
555
        else:
556
            # no error is ok, and expected on POSIX (because POSIX allows
557
            # os.rename over an empty directory).
558
            lf.unlock()
559
        # Currently raises TokenMismatch, but LockCorrupt would be reasonable
560
        # too.
561
        self.assertRaises(
562
            (errors.TokenMismatch, errors.LockCorrupt),
563
            lf.validate_token, 'fake token')
564
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
565
566
class TestLockDirHooks(TestCaseWithTransport):
567
568
    def setUp(self):
569
        super(TestLockDirHooks, self).setUp()
570
        self._calls = []
571
572
    def get_lock(self):
573
        return LockDir(self.get_transport(), 'test_lock')
574
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
575
    def record_hook(self, result):
576
        self._calls.append(result)
577
3331.3.12 by Martin Pool
Remove PhysicalLock class
578
    def test_LockDir_acquired_success(self):
579
        # the LockDir.lock_acquired hook fires when a lock is acquired.
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
580
        LockDir.hooks.install_named_hook('lock_acquired',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
581
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
582
        ld = self.get_lock()
583
        ld.create()
584
        self.assertEqual([], self._calls)
585
        result = ld.attempt_lock()
3331.3.2 by Robert Collins
Polish on lock hooks to be easier to use.
586
        lock_path = ld.transport.abspath(ld.path)
587
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
588
        ld.unlock()
3331.3.2 by Robert Collins
Polish on lock hooks to be easier to use.
589
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
590
3331.3.12 by Martin Pool
Remove PhysicalLock class
591
    def test_LockDir_acquired_fail(self):
592
        # 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
593
        ld = self.get_lock()
594
        ld.create()
595
        ld2 = self.get_lock()
596
        ld2.attempt_lock()
597
        # install a lock hook now, when the disk lock is locked
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
598
        LockDir.hooks.install_named_hook('lock_acquired',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
599
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
600
        self.assertRaises(errors.LockContention, ld.attempt_lock)
601
        self.assertEqual([], self._calls)
602
        ld2.unlock()
603
        self.assertEqual([], self._calls)
604
3331.3.12 by Martin Pool
Remove PhysicalLock class
605
    def test_LockDir_released_success(self):
606
        # the LockDir.lock_released hook fires when a lock is acquired.
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
607
        LockDir.hooks.install_named_hook('lock_released',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
608
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
609
        ld = self.get_lock()
610
        ld.create()
611
        self.assertEqual([], self._calls)
612
        result = ld.attempt_lock()
613
        self.assertEqual([], self._calls)
614
        ld.unlock()
3331.3.2 by Robert Collins
Polish on lock hooks to be easier to use.
615
        lock_path = ld.transport.abspath(ld.path)
616
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
617
3331.3.12 by Martin Pool
Remove PhysicalLock class
618
    def test_LockDir_released_fail(self):
619
        # 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
620
        ld = self.get_lock()
621
        ld.create()
622
        ld2 = self.get_lock()
623
        ld.attempt_lock()
624
        ld2.force_break(ld2.peek())
3331.3.11 by Martin Pool
Move LockDir hooks onto LockDir
625
        LockDir.hooks.install_named_hook('lock_released',
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
626
                                         self.record_hook, 'record_hook')
3331.3.1 by Robert Collins
* ``LockDir`` lock acquisition and release now trigger hooks allowing
627
        self.assertRaises(LockBroken, ld.unlock)
628
        self.assertEqual([], self._calls)
4327.1.2 by Vincent Ladeuil
Introduce a new lock_broken hook.
629
630
    def test_LockDir_broken_success(self):
631
        # the LockDir.lock_broken hook fires when a lock is broken.
632
        ld = self.get_lock()
633
        ld.create()
634
        ld2 = self.get_lock()
635
        result = ld.attempt_lock()
636
        LockDir.hooks.install_named_hook('lock_broken',
637
                                         self.record_hook, 'record_hook')
638
        ld2.force_break(ld2.peek())
639
        lock_path = ld.transport.abspath(ld.path)
640
        self.assertEqual([lock.LockResult(lock_path, result)], self._calls)
641
642
    def test_LockDir_broken_failure(self):
643
        # the LockDir.lock_broken hook does not fires when a lock is already
644
        # released.
645
        ld = self.get_lock()
646
        ld.create()
647
        ld2 = self.get_lock()
648
        result = ld.attempt_lock()
649
        holder_info = ld2.peek()
650
        ld.unlock()
651
        LockDir.hooks.install_named_hook('lock_broken',
652
                                         self.record_hook, 'record_hook')
653
        ld2.force_break(holder_info)
654
        lock_path = ld.transport.abspath(ld.path)
655
        self.assertEqual([], self._calls)
5425.4.1 by Martin Pool
add xfail tests for stale locks
656
657
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
658
class TestLockHeldInfo(TestCase):
659
    """Can get information about the lock holder, and detect whether they're
660
    still alive."""
5425.4.1 by Martin Pool
add xfail tests for stale locks
661
5425.4.17 by Martin Pool
Add test for LockHeldInfo repr
662
    def test_repr(self):
663
        info = LockHeldInfo.for_this_process(None)
5425.4.18 by Martin Pool
Cleaner and consistent user-orientend representation of LockHeldInfo
664
        self.assertContainsRe(repr(info), r"LockHeldInfo\(.*\)")
665
666
    def test_unicode(self):
667
        info = LockHeldInfo.for_this_process(None)
668
        self.assertContainsRe(unicode(info),
669
            r'held by .* on .* \(process #\d+\), acquired .* ago')
5425.4.17 by Martin Pool
Add test for LockHeldInfo repr
670
5425.4.7 by Martin Pool
Add and test is_locked_by_this_process
671
    def test_is_locked_by_this_process(self):
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
672
        info = LockHeldInfo.for_this_process(None)
5425.4.7 by Martin Pool
Add and test is_locked_by_this_process
673
        self.assertTrue(info.is_locked_by_this_process())
5425.4.6 by Martin Pool
Move preparation of new locks into LockHeldInfo
674
5425.4.8 by Martin Pool
Basic detection of known dead lock holders
675
    def test_is_not_locked_by_this_process(self):
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
676
        info = LockHeldInfo.for_this_process(None)
5425.4.27 by Martin Pool
pep8 cleanups
677
        info.info_dict['pid'] = '123123123123123'
5425.4.8 by Martin Pool
Basic detection of known dead lock holders
678
        self.assertFalse(info.is_locked_by_this_process())
679
680
    def test_lock_holder_live_process(self):
681
        """Detect that the holder (this process) is still running."""
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
682
        info = LockHeldInfo.for_this_process(None)
5425.4.8 by Martin Pool
Basic detection of known dead lock holders
683
        self.assertFalse(info.is_lock_holder_known_dead())
5425.4.27 by Martin Pool
pep8 cleanups
684
5425.4.8 by Martin Pool
Basic detection of known dead lock holders
685
    def test_lock_holder_dead_process(self):
686
        """Detect that the holder (this process) is still running."""
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
687
        info = LockHeldInfo.for_this_process(None)
5425.4.27 by Martin Pool
pep8 cleanups
688
        info.info_dict['pid'] = '123123123'
5425.4.8 by Martin Pool
Basic detection of known dead lock holders
689
        self.assertTrue(info.is_lock_holder_known_dead())
5425.4.27 by Martin Pool
pep8 cleanups
690
5425.4.1 by Martin Pool
add xfail tests for stale locks
691
    def test_lock_holder_other_machine(self):
692
        """The lock holder isn't here so we don't know if they're alive."""
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
693
        info = LockHeldInfo.for_this_process(None)
5425.6.1 by Martin Pool
Only detect as stale locks from the same user
694
        info.info_dict['hostname'] = 'egg.example.com'
5425.4.27 by Martin Pool
pep8 cleanups
695
        info.info_dict['pid'] = '123123123'
5425.6.1 by Martin Pool
Only detect as stale locks from the same user
696
        self.assertFalse(info.is_lock_holder_known_dead())
697
698
    def test_lock_holder_other_user(self):
699
        """Only auto-break locks held by this user."""
700
        info = LockHeldInfo.for_this_process(None)
701
        info.info_dict['user'] = 'notme@example.com'
5425.4.27 by Martin Pool
pep8 cleanups
702
        info.info_dict['pid'] = '123123123'
5425.4.8 by Martin Pool
Basic detection of known dead lock holders
703
        self.assertFalse(info.is_lock_holder_known_dead())
5425.4.1 by Martin Pool
add xfail tests for stale locks
704
705
    def test_no_good_hostname(self):
706
        """Correctly handle ambiguous hostnames.
707
708
        If the lock's recorded with just 'localhost' we can't really trust
5425.4.9 by Martin Pool
Defensively handle 'localhost' in detecting dead locks
709
        it's the same 'localhost'.  (There are quite a few of them. :-)
710
        So even if the process is known not to be alive, we can't say that's
711
        known for sure.
5425.4.1 by Martin Pool
add xfail tests for stale locks
712
        """
5425.4.9 by Martin Pool
Defensively handle 'localhost' in detecting dead locks
713
        self.overrideAttr(lockdir, 'get_host_name',
714
            lambda: 'localhost')
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
715
        info = LockHeldInfo.for_this_process(None)
5425.4.9 by Martin Pool
Defensively handle 'localhost' in detecting dead locks
716
        info.info_dict['pid'] = '123123123'
717
        self.assertFalse(info.is_lock_holder_known_dead())
5425.4.1 by Martin Pool
add xfail tests for stale locks
718
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
719
720
class TestStaleLockDir(TestCaseWithTransport):
721
    """Can automatically break stale locks.
722
723
    :see: https://bugs.launchpad.net/bzr/+bug/220464
724
    """
5425.4.25 by Martin Pool
Support auto-stealing dead locks, but turn it off by default.
725
5425.4.1 by Martin Pool
add xfail tests for stale locks
726
    def test_auto_break_stale_lock(self):
727
        """Locks safely known to be stale are just cleaned up.
728
729
        This generates a warning but no other user interaction.
730
        """
5425.4.25 by Martin Pool
Support auto-stealing dead locks, but turn it off by default.
731
        # This is off by default at present; see the discussion in the bug.
732
        # If you change the default, don't forget to update the docs.
733
        config.GlobalConfig().set_user_option('locks.steal_dead', True)
5425.4.10 by Martin Pool
Allow storing extra info into lockdir info files to aid testing
734
        # Create a lock pretending to come from a different nonexistent
735
        # process on the same machine.
736
        l1 = LockDir(self.get_transport(), 'a',
737
            extra_holder_info={'pid': '12312313'})
738
        token_1 = l1.attempt_lock()
739
        l2 = LockDir(self.get_transport(), 'a')
5425.4.13 by Martin Pool
Steal locks of dead local processes, with a user warning
740
        token_2 = l2.attempt_lock()
741
        # l1 will notice its lock was stolen.
742
        self.assertRaises(errors.LockBroken,
743
            l1.unlock)
744
        l2.unlock()
5425.4.1 by Martin Pool
add xfail tests for stale locks
745
746
    def test_auto_break_stale_lock_configured_off(self):
5425.4.15 by Martin Pool
Add steal_dead_locks option letting automatic lock-breaking be turned off
747
        """Automatic breaking can be turned off"""
748
        l1 = LockDir(self.get_transport(), 'a',
749
            extra_holder_info={'pid': '12312313'})
750
        token_1 = l1.attempt_lock()
751
        self.addCleanup(l1.unlock)
752
        l2 = LockDir(self.get_transport(), 'a')
5425.4.25 by Martin Pool
Support auto-stealing dead locks, but turn it off by default.
753
        # This fails now, because dead lock breaking is off by default.
5425.4.15 by Martin Pool
Add steal_dead_locks option letting automatic lock-breaking be turned off
754
        self.assertRaises(LockContention,
755
            l2.attempt_lock)
756
        # and it's in fact not broken
757
        l1.confirm()