lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Muir (JIRA)" <>
Subject [jira] Created: (LUCENE-2804) check all tests that use
Date Mon, 06 Dec 2010 15:55:11 GMT
check all tests that use

                 Key: LUCENE-2804
             Project: Lucene - Java
          Issue Type: Test
            Reporter: Robert Muir

In LUCENE-2471 we were discussing the copyBytes issue, and Shai and I had a discussion about
how we could prevent such bugs in the future.

One thing that lead to the bug existing in our code for so long, was that it only happened
on windows (e.g. never failed in hudson!)
This was because the bug only happened if you were copying from SimpleFSDirectory, and the
test used

Today the situation is improving: most tests use newDirectory() which is random by default
and never use,
it always uses SimpleFS or NIOFS so that the same random seed will reproduce across both windows
and unix.

So I think we need to review all uses of in our tests, and minimize these.
In general tests should use newDirectory().
If the test comes with say a zip-file and wants to explicitly open stuff from disk, I think
it should open the contents with say SimpleFSDir,
and then call newDirectory(Directory) to copy into a new "random" implementation for actual
testing. This method already exists:
   * Returns a new Dictionary instance, with contents copied from the
   * provided directory. See {@link #newDirectory()} for more
   * information.
  public static MockDirectoryWrapper newDirectory(Directory d) throws IOException {

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message