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
+}