> +
> + HttpResponse listResponse = HttpResponse.builder().statusCode(200)
> +
> .payload(payloadFromResource("/listAvailabilityZones.json")).build();
> +
> + NovaApi availabilityZonesApi =
> requestsSendResponses(keystoneAuthWithUsernameAndPasswordAndTenantName,
> + responseWithKeystoneAccess, extensionsOfNovaRequest,
> extensionsOfNovaResponse, list, listResponse);
> +
> + assertEquals(availabilityZonesApi.getConfiguredZones(),
> ImmutableSet.of("az-1.region-a.geo-1", "az-2.region-a.geo-1",
> "az-3.region-a.geo-1"));
> +
> + FluentIterable<? extends AvailabilityZone> zones =
> availabilityZonesApi.getAvailabilityZoneApi("az-1.region-a.geo-1").list();
> +
> + Optional<? extends AvailabilityZone> zone = zones.first();
> +
> + assertTrue(zone.isPresent());
> + Assert.assertTrue(zone.get().getName().equals("nova"));
> + Assert.assertTrue(zone.get().getZoneState().isAvailable());
Remove the `Assert.` bit? And please add a message to the `assertTrue` that
explains what is expected - this makes it easier to understand test failures.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/179/files#r6973678
