>     public void testStopContainer() {
>        api().stopContainer(container.id());
>        assertFalse(api().inspectContainer(container.id()).state().running());
>     }
>  
> +   @Test(dependsOnMethods = "testStopContainer")
> +   public void testRestartContainer() {
> +      api().restart(container.id(), RestartOptions.NONE);
> +      assertTrue(api().inspectContainer(container.id()).state().running());
> +   }
> +
> +   @Test(dependsOnMethods = "testRestartContainer")
> +   public void testWaitContainer() {
> +      api().stopContainer(container.id(), StopOptions.Builder.t("1"));
> +      assertEquals(api().wait(container.id()).statusCode(), -1);
> +   }
> +
> +   @Test(dependsOnMethods = "testWaitContainer", expectedExceptions = 
> NullPointerException.class)

NullPointerException is never something we should expect. make the test 
assertNull or something.

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

Reply via email to