jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject Re: [jclouds-labs] CloudSigma2 ComputeService and SecurityGroupExtension (#70)
Date Mon, 20 Oct 2014 14:29:28 GMT
> +      String uuid = "9001b532-857a-405a-8e50-54e342871e77";
> +
> +      CloudSigma2Api api = requestsSendResponses(
> +            deleteBuilder()
> +                  .endpoint(endpoint + "fwpolicies/" + uuid + "/")
> +                  .build(),
> +            responseBuilder()
> +                  .build(),
> +            deleteBuilder()
> +                  .endpoint(endpoint + "fwpolicies/failure")
> +                  .build(),
> +            HttpResponse.builder()
> +                  .statusCode(404)
> +                  .build());
> +
> +      api.deleteFirewallPolicy(uuid);

Nope. In contrast to mockwebserver tests, expect tests don't record the requests. This method
returns void, so there's nothing to assert, and due to how the expect tests work, if the method
succeeds, that means there is a match between the generated request and the expected one,
so the test is fine as-is.

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