> +               newSecGroupRule = sgr;
> +               break;
> +            }
> +         }
> +         assertNotNull(newSecGroupRule, "Did not find the new rule in the 
> group.");
> +
> +         assertEquals(rule, newSecGroupRule);
> +
> +         assertEquals(rule.getEthertype(), RuleEthertype.IPV6);
> +         assertEquals(rule.getProtocol(), RuleProtocol.TCP);
> +         assertEquals(rule.getPortRangeMax().intValue(), 90);
> +         assertEquals(rule.getPortRangeMin().intValue(), 80);
> +         assertEquals(rule.getDirection(), RuleDirection.EGRESS);
> +
> +         assertTrue(sgApi.deleteRule(rule.getId()));
> +         assertTrue(sgApi.deleteSecurityGroup(securityGroup.getId()));

The cleanup won't be done if the test fails earlier. Is that intentional?

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

Reply via email to