> +   @Test(groups = { "integration", "live" })
> +   public void testListVaults() throws Exception {
> +      GlacierClient client = getGlacierClient();
> +      client.createVault("testListVaults1");
> +      client.createVault("testListVaults2");
> +      client.createVault("testListVaults3");
> +      PaginatedVaultCollection vaults = getGlacierClient().listVaults();
> +      Iterator<VaultMetadata> i = vaults.iterator();
> +      assertEquals(i.next().getVaultName(), "testListVaults1");
> +      assertEquals(i.next().getVaultName(), "testListVaults2");
> +      assertEquals(i.next().getVaultName(), "testListVaults3");
> +      client.deleteVaultIfEmpty("testListVaults1");
> +      client.deleteVaultIfEmpty("testListVaults2");
> +      client.deleteVaultIfEmpty("testListVaults3");
> +   }
> +}

Instead of creating N vaults, could you store the created ones in local 
variables, and make the `list` and `get` tests depend on the create one? That 
could save some calls to the provider API and save a valuable time when running 
the live tests (we have providers where live tests take more than 30 min. to 
run).

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/4/files#r12843971

Reply via email to