lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simon Willnauer (JIRA)" <>
Subject [jira] Updated: (LUCENE-2652) Rethink LocalizedTestCaseRunner with JUnit 4 - Clover OOM
Date Sun, 19 Sep 2010 11:44:32 GMT


Simon Willnauer updated LUCENE-2652:

    Attachment: LUCENE-2652.patch

I added support for tests.locale=all && tests.codec=all while still utilizing the
TestRunners we used before. With this patch we preserve all functionality we had before  -
being able to run with all locale / codec types for those tests which have been marked with
the corresponding runners. Should we commit that until we change to a single runner?

Yet, i think what we should ultimately do is remove the runners and add support to run all
permutations like a test with all codecs, locale , timezone etc. I don't think that it makes
lot of sense to do that for every test but maybe we want to offer the test to tell if it wants
to run with all locale and / or all codec .... 

maybe something like 


enum TestProperty {
  Locale, Codec, Timezone



setTestProperties(TestProperty ... properties) {


> Rethink LocalizedTestCaseRunner with JUnit 4 - Clover OOM
> ---------------------------------------------------------
>                 Key: LUCENE-2652
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Test
>          Components: Tests
>            Reporter: Simon Willnauer
>            Assignee: Simon Willnauer
>             Fix For: 3.1, 4.0
>         Attachments: LUCENE-2652.patch, LUCENE-2652.patch, LUCENE-2652.patch
> As a spinn off from this [conversation|]
we should rethink the way how we execute testcases with different locals since glover reports
appears to throw OOM errors b/c Junit treats each local as a single test case run.
> Here are some options:
> * select the local at random only run the test with a single local
> * set the local via system property -Dtest.locale=en.EN
> * run with the default locale only -Dtest.skiplocale=true
> * one from the above but only if instrumented with clover (let common tests run all the

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