[ 
https://issues.apache.org/jira/browse/LUCENE-3273?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Muir updated LUCENE-3273:
--------------------------------

    Attachment: LUCENE-3273_testboolean2.patch

here's my example, TestBoolean2.

in my opinion building the queries like this makes the test much more readable.

it adds 48 lines and deletes 29 lines of code... 

I think adding these 19 lines of code to this 343 line test case is worth every 
penny, because its much easier to see what any given test does, e.g. just 
glance real quick at testQueries06 and you see its a BQ with one MUST and two 
MUST_NOTS, no parsing by the brain required.


> Convert Lucene Core tests over to a simple MockQueryParser
> ----------------------------------------------------------
>
>                 Key: LUCENE-3273
>                 URL: https://issues.apache.org/jira/browse/LUCENE-3273
>             Project: Lucene - Java
>          Issue Type: Sub-task
>          Components: core/other
>            Reporter: Chris Male
>         Attachments: LUCENE-3273.patch, LUCENE-3273_testboolean2.patch
>
>
> Most tests use Lucene Core's QueryParser for convenience.  We want to 
> consolidate it into a QP module which we can't have as a dependency.  We 
> should add a simple MockQueryParser which does String.split() on the query 
> string, analyzers the terms and builds a BooleanQuery if necessary.  Any more 
> complex Queries (such as phrases) should be done programmatically. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to