> +
> +      try {
> +         api.removeNetwork(networkId);
> +         assertSent(server, "DELETE", "/networks/" + networkId);
> +
> +      } finally {
> +         server.shutdown();
> +      }
> +   }
> +
> +   public void testConnectContainerToNetwork() throws Exception {
> +      MockWebServer server = mockWebServer(new 
> MockResponse().setResponseCode(200));
> +      NetworkApi api = api(DockerApi.class, 
> server.getUrl("/").toString()).getNetworkApi();
> +      try {
> +         api.connectContainerToNetwork("1", "containerName");
> +         assertSent(server, "POST", "/networks/1/connect");

I know that it is not done in any of the mock tests, but when testing methods 
that produce a body, we should also verify the sent body is the expected one. 
Let's add these checks to this test and fix the other mock tests in upcoming 
PRs.

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

Reply via email to