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

Shai Erera resolved LUCENE-4691.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 5.0
                   4.2

Committed to trunk and 4x.
                
> Facet tests should randomly pick IntEncoder and other stuff
> -----------------------------------------------------------
>
>                 Key: LUCENE-4691
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4691
>             Project: Lucene - Core
>          Issue Type: Test
>          Components: modules/facet
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>             Fix For: 4.2, 5.0
>
>         Attachments: LUCENE-4691.patch
>
>
> Facets have many encoders already, but only one of them is tested by all 
> facet tests (the default). While {{EncodingTest}} covers all encoders, I 
> already fixed it because it didn't catch a bug where one encoder didn't reset 
> the given {{BytesRef}}. It will be useful if tests could choose a 
> randomCategoryListParams(), or randomFacetIndexingParams() etc., to swap in 
> random encoders.
> Also, I wrote an AssertingCategoryListIterator which asserts all kind of 
> things (e.g. that if setNextReader returns false, getOrdinals isn't called). 
> Would be good to use it by all tests.
> To do that, I think that it would be best if all facet tests extend a 
> FacetTestCase with such utility methods.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to