Failing Test: Lucene.Net.Index.TestIndexWriter::TestTwoThreadsInterruptDeadlock()

[NightOwl888]

For information about how to help us debug tests, see #269.

This test was previously reported as failing in #269. However, the behavior is slightly different now after the patch was applied in #525.

This test only started failing again after fixing the test framework to randomize the codecs, culture and time zone (NUnit's Randomizer was always initialized with a seed of 0, so none of these were actually random).

Lucene.Net.Util.LuceneSystemException : MockDirectoryWrapper: cannot close: there are still open locks: [write.lock]
Data:
OriginalMessage: Lucene.Net.Util.LuceneSystemException: MockDirectoryWrapper: cannot close: there are still open locks: [write.lock]
at Lucene.Net.Store.MockDirectoryWrapper.Dispose(Boolean disposing) in D:\a\1\s\src\Lucene.Net.TestFramework\Store\MockDirectoryWrapper.cs:line 1096
at Lucene.Net.Store.Directory.Dispose() in D:\a\1\s\src\Lucene.Net\Store\Directory.cs:line 134
at Lucene.Net.Util.IOUtils.ReThrowUnchecked(Exception th) in D:\a\1\s\src\Lucene.Net\Util\IOUtils.cs:line 530
at Lucene.Net.Index.TestIndexWriter.IndexerThreadInterrupt.Run() in D:\a\1\s\src\Lucene.Net.Tests\Index\TestIndexWriter.cs:line 1434
at J2N.Threading.ThreadJob.SafeRun(ThreadStart start)

To reproduce this test result:

Option 1:

Apply the following assembly-level attributes:

[assembly: Lucene.Net.Util.RandomSeed(0x988353ef90a15617L)]
[assembly: NUnit.Framework.SetCulture("fr-CA")]

Option 2:

Use the following .runsettings file:

<RunSettings>
  <TestRunParameters>
    <Parameter name="tests:seed" value="0x988353ef90a15617" />
    <Parameter name="tests:culture" value="fr-CA" />
  </TestRunParameters>
</RunSettings>

See the .runsettings documentation at: https://docs.microsoft.com/en-us/visualstudio/test/configure-unit-tests-by-using-a-dot-runsettings-file.
at Lucene.Net.Store.MockDirectoryWrapper.Dispose(Boolean disposing) in D:\a\1\s\src\Lucene.Net.TestFramework\Store\MockDirectoryWrapper.cs:line 1096
at Lucene.Net.Store.Directory.Dispose() in D:\a\1\s\src\Lucene.Net\Store\Directory.cs:line 134
at Lucene.Net.Util.IOUtils.ReThrowUnchecked(Exception th) in D:\a\1\s\src\Lucene.Net\Util\IOUtils.cs:line 530
at Lucene.Net.Index.TestIndexWriter.IndexerThreadInterrupt.Run() in D:\a\1\s\src\Lucene.Net.Tests\Index\TestIndexWriter.cs:line 1434
at J2N.Threading.ThreadJob.SafeRun(ThreadStart start)
--- End of stack trace from previous location ---
at Lucene.Net.Index.TestIndexWriter.TestTwoThreadsInterruptDeadlock() in D:\a\1\s\src\Lucene.Net.Tests\Index\TestIndexWriter.cs:line 1556

The test is not repeatable with the random seed, which likely indicates we have an actual concurrency issue this time. We also do not see Lucene.Net.Index.TestIndexWriter.TestThreadInterruptDeadlock() fail anymore, which indicates likewise.

NOTE: I am working on a PR to fix the random seed functionality #288 now, and the settings in the error message don't exist in master yet.

It is failing on every target except .NET Framework, every OS and both in x86 and x64, but rarely (less than 1 run out of 20).