lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Busch (JIRA)" <>
Subject [jira] Commented: (LUCENE-486) Core Test should not have dependencies on the Demo code
Date Wed, 23 Sep 2009 12:27:16 GMT


Michael Busch commented on LUCENE-486:

I still think we should remove the dependency that test has on demo. Looking into the code
you can replace the call of FileDocument in TestDoc with just 2 lines:
Document doc = new Document();
doc.add(new Field("contents", new FileReader(file)));

With this change alone we could remove the dependency. I think we should do that because it
makes the build cleaner and I don't think we'll pay less attention to the demo code in the
future just because we remove this classpath dependency. Any objections? Otherwise I'll reopen
this issue and attach a patch.

> Core Test should not have dependencies on the Demo code
> -------------------------------------------------------
>                 Key: LUCENE-486
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Test
>    Affects Versions: 1.4
>            Reporter: Grant Ingersoll
>            Priority: Minor
>         Attachments:, testdoc.patch
> The Test file has a dependency on the Demo FileDocument code.  Some of us
don't keep the Demo code around after downloading, so this breaks the build.
> Patch will be along shortly

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