> +import static org.testng.Assert.assertNotNull;
> +
> +import org.jclouds.shipyard.domain.roles.RoleInfo;
> +import org.jclouds.shipyard.internal.BaseShipyardApiLiveTest;
> +import org.testng.annotations.AfterClass;
> +import org.testng.annotations.Test;
> +
> +@Test(groups = "live", testName = "RolesApiLiveTest", singleThreaded = true)
> +public class RolesApiLiveTest extends BaseShipyardApiLiveTest {
> +
> +   private final String roleName = "jclouds-shipyard-test-" + 
> UUID.randomUUID().toString().replaceAll("-", "");
> +   
> +   @AfterClass (alwaysRun = true)
> +   protected void tearDown() {
> +      boolean removed = api().deleteRole(roleName);
> +      assertTrue(removed);

This will fail if the `testCreateRole` test fails. Better leave the assertions 
for the tests, not the tearDown methods.

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

Reply via email to