aries-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ARIES-1188) Remove pax exam dependency from org.apache.aries.testsupport.unit
Date Fri, 16 May 2014 10:53:05 GMT

    [ https://issues.apache.org/jira/browse/ARIES-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999048#comment-13999048
] 

ASF GitHub Bot commented on ARIES-1188:
---------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/aries/pull/10


> Remove pax exam dependency from org.apache.aries.testsupport.unit
> -----------------------------------------------------------------
>
>                 Key: ARIES-1188
>                 URL: https://issues.apache.org/jira/browse/ARIES-1188
>             Project: Aries
>          Issue Type: Improvement
>            Reporter: Christian Schneider
>
> Like described in http://mail-archives.apache.org/mod_mbox/aries-dev/201405.mbox/%3C5370E1EA.9040008%40die-schneider.net%3E
> I propose to remove the pax exam dependency from org.apache.aries.testsupport.unit. So
some reasons why I think this makes sense:
> - testsupport is used in many aries projects. Many of these do not need the pax exam
depdenency. So the dependency binds all these project to a particular pax exam version and
increases coupling
> - pax exam is only needed for two classes in test support. ExtraOptions and AbstractIntegrationTest.
ExtraOptions is not really necessary with newer pax exam versions as most of the features
are available directly in the pax exam DSL. AbstractIntegrationTest just uses the pax exam
@Inject. In pax exam 3 this is replaced with javax @Inject. So this dependency is not necessary
anymore.
> As far as I know this means we need a new major version of testsupport as we remove a
public class.
> I will provide a patch.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message