> + MockWebServer server = mockShipyardWebServer();
> + server.enqueue(new MockResponse().setResponseCode(204));
> + ShipyardApi shipyardApi = api(server.getUrl("/"));
> + ServiceKeysApi api = shipyardApi.serviceKeysApi();
> + try {
> + DeleteServiceKey deleteServiceKey =
> DeleteServiceKey.create("1111222233334444");
> + api.deleteServiceKey(deleteServiceKey);
> + assertSent(server, "DELETE", "/api/servicekeys");
> + } finally {
> + shipyardApi.close();
> + server.shutdown();
> + }
> + }
> +
> + @Test (expectedExceptions = HttpResponseException.class)
> + public void testDeleteNonExistentServiceKey() throws Exception {
Don't add tests for paths that are not explicitly present in the code. If you
don't declare a fallback, then remove the test. There's no need to test the
jclouds default behavior.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/149/files#r26168317
