You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pd...@apache.org on 2010/04/25 11:36:43 UTC

svn commit: r937770 - /felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FactoryConfigurationAdapterTest.java

Author: pderop
Date: Sun Apr 25 09:36:42 2010
New Revision: 937770

URL: http://svn.apache.org/viewvc?rev=937770&view=rev
Log:
added an assert for detecting wrong CM update notification

Modified:
    felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FactoryConfigurationAdapterTest.java

Modified: felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FactoryConfigurationAdapterTest.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FactoryConfigurationAdapterTest.java?rev=937770&r1=937769&r2=937770&view=diff
==============================================================================
--- felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FactoryConfigurationAdapterTest.java (original)
+++ felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FactoryConfigurationAdapterTest.java Sun Apr 25 09:36:42 2010
@@ -82,7 +82,7 @@ public class FactoryConfigurationAdapter
             .setRequired(true)
             .setAutoConfig(true));
         
-        // Create extra adapter service dependency which our adapter depends on.
+        // Create extra adapter service dependency upon which our adapter depends on.
         Service s3 = m.createService()
             .setImplementation(new AdapterExtraDependency())
             .setInterface(AdapterExtraDependency.class.getName(), null);
@@ -185,6 +185,8 @@ public class FactoryConfigurationAdapter
                 m_ensure.step(9);
                 Assert.assertEquals(true, "value2".equals(settings.get("key")));
                 m_ensure.step(10);
+            } else {
+                Assert.fail("wrong call to updated method: count=" + updateCount);
             }
         }
 
@@ -237,4 +239,4 @@ public class FactoryConfigurationAdapter
             m_ensure.step(15);
         }
     }
-}
\ No newline at end of file
+}