remove WIP unit test

Author: dhlaluku

server/src/test/java/org/apache/cloudstack/diagnostics/DiagnosticsServiceImplTest.java

@@ -18,27 +18,12 @@
 //
 package org.apache.cloudstack.diagnostics;
 
-import com.cloud.agent.AgentManager;
-import com.cloud.agent.api.Answer;
-import com.cloud.agent.api.routing.NetworkElementCommand;
-import com.cloud.exception.InvalidParameterValueException;
-import com.cloud.hypervisor.Hypervisor;
-import com.cloud.server.StatsCollector;
-import com.cloud.utils.exception.CloudRuntimeException;
-import com.cloud.vm.VMInstanceVO;
-import com.cloud.vm.VirtualMachine;
-import com.cloud.vm.VirtualMachineManager;
-import com.cloud.vm.dao.VMInstanceDao;
-import junit.framework.TestCase;
+import java.util.HashMap;
+import java.util.Map;
+
 import org.apache.cloudstack.api.ApiConstants;
-import org.apache.cloudstack.api.command.admin.diagnostics.GetDiagnosticsDataCmd;
 import org.apache.cloudstack.api.command.admin.diagnostics.RunDiagnosticsCmd;
 import org.apache.cloudstack.engine.orchestration.service.NetworkOrchestrationService;
-import org.apache.cloudstack.engine.subsystem.api.storage.DataStore;
-import org.apache.cloudstack.engine.subsystem.api.storage.DataStoreManager;
-import org.apache.cloudstack.engine.subsystem.api.storage.ZoneScope;
-import org.apache.cloudstack.storage.image.datastore.ImageStoreEntity;
-import org.apache.cxf.management.annotation.ManagedOperationParameter;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
@@ -48,10 +33,16 @@
 import org.mockito.Mockito;
 import org.mockito.runners.MockitoJUnitRunner;
 
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
+import com.cloud.agent.AgentManager;
+import com.cloud.agent.api.routing.NetworkElementCommand;
+import com.cloud.exception.InvalidParameterValueException;
+import com.cloud.utils.exception.CloudRuntimeException;
+import com.cloud.vm.VMInstanceVO;
+import com.cloud.vm.VirtualMachine;
+import com.cloud.vm.VirtualMachineManager;
+import com.cloud.vm.dao.VMInstanceDao;
+
+import junit.framework.TestCase;
 
 @RunWith(MockitoJUnitRunner.class)
 public class DiagnosticsServiceImplTest extends TestCase {
@@ -71,20 +62,6 @@ public class DiagnosticsServiceImplTest extends TestCase {
     @Mock
     private NetworkOrchestrationService networkManager;
 
-    // Get Diagnostics Data additions
-    @Mock
-    private GetDiagnosticsDataCmd getDiagnosticsDataCmd;
-    @Mock
-    private DataStoreManager storeMgr;
-    @Mock
-    private DataStore store;
-    @Mock
-    private StatsCollector statsCollector;
-    @Mock
-    private PrepareFilesCommand filesCommand;
-    @Mock
-    private ImageStoreEntity storeEntity;
-
     @InjectMocks
     private DiagnosticsServiceImpl serviceImpl = new DiagnosticsServiceImpl();
 
@@ -94,14 +71,6 @@ public void setUp() throws Exception {
         Mockito.when(runDiagnosticsCmd.getType()).thenReturn(DiagnosticsType.PING);
         Mockito.when(instanceDao.findByIdTypes(Mockito.anyLong(), Mockito.any(VirtualMachine.Type.class),
                 Mockito.any(VirtualMachine.Type.class), Mockito.any(VirtualMachine.Type.class))).thenReturn(vmInstanceVO);
-
-        // GetDiagnosticsDataCmd
-        Map<String, String> accessDetailsMap = new HashMap<>();
-        accessDetailsMap.put(NetworkElementCommand.ROUTER_IP, "169.20.175.10");
-        Mockito.when(networkManager.getSystemVMAccessDetails(Mockito.any(VMInstanceVO.class))).thenReturn(accessDetailsMap);
-        Mockito.when(getDiagnosticsDataCmd.getId()).thenReturn(1L);
-        Mockito.when(vmInstanceVO.getHostId()).thenReturn(1L);
-
     }
 
     @After
@@ -225,49 +194,4 @@ public void testValidCharsInParams() throws Exception {
         assertTrue(serviceImpl.hasValidChars("-c 5 -4"));
         assertTrue(serviceImpl.hasValidChars("-c 5 -4 -AbDfhqUV"));
     }
-
-    @Test
-    public void testGetDiagnosticsDataCommandFromDefaults() {
-        Mockito.when(getDiagnosticsDataCmd.getFilesList()).thenReturn(null);
-        Mockito.when(vmInstanceVO.getType()).thenReturn(VirtualMachine.Type.ConsoleProxy);
-        List<DataStore> mockList = new ArrayList<>();
-        mockList.add(store);
-        mockList.add(store);
-        mockList.add(store);
-        Mockito.when(storeMgr.getImageStoresByScope(Mockito.any(ZoneScope.class))).thenReturn(mockList);
-
-        Long zoneId = 2L;
-
-        Mockito.when(vmInstanceVO.getDataCenterId()).thenReturn(zoneId);
-
-        Mockito.when(statsCollector.imageStoreHasEnoughCapacity(store, serviceImpl.DiskQuotaPercentageThreshold.value())).thenReturn(true);
-
-        List<VMInstanceVO> ssvmList = new ArrayList<>();
-        ssvmList.add(vmInstanceVO);
-        Mockito.when(instanceDao.listByZoneIdAndType(zoneId, VirtualMachine.Type.SecondaryStorageVm)).thenReturn(ssvmList);
-        // Get Default Files from global settings
-        List<String> fileList = DiagnosticsFilesListFactory.getDiagnosticsFilesList(getDiagnosticsDataCmd.getFilesList(),
-                vmInstanceVO).generateFileList();
-
-        String zipFile = "/root/diagnostics_xxxx.tar\n";
-
-        // Mock Agent Answer
-        Mockito.when(agentManager.easySend(Mockito.anyLong(), Mockito.any(PrepareFilesCommand.class))).thenReturn(new Answer(filesCommand, true, zipFile));
-
-        Mockito.when(vmInstanceVO.getHypervisorType()).thenReturn(Hypervisor.HypervisorType.KVM);
-
-        String url = serviceImpl.getDiagnosticsDataCommand(getDiagnosticsDataCmd);
-    }
-
-    @Test
-    public void getSystemVMInstance() {
-    }
-
-    @Test
-    public void prepareNetworkElementCommand() {
-    }
-
-    @Test
-    public void getVMSshIp() {
-    }
 }