Add Mojo Test

pom.xml

@@ -16,13 +16,15 @@
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<!-- change the versions at your own peril -->
-		<junit.version>3.8.1</junit.version>
+		<junit.version>4.12</junit.version>
 		<maven.version>3.2.2</maven.version>
 		<maven-plugin-annotations.version>3.5</maven-plugin-annotations.version>
 		<java.version>1.8</java.version>
 		<maven-compiler-plugin.version>3.5.1</maven-compiler-plugin.version>
 		<maven-compat.version>3.2.1</maven-compat.version>
 		<maven-plugin-plugin.version>3.2</maven-plugin-plugin.version>
+	    <mockito.version>1.10.19</mockito.version>
+
 	</properties>
 
 	<scm>
@@ -102,6 +104,12 @@
                 </exclusion>                
             </exclusions>
 		</dependency>
+		<dependency>
+            <groupId>org.mockito</groupId>
+            <artifactId>mockito-core</artifactId>
+            <version>${mockito.version}</version>
+            <scope>test</scope>
+        </dependency>
 	</dependencies>
 
 	<build>

src/main/java/uk/gov/justice/plugin/StartServerMojo.java

@@ -92,22 +92,24 @@ public void setServerClass(String serverClass) {
 
     private Thread waitThread;
 
+    private File java;
+
     @Override
     public void execute() throws MojoExecutionException {
         try {
-            final File java = new File(new File(System.getProperty("java.home"), "bin"), "java");
+            java = new File(new File(System.getProperty("java.home"), "bin"), "java");
 
             final List<String> args = new ArrayList<String>();
             args.add(java.getAbsolutePath());
             args.add("-cp");
             args.add(buildClasspath());
             args.add(getServerClass());
 
-            final Process p = new ProcessBuilder(args).start();
-            dumpStream(p.getInputStream(), System.out);
-            dumpStream(p.getErrorStream(), System.err);
-            addShutdownHook(p);
-            waitOnStopCommand(p);
+            final Process process = new ProcessBuilder(args).start();
+            dumpStream(process.getInputStream(), System.out);
+            dumpStream(process.getErrorStream(), System.err);
+            addShutdownHook(process);
+            waitOnStopCommand(process);
 
         } catch (Exception e) {
             throw new RuntimeException(e);
@@ -166,7 +168,6 @@ public void run() {
     }
 
     private void addShutdownHook(final Process process) {
-
         Runtime.getRuntime().addShutdownHook(new Thread(new Runnable() {
             public void run() {
                 try {
@@ -178,13 +179,13 @@ public void run() {
         }));
     }
 
-    private void destroyIfAlive(final Process process) {
+    private synchronized void destroyIfAlive(final Process process) {
         if (!Objects.isNull(process) && process.isAlive()) {
             process.destroy();
         }
     }
 
-    private void destroyForciblyIfAlive(final Process process) {
+    private synchronized void destroyForciblyIfAlive(final Process process) {
         if (!Objects.isNull(process) && process.isAlive()) {
             process.destroyForcibly();
         }
@@ -201,4 +202,13 @@ public void run() {
             }
         }).start();
     }
+
+    public File getJava() {
+        return java;
+    }
+
+    public void setJava(File java) {
+        this.java = java;
+    }
+
 }

src/main/java/uk/gov/justice/plugin/StopServerMojo.java

@@ -20,6 +20,7 @@ public class StopServerMojo extends AbstractMojo {
     @Parameter(defaultValue = "${project}", readonly = true)
     protected MavenProject project;
 
+
     public MavenProject getProject() {
         return project;
     }
@@ -36,6 +37,7 @@ public void setPort(int port) {
         this.port = port;
     }
 
+
     @Override
     public void execute() throws MojoExecutionException {
         try (final Socket clientSocket = new Socket("localhost", getPort())) {

src/test/java/uk/gov/justice/plugin/StartServerMojoTest.java

@@ -0,0 +1,56 @@
+package uk.gov.justice.plugin;
+
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+import java.io.File;
+import java.util.ArrayList;
+import java.util.HashMap;
+
+import org.apache.maven.artifact.Artifact;
+import org.apache.maven.plugin.descriptor.PluginDescriptor;
+import org.apache.maven.project.MavenProject;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.Mock;
+import org.mockito.runners.MockitoJUnitRunner;
+
+
+@RunWith(MockitoJUnitRunner.class)
+public class StartServerMojoTest {
+
+    @Mock
+    MavenProject project;
+
+    @Mock
+    File targetOutputDirectory;
+
+    @Mock
+    File targetTestOutputDirectory;
+
+    @Mock
+    PluginDescriptor plugin;
+
+    @InjectMocks
+    StartServerMojo startServerMojo;
+
+    @Test
+    public void shouldExecuteMojo() throws Exception {
+
+        when(project.getRuntimeClasspathElements()).thenReturn(new ArrayList<>());
+
+        when(targetOutputDirectory.getAbsolutePath()).thenReturn("");
+
+        when(targetTestOutputDirectory.getAbsolutePath()).thenReturn("");
+
+        startServerMojo.setServerClass(
+                        mock(StartServerMojoTest.class).getClass().getCanonicalName());
+
+        when(plugin.getArtifactMap()).thenReturn(new HashMap<String, Artifact>());
+
+        startServerMojo.execute();
+
+    }
+
+}