search

eclipse-archived / concierge

Eclipse Concierge™ project
https://www.eclipse.org/concierge/
Eclipse Public License 1.0
33 stars 22 forks source link

Use pax-exam for Concierge integration tests? #14

Open JochenHiller opened 7 years ago

JochenHiller commented 7 years ago

Migrated from Eclipse Bugzilla #480566

Jochen Hiller 2015-10-24 07:12:43 EDT

We have yet some integration tests in Concierge (see http://git.eclipse.org/c/concierge/org.eclipse.concierge.git/tree/org.eclipse.concierge.tests.integration). These tests are built on a "proprietary" solution to download and install bundles from online URLs, like Apache Felix, Eclipse Orbit and others.

A similar solution will be provided by the pax-exam project (see https://ops4j1.jira.com/wiki/display/PAXEXAM4/Pax+Exam, latest version 4.x). This framework will allow easy writing test cases installing bundles from different sources (maven, p2 repo, local URLs, ...). It also allows to run tests against different OSGi frameworks, so same tests could also be applied to Equinox, Felix, to check about same or different behavior. The pax-exam OSGi container integration tests requires minimum OSGi R4.3 and does support the Launcher API, so it should work out-of-the-box even with Concierge.

The problem: pax-exam is not yet in Orbit. It is hosted in Maven central at http://mvnrepository.com/artifact/org.ops4j.pax.exam Pax-exam is heavily based on other op4j projects, like ops4j base, pax-url, pax-links, ...

As these jars are only used for regression testing, that is a test-only dependency.

What do you think? Does it makes sense to invest time on getting that running, or does the license problem is a show-stopper?

JochenHiller commented 7 years ago

@rellermeyer @tverbele Coming back to my proposal to introduce pax-exam

I would like to do:

Agreed? I will prepare a PR to at least cleanup this old code. And I will check about the pax-exam version from Orbit which can be used for that, or create a CQ to add that to Orbit

rellermeyer commented 7 years ago

Sounds good to me.

—Jan.

On Oct 28, 2016, at 3:17 PM, Jochen Hiller notifications@github.com<mailto:notifications@github.com> wrote:

@rellermeyerhttps://github.com/rellermeyer @tverbelehttps://github.com/tverbele Coming back to my proposal to introduce pax-exam

I would like to do:

Agreed? I will prepare a PR to at least cleanup this old code. And I will check about the pax-exam version from Orbit which can be used for that, or create a CQ to add that to Orbit

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/eclipse/concierge/issues/14#issuecomment-256917537, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGMmfx1VGU9dgv2fCVTqcQp4FAv9inyTks5q4fXdgaJpZM4KjZ2O.