[LUCENE-3152] MockDirectoryWrapper should wrap the lockfactory

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.3, 4.0-ALPHA
• Component/s: general/test
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Robert Muir
• Created: 28/May/11 22:54
• Updated: 28/Aug/22 12:48
• Resolved: 09/Jun/11 16:21

Description

After applying the patch from LUCENE-3147, I added a line to make the test fail if it cannot remove its temporary directory.

I ran 'ant test' on linux 50 times, and it passed all 50 times.
But on windows, it failed often because of write.lock... this is because of unclosed writers in the test.

MockDirectoryWrapper is currently unaware of this write.lock, I think it should wrap the lockfactory so that .close() will fail if there are any outstanding locks.
Then hopefully these tests would fail on linux too.

Issue Links

• is part of: LUCENE-3147 MockDirectoryWrapper should track open file handles of IndexOutput too