fix issues

Author: Pearl1594

plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java

@@ -72,6 +72,7 @@
 import org.joda.time.Duration;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.libvirt.Connect;
@@ -5607,6 +5608,7 @@ public void testAddExtraConfigComponentNotEmptyExtraConfig() {
         Mockito.verify(vmDef, times(1)).addComp(Mockito.any());
     }
 
+    @Ignore
     public void validateGetCurrentMemAccordingToMemBallooningWithoutMemBalooning(){
         VirtualMachineTO vmTo = Mockito.mock(VirtualMachineTO.class);
         Mockito.when(vmTo.getType()).thenReturn(Type.User);
@@ -5692,6 +5694,7 @@ public void validateCountDomainRunningVcpus() throws LibvirtException{
         Assert.assertEquals(valueExpected, result);
     }
 
+    @Ignore
     public void setDiskIoDriverTestIoUring() {
         DiskDef diskDef = configureAndTestSetDiskIoDriverTest(HYPERVISOR_LIBVIRT_VERSION_SUPPORTS_IOURING, HYPERVISOR_QEMU_VERSION_SUPPORTS_IOURING);
         Assert.assertEquals(IoDriverPolicy.IO_URING, diskDef.getIoDriver());

plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ScaleIOStoragePoolTest.java

@@ -35,6 +35,7 @@
 import org.apache.cloudstack.utils.qemu.QemuImg.PhysicalDiskFormat;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.MockedConstruction;
@@ -140,6 +141,7 @@ public void testDefaults() {
         assertTrue(pool.isExternalSnapshot());
     }
 
+    @Ignore
     public void testGetPhysicalDiskWithWildcardFileFilter() throws Exception {
         final String volumePath = "6c3362b500000001:vol-139-3d2c-12f0";
         final String systemId = "218ce1797566a00f";

plugins/hypervisors/vmware/src/test/java/com/cloud/hypervisor/vmware/VmwareDatacenterApiUnitTest.java

@@ -79,6 +79,7 @@
 import org.apache.cloudstack.test.utils.SpringUtils;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.Mock;
@@ -240,12 +241,16 @@ public void tearDown() throws Exception {
     }
 
     //@Test(expected = InvalidParameterValueException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcToInvalidZone() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(addCmd.getZoneId()).thenReturn(2L);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
     }
 
     //@Test(expected = ResourceInUseException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcToZoneWithClusters() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(_clusterDao.listByZoneId(1L)).thenReturn(clusterList);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
@@ -264,36 +269,48 @@ public void testRemoveVmwareDcToZoneWithClusters() throws ResourceInUseException
     }
 
     //@Test(expected = ResourceInUseException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcToZoneWithVmwareDc() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(_vmwareDcDao.getVmwareDatacenterByNameAndVcenter(vmwareDcName, vCenterHost)).thenReturn(vmwareDcs);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
     }
 
     //@Test(expected = InvalidParameterValueException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcWithNullUser() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(addCmd.getUsername()).thenReturn(null);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
     }
 
     //@Test(expected = InvalidParameterValueException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcWithNullPassword() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(addCmd.getPassword()).thenReturn(null);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
     }
 
     //@Test(expected = InvalidParameterValueException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcWithNullUrl() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(addCmd.getVcenter()).thenReturn(null);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
     }
 
     //@Test(expected = InvalidParameterValueException.class)
+    @Test
+    @Ignore
     public void testAddVmwareDcWithNullDcName() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(addCmd.getName()).thenReturn(null);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);
     }
 
     //@Test(expected = CloudRuntimeException.class)
