[Engine-patches] Change in ovirt-engine[master]: core: GetHostListFromExternalProvider brackets

[amureini]

Allon Mureinik has uploaded a new change for review.

Change subject: core: GetHostListFromExternalProvider brackets
......................................................................

core: GetHostListFromExternalProvider brackets

Removed redundant brackets.

Change-Id: I18a04efaf48f96b0b2acd7073a0c5d54764da9ea
Signed-off-by: Allon Mureinik <amure...@redhat.com>
---
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQueryTest.java
1 file changed, 2 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/63/27763/1

diff --git 
a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQueryTest.java
 
b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQueryTest.java
index 392fb6e..74edf6c 100644
--- 
a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQueryTest.java
+++ 
b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQueryTest.java
@@ -10,7 +10,6 @@
 import org.junit.Before;
 import static org.junit.Assert.assertEquals;
 import org.ovirt.engine.core.common.businessentities.Provider;
-import org.ovirt.engine.core.common.businessentities.ProviderType;
 import org.ovirt.engine.core.common.businessentities.VDS;
 import 
org.ovirt.engine.core.common.queries.GetHostListFromExternalProviderParameters;
 
@@ -30,11 +29,6 @@
         host2.setHostName("host2");
         existingHost1.setHostName("existingHost1");
         existingHost2.setHostName("existingHost2");
-        Provider hostProvider = new Provider();
-        hostProvider.setUsername("admin");
-        hostProvider.setPassword("password");
-        hostProvider.setUrl("http://provider.com";);
-        hostProvider.setType(ProviderType.FOREMAN);
     }
 
     @Test
@@ -65,7 +59,7 @@
         doReturn(allHosts).when(getQuery()).getProviderHosts(hostProvider, 
null);
         doReturn(existingHosts).when(getQuery()).getExistingHosts();
         getQuery().executeQueryCommand();
-        assertEquals(1, 
((List<VDS>)(getQuery().getQueryReturnValue().getReturnValue())).size());
+        assertEquals(1, ((List<VDS>) 
getQuery().getQueryReturnValue().getReturnValue()).size());
     }
 
     @Test
@@ -96,7 +90,7 @@
         doReturn(allHosts).when(getQuery()).getProviderHosts(hostProvider, 
null);
         doReturn(existingHosts).when(getQuery()).getExistingHosts();
         getQuery().executeQueryCommand();
-        assertEquals(2, 
((List<VDS>)(getQuery().getQueryReturnValue().getReturnValue())).size());
+        assertEquals(2, ((List<VDS>) 
getQuery().getQueryReturnValue().getReturnValue()).size());
     }
 
 }


-- 
To view, visit http://gerrit.ovirt.org/27763
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I18a04efaf48f96b0b2acd7073a0c5d54764da9ea
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches