> + return requestSendsResponse(requestBuilder.build(),
> + HttpResponse.builder().statusCode(200).payload(
> +
> payloadFromResourceWithContentType(payloadFile,
> +
> normalize(UsersDto.MEDIA_TYPE))) //
> + .build());
> + }
> +
> + public Builder<?> getRequestBuilder() {
> + return HttpRequest.builder() //
> + .method("GET")
> +
> .endpoint(URI.create("http://localhost/api/admin/enterprises/1/users";))
> + .addHeader("Cookie", tokenAuth)
> + .addHeader("Accept",
> normalize(UsersDto.MEDIA_TYPE));
> + }
> +
> + public void testListUsersReturns3xx() {
Method names like `fooReturns2xx` refer to the expected response codes (200
codes mean successful responses, for example). I'd change the name of the test
methods to reflect that, or just to `testListUsersWithoutPagination` (or
similar) to reflect what is being tested.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/22/files#r5705665
