incubator-sling-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bdelacre...@apache.org
Subject svn commit: r810983 - /sling/trunk/installer/osgi/it/src/test/java/org/apache/sling/osgi/installer/it/ConfigInstallTest.java
Date Thu, 03 Sep 2009 14:52:44 GMT
Author: bdelacretaz
Date: Thu Sep  3 14:52:43 2009
New Revision: 810983

URL: http://svn.apache.org/viewvc?rev=810983&view=rev
Log:
SLING-1078 - more robust testing

Modified:
    sling/trunk/installer/osgi/it/src/test/java/org/apache/sling/osgi/installer/it/ConfigInstallTest.java

Modified: sling/trunk/installer/osgi/it/src/test/java/org/apache/sling/osgi/installer/it/ConfigInstallTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/installer/osgi/it/src/test/java/org/apache/sling/osgi/installer/it/ConfigInstallTest.java?rev=810983&r1=810982&r2=810983&view=diff
==============================================================================
--- sling/trunk/installer/osgi/it/src/test/java/org/apache/sling/osgi/installer/it/ConfigInstallTest.java
(original)
+++ sling/trunk/installer/osgi/it/src/test/java/org/apache/sling/osgi/installer/it/ConfigInstallTest.java
Thu Sep  3 14:52:43 2009
@@ -167,7 +167,8 @@
     	final Configuration c = ca.getConfiguration(cfgPid);
     	c.update(cfgData);
         waitForConfigValue("After manual installation", cfgPid, TIMEOUT, "foo", "bar");
-        assertEquals("Expected one ConfigurationEvent after manual install", 1, events.size());
+		Condition cond = new Condition() { public boolean isTrue() { return events.size() == 1;
}};
+        waitForCondition("Expected two ConfigurationEvents since beginning of test", TIMEOUT,
cond);
     	
         long nOps = installer.getCounters()[OsgiInstaller.OSGI_TASKS_COUNTER];
         installer.addResource(getInstallableResource(cfgPid, cfgData));
@@ -180,7 +181,7 @@
         cfgData.put("foo", "changed");
         installer.addResource(getInstallableResource(cfgPid, cfgData));
         waitForConfigValue("After changing value", cfgPid, TIMEOUT, "foo", "changed");
-		final Condition cond = new Condition() { public boolean isTrue() { return events.size()
== 2; }};
+		cond = new Condition() { public boolean isTrue() { return events.size() == 2; }};
         waitForCondition("Expected two ConfigurationEvents since beginning of test", TIMEOUT,
cond);
     }
 }



Mime
View raw message