From d6830bd5ff8da00ab3c4c9876c85f47f04a27ac7 Mon Sep 17 00:00:00 2001 From: Andrew Bayer Date: Tue, 1 Oct 2013 14:36:34 -0700 Subject: [PATCH] Fix assertEquals arg ordering in BaseSecurityGroupExtensionLiveTest --- .../internal/BaseSecurityGroupExtensionLiveTest.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseSecurityGroupExtensionLiveTest.java b/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseSecurityGroupExtensionLiveTest.java index 3f775d2db9..022b670c65 100644 --- a/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseSecurityGroupExtensionLiveTest.java +++ b/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseSecurityGroupExtensionLiveTest.java @@ -137,7 +137,8 @@ public abstract class BaseSecurityGroupExtensionLiveTest extends BaseComputeServ SecurityGroup newGroup = securityGroupExtension.get().addIpPermission(perm, group); - assertEquals(perm, Iterables.getOnlyElement(newGroup.getIpPermissions())); + assertEquals(Iterables.getOnlyElement(newGroup.getIpPermissions()), perm, + "Expecting IpPermission " + perm + " but group was " + newGroup); } @Test(groups = { "integration", "live" }, singleThreaded = true, dependsOnMethods = "testAddIpPermission") @@ -165,7 +166,8 @@ public abstract class BaseSecurityGroupExtensionLiveTest extends BaseComputeServ SecurityGroup newGroup = securityGroupExtension.get().removeIpPermission(perm, group); - assertEquals(0, Iterables.size(newGroup.getIpPermissions())); + assertEquals(Iterables.size(newGroup.getIpPermissions()), 0, + "Group should have no permissions, but has " + Iterables.size(newGroup.getIpPermissions())); } @Test(groups = { "integration", "live" }, singleThreaded = true, dependsOnMethods = "testRemoveIpPermission")