openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Dick (JIRA)" <j...@apache.org>
Subject [jira] Commented: (OPENJPA-949) Allow override of Surefire test excludes from cmdline
Date Mon, 02 Mar 2009 22:38:56 GMT

    [ https://issues.apache.org/jira/browse/OPENJPA-949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12678138#action_12678138
] 

Michael Dick commented on OPENJPA-949:
--------------------------------------

Just MHO : 

1. I like Don's patch. It's much easier to edit a single file than it is to pore through pom.xml.
I never would have thought of using the groovy plugin that way. 

2. I think a property that may be overridden by something in settings.xml is probably a better
approach. Settings.xml is never committed so there's no chance of an accidental commit going
unnoticed. We currently exclude a lot of tests though so the property would have to be rather
large and surefire / maven may have trouble with them in a single property. 
$ mvn -Dopenjpa.excludedTests=org/apache/openjpa/persistence/kernel/TestPMMemory.java,org/apache/openjpa/persistence/jpql/clause/TestSomething.java,org/apache/openjpa/datacache/TestSomethingElse.java,
. . . , { new exclusions go here }

3. I'm ambivalent re @AllowFailure. In concept it's a nice idea, but I don't like the idea
that svn revision 101 might pass a certain test and revision 120 regresses it and we'd never
know. It also seems to take a lot of time to run tests that don't impact whether the build
is "good". 

4. I might be in the minority about @AllowFailure. :-)

5. Pinaki's comments about having the maven build process restore the filter file are great
if you're doing the build on your system. If you do it on TeamCity then having maven restore
the file won't do you any good. TeamCity will restore the copy on the TeamCity server but
the commit comes from the client. So you'll have to manually revert the file or do your own
local build.

6. The best current solution might be to take Don's patch and make the exclusions filename
a property. Different profiles use different (local and unversion controlled) exclusion files.
Local users make their own exclusion file at least until we reduce the number or come up with
a shorter list. 


> Allow override of Surefire test excludes from cmdline
> -----------------------------------------------------
>
>                 Key: OPENJPA-949
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-949
>             Project: OpenJPA
>          Issue Type: Improvement
>          Components: build / infrastructure
>            Reporter: Donald Woods
>            Assignee: Donald Woods
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: OPENJPA-949-trunk-20090302.patch
>
>
> Allow users to provide their own Surefire excludes list through an external file.
> This can be accomplished by using the gmaven-plugin and some Groovy code.
> The default set of excludes will still be provided and used by default in the builds,
but will be moved to src/test/resources/excludes.xml.
> Users can provide their own version of excludes file by:
> mvn -DsurefireExcludesFile=<filename>
> where <filename> could be a local file like:
> -DsurefireExcludesFile=/tmp/excludes.xml
> or from a network share like:
> -DsurefireExcludesFile=/mnt/public/openjpa-persistence-jdbc/excludes.xml

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message