> @@ -178,6 +210,15 @@ protected void checkFirewallRule(FirewallRule rule) {
>        assert rule.getProtocol() != null;
>     }
>  
> +   protected void checkEgressFirewallRule(FirewallRule rule) {
> +      assertEquals(rule,
> +              client.getFirewallApi().getEgressFirewallRule(rule.getId()));
> +      assert rule.getId() != null : rule;
> +      assert rule.getStartPort() > 0 : rule;
> +      assert rule.getEndPort() >= rule.getStartPort() : rule;
> +      assert rule.getProtocol() != null;

No good reason for assert. Just what we were doing for ingress rules, and I 
copy-and-pasted, since the content's the same.

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

Reply via email to