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);