> +      assertNicRemoved(testNic);
> +   } 
> +   
> +   private void assertNicAvailable(Nic nic) {
> +      assertPredicate(new Predicate<String>() {
> +         @Override
> +         public boolean apply(String args) {
> +            String[] params = args.split(",");
> +            Nic nic = nicApi().getNic(params[0], params[1], params[2]);
> +            
> +            if (nic == null || nic.metadata() == null)
> +               return false;
> +            
> +            return nic.metadata().state() == State.AVAILABLE;
> +         }
> +      }, complexId(nic.dataCenterId(), testServer.id(), nic.id()));

Why assembling this into a String to just split it immediately? Why not passing 
directly the Nic object to the predicate?

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/249/files/a17eac2160481358b8735aab30ecf12d672b341b#r56996964

Reply via email to