[ 
https://issues.apache.org/jira/browse/LUCENE-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13561972#comment-13561972
 ] 

Commit Tag Bot commented on LUCENE-4691:
----------------------------------------

[trunk commit] Shai Erera
http://svn.apache.org/viewvc?view=revision&revision=1438188

LUCENE-4691: Facet tests should randomly pick IntEncoder and other stuff

                
> 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
>         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: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to