> +         }
> +
> +         for(Subnet subnet : subnetApi.list().concat().toList()) {
> +            subnetApi.delete(subnet.getId());
> +         }
> +
> +         for(Port port : portApi.list().concat().toList()) {
> +            portApi.delete(port.getId());
> +         }
> +
> +         for(Network network : networkApi.list().concat().toList()) {
> +            if("private".equals(network.getName()) || 
> "public".equals(network.getName()))continue;
> +            networkApi.delete(network.getId());
> +         }
> +      }
> +   }*/

The new live tests pretty much are a 1:1 rewrite of the old neutron tests. 
Unfortunately, they might need to be refactored more to use proper 
setup/teardown - in some cases, especially failures, you will have left-over 
entities. Running this test is a nice way to cleanup your account, but doesn't 
make sense leaving it enabled. It might make sense to control it through some 
sort of property instead of just commenting it out, though. Best solution would 
be to refactor the tests though.
Thoughts?

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

Reply via email to