karaf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andreas Pieber <anpie...@gmail.com>
Subject Re: svn commit: r1051896 - in /karaf/trunk/itests: dependencies/pom.xml tests/src/test/java/org/apache/karaf/shell/itests/CoreTest.java tests/src/test/java/org/apache/karaf/shell/itests/FeaturesTest.javar
Date Thu, 23 Dec 2010 06:52:39 GMT
Thank you very much; I'll give it a shot tomorrow

kind regards,
andreas

On Thu, Dec 23, 2010 at 07:27:52AM +0100, Guillaume Nodet wrote:
> On Wed, Dec 22, 2010 at 23:32, Andreas Pieber <anpieber@gmail.com> wrote:
> > +1 for exam2; I've took a look at exam2 some weeks ago (and had a similar
> > thought). Can you push your local branch (into a dev-branch) to the github repo?
> > I would like to take a look at it.
> 
> Done, see https://github.com/gnodet/karaf/tree/paxexam2 (i've given
> you karma on that repo)
> It's a few weeks old, so it may need to be rebased on trunk.
> I've also pushed a gnodet branch to pax-exam which contains a few
> improvements, but
> not sure how stable it is (hence the branch).
> 
> >
> > @own platform.xml: I've tried this already some time ago for karaf-2.x and it
> > was anything but straight forward... I think I would prefer to push exam2 to
> > a 2.0.0.M2 release we can use for Karaf :)
> 
> I had the exact same reaction.  I'd like to make sure such things are
> way easier in pax-exam2.
> 
> >
> > On Wed, Dec 22, 2010 at 11:05:53PM +0100, Guillaume Nodet wrote:
> >> Ideally, we should have our own platform.
> >> Though my real goal would be to start moving to pax-exam2 and enhance
> >> pax-exam2 in order to fix some those problems.
> >> I've already started on that in a local branch a few weeks ago, but
> >> haven't had any time to get back to it yet.
> >> In the case of karaf, we don't really need pax-runner, as we have our
> >> own provisioning mechanism, and i'd like the integration tests to be
> >> able to be run offline without too much pain.
> >> I didn't want to spent time on writing our own platform.xml if if
> >> you're fancy doing it, you'd be more than welcome.
> >>
> >> On Wed, Dec 22, 2010 at 22:54, Andreas Pieber <anpieber@gmail.com> wrote:
> >> > Shouldn't we upgrade pax-runner (or provide our own platform.xml) instead?
> >> >
> >> > kind regards,
> >> > andreas
> >> >
> >> > On Wed, Dec 22, 2010 at 01:46:13PM -0000, gnodet@apache.org wrote:
> >> >> Author: gnodet
> >> >> Date: Wed Dec 22 13:46:13 2010
> >> >> New Revision: 1051896
> >> >>
> >> >> URL: http://svn.apache.org/viewvc?rev=1051896&view=rev
> >> >> Log:
> >> >> Make sure we have the felix main in the local repo for pax-runner
> >> >>
> >> >> Modified:
> >> >>     karaf/trunk/itests/dependencies/pom.xml
> >> >>     karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/CoreTest.java
> >> >>     karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/FeaturesTest.java
> >> >>
> >> >> Modified: karaf/trunk/itests/dependencies/pom.xml
> >> >> URL: http://svn.apache.org/viewvc/karaf/trunk/itests/dependencies/pom.xml?rev=1051896&r1=1051895&r2=1051896&view=diff
> >> >> ==============================================================================
> >> >> --- karaf/trunk/itests/dependencies/pom.xml (original)
> >> >> +++ karaf/trunk/itests/dependencies/pom.xml Wed Dec 22 13:46:13 2010
> >> >> @@ -39,12 +39,12 @@
> >> >>      </repositories>
> >> >>
> >> >>      <dependencies>
> >> >> -        <!--
> >> >>          <dependency>
> >> >>                       <groupId>org.apache.felix</groupId>
> >> >>                       <artifactId>org.apache.felix.main</artifactId>
> >> >> -                     <version>2.0.2</version>
> >> >> +                     <version>3.0.2</version>
> >> >>          </dependency>
> >> >> +        <!--
> >> >>          <dependency>
> >> >>                       <groupId>org.eclipse</groupId>
> >> >>                       <artifactId>osgi</artifactId>
> >> >>
> >> >> Modified: karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/CoreTest.java
> >> >> URL: http://svn.apache.org/viewvc/karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/CoreTest.java?rev=1051896&r1=1051895&r2=1051896&view=diff
> >> >> ==============================================================================
> >> >> --- karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/CoreTest.java
(original)
> >> >> +++ karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/CoreTest.java
Wed Dec 22 13:46:13 2010
> >> >> @@ -130,7 +130,10 @@ public class CoreTest extends AbstractIn
> >> >>              waitForFrameworkStartup(),
> >> >>
> >> >>              // Test on both equinox and felix
> >> >> -            equinox(), felix(),
> >> >> +            // TODO: pax-exam does not support the latest felix
version :-(
> >> >> +            // TODO: so we use the higher supported which should
be the same
> >> >> +            // TODO: as the one specified in itests/dependencies/pom.xml
> >> >> +            equinox(), felix().version("3.0.2"),
> >> >>
> >> >>              felixProvisionalApis()
> >> >>          );
> >> >>
> >> >> Modified: karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/FeaturesTest.java
> >> >> URL: http://svn.apache.org/viewvc/karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/FeaturesTest.java?rev=1051896&r1=1051895&r2=1051896&view=diff
> >> >> ==============================================================================
> >> >> --- karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/FeaturesTest.java
(original)
> >> >> +++ karaf/trunk/itests/tests/src/test/java/org/apache/karaf/shell/itests/FeaturesTest.java
Wed Dec 22 13:46:13 2010
> >> >> @@ -81,7 +81,10 @@ public class FeaturesTest extends Abstra
> >> >>              waitForFrameworkStartup(),
> >> >>
> >> >>              // Test on both equinox and felix
> >> >> -            equinox(), felix(),
> >> >> +            // TODO: pax-exam does not support the latest felix
version :-(
> >> >> +            // TODO: so we use the higher supported which should
be the same
> >> >> +            // TODO: as the one specified in itests/dependencies/pom.xml
> >> >> +            equinox(), felix().version("3.0.2"),
> >> >>
> >> >>              felixProvisionalApis()
> >> >>          );
> >> >>
> >> >>
> >> >
> >>
> >>
> >>
> >> --
> >> Cheers,
> >> Guillaume Nodet
> >> ------------------------
> >> Blog: http://gnodet.blogspot.com/
> >> ------------------------
> >> Open Source SOA
> >> http://fusesource.com
> >
> 
> 
> 
> -- 
> Cheers,
> Guillaume Nodet
> ------------------------
> Blog: http://gnodet.blogspot.com/
> ------------------------
> Open Source SOA
> http://fusesource.com

Mime
View raw message