> @@ -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;

Why not `assertTrue` or `assertNotNull` for these?

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

Reply via email to