Remove test cases duplicated code

Removed auto generated code comment

Author: rafaelweingartner

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

@@ -19,14 +19,22 @@
 import java.util.List;
 
 import org.junit.Assert;
+import org.junit.Test;
 import org.mockito.Mockito;
 import org.powermock.api.mockito.PowerMockito;
 
 import com.cloud.utils.script.Script;
 
-public abstract class CitrixResourceBaseTest {
+public class CitrixResourceBaseTest {
 
-    public void testGetPathFilesExeption(CitrixResourceBase citrixResourceBase) {
+    protected CitrixResourceBase citrixResourceBase = new CitrixResourceBase() {
+        @Override
+        protected String getPatchFilePath() {
+            return null;
+        }
+    };
+
+    public void testGetPathFilesExeption() {
         String patch = citrixResourceBase.getPatchFilePath();
 
         PowerMockito.mockStatic(Script.class);
@@ -36,7 +44,7 @@ public void testGetPathFilesExeption(CitrixResourceBase citrixResourceBase) {
 
     }
 
-    public void testGetPathFilesListReturned(CitrixResourceBase citrixResourceBase) {
+    public void testGetPathFilesListReturned() {
         String patch = citrixResourceBase.getPatchFilePath();
 
         PowerMockito.mockStatic(Script.class);
@@ -50,31 +58,35 @@ public void testGetPathFilesListReturned(CitrixResourceBase citrixResourceBase)
         Assert.assertEquals(receivedPath, pathExpected);
     }
 
-    public void testGetGuestOsTypeNull(CitrixResourceBase citrixResourceBase) {
+    @Test
+    public void testGetGuestOsTypeNull() {
         String platformEmulator = null;
 
         String expected = "Other install media";
         String guestOsType = citrixResourceBase.getGuestOsType(platformEmulator);
         Assert.assertEquals(expected, guestOsType);
     }
 
-    public void testGetGuestOsTypeEmpty(CitrixResourceBase citrixResourceBase) {
+    @Test
+    public void testGetGuestOsTypeEmpty() {
         String platformEmulator = "";
 
         String expected = "Other install media";
         String guestOsType = citrixResourceBase.getGuestOsType(platformEmulator);
         Assert.assertEquals(expected, guestOsType);
     }
 
-    public void testGetGuestOsTypeBlank(CitrixResourceBase citrixResourceBase) {
+    @Test
+    public void testGetGuestOsTypeBlank() {
         String platformEmulator = "    ";
 
         String expected = "Other install media";
         String guestOsType = citrixResourceBase.getGuestOsType(platformEmulator);
         Assert.assertEquals(expected, guestOsType);
     }
 
-    public void testGetGuestOsTypeOther(CitrixResourceBase citrixResourceBase) {
+    @Test
+    public void testGetGuestOsTypeOther() {
         String platformEmulator = "My Own Linux Distribution Y.M (64-bit)";
 
         String guestOsType = citrixResourceBase.getGuestOsType(platformEmulator);

plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/XcpOssResourceTest.java

@@ -16,6 +16,7 @@
 package com.cloud.hypervisor.xenserver.resource;
 
 import org.junit.Assert;
+import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.powermock.core.classloader.annotations.PrepareForTest;
@@ -26,11 +27,14 @@
 @RunWith(PowerMockRunner.class)
 public class XcpOssResourceTest extends CitrixResourceBaseTest{
 
-    private XcpOssResource xcpOssResource = new XcpOssResource();
+    @Before
+    public void beforeTest() {
+        super.citrixResourceBase = new XcpOssResource();
+    }
 
     @Test
     public void testPatchFilePath() {
-        String patchFilePath = xcpOssResource.getPatchFilePath();
+        String patchFilePath = citrixResourceBase.getPatchFilePath();
         String patch = "scripts/vm/hypervisor/xenserver/xcposs/patch";
 
         Assert.assertEquals(patch, patchFilePath);
@@ -39,29 +43,12 @@ public void testPatchFilePath() {
     @Test(expected = CloudRuntimeException.class)
     @PrepareForTest(Script.class )
     public void testGetFiles(){
-        testGetPathFilesExeption(xcpOssResource);
+        testGetPathFilesExeption();
     }
 
     @Test
     @PrepareForTest(Script.class )
     public void testGetFilesListReturned(){
-        testGetPathFilesListReturned(xcpOssResource);
-    }
-
-    @Test
-    public void testGetOsTypeNull() {
-        testGetGuestOsTypeNull(xcpOssResource);
-    }
-    @Test
-    public void testGetOsTypeEmpty() {
-        testGetGuestOsTypeEmpty(xcpOssResource);
-    }
-    @Test
-    public void testGetOsTypeBlank() {
-        testGetGuestOsTypeBlank(xcpOssResource);
-    }
-    @Test
-    public void testGetOsTypeOther() {
-        testGetGuestOsTypeOther(xcpOssResource);
+        testGetPathFilesListReturned();
     }
 }

plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/XcpServerResourceTest.java

@@ -16,6 +16,7 @@
 package com.cloud.hypervisor.xenserver.resource;
 
 import org.junit.Assert;
+import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.powermock.core.classloader.annotations.PrepareForTest;
@@ -27,11 +28,14 @@
 @RunWith(PowerMockRunner.class)
 public class XcpServerResourceTest extends CitrixResourceBaseTest{
 
-    private XcpServerResource xcpServerResource = new XcpServerResource();
+    @Before
+    public void beforeTest() {
+        super.citrixResourceBase = new XcpServerResource();
+    }
 
     @Test
     public void testPatchFilePath() {
-        String patchFilePath = xcpServerResource.getPatchFilePath();
+        String patchFilePath = citrixResourceBase.getPatchFilePath();
         String patch = "scripts/vm/hypervisor/xenserver/xcpserver/patch";
 
         Assert.assertEquals(patch, patchFilePath);
@@ -40,29 +44,12 @@ public void testPatchFilePath() {
     @Test(expected = CloudRuntimeException.class)
     @PrepareForTest(Script.class )
     public void testGetFilesExeption(){
-        testGetPathFilesExeption(xcpServerResource);
+        testGetPathFilesExeption();
     }
 
     @Test
     @PrepareForTest(Script.class )
     public void testGetFilesListReturned(){
-        testGetPathFilesListReturned(xcpServerResource);
-    }
-
-    @Test
-    public void testGetOsTypeNull() {
-        testGetGuestOsTypeNull(xcpServerResource);
-    }
-    @Test
-    public void testGetOsTypeEmpty() {
-        testGetGuestOsTypeEmpty(xcpServerResource);
-    }
-    @Test
-    public void testGetOsTypeBlank() {
-        testGetGuestOsTypeBlank(xcpServerResource);
-    }
-    @Test
-    public void testGetOsTypeOther() {
-        testGetGuestOsTypeOther(xcpServerResource);
+        testGetPathFilesListReturned();
     }
 }

plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/XenServer56FP1ResourceTest.java

@@ -16,50 +16,40 @@
 package com.cloud.hypervisor.xenserver.resource;
 
 import org.junit.Assert;
+import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;
 
 import com.cloud.utils.exception.CloudRuntimeException;
 import com.cloud.utils.script.Script;
+
 @RunWith(PowerMockRunner.class)
 public class XenServer56FP1ResourceTest extends CitrixResourceBaseTest{
 
-    private XenServer56FP1Resource xenServer56FP1Resource = new XenServer56FP1Resource();
+    @Before
+    public void beforeTest() {
+        super.citrixResourceBase = new XenServer56FP1Resource();
+    }
 
     @Test
     public void testPatchFilePath() {
-        String patchFilePath = xenServer56FP1Resource.getPatchFilePath();
+        String patchFilePath = citrixResourceBase.getPatchFilePath();
         String patch = "scripts/vm/hypervisor/xenserver/xenserver56fp1/patch";
 
         Assert.assertEquals(patch, patchFilePath);
     }
+
     @Test(expected = CloudRuntimeException.class)
     @PrepareForTest(Script.class )
     public void testGetFiles(){
-        testGetPathFilesExeption(xenServer56FP1Resource);
+        testGetPathFilesExeption();
     }
+
     @Test
     @PrepareForTest(Script.class )
     public void testGetFilesListReturned(){
-        testGetPathFilesListReturned(xenServer56FP1Resource);
-    }
-
-    @Test
-    public void testGetOsTypeNull() {
-        testGetGuestOsTypeNull(xenServer56FP1Resource);
-    }
-    @Test
-    public void testGetOsTypeEmpty() {
-        testGetGuestOsTypeEmpty(xenServer56FP1Resource);
-    }
-    @Test
-    public void testGetOsTypeBlank() {
-        testGetGuestOsTypeBlank(xenServer56FP1Resource);
-    }
-    @Test
-    public void testGetOsTypeOther() {
-        testGetGuestOsTypeOther(xenServer56FP1Resource);
+        testGetPathFilesListReturned();
     }
 }

plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/XenServer56ResourceTest.java

@@ -16,6 +16,7 @@
 package com.cloud.hypervisor.xenserver.resource;
 
 import org.junit.Assert;
+import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.powermock.core.classloader.annotations.PrepareForTest;
@@ -26,11 +27,14 @@
 @RunWith(PowerMockRunner.class)
 public class XenServer56ResourceTest extends CitrixResourceBaseTest {
 
-    private XenServer56Resource xenServer56Resource = new XenServer56Resource();
+    @Before
+    public void beforeTest() {
+        super.citrixResourceBase = new XenServer56Resource();
+    }
 
     @Test
     public void testPatchFilePath() {
-        String patchFilePath = xenServer56Resource.getPatchFilePath();
+        String patchFilePath = citrixResourceBase.getPatchFilePath();
         String patch = "scripts/vm/hypervisor/xenserver/xenserver56/patch";
 
         Assert.assertEquals(patch, patchFilePath);
@@ -39,28 +43,12 @@ public void testPatchFilePath() {
     @Test(expected = CloudRuntimeException.class)
     @PrepareForTest(Script.class )
     public void testGetFiles(){
-        testGetPathFilesExeption(xenServer56Resource);
+        testGetPathFilesExeption();
     }
+
     @Test
     @PrepareForTest(Script.class )
     public void testGetFilesListReturned(){
-        testGetPathFilesListReturned(xenServer56Resource);
-    }
-
-    @Test
-    public void testGetOsTypeNull() {
-        testGetGuestOsTypeNull(xenServer56Resource);
-    }
-    @Test
-    public void testGetOsTypeEmpty() {
-        testGetGuestOsTypeEmpty(xenServer56Resource);
-    }
-    @Test
-    public void testGetOsTypeBlank() {
-        testGetGuestOsTypeBlank(xenServer56Resource);
-    }
-    @Test
-    public void testGetOsTypeOther() {
-        testGetGuestOsTypeOther(xenServer56Resource);
+        testGetPathFilesListReturned();
     }
 }

plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/XenServer56SP2ResourceTest.java

@@ -16,6 +16,7 @@
 package com.cloud.hypervisor.xenserver.resource;
 
 import org.junit.Assert;
+import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.powermock.core.classloader.annotations.PrepareForTest;
@@ -26,40 +27,28 @@
 @RunWith(PowerMockRunner.class)
 public class XenServer56SP2ResourceTest extends CitrixResourceBaseTest{
 
-    private XenServer56SP2Resource xenServer56SP2Resource = new XenServer56SP2Resource();
+    @Before
+    public void beforeTest() {
+        super.citrixResourceBase = new XenServer56SP2Resource();
+    }
 
     @Test
     public void testPatchFilePath() {
-        String patchFilePath = xenServer56SP2Resource.getPatchFilePath();
+        String patchFilePath = citrixResourceBase.getPatchFilePath();
         String patch = "scripts/vm/hypervisor/xenserver/xenserver56fp1/patch";
 
         Assert.assertEquals(patch, patchFilePath);
     }
+
     @Test(expected = CloudRuntimeException.class)
     @PrepareForTest(Script.class )
     public void testGetFiles(){
-        testGetPathFilesExeption(xenServer56SP2Resource);
+        testGetPathFilesExeption();
     }
+
     @Test
     @PrepareForTest(Script.class )
     public void testGetFilesListReturned(){
-        testGetPathFilesListReturned(xenServer56SP2Resource);
-    }
-
-    @Test
-    public void testGetOsTypeNull() {
-        testGetGuestOsTypeNull(xenServer56SP2Resource);
-    }
-    @Test
-    public void testGetOsTypeEmpty() {
-        testGetGuestOsTypeEmpty(xenServer56SP2Resource);
-    }
-    @Test
-    public void testGetOsTypeBlank() {
-        testGetGuestOsTypeBlank(xenServer56SP2Resource);
-    }
-    @Test
-    public void testGetOsTypeOther() {
-        testGetGuestOsTypeOther(xenServer56SP2Resource);
+        testGetPathFilesListReturned();
     }
 }