> @@ -247,7 +247,7 @@ public void testReserveWhenResponseIs4xx() throws 
> Exception {
>              .addHeader("Accept", MediaType.APPLICATION_JSON)
>              .addHeader("X-Auth-Token", authToken).build();
>  
> -      HttpResponse createResponse = HttpResponse.builder().statusCode(401)
> +      HttpResponse createResponse = HttpResponse.builder().statusCode(403)

> Anyway, I merged this because it literally made the build take forever, and 
> that blocked not only the build but all subsequent PRs. Fix 
> the build, then fix the feature (if needed).

Oh, fully understand. And agree that if multiple 401s is not a "realistic" 
scenario then we don't need expect tests for it, at least not now. Thanks!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/16/files#r5136694

Reply via email to