> @@ -149,6 +149,37 @@ public void testAddIpPermission() {
>        assertEquals(Iterables.getOnlyElement(newGroup.getIpPermissions()), 
> perm,
>                "Expecting IpPermission " + perm + " but group was " + 
> newGroup);
>     }
> +
> +   @Test(groups = { "integration", "live" }, singleThreaded = true, 
> dependsOnMethods = "testGetSecurityGroupById")

Fine with having this as a separate method, but I think we could also fold this 
in as a second "create/check" in `testAddIpPermission`, which would avoid a lot 
of the boilerplate.

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

Reply via email to