add tests for full coveraga + fix synchronization issue with refresh

command
asciidoctor · Aug 1, 2020 · 9a11bc1 · 9a11bc1
1 parent d90e637
commit 9a11bc1
Show file tree

Hide file tree

Showing 5 changed files with 266 additions and 39 deletions.
diff --git a/src/main/java/org/asciidoctor/maven/AsciidoctorHttpMojo.java b/src/main/java/org/asciidoctor/maven/AsciidoctorHttpMojo.java
@@ -46,6 +46,7 @@ protected void doWork() throws MojoFailureException, MojoExecutionException {
         server.start();
 
         super.doWork();
+        super.doWait();
 
         server.stop();
     }

diff --git a/src/main/java/org/asciidoctor/maven/AsciidoctorRefreshMojo.java b/src/main/java/org/asciidoctor/maven/AsciidoctorRefreshMojo.java
@@ -48,6 +48,7 @@ public class AsciidoctorRefreshMojo extends AsciidoctorMojo {
     public void execute() throws MojoExecutionException, MojoFailureException {
         startPolling();
         doWork();
+        doWait();
         stopMonitors();
     }
 
@@ -60,7 +61,6 @@ protected void doWork() throws MojoFailureException, MojoExecutionException {
             }
         });
         getLog().info("Converted document(s) in " + timeInMillis + "ms");
-        doWait();
     }
 
     protected void doWait() throws MojoExecutionException, MojoFailureException {
@@ -71,7 +71,7 @@ protected void doWait() throws MojoExecutionException, MojoFailureException {
         while ((line = scanner.nextLine()) != null) {
             line = line.trim();
             if ("exit".equalsIgnoreCase(line) || "quit".equalsIgnoreCase(line)) {
-                break;
+                return;
             }
 
             if ("refresh".equalsIgnoreCase(line)) {