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

Steve Rowe updated LUCENE-5930:
-------------------------------
    Attachment: LUCENE-5930.patch

bq. My only concern is adding the @Ignore. That breaks running 
CreateBackwardsCompatibilityIndex from ant. I think intellij should be using 
the same test file pattern ant does?

Thanks Ryan, I didn't realize the intent was to invoke 
{{CreateBackwardsCompatibilityIndex}} as a test from ant.

This version of the patch doesn't add the @Ignore, and for the backward-codecs 
module test uses a pattern (regex against the fully qualified class name) 
{{.\*\.Test\[^.\]\*$|.\*Test$}}, matching test class names that start or end 
with "Test", which finds all tests in the module except 
{{CreateBackwardsCompatibilityIndex}}.

Committing shortly.


> IntelliJ config: drop resource-only modules, add module groups, and add 
> module for lucene/backward-codecs
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-5930
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5930
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Ryan Ernst
>         Attachments: LUCENE-5930.patch, LUCENE-5930.patch, LUCENE-5930.patch, 
> LUCENE-5930.patch
>
>
> The number of intellij modules is getting out of hand.  Intellij supports 
> marking subdirectories within a module as 
> source/resources/tests/test-resources.  I think we should consolidate these 
> modules so we have just one per lucene module.  Is there some reason I'm 
> missing that this was not done in the first place?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to