nacx commented on this pull request.


> @@ -173,15 +170,6 @@ public void testExtendedOptionsAndLogin() throws 
> Exception {
             monitoringApi.close();
          }
 
-         // make sure we made our dummy group and also let in the user's group
-         assertEquals(newTreeSet(instance.getGroupNames()), 
ImmutableSortedSet.<String> of("jclouds#" + group, group));
-
-         // make sure our dummy group has no rules
-         SecurityGroup secgroup = 
getOnlyElement(securityGroupApi.describeSecurityGroupsInRegion(instance
-                  .getRegion(), "jclouds#" + group));
-
-         assert secgroup.size() == 0 : secgroup;

Note: This is not arbitrary. This should have been removed in 
https://github.com/jclouds/jclouds/pull/947

-- 
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/pull/1026#pullrequestreview-5372233

Reply via email to