From 09975143afb2974b724ce42f9e74dbd00838ef2e Mon Sep 17 00:00:00 2001 From: Dominik Przybysz Date: Mon, 24 Feb 2025 11:38:42 +0100 Subject: [PATCH] [MAINTENANCE] Restore sample blueprint hello world --- README.md | 2 +- .../helloworld/helloworld-itests/pom.xml | 60 ++++++------------- .../itests/AbstractIntegrationTest.java | 5 +- .../itests/HelloworldSampleTest.java | 60 +++++++++---------- samples/blueprint/pom.xml | 2 +- 5 files changed, 51 insertions(+), 78 deletions(-) diff --git a/README.md b/README.md index 8f0474949b..c8ee92a028 100644 --- a/README.md +++ b/README.md @@ -17,7 +17,7 @@ See [Apache Aries Website](http://aries.apache.org/). - [![Proxy - CI Build](https://github.com/apache/aries/actions/workflows/proxy.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/proxy.yml) - [![Pushstream - CI Build](https://github.com/apache/aries/actions/workflows/pushstream.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/pushstream.yml) - [![Quiesce - CI Build](https://github.com/apache/aries/actions/workflows/quiesce.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/quiesce.yml) -- Samples - Missing +- [![Samples - CI Build](https://github.com/apache/aries/actions/workflows/samples.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/samples.yml) - [![SPI Fly - CI Build](https://github.com/apache/aries/actions/workflows/spi-fly.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/spi-fly.yml) - [![Subsystem - CI Build](https://github.com/apache/aries/actions/workflows/subsystem.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/subsystem.yml) - [![Transaction - CI Build](https://github.com/apache/aries/actions/workflows/transaction.yml/badge.svg)](https://github.com/apache/aries/actions/workflows/transaction.yml) diff --git a/samples/blueprint/helloworld/helloworld-itests/pom.xml b/samples/blueprint/helloworld/helloworld-itests/pom.xml index 08392ae49b..ea1e1608b4 100644 --- a/samples/blueprint/helloworld/helloworld-itests/pom.xml +++ b/samples/blueprint/helloworld/helloworld-itests/pom.xml @@ -32,35 +32,33 @@ - 4.0 - 0.0.357 + 5.0.3 3.2.0-v20070116 1.0.0 - 3.5.0.v20090520 - 1.2.4 - 1.4.0 - 1.2.0 - 1.3.5 + 3.8.0.v20120529-1548 + 4.13.5 + 1.7.2 + 2.6.16 3.1.200-v20070605 - org.osgi - org.osgi.core + org.eclipse + org.eclipse.osgi + ${org.eclipse.osgi.version} provided org.osgi - org.osgi.compendium + org.osgi.core provided - org.eclipse - osgi - ${osgi.version} + org.osgi + org.osgi.compendium provided @@ -70,12 +68,6 @@ test - - biz.aQute - bndlib - ${bndlib.version} - - org.apache.aries.proxy org.apache.aries.proxy @@ -113,38 +105,29 @@ ${pax-exam.version} test - org.ops4j.pax.exam - pax-exam-junit + pax-exam-junit4 ${pax-exam.version} test - org.ops4j.pax.exam - pax-exam-container-default + pax-exam-container-native ${pax-exam.version} test - org.ops4j.pax.exam - pax-exam-junit-extender-impl + pax-exam-link-mvn ${pax-exam.version} test - - - org.ops4j.pax.runner - pax-runner-no-jcl - ${pax-runner-no-jcl.version} - - - org.ops4j.pax.swissbox - pax-swissbox-tinybundles - ${pax-swissbox-tinybundles.version} + org.ops4j.pax.url + pax-url-aether + ${pax-url.version} + test @@ -171,13 +154,6 @@ test - - org.ops4j.pax.url - pax-url-mvn - ${pax-url-mvn.version} - test - - org.apache.servicemix.bundles org.apache.servicemix.bundles.cglib diff --git a/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/AbstractIntegrationTest.java b/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/AbstractIntegrationTest.java index d5c42e3863..d99e860fc7 100644 --- a/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/AbstractIntegrationTest.java +++ b/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/AbstractIntegrationTest.java @@ -37,7 +37,6 @@ import org.junit.After; import org.junit.Before; import org.ops4j.pax.exam.CoreOptions; -import org.ops4j.pax.exam.Inject; import org.ops4j.pax.exam.Option; import org.ops4j.pax.exam.options.MavenArtifactProvisionOption; import org.ops4j.pax.url.mvn.Handler; @@ -51,6 +50,8 @@ import org.osgi.framework.ServiceReference; import org.osgi.util.tracker.ServiceTracker; +import javax.inject.Inject; + public abstract class AbstractIntegrationTest { @@ -272,7 +273,7 @@ protected T getOsgiService(BundleContext bc, Class type, public static URL getUrlToEba(String groupId, String artifactId) throws MalformedURLException { String artifactVersion = getArtifactVersion(groupId, artifactId); - // Need to use handler from org.ops4j.pax.url.mvn + // Need to use handler from org.ops4j.pax.url.aether URL urlToEba = new URL(null, ServiceConstants.PROTOCOL + ":" + groupId + "/" +artifactId + "/" + artifactVersion + "/eba", new Handler()); diff --git a/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/HelloworldSampleTest.java b/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/HelloworldSampleTest.java index 21f280e5a1..5e64f15661 100644 --- a/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/HelloworldSampleTest.java +++ b/samples/blueprint/helloworld/helloworld-itests/src/test/java/org/apache/aries/samples/blueprint/helloworld/itests/HelloworldSampleTest.java @@ -22,20 +22,23 @@ import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertNull; import static org.ops4j.pax.exam.CoreOptions.bootDelegationPackages; -import static org.ops4j.pax.exam.CoreOptions.equinox; +import static org.ops4j.pax.exam.CoreOptions.*; import static org.ops4j.pax.exam.CoreOptions.options; import static org.ops4j.pax.exam.CoreOptions.systemProperty; -import static org.ops4j.pax.exam.container.def.PaxRunnerOptions.vmOption; +import org.ops4j.pax.exam.Configuration; +import org.ops4j.pax.exam.junit.PaxExam; +import org.ops4j.pax.exam.spi.reactors.ExamReactorStrategy; +import org.ops4j.pax.exam.spi.reactors.PerClass; import org.osgi.service.blueprint.container.BlueprintContainer; import org.junit.Test; import org.junit.runner.RunWith; import org.ops4j.pax.exam.Option; -import org.ops4j.pax.exam.junit.JUnit4TestRunner; import org.osgi.framework.Bundle; -@RunWith(JUnit4TestRunner.class) +@RunWith(PaxExam.class) +@ExamReactorStrategy(PerClass.class) public class HelloworldSampleTest extends AbstractIntegrationTest { @Test @@ -44,7 +47,7 @@ public void testBundlesStart() throws Exception { /* Check that the HelloWorld Sample bundles are present an started */ Bundle bapi = getInstalledBundle("org.apache.aries.samples.blueprint.helloworld.api"); assertNotNull(bapi); - + failInBundleNotActiveInFiveSeconds(bapi); assertEquals(Bundle.ACTIVE, bapi.getState()); @@ -56,7 +59,7 @@ public void testBundlesStart() throws Exception { assertNotNull(bser); failInBundleNotActiveInFiveSeconds(bser); } - + @Test public void testClientBlueprintContainerOnlyStartsWhenServiceStarted() throws Exception { @@ -69,32 +72,32 @@ public void testClientBlueprintContainerOnlyStartsWhenServiceStarted() throws Ex assertNotNull(bser); bser.stop(); - // Wait for everything to shut down + // Wait for everything to shut down Thread.sleep(1000); - - // When everything is stopped, there should be no blueprint container for either the client or the server - + + // When everything is stopped, there should be no blueprint container for either the client or the server + assertClientBlueprintContainerNull(); assertServerBlueprintContainerNull(); // If we start the client first, it shouldn't have a blueprint container bcli.start(); - // Wait for everything to get started + // Wait for everything to get started Thread.sleep(1000); assertClientBlueprintContainerNull(); - + // Then when we start the server both it and the client should have blueprint containers bser.start(); - // Wait for everything to get started + // Wait for everything to get started Thread.sleep(1000); assertClientBlueprintContainerNotNull(); assertServerBlueprintContainerNotNull(); } - + private BlueprintContainer getBlueprintContainer(String bundleName) - { + { BlueprintContainer container = null; try { container = getOsgiService(BlueprintContainer.class, "(osgi.blueprint.container.symbolicname=" + bundleName + ")", 500); @@ -104,17 +107,17 @@ private BlueprintContainer getBlueprintContainer(String bundleName) } return container; } - + private BlueprintContainer getClientBlueprintContainer() { return getBlueprintContainer("org.apache.aries.samples.blueprint.helloworld.client"); } - + private BlueprintContainer getServerBlueprintContainer() { return getBlueprintContainer("org.apache.aries.samples.blueprint.helloworld.server"); } - + private void assertClientBlueprintContainerNotNull() { assertNotNull("There was no blueprint container for the client bundle.", getClientBlueprintContainer()); @@ -146,21 +149,23 @@ private void failInBundleNotActiveInFiveSeconds(Bundle bapi) e.printStackTrace(); } } - + assertEquals("The bundle " + bapi.getSymbolicName() + " " + bapi.getVersion() + " is not active", Bundle.ACTIVE, bapi.getState()); } - @org.ops4j.pax.exam.junit.Configuration + @Configuration public static Option[] configuration() { Option[] options = options( bootDelegationPackages("javax.transaction", "javax.transaction.*"), vmOption("-Dorg.osgi.framework.system.packages=javax.accessibility,javax.activation,javax.activity,javax.annotation,javax.annotation.processing,javax.crypto,javax.crypto.interfaces,javax.crypto.spec,javax.imageio,javax.imageio.event,javax.imageio.metadata,javax.imageio.plugins.bmp,javax.imageio.plugins.jpeg,javax.imageio.spi,javax.imageio.stream,javax.jws,javax.jws.soap,javax.lang.model,javax.lang.model.element,javax.lang.model.type,javax.lang.model.util,javax.management,javax.management.loading,javax.management.modelmbean,javax.management.monitor,javax.management.openmbean,javax.management.relation,javax.management.remote,javax.management.remote.rmi,javax.management.timer,javax.naming,javax.naming.directory,javax.naming.event,javax.naming.ldap,javax.naming.spi,javax.net,javax.net.ssl,javax.print,javax.print.attribute,javax.print.attribute.standard,javax.print.event,javax.rmi,javax.rmi.CORBA,javax.rmi.ssl,javax.script,javax.security.auth,javax.security.auth.callback,javax.security.auth.kerberos,javax.security.auth.login,javax.security.auth.spi,javax.security.auth.x500,javax.security.cert,javax.security.sasl,javax.sound.midi,javax.sound.midi.spi,javax.sound.sampled,javax.sound.sampled.spi,javax.sql,javax.sql.rowset,javax.sql.rowset.serial,javax.sql.rowset.spi,javax.swing,javax.swing.border,javax.swing.colorchooser,javax.swing.event,javax.swing.filechooser,javax.swing.plaf,javax.swing.plaf.basic,javax.swing.plaf.metal,javax.swing.plaf.multi,javax.swing.plaf.synth,javax.swing.table,javax.swing.text,javax.swing.text.html,javax.swing.text.html.parser,javax.swing.text.rtf,javax.swing.tree,javax.swing.undo,javax.tools,javax.xml,javax.xml.bind,javax.xml.bind.annotation,javax.xml.bind.annotation.adapters,javax.xml.bind.attachment,javax.xml.bind.helpers,javax.xml.bind.util,javax.xml.crypto,javax.xml.crypto.dom,javax.xml.crypto.dsig,javax.xml.crypto.dsig.dom,javax.xml.crypto.dsig.keyinfo,javax.xml.crypto.dsig.spec,javax.xml.datatype,javax.xml.namespace,javax.xml.parsers,javax.xml.soap,javax.xml.stream,javax.xml.stream.events,javax.xml.stream.util,javax.xml.transform,javax.xml.transform.dom,javax.xml.transform.sax,javax.xml.transform.stax,javax.xml.transform.stream,javax.xml.validation,javax.xml.ws,javax.xml.ws.handler,javax.xml.ws.handler.soap,javax.xml.ws.http,javax.xml.ws.soap,javax.xml.ws.spi,javax.xml.xpath,org.ietf.jgss,org.omg.CORBA,org.omg.CORBA.DynAnyPackage,org.omg.CORBA.ORBPackage,org.omg.CORBA.TypeCodePackage,org.omg.CORBA.portable,org.omg.CORBA_2_3,org.omg.CORBA_2_3.portable,org.omg.CosNaming,org.omg.CosNaming.NamingContextExtPackage,org.omg.CosNaming.NamingContextPackage,org.omg.Dynamic,org.omg.DynamicAny,org.omg.DynamicAny.DynAnyFactoryPackage,org.omg.DynamicAny.DynAnyPackage,org.omg.IOP,org.omg.IOP.CodecFactoryPackage,org.omg.IOP.CodecPackage,org.omg.Messaging,org.omg.PortableInterceptor,org.omg.PortableInterceptor.ORBInitInfoPackage,org.omg.PortableServer,org.omg.PortableServer.CurrentPackage,org.omg.PortableServer.POAManagerPackage,org.omg.PortableServer.POAPackage,org.omg.PortableServer.ServantLocatorPackage,org.omg.PortableServer.portable,org.omg.SendingContext,org.omg.stub.java.rmi,org.w3c.dom,org.w3c.dom.bootstrap,org.w3c.dom.css,org.w3c.dom.events,org.w3c.dom.html,org.w3c.dom.ls,org.w3c.dom.ranges,org.w3c.dom.stylesheets,org.w3c.dom.traversal,org.w3c.dom.views,org.xml.sax,org.xml.sax.ext,org.xml.sax.helpers,javax.transaction;partial=true;mandatory:=partial,javax.transaction.xa;partial=true;mandatory:=partial"), + junitBundles(), + // Log mavenBundle("org.ops4j.pax.logging", "pax-logging-api"), mavenBundle("org.ops4j.pax.logging", "pax-logging-service"), - // Felix mvn url handler - do we need this? - mavenBundle("org.ops4j.pax.url", "pax-url-mvn"), + + mavenBundle("org.ops4j.pax.url", "pax-url-aether"), // this is how you set the default log level when using pax // logging (logProfile) @@ -177,16 +182,7 @@ public static Option[] configuration() { mavenBundle("org.ow2.asm", "asm-all" ), mavenBundle("org.apache.aries.samples.blueprint.helloworld", "org.apache.aries.samples.blueprint.helloworld.api"), mavenBundle("org.apache.aries.samples.blueprint.helloworld", "org.apache.aries.samples.blueprint.helloworld.server"), - mavenBundle("org.apache.aries.samples.blueprint.helloworld", "org.apache.aries.samples.blueprint.helloworld.client"), - /* For debugging, uncomment the next two lines */ - /*vmOption ("-Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=7777"), - waitForFrameworkStartup(), -*/ - /* For debugging, add these imports: - import static org.ops4j.pax.exam.CoreOptions.waitForFrameworkStartup; - import static org.ops4j.pax.exam.container.def.PaxRunnerOptions.vmOption; - */ - equinox().version("3.5.0") + mavenBundle("org.apache.aries.samples.blueprint.helloworld", "org.apache.aries.samples.blueprint.helloworld.client") ); options = updateOptions(options); return options; diff --git a/samples/blueprint/pom.xml b/samples/blueprint/pom.xml index d110036f28..3f4356ba8c 100644 --- a/samples/blueprint/pom.xml +++ b/samples/blueprint/pom.xml @@ -44,7 +44,7 @@ - + helloworld idverifier