+    @Test
+    @Ignore
     public void testReAddVmwareDc() throws ResourceInUseException, IllegalArgumentException, DiscoveryException, Exception {
         Mockito.when(_vmwareDcZoneMapDao.findByZoneId(1L)).thenReturn(dcZoneMap);
         _vmwareDatacenterService.addVmwareDatacenter(addCmd);

plugins/hypervisors/xenserver/src/test/java/com/cloud/hypervisor/xenserver/resource/CitrixResourceBaseTest.java

@@ -37,6 +37,7 @@
 import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.BDDMockito;
@@ -112,6 +113,7 @@ public void beforeTest() throws XenAPIException, XmlRpcException {
 
     }
 
+    @Ignore
     public void testGetPathFilesException() {
         String patch = citrixResourceBase.getPatchFilePath();
 
@@ -122,6 +124,7 @@ public void testGetPathFilesException() {
         }
     }
 
+    @Ignore
     public void testGetPathFilesListReturned() {
         String patch = citrixResourceBase.getPatchFilePath();
 

plugins/network-elements/cisco-vnmc/src/test/java/com/cloud/network/cisco/CiscoVnmcConnectionTest.java

@@ -16,6 +16,7 @@
 // under the License.
 package com.cloud.network.cisco;
 
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
 import java.util.Map;
@@ -136,7 +137,7 @@ public void testCreateEdgeFirewall() throws ExecutionException {
     @Test
     public void testListUnassocAsa1000v() throws ExecutionException {
         Map<String, String> response = connection.listUnAssocAsa1000v();
-        assertTrue(response.size() >= 0);
+        assertFalse(response.isEmpty());
         fwDns = response;
     }
 

plugins/network-elements/cisco-vnmc/src/test/java/com/cloud/network/resource/CiscoVnmcResourceTest.java

@@ -32,6 +32,7 @@
 import javax.naming.ConfigurationException;
 
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 
 import com.cloud.agent.api.Answer;
@@ -79,13 +80,15 @@ public void resourceConfigureFailure() throws ConfigurationException {
     }
 
     //@Test
+    @Ignore
     public void resourceConfigure() throws ConfigurationException {
         _resource.configure("CiscoVnmcResource", _parameters);
         assertTrue("CiscoVnmc".equals(_resource.getName()));
         assertTrue(_resource.getType() == Host.Type.ExternalFirewall);
     }
 
     //@Test
+    @Ignore
     public void testInitialization() throws ConfigurationException {
         _resource.configure("CiscoVnmcResource", _parameters);
         StartupCommand[] sc = _resource.initialize();

plugins/network-elements/nsx/src/main/java/org/apache/cloudstack/agent/api/NsxNetworkCommand.java

@@ -105,7 +105,7 @@ public boolean equals(Object o) {
         if (o == null || getClass() != o.getClass()) return false;
         if (!super.equals(o)) return false;
         NsxNetworkCommand that = (NsxNetworkCommand) o;
-        return networkResourceId == that.networkResourceId && vmId == that.vmId &&
+        return networkResourceId.equals(that.networkResourceId) && vmId.equals(that.vmId) &&
                 Objects.equals(networkResourceName, that.networkResourceName) && Objects.equals(publicIp, that.publicIp)
                 && Objects.equals(vmIp, that.vmIp);
     }

plugins/network-elements/tungsten/src/test/java/org/apache/cloudstack/network/tungsten/api/command/AddTungstenFabricPolicyRuleCmdTest.java

@@ -24,6 +24,7 @@
 import org.apache.cloudstack.api.ServerApiException;
 import org.apache.cloudstack.network.tungsten.api.response.TungstenFabricRuleResponse;
 import org.apache.cloudstack.network.tungsten.service.TungstenService;
+import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
@@ -67,6 +68,7 @@ public void setup() {
         ReflectionTestUtils.setField(addTungstenFabricPolicyRuleCmd, "destEndPort", 1);
     }
 
+    @After
     public void tearDown() throws Exception {
         closeable.close();
     }

plugins/network-elements/tungsten/src/test/java/org/apache/cloudstack/network/tungsten/api/command/CreateTungstenFabricFirewallRuleCmdTest.java

@@ -24,6 +24,7 @@
 import org.apache.cloudstack.api.ServerApiException;
 import org.apache.cloudstack.network.tungsten.api.response.TungstenFabricFirewallRuleResponse;
 import org.apache.cloudstack.network.tungsten.service.TungstenService;
+import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
@@ -66,6 +67,7 @@ public void setup() {
         ReflectionTestUtils.setField(createTungstenFabricFirewallRuleCmd, "sequence", 1);
     }
 
+    @After
     public void tearDown() throws Exception {
         closeable.close();
     }

plugins/network-elements/tungsten/src/test/java/org/apache/cloudstack/network/tungsten/api/command/CreateTungstenFabricProviderCmdTest.java

@@ -25,6 +25,7 @@
 import org.apache.cloudstack.api.ServerApiException;
 import org.apache.cloudstack.network.tungsten.api.response.TungstenFabricProviderResponse;
 import org.apache.cloudstack.network.tungsten.service.TungstenProviderService;
+import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
@@ -60,6 +61,7 @@ public void setup() {
         ReflectionTestUtils.setField(createTungstenFabricProviderCmd, "introspectPort", "test");
     }
 
+    @After
     public void tearDown() throws Exception {
         closeable.close();
     }