You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ss...@apache.org on 2017/03/14 21:47:48 UTC

svn commit: r1786956 - in /sling/trunk/testing/mocks/osgi-mock/src: main/java/org/apache/sling/testing/mock/osgi/ test/java/org/apache/sling/testing/mock/osgi/

Author: sseifert
Date: Tue Mar 14 21:47:48 2017
New Revision: 1786956

URL: http://svn.apache.org/viewvc?rev=1786956&view=rev
Log:
SLING-6641 osgi-mock: MockOsgi.setConfigForPid convenience method

Modified:
    sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/MockOsgi.java
    sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/package-info.java
    sling/trunk/testing/mocks/osgi-mock/src/test/java/org/apache/sling/testing/mock/osgi/MockConfigurationAdminTest.java

Modified: sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/MockOsgi.java
URL: http://svn.apache.org/viewvc/sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/MockOsgi.java?rev=1786956&r1=1786955&r2=1786956&view=diff
==============================================================================
--- sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/MockOsgi.java (original)
+++ sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/MockOsgi.java Tue Mar 14 21:47:48 2017
@@ -22,12 +22,14 @@ import static org.apache.sling.testing.m
 import static org.apache.sling.testing.mock.osgi.MapUtil.toDictionary;
 import static org.apache.sling.testing.mock.osgi.MapUtil.toMap;
 
+import java.io.IOException;
 import java.util.Dictionary;
 import java.util.Map;
 
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.BundleEvent;
 import org.osgi.framework.ServiceReference;
+import org.osgi.service.cm.Configuration;
 import org.osgi.service.cm.ConfigurationAdmin;
 import org.osgi.service.component.ComponentContext;
 import org.osgi.service.log.LogService;
@@ -272,6 +274,40 @@ public final class MockOsgi {
     }
     
     /**
+     * Set configuration via ConfigurationAdmin service in bundle context for component with given pid.
+     * @param bundleContext Bundle context
+     * @param pid PID
+     * @param properties Configuration properties
+     */
+    public static void setConfigForPid(BundleContext bundleContext, String pid, Map<String,Object> properties) {
+        setConfigForPid(bundleContext, pid, toDictionary(properties));
+    }
+    
+    /**
+     * Set configuration via ConfigurationAdmin service in bundle context for component with given pid.
+     * @param bundleContext Bundle context
+     * @param pid PID
+     * @param properties Configuration properties
+     */
+    public static void setConfigForPid(BundleContext bundleContext, String pid, Object... properties) {
+        setConfigForPid(bundleContext, pid, toDictionary(properties));
+    }
+    
+    private static void setConfigForPid(BundleContext bundleContext, String pid, Dictionary<String, Object> properties) {
+        ConfigurationAdmin configAdmin = getConfigAdmin(bundleContext);
+        if (configAdmin == null) {
+            throw new RuntimeException("ConfigurationAdmin service is not registered in bundle context.");
+        }
+        try {
+            Configuration config = configAdmin.getConfiguration(pid);
+            config.update(properties);
+        }
+        catch (IOException ex) {
+            throw new RuntimeException("Unable to update configuration for pid '" + pid + "'.", ex);
+        }
+    }
+    
+    /**
      * Deactivates all bundles registered in the mocked bundle context.
      * @param bundleContext Bundle context
      */

Modified: sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/package-info.java
URL: http://svn.apache.org/viewvc/sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/package-info.java?rev=1786956&r1=1786955&r2=1786956&view=diff
==============================================================================
--- sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/package-info.java (original)
+++ sling/trunk/testing/mocks/osgi-mock/src/main/java/org/apache/sling/testing/mock/osgi/package-info.java Tue Mar 14 21:47:48 2017
@@ -19,5 +19,5 @@
 /**
  * Mock implementation of selected OSGi APIs.
  */
-@org.osgi.annotation.versioning.Version("3.2")
+@org.osgi.annotation.versioning.Version("3.3")
 package org.apache.sling.testing.mock.osgi;

Modified: sling/trunk/testing/mocks/osgi-mock/src/test/java/org/apache/sling/testing/mock/osgi/MockConfigurationAdminTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/testing/mocks/osgi-mock/src/test/java/org/apache/sling/testing/mock/osgi/MockConfigurationAdminTest.java?rev=1786956&r1=1786955&r2=1786956&view=diff
==============================================================================
--- sling/trunk/testing/mocks/osgi-mock/src/test/java/org/apache/sling/testing/mock/osgi/MockConfigurationAdminTest.java (original)
+++ sling/trunk/testing/mocks/osgi-mock/src/test/java/org/apache/sling/testing/mock/osgi/MockConfigurationAdminTest.java Tue Mar 14 21:47:48 2017
@@ -22,8 +22,6 @@ import static org.junit.Assert.assertArr
 import static org.junit.Assert.assertEquals;
 
 import java.io.IOException;
-import java.util.Dictionary;
-import java.util.Hashtable;
 
 import org.apache.sling.testing.mock.osgi.OsgiMetadataUtilTest.ServiceWithMetadata;
 import org.apache.sling.testing.mock.osgi.junit.OsgiContext;
@@ -31,8 +29,6 @@ import org.junit.Rule;
 import org.junit.Test;
 import org.osgi.framework.Constants;
 import org.osgi.framework.ServiceReference;
-import org.osgi.service.cm.Configuration;
-import org.osgi.service.cm.ConfigurationAdmin;
 
 import com.google.common.collect.ImmutableMap;
 
@@ -48,14 +44,10 @@ public class MockConfigurationAdminTest
 
     @Test
     public void testGetConfigurationString() throws IOException {
-        ConfigurationAdmin configAdmin = context.getService(ConfigurationAdmin.class);
-        
-        Configuration config = configAdmin.getConfiguration("org.apache.sling.testing.mock.osgi.OsgiMetadataUtilTest$ServiceWithMetadata");
-        Dictionary<String, Object> configProps = new Hashtable<String, Object>();
-        configProps.put(Constants.SERVICE_RANKING, 3000);
-        configProps.put("adaptables", TEST_ADAPTABLES);
-        configProps.put("prop2", 2);
-        config.update(configProps);
+        MockOsgi.setConfigForPid(context.bundleContext(), "org.apache.sling.testing.mock.osgi.OsgiMetadataUtilTest$ServiceWithMetadata",
+                Constants.SERVICE_RANKING, 3000,
+                "adaptables", TEST_ADAPTABLES,
+                "prop2", 2);
         
         context.registerInjectActivateService(new ServiceWithMetadata(), ImmutableMap.<String, Object>builder()
                 .put(Constants.SERVICE_RANKING, 4000)