lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Robert Muir <>
Subject Re: svn commit: r1351829 - in /lucene/dev/trunk/lucene: analysis/kuromoji/src/test/org/apache/lucene/analysis/ja/ test-framework/src/java/org/apache/lucene/analysis/
Date Tue, 19 Jun 2012 22:18:22 GMT
On Tue, Jun 19, 2012 at 6:13 PM, Michael McCandless
<> wrote:
> Ahhh good point: using the annotation, LuceneTestCase will simply pick
> a codec that's allowed, and the test runs.
> But using an Assume somewhere inside means ... LuceneTestCase can pick
> the "wrong" codec and then silently the test doesn't run.
> And, I hadn't realized I can add the annot to BaseTokenStreamTestCase
> and all tests inheriting from it will respect that.  Though I guess
> that means any test inheriting from BTSTC and not using
> checkRandomData with biggish args loses some test coverage ... but
> that seems the lesser evil here.

Looking at your commit, i fixed it with a different approach: if your
logic doesn't match, we just dont index anything (but still test the
actual analyzer).


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

View raw message