Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-3152

MockDirectoryWrapper should wrap the lockfactory

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3, 4.0-ALPHA
    • Component/s: general/test
    • Labels:
      None
    • Lucene Fields:
      New

      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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                rcmuir Robert Muir
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: