[slf4j-dev] svn commit: r717 - in slf4j/trunk/slf4j-osgi-integration-test/src/test: java/org/slf4j/osgi/integration java/org/slf4j/osgi/integration/jdk java/org/slf4j/osgi/integration/jdk/test java/org/slf4j/osgi/integration/log4j/test java/org/slf4j/osgi/integration/nop/test java/org/slf4j/osgi/integration/simple/test resources/org/slf4j/osgi/integration/jdk resources/org/slf4j/osgi/integration/jdk/test

jconlon at slf4j.org jconlon at slf4j.org
Wed Feb 7 18:12:17 CET 2007


Author: jconlon
Date: Wed Feb  7 18:12:17 2007
New Revision: 717

Added:
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/jdk/
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/jdk/test/
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/jdk/test/JdkBundleTest.java
   slf4j/trunk/slf4j-osgi-integration-test/src/test/resources/org/slf4j/osgi/integration/jdk/
   slf4j/trunk/slf4j-osgi-integration-test/src/test/resources/org/slf4j/osgi/integration/jdk/test/
   slf4j/trunk/slf4j-osgi-integration-test/src/test/resources/org/slf4j/osgi/integration/jdk/test/MANIFEST.MF
Modified:
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/IntegrationTestConstants.java
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/log4j/test/Log4JBundleTest.java
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/nop/test/NopBundleTest.java
   slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/simple/test/SimpleBundleTest.java

Log:
Added tests for the jdk104 binding bundle.
Refactored more strings to constants.

Modified: slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/IntegrationTestConstants.java
==============================================================================
--- slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/IntegrationTestConstants.java	(original)
+++ slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/IntegrationTestConstants.java	Wed Feb  7 18:12:17 2007
@@ -32,18 +32,42 @@
 package org.slf4j.osgi.integration;
 
 public interface IntegrationTestConstants {
-
-	public static final String SLF4J_NOP = "org.slf4j.slf4j-nop";
-
-	public static final String SLF4J_OSGI_TEST = "org.slf4j.slf4j-osgi-test-bundle";
-
-	public static final String SLF4J_LOG4J = "org.slf4j.slf4j-log4j12";
+    
+    /**
+     * Versions of this release.
+     */
+    public static final String SLF4J_VERSION_UNDER_TEST = "1.3.0-SNAPSHOT";
+    
+    public static final String SLF4J_GROUP_ID = "org.slf4j";
+    
+    public static final String SPRINGFRAMEWORK_OSGI_GROUP_NAME = "org.springframework.osgi";
+    
+
+    public static final String JCL104_ADAPTER_BUNDLE_ARTIFACT_ID = "jcl104-over-slf4j";
+
+    public static final String JDK14_BINDING_BUNDLE_ARTIFACT_ID = "slf4j-jdk14";
+
+    public static final String SIMPLE_BINDING_BUNDLE_ARTIFACT_ID = "slf4j-simple";
+
+    public static final String LOG4J12_BINDING_BUNDLE_ARTIFACT_ID = "slf4j-log4j12";
+
+    public static final String NOP_BINDING_BUNDLE_ARTIFACT_ID = "slf4j-nop";
+
+    public static final String TEST_BUNDLE_ARTIFACT_ID = "slf4j-osgi-test-bundle";
+
+    
+    public static final String TEST_BUNDLE_SYM_NAME = SLF4J_GROUP_ID+'.'+TEST_BUNDLE_ARTIFACT_ID;
+    
+	public static final String NOP_BINDING_BUNDLE_SYM_NAME = SLF4J_GROUP_ID+'.'+NOP_BINDING_BUNDLE_ARTIFACT_ID;
+   
+	public static final String LOG4J_BINDING_BUNDLE_SYM_NAME = SLF4J_GROUP_ID+'.'+LOG4J12_BINDING_BUNDLE_ARTIFACT_ID;
 	
-	public static final String SLF4J_SIMPLE = "org.slf4j.slf4j-simple";
-	
-	public static final String SLF4J_JCL = "org.slf4j.jcl104-over-slf4j";
-
-
+	public static final String SIMPLE_BINDING_BUNDLE_SYM_NAME = SLF4J_GROUP_ID+'.'+SIMPLE_BINDING_BUNDLE_ARTIFACT_ID;
+    
+    public static final String JDK_BINDING_BUNDLE_SYM_NAME = SLF4J_GROUP_ID+'.'+JDK14_BINDING_BUNDLE_ARTIFACT_ID;
 	
-	public static final String VERSION = "1.3.0-SNAPSHOT";
+	public static final String JCL_ADAPTER_BUNDLE_SYM_NAME = SLF4J_GROUP_ID+'.'+JCL104_ADAPTER_BUNDLE_ARTIFACT_ID;
+   
+    
+    
 }

Added: slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/jdk/test/JdkBundleTest.java
==============================================================================
--- (empty file)
+++ slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/jdk/test/JdkBundleTest.java	Wed Feb  7 18:12:17 2007
@@ -0,0 +1,175 @@
+/* 
+ * Copyright (c) 2004-2005 SLF4J.ORG
+ * 
+ * All rights reserved.
+ * 
+ * Permission is hereby granted, free of charge, to any person obtaining
+ * a copy of this software and associated documentation files (the
+ * "Software"), to  deal in  the Software without  restriction, including
+ * without limitation  the rights to  use, copy, modify,  merge, publish,
+ * distribute, and/or sell copies of  the Software, and to permit persons
+ * to whom  the Software is furnished  to do so, provided  that the above
+ * copyright notice(s) and this permission notice appear in all copies of
+ * the  Software and  that both  the above  copyright notice(s)  and this
+ * permission notice appear in supporting documentation.
+ * 
+ * THE  SOFTWARE IS  PROVIDED  "AS  IS", WITHOUT  WARRANTY  OF ANY  KIND,
+ * EXPRESS OR  IMPLIED, INCLUDING  BUT NOT LIMITED  TO THE  WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR  A PARTICULAR PURPOSE AND NONINFRINGEMENT
+ * OF  THIRD PARTY  RIGHTS. IN  NO EVENT  SHALL THE  COPYRIGHT  HOLDER OR
+ * HOLDERS  INCLUDED IN  THIS  NOTICE BE  LIABLE  FOR ANY  CLAIM, OR  ANY
+ * SPECIAL INDIRECT  OR CONSEQUENTIAL DAMAGES, OR  ANY DAMAGES WHATSOEVER
+ * RESULTING FROM LOSS  OF USE, DATA OR PROFITS, WHETHER  IN AN ACTION OF
+ * CONTRACT, NEGLIGENCE  OR OTHER TORTIOUS  ACTION, ARISING OUT OF  OR IN
+ * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+ * 
+ * Except as  contained in  this notice, the  name of a  copyright holder
+ * shall not be used in advertising or otherwise to promote the sale, use
+ * or other dealings in this Software without prior written authorization
+ * of the copyright holder.
+ *
+ */
+
+package org.slf4j.osgi.integration.jdk.test;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.osgi.framework.Bundle;
+import org.osgi.framework.BundleContext;
+import org.osgi.framework.Constants;
+import org.osgi.framework.ServiceReference;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.slf4j.osgi.integration.IntegrationTestConstants;
+import org.slf4j.osgi.test.service.Probe;
+import org.springframework.osgi.test.ConfigurableBundleCreatorTests;
+
+/**
+ * 
+ * <code>JdkBundleTest</code>  starts up an OSGi environment (equinox,
+ * knopflerfish, or felix according to the profile selected) and installs the
+ * slf4j.osgi.test.bundle, the slf4j.jdk bundle and the bundles they depend on. 
+ * 
+ * The test classes in this project will be turned into a virtual bundle which is 
+ * also installed and the tests are then run inside the OSGi runtime.
+ * 
+ * The tests have access to a BundleContext, which we use to test that all bundles have
+ * been started.
+ * 
+ * 
+ * @author John Conlon
+ */
+public class JdkBundleTest extends ConfigurableBundleCreatorTests implements IntegrationTestConstants{
+
+
+
+	
+
+    /**
+	 * The manifest to use for the "virtual bundle" created out of the test
+	 * classes and resources in this project
+	 * 
+	 * This is actually the boilerplate manifest with one additional
+	 * import-package added. We should provide a simpler customization point for
+	 * such use cases that doesn't require duplication of the entire manifest...
+	 */
+	protected String getManifestLocation() {
+		return "classpath:org/slf4j/osgi/integration/jdk/test/MANIFEST.MF";
+	}
+	
+	/**
+	 * The location of the packaged OSGi bundles to be installed for this test.
+	 * Values are Spring resource paths. The bundles we want to use are part of
+	 * the same multi-project maven build as this project is. Hence we use the
+	 * localMavenArtifact helper method to find the bundles produced by the
+	 * package phase of the maven build (these tests will run after the
+	 * packaging phase, in the integration-test phase).
+	 * 
+	 * JUnit, commons-logging, spring-core and the spring OSGi test bundle are
+	 * automatically included so they do not need to be specified here.
+	 * 
+	 * Our test bundles are using package import and export versions to keep
+	 * these other logging bundles from getting mixed up with our test bundles. 
+	 */
+	protected String[] getBundleLocations() {
+		return new String[] {
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
+						"aopalliance.osgi", "1.0-SNAPSHOT"),
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
+						"spring-context", "2.1-SNAPSHOT"),
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-beans",
+						"2.1-SNAPSHOT"),
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
+						"spring-osgi-core", "1.0-SNAPSHOT"),
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-aop",
+						"2.1-SNAPSHOT"),
+						localMavenArtifact(SLF4J_GROUP_ID, JDK14_BINDING_BUNDLE_ARTIFACT_ID, SLF4J_VERSION_UNDER_TEST),
+						localMavenArtifact(SLF4J_GROUP_ID, TEST_BUNDLE_ARTIFACT_ID,
+								SLF4J_VERSION_UNDER_TEST) };
+	}
+
+	/**
+	 * The superclass provides us access to the root bundle context via the
+	 * 'getBundleContext' operation. Make sure it is not null.
+	 */
+	public void testOSGiStartedOk() {
+		BundleContext bundleContext = getBundleContext();
+		assertNotNull(bundleContext);
+
+	}
+
+	/**
+	 * Makes sure our bundles are in the OSGi runtime and their state is Active.
+	 * 
+	 */
+	public void testSlf4jNopBundleIntegration() {
+		Logger log = LoggerFactory.getLogger(JdkBundleTest.class);
+		assertNotNull(log);
+		BundleContext context = getBundleContext();
+		List symNames = new ArrayList();
+
+		Bundle[] bundles = context.getBundles();
+		log.info("Loaded bundles:");
+		for (int i = 0; i < bundles.length; i++) {
+			Bundle bundle = bundles[i];
+			assertEquals("Bundle " + bundle.getSymbolicName()
+					+ " is not active.", Bundle.ACTIVE, bundle.getState());
+			symNames.add(bundle.getSymbolicName());
+			if (bundle.getHeaders().get(Constants.BUNDLE_ACTIVATOR) != null) {
+				log.info("Symbolic Name:" + bundle.getSymbolicName()
+						+ ", Activator:"
+						+ bundle.getHeaders().get(Constants.BUNDLE_ACTIVATOR));
+			} else {
+				log.info("Symbolic Name:" + bundle.getSymbolicName());
+			}
+		}
+
+		assertTrue(symNames.contains(JDK_BINDING_BUNDLE_SYM_NAME));
+		assertTrue(symNames.contains(TEST_BUNDLE_SYM_NAME));
+
+	}
+	
+	public void testProbeService(){
+		Logger log = LoggerFactory.getLogger(JdkBundleTest.class);
+		log.debug("Testing probe");
+		BundleContext context = getBundleContext();
+		ServiceReference ref = context.getServiceReference( Probe.class.getName() );
+		assertNotNull( "Service Reference is null", ref );
+		Probe probe = ( Probe ) context.getService( ref );
+		assertNotNull( "Cannot find the probe service", probe );
+
+		try {
+			probe.testCommonslogging();
+			fail("Expected exception while testing commons logging.");
+		} catch (Throwable t){
+			log.debug("Failed to execute the probe.testCommonsLogging. "+t);
+
+		}
+
+		context.ungetService( ref );
+
+		log.debug("Tested probe.");
+	}
+
+}

Modified: slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/log4j/test/Log4JBundleTest.java
==============================================================================
--- slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/log4j/test/Log4JBundleTest.java	(original)
+++ slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/log4j/test/Log4JBundleTest.java	Wed Feb  7 18:12:17 2007
@@ -101,19 +101,19 @@
 	 */
 	protected String[] getBundleLocations() {
 		return new String[] {
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"aopalliance.osgi", "1.0-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"spring-context", "2.1-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi", "spring-beans",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-beans",
 						"2.1-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"spring-osgi-core", "1.0-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi", "spring-aop",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-aop",
 						"2.1-SNAPSHOT"),
-				localMavenArtifact("org.slf4j", "slf4j-log4j12", VERSION),
-				localMavenArtifact("org.slf4j", "slf4j-osgi-test-bundle",
-						VERSION) };
+				localMavenArtifact(SLF4J_GROUP_ID, LOG4J12_BINDING_BUNDLE_ARTIFACT_ID, SLF4J_VERSION_UNDER_TEST),
+				localMavenArtifact(SLF4J_GROUP_ID, TEST_BUNDLE_ARTIFACT_ID,
+						SLF4J_VERSION_UNDER_TEST) };
 	}
 
 	/**
@@ -152,8 +152,8 @@
 			}
 		}
 
-		assertTrue(symNames.contains(SLF4J_LOG4J));
-		assertTrue(symNames.contains(SLF4J_OSGI_TEST));
+		assertTrue(symNames.contains(LOG4J_BINDING_BUNDLE_SYM_NAME));
+		assertTrue(symNames.contains(TEST_BUNDLE_SYM_NAME));
 
 	}
 

Modified: slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/nop/test/NopBundleTest.java
==============================================================================
--- slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/nop/test/NopBundleTest.java	(original)
+++ slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/nop/test/NopBundleTest.java	Wed Feb  7 18:12:17 2007
@@ -47,7 +47,7 @@
 
 /**
  * 
- * <code>NopBundleTest</code>  starts up an OSGi environment (equinox,
+ * <code>JdkBundleTest</code>  starts up an OSGi environment (equinox,
  * knopflerfish, or felix according to the profile selected) and installs the
  * slf4j.osgi.test.bundle, the slf4j.nop bundle and the bundles they depend on. 
  * 
@@ -92,19 +92,19 @@
 	 */
 	protected String[] getBundleLocations() {
 		return new String[] {
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"aopalliance.osgi", "1.0-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"spring-context", "2.1-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi", "spring-beans",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-beans",
 						"2.1-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"spring-osgi-core", "1.0-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi", "spring-aop",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-aop",
 						"2.1-SNAPSHOT"),
-						localMavenArtifact("org.slf4j", "slf4j-nop", VERSION),
-						localMavenArtifact("org.slf4j", "slf4j-osgi-test-bundle",
-								VERSION) };
+						localMavenArtifact(SLF4J_GROUP_ID, NOP_BINDING_BUNDLE_ARTIFACT_ID, SLF4J_VERSION_UNDER_TEST),
+						localMavenArtifact(SLF4J_GROUP_ID, TEST_BUNDLE_ARTIFACT_ID,
+								SLF4J_VERSION_UNDER_TEST) };
 	}
 
 	/**
@@ -143,8 +143,8 @@
 			}
 		}
 
-		assertTrue(symNames.contains(SLF4J_NOP));
-		assertTrue(symNames.contains(SLF4J_OSGI_TEST));
+		assertTrue(symNames.contains(NOP_BINDING_BUNDLE_SYM_NAME));
+		assertTrue(symNames.contains(TEST_BUNDLE_SYM_NAME));
 
 	}
 	

Modified: slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/simple/test/SimpleBundleTest.java
==============================================================================
--- slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/simple/test/SimpleBundleTest.java	(original)
+++ slf4j/trunk/slf4j-osgi-integration-test/src/test/java/org/slf4j/osgi/integration/simple/test/SimpleBundleTest.java	Wed Feb  7 18:12:17 2007
@@ -95,20 +95,20 @@
 	 */
 	protected String[] getBundleLocations() {
 		return new String[] {
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"aopalliance.osgi", "1.0-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"spring-context", "2.1-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi", "spring-beans",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-beans",
 						"2.1-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME,
 						"spring-osgi-core", "1.0-SNAPSHOT"),
-				localMavenArtifact("org.springframework.osgi", "spring-aop",
+				localMavenArtifact(SPRINGFRAMEWORK_OSGI_GROUP_NAME, "spring-aop",
 						"2.1-SNAPSHOT"),
-				localMavenArtifact("org.slf4j", "slf4j-simple", VERSION),
-				localMavenArtifact("org.slf4j", "jcl104-over-slf4j", VERSION),
-				localMavenArtifact("org.slf4j", "slf4j-osgi-test-bundle",
-						VERSION) };
+				localMavenArtifact(SLF4J_GROUP_ID, SIMPLE_BINDING_BUNDLE_ARTIFACT_ID, SLF4J_VERSION_UNDER_TEST),
+				localMavenArtifact(SLF4J_GROUP_ID, JCL104_ADAPTER_BUNDLE_ARTIFACT_ID, SLF4J_VERSION_UNDER_TEST),
+				localMavenArtifact(SLF4J_GROUP_ID, TEST_BUNDLE_ARTIFACT_ID,
+						SLF4J_VERSION_UNDER_TEST) };
 	}
 
 	/**
@@ -147,9 +147,9 @@
 			}
 		}
 
-		assertTrue(symNames.contains(SLF4J_SIMPLE));
-		assertTrue(symNames.contains(SLF4J_OSGI_TEST));
-		assertTrue(symNames.contains(SLF4J_JCL));
+		assertTrue(symNames.contains(SIMPLE_BINDING_BUNDLE_SYM_NAME));
+		assertTrue(symNames.contains(TEST_BUNDLE_SYM_NAME));
+		assertTrue(symNames.contains(JCL_ADAPTER_BUNDLE_SYM_NAME));
 
 	}
 

Added: slf4j/trunk/slf4j-osgi-integration-test/src/test/resources/org/slf4j/osgi/integration/jdk/test/MANIFEST.MF
==============================================================================
--- (empty file)
+++ slf4j/trunk/slf4j-osgi-integration-test/src/test/resources/org/slf4j/osgi/integration/jdk/test/MANIFEST.MF	Wed Feb  7 18:12:17 2007
@@ -0,0 +1,15 @@
+Manifest-Version: 1.0
+Bundle-Name: simple-service-integration-tests
+Bundle-SymbolicName: org.slf4j.osgi.integration.jdk.test
+Bundle-Vendor: SLF4J ORG
+Bundle-Activator: org.springframework.osgi.test.JUnitTestActivator
+Import-Package: junit.framework,
+ org.osgi.framework;specification-version="1.3.0",
+ org.springframework.core.io;specification-version="2.1.0",
+ org.springframework.osgi.test,
+ org.springframework.osgi.test.runner,
+ org.slf4j,
+ org.slf4j.osgi.test.service
+ 
+ 
+      



More information about the slf4j-dev mailing list