[MERGE] Cleanup and test Lock objects

[John Arbash Meinel]

John Arbash Meinel wrote:
> Martin Pool wrote:
>> Martin Pool has voted +1.
>> Status is now: Approved
>> Comment:
>> +    _fmt = "Cannot acquire write lock on %(fname)s. File is readonly."
>>
>> Adding a specific exception class sounds good.  I think you could get
>> eaccess for other reasons when the file is not readonly?  Maybe it
>> would be better to include the message from the IOError.
>>
>> I agree with Alexander that we should perhaps have per-implementation
>> tests.
> 

Well to give myself some more experience with writing adapted tests, I
have went ahead and done this in the attached patch.

It is a large enough change, that I would like someone to look it over,
even though the other one was +1.

John
=:->
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: locks_with_implementation_tests.patch
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20070314/c9252406/attachment-0001.diff