openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Albert Lee (JIRA)" <>
Subject [jira] Commented: (OPENJPA-770) Use annotations instead of excluding tests in pom.xml
Date Fri, 13 Feb 2009 15:17:59 GMT


Albert Lee commented on OPENJPA-770:

Tim, I took a look at the patch, It does what is asked for in the original request. 

To keep the changes cleaner and to a minimum, I wonder a class level @AllowFailure can be
applied to some of these tests instead of annotating in every method. Also consider the use
of @AllowFailure(value=false) to negate test selection and/or exclusions.

Thanks for the patch.
Albert Lee.

> Use annotations instead of excluding tests in pom.xml
> -----------------------------------------------------
>                 Key: OPENJPA-770
>                 URL:
>             Project: OpenJPA
>          Issue Type: Sub-task
>    Affects Versions: 2.0.0
>            Reporter: Michael Dick
>            Assignee: Tim McConnell
>             Fix For: 2.0.0
>         Attachments: OPENJPA-770.patch
>   Original Estimate: 3h
>  Remaining Estimate: 3h
> In openjpa-persistence-jdbc/pom.xml there are a lot of excluded testcases. Instead of
hard coding these in pom.xml we should use the new annotation that Pinaki introduced in the
"parent" JIRA issue. 
> The list of excluded tests can be found in pom.xml, here's a subsection of them
> <!-- exclude new tests that aren't passing yet -->
>                         <exclude>org/apache/openjpa/persistence/jpql/clauses/</exclude>
>                         <exclude>org/apache/openjpa/persistence/kernel/</exclude>
>                         <exclude>org/apache/openjpa/persistence/kernel/</exclude>
>                         <exclude>org/apache/openjpa/persistence/query/</exclude>

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

View raw message