sling-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cziege...@apache.org
Subject svn commit: r1649504 - /sling/trunk/installer/it/src/test/java/org/apache/sling/installer/it/ConfigInstallTest.java
Date Mon, 05 Jan 2015 11:25:15 GMT
Author: cziegeler
Date: Mon Jan  5 11:25:15 2015
New Revision: 1649504

URL: http://svn.apache.org/r1649504
Log:
Add todos to sleep command

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

Modified: sling/trunk/installer/it/src/test/java/org/apache/sling/installer/it/ConfigInstallTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/installer/it/src/test/java/org/apache/sling/installer/it/ConfigInstallTest.java?rev=1649504&r1=1649503&r2=1649504&view=diff
==============================================================================
--- sling/trunk/installer/it/src/test/java/org/apache/sling/installer/it/ConfigInstallTest.java
(original)
+++ sling/trunk/installer/it/src/test/java/org/apache/sling/installer/it/ConfigInstallTest.java
Mon Jan  5 11:25:15 2015
@@ -173,7 +173,7 @@ public class ConfigInstallTest extends O
                 null, secondData, null, InstallableResource.TYPE_PROPERTIES, 20);
         installer.updateResources(URL_SCHEME, new InstallableResource[] {rsrc2}, null);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated config
         final Configuration secondCfg = waitForConfiguration("After updating", cfgPid, TIMEOUT,
true);
@@ -182,7 +182,7 @@ public class ConfigInstallTest extends O
         // remove config
         installer.updateResources(URL_SCHEME, null, new String[] {"/configB/" + cfgPid});
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration origCfg = waitForConfiguration("After deleting", cfgPid, TIMEOUT,
true);
         assertEquals("Config value must match", "bar", origCfg.getProperties().get("foo"));
@@ -212,7 +212,7 @@ public class ConfigInstallTest extends O
                 null, secondData, null, InstallableResource.TYPE_PROPERTIES, 20);
         installer.updateResources(URL_SCHEME, new InstallableResource[] {rsrc2}, null);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated config
         final Configuration secondCfg = waitForConfiguration("After updating", cfgPid, TIMEOUT,
true);
@@ -221,7 +221,7 @@ public class ConfigInstallTest extends O
         // remove config
         installer.updateResources(URL_SCHEME, null, new String[] {"/configB/" + cfgPid});
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration noCfg = waitForConfiguration("After deleting", cfgPid, TIMEOUT,
false);
         assertNull("Configuration should be removed", noCfg);
@@ -251,7 +251,7 @@ public class ConfigInstallTest extends O
                 null, secondData, null, InstallableResource.TYPE_PROPERTIES, 20);
         installer.updateResources(URL_SCHEME, new InstallableResource[] {rsrc2}, null);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated factory config
         final Configuration secondCfg = waitForFactoryConfiguration("After updating", cfgFactoryPid,
TIMEOUT, true);
@@ -260,7 +260,7 @@ public class ConfigInstallTest extends O
         // remove factory config
         installer.updateResources(URL_SCHEME, null, new String[] {"/configB/" + cfgFactoryPid
+ "-" + alias});
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration origCfg = waitForFactoryConfiguration("After deleting", cfgFactoryPid,
TIMEOUT, true);
         assertEquals("Config value must match", "bar", origCfg.getProperties().get("foo"));
@@ -291,7 +291,7 @@ public class ConfigInstallTest extends O
                 null, secondData, null, InstallableResource.TYPE_PROPERTIES, 20);
         installer.updateResources(URL_SCHEME, new InstallableResource[] {rsrc2}, null);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated factory config
         final Configuration secondCfg = waitForFactoryConfiguration("After updating", cfgFactoryPid,
TIMEOUT, true);
@@ -300,7 +300,7 @@ public class ConfigInstallTest extends O
         // remove config
         installer.updateResources(URL_SCHEME, null, new String[] {"/configB/" + cfgFactoryPid
+ "-" + alias});
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration noCfg = waitForFactoryConfiguration("After deleting", cfgFactoryPid,
TIMEOUT, false);
         assertNull("Factory configuration should be removed", noCfg);
@@ -327,7 +327,7 @@ public class ConfigInstallTest extends O
         secondData.put("foo", "bla");
         cfg.update(secondData);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated config
         final Configuration secondCfg = waitForConfiguration("After updating", cfgPid, TIMEOUT,
true);
@@ -336,7 +336,7 @@ public class ConfigInstallTest extends O
         // remove config
         secondCfg.delete();
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration origCfg = waitForConfiguration("After deleting", cfgPid, TIMEOUT,
true);
         assertEquals("Config value must match", "bar", origCfg.getProperties().get("foo"));
@@ -364,7 +364,7 @@ public class ConfigInstallTest extends O
         secondData.put("foo", "bla");
         cfg.update(secondData);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated config
         final Configuration secondCfg = waitForConfiguration("After updating", cfgPid, TIMEOUT,
true);
@@ -373,7 +373,7 @@ public class ConfigInstallTest extends O
         // remove config
         secondCfg.delete();
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration noCfg = waitForConfiguration("After deleting", cfgPid, TIMEOUT,
false);
         assertNull("Configuration should be removed", noCfg);
@@ -401,7 +401,7 @@ public class ConfigInstallTest extends O
         secondData.put("foo", "bla");
         cfg.update(secondData);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated factory config
         final Configuration secondCfg = waitForFactoryConfiguration("After updating", cfgFactoryPid,
TIMEOUT, true);
@@ -409,7 +409,7 @@ public class ConfigInstallTest extends O
 
         // remove factory config
         secondCfg.delete();
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration origCfg = waitForFactoryConfiguration("After deleting", cfgFactoryPid,
TIMEOUT, true);
         assertEquals("Config value must match", "bar", origCfg.getProperties().get("foo"));
@@ -438,7 +438,7 @@ public class ConfigInstallTest extends O
         secondData.put("foo", "bla");
         cfg.update(secondData);
 
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         // get updated factory config
         final Configuration secondCfg = waitForFactoryConfiguration("After updating", cfgFactoryPid,
TIMEOUT, true);
@@ -446,7 +446,7 @@ public class ConfigInstallTest extends O
 
         // remove config
         secondCfg.delete();
-        sleep(200);
+        sleep(200); // TODO replace with wait condition
 
         final Configuration noCfg = waitForFactoryConfiguration("After deleting", cfgFactoryPid,
TIMEOUT, false);
         assertNull("Factory configuration should be removed", noCfg);



Mime
View raw message