jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Phillips <notificati...@github.com>
Subject Re: [jclouds-labs-aws] JCLOUDS-457: Add polling strategy (#44)
Date Sat, 02 Aug 2014 01:24:10 GMT
> @@ -112,11 +111,8 @@ public void testListJobs() {
>  
>     @Test(groups = {"live", "livelong", "longtest"}, dependsOnMethods = {"testInitiateJob",
"testDescribeJob", "testListJobs"})
>     public void testWaitForSucceed() throws InterruptedException {
> -      Thread.sleep(INITIAL_WAIT);
> -      while (api.describeJob(VAULT_NAME, archiveRetrievalJob).getStatusCode() == JobStatus.IN_PROGRESS
||
> -            api.describeJob(VAULT_NAME, inventoryRetrievalJob).getStatusCode() == JobStatus.IN_PROGRESS)
{
> -         Thread.sleep(TIME_BETWEEN_POLLS);
> -      }
> +      new BasePollingStrategy(api).waitForSuccess(VAULT_NAME, archiveRetrievalJob);

This will wait for **three hours**? Does it make sense to have a live test like that?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/44/files#r15726059
Mime
View raw message