Merge branch 'master' into dev

Signed-off-by: Emmanuel Bigeon <emmanuel@bigeon.fr>

.gitignore

@@ -0,0 +1,6 @@
+# maven files
+target/
+# Eclipse files
+.settings/
+.classpath
+.project

gclc-process/pom.xml

@@ -3,12 +3,12 @@
 	<parent>
 		<groupId>net.bigeon.config</groupId>
 		<artifactId>ebigeon-config</artifactId>
-		<version>1.8.21</version>
+		<version>1.8.29</version>
 	</parent>
 
 	<groupId>net.bigeon.gclc</groupId>
 	<artifactId>process</artifactId>
-	<version>0.0.6-SNAPSHOT</version>
+	<version>0.0.7-SNAPSHOT</version>
 	<packaging>jar</packaging>
 
 	<name>process</name>
@@ -49,12 +49,12 @@
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>gclc</artifactId>
-			<version>2.0.11</version>
+			<version>2.1.2</version>
 		</dependency>
 		<dependency>
 			<groupId>net.bigeon.test</groupId>
 			<artifactId>junitmt</artifactId>
-			<version>1.0.2</version>
+			<version>1.0.4</version>
 			<scope>test</scope>
 		</dependency>
 	</dependencies>

gclc-process/src/main/java/net/bigeon/gclc/process/ForkTask.java

@@ -100,7 +100,7 @@ public abstract class ForkTask implements Task {
     /** Create the task.
      * 
      * @param lines the number of print to store in the output */
-    public ForkTask(final int lines) {
+    protected ForkTask(final int lines) {
         out = new ConnectingConsoleOutput(ConnectingConsoleOutput.PERSIST, lines);
     }
 
@@ -142,15 +142,18 @@ public abstract class ForkTask implements Task {
      * @param timeout the maximal time to join for (0 for ever) */
     public final void join(final ConsoleOutput out, final ConsoleInput in,
             final long timeout) {
+        final long tic = System.currentTimeMillis();
         synchronized (runLock) {
             this.out.connect(out);
             this.in.connect(in);
-            try {
-                if (running) {
+            long tac = System.currentTimeMillis() - tic;
+            while (running && tac < timeout) {
+                try {
                     runLock.wait(timeout);
+                } catch (final InterruptedException e) {
+                    Thread.currentThread().interrupt();
                 }
-            } catch (final InterruptedException e) {
-                Thread.currentThread().interrupt();
+                tac = System.currentTimeMillis() - tic;
             }
             this.out.disconnect();
             this.in.disconnect();

gclc-process/src/main/java/net/bigeon/gclc/process/TaskSpawner.java

@@ -84,8 +84,8 @@ public abstract class TaskSpawner extends Command {
     private final ExecutorService threadPool;
 
     /** @param name the command name
-     * @param pool the pool */
-    public TaskSpawner(final String name, final TaskPool pool,
+     * @param threadPool the pool */
+    protected TaskSpawner(final String name, final TaskPool pool,
                        final ExecutorService threadPool) {
         super(name);
         this.pool = pool;

gclc-process/src/main/java/net/bigeon/gclc/process/io/ConnectingConsoleInput.java

@@ -68,43 +68,38 @@ package net.bigeon.gclc.process.io;
  * #L%
  */
 import java.io.IOException;
-import java.util.logging.Logger;
+import java.io.InterruptedIOException;
+import java.util.function.Supplier;
 
 import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.tools.ConstantString;
-import net.bigeon.gclc.tools.StringProvider;
 
 /** A console input that can be connected to and diconnected from.
  * 
  * @author Emmanuel Bigeon */
 public final class ConnectingConsoleInput implements ConsoleInput {
 
-    /** The empty string provider. */
-    private static final ConstantString EMPTY_STRING = new ConstantString("");
-    /** The logger. */
-    private static final Logger         LOGGER       = Logger
-            .getLogger(ConnectingConsoleInput.class.getName());
-
     /** If the input is closed. */
-    private boolean        close          = false;
+    private boolean          close          = false;
     /** The prompt string. */
-    private StringProvider prompt         = EMPTY_STRING;
+    private Supplier<String> prompt         = () -> "";
     /** If the input is currently in prompting state.
      * <p>
      * To change it you should be in a promptLock. */
-    private boolean        prompting      = false;
+    private boolean          prompting      = false;
     /** The synchronization lock for the prompting status. */
-    private final Object   promptLock     = new Object();
+    private final Object     promptLock     = new Object();
     /** The synchronization lock for the connection status. */
-    private final Object   connectionLock = new Object();
+    private final Object     connectionLock = new Object();
     /** The connected console input.
      * <p>
      * To use it, you should be in a promptLock and connectionLock. */
-    private ConsoleInput   connected      = null;
+    private ConsoleInput     connected      = null;
     /** The connection state.
      * <p>
      * To read or modify it, you should be in a connectionLock synchronize block. */
-    private boolean        disconnection  = false;
+    private boolean          disconnection  = false;
+
+    private boolean interrupting = false;
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#close() */
@@ -143,7 +138,7 @@ public final class ConnectingConsoleInput implements ConsoleInput {
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#getPrompt() */
     @Override
-    public StringProvider getPrompt() {
+    public Supplier<String> getPrompt() {
         return prompt;
     }
 
@@ -153,6 +148,9 @@ public final class ConnectingConsoleInput implements ConsoleInput {
     public void interruptPrompt() {
         synchronized (connectionLock) {
             synchronized (promptLock) {
+                if (prompting) {
+                    interrupting = true;
+                }
                 connectionLock.notifyAll();
                 if (connected != null) {
                     connected.interruptPrompt();
@@ -174,14 +172,14 @@ public final class ConnectingConsoleInput implements ConsoleInput {
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt() */
     @Override
     public String prompt() throws IOException {
-        return prompt(prompt.apply());
+        return prompt(prompt.get());
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(long) */
     @Override
     public String prompt(final long timeout) throws IOException {
-        return prompt(prompt.apply(), timeout);
+        return prompt(prompt.get(), timeout);
     }
 
     /* (non-Javadoc)
@@ -191,32 +189,7 @@ public final class ConnectingConsoleInput implements ConsoleInput {
         synchronized (promptLock) {
             prompting = true;
         }
-        while (true) {
-            synchronized (promptLock) {
-                if (!prompting) {
-                    return null;
-                }
-            }
-            getConnection(0);
-            boolean connect;
-            ConsoleInput actualConnected;
-            synchronized (connectionLock) {
-                connect = connected != null;
-                actualConnected = connected;
-            }
-            if (connect) {
-                final String res = actualConnected.prompt(message);
-                synchronized (promptLock) {
-                    if (disconnection) {
-                        disconnection = false;
-                    } else if (prompting) {
-                        return res;
-                    } else {
-                        // prompt interrupted, lose the result.
-                    }
-                }
-            }
-        }
+        return doPrompt(message, 0, 0);
     }
 
     /* (non-Javadoc)
@@ -226,52 +199,114 @@ public final class ConnectingConsoleInput implements ConsoleInput {
         if (timeout <= 0) {
             return prompt(message);
         }
-        final long end = System.currentTimeMillis() + timeout;
         synchronized (promptLock) {
             prompting = true;
         }
+        final long tic = System.currentTimeMillis();
+        return doPrompt(message, timeout, tic);
+    }
+
+    /** Actually do the prompting
+     * 
+     * @param message the prompt message
+     * @param timeout the time to wait for an answer
+     * @param tic the moment the wait started
+     * @return the message provided through the prompting
+     * @throws IOException if an IO error occurred while prompting. */
+    private String doPrompt(final String message, final long timeout, final long tic)
+            throws IOException {
         do {
-            synchronized (promptLock) {
-                if (!prompting) {
-                    break;
-                }
+            if (!checkPrompt()) {
+                // We are not prompting... lets stop here.
+                break;
             }
-            getConnection(timeout);
+            getConnection(getTimeoutLeft(tic, timeout));
             boolean connect;
             ConsoleInput actualConnected;
             synchronized (connectionLock) {
                 connect = connected != null;
                 actualConnected = connected;
             }
-            if (connect) {
+            if (!connect) {
+                // There is no provide of input... So lets start again, until we actually
+                // DO get a provider of input.
+                continue;
+            }
+            try {
+                final String res = actualPrompt(message, timeout, tic, actualConnected);
+                // We got something from the prompting
                 synchronized (promptLock) {
-                    final String res = actualConnected.prompt(message,
-                            end - System.currentTimeMillis());
-                    if (disconnection) {
-                        disconnection = false;
-                    } else if (prompting) {
+                    if (prompting) {
+                        prompting = false;
                         return res;
-                    } else {
-                        // prompt interrupted, lose the result.
                     }
                 }
+            } catch (final InterruptedIOException e) {
+                // The inner console was interrupted. This can mean we are
+                // disconnecting or actually interrupted.
+                if (disconnection) {
+                    disconnection = false;
+                } else {
+                    interrupting = false;
+                    throw e;
+                }
             }
-        } while (System.currentTimeMillis() < end);
+        } while (checkTimeout(tic, timeout));
         return null;
-
     }
 
-    private void getConnection(final long timeout) {
-        boolean connect;
+    private static String actualPrompt(final String message, final long timeout,
+            final long tic, final ConsoleInput actualConnected) throws IOException {
+        final long timeoutLeft = getTimeoutLeft(tic, timeout);
+        if (timeoutLeft == 0) {
+            return actualConnected.prompt(message);
+        }
+        return actualConnected.prompt(message, timeoutLeft);
+    }
+
+    private static boolean checkTimeout(final long tic, final long timeout) {
+        return timeout <= 0 || tic + timeout > System.currentTimeMillis();
+    }
+
+    private static long getTimeoutLeft(final long tic, final long timeout) {
+        if (timeout > 0) {
+            return Math.max(timeout + tic - System.currentTimeMillis(), 1);
+        }
+        if (timeout < 0) {
+            return 1;
+        }
+        return 0;
+    }
+
+    /** Test if we are in prompting state.
+     * 
+     * @return if the process is currently in prompting state.
+     * @throws InterruptedIOException if the prompting state has been interrupted */
+    private boolean checkPrompt() throws InterruptedIOException {
+        synchronized (promptLock) {
+            if (!prompting && interrupting) {
+                interrupting = false;
+                throw new InterruptedIOException("Prompt was interrupted");
+            }
+            return prompting;
+        }
+    }
+
+    private void getConnection(final long timeout) throws InterruptedIOException {
+        final long tic = System.currentTimeMillis();
         synchronized (connectionLock) {
-            connect = connected != null;
-            if (!connect) {
+            while ((connected == null || !interrupting)
+                    && (tic + timeout) > System.currentTimeMillis()) {
                 try {
                     connectionLock.wait(timeout);
                 } catch (final InterruptedException e) {
                     Thread.currentThread().interrupt();
                 }
             }
+            if (interrupting) {
+                interrupting = false;
+                throw new InterruptedIOException("Prompt ws interrupted");
+            }
         }
     }
 
@@ -279,14 +314,14 @@ public final class ConnectingConsoleInput implements ConsoleInput {
      * @see fr.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
     @Override
     public void setPrompt(final String prompt) {
-        this.prompt = new ConstantString(prompt);
+        this.prompt = () -> prompt;
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#setPrompt(fr.bigeon.gclc.tools.
      * StringProvider) */
     @Override
-    public void setPrompt(final StringProvider string) {
+    public void setPrompt(final Supplier<String> string) {
         prompt = string;
     }
 }

gclc-process/src/test/java/net/bigeon/gclc/process/ForkTaskTest.java

@@ -5,15 +5,18 @@ package net.bigeon.gclc.process;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
 
 import java.io.IOException;
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import org.junit.Test;
 
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.CommandRunExceptionType;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /** @author Emmanuel Bigeon */
 public class ForkTaskTest {
@@ -36,27 +39,118 @@ public class ForkTaskTest {
                     throw new CommandRunException(CommandRunExceptionType.INTERACTION,
                             "Unable to prompt user");
                 }
-                if ("ok".equals(msg)) {
-                    out.println("Message");
-                } else {
-                    out.println("fail");
-                }
+                out.println(msg);
             }
         };
 
-        final Thread execThread = new Thread(task);
+        final Thread execThread = new Thread(task, "Task");
         execThread.start();
+        try {
+            execThread.join(100);
+        } catch (final InterruptedException e) {
+            // TODO Auto-generated catch block
+            e.printStackTrace();
+        }
 
         try (PipedConsoleOutput pco = new PipedConsoleOutput();
                 PipedConsoleInput pci = new PipedConsoleInput(null)) {
-            pci.type("ok");
             while (!pco.available()) {
+                pci.type("ok");
                 task.join(pco, pci, 1000);
             }
-            assertEquals("Execution should work", "Message", pco.readNextLine());
+            assertEquals("Positive execution", "ok", pco.readNextLine());
         }
         assertFalse("Running state should be updated by task on its completion",
                 task.isRunning());
     }
 
+    @Test
+    public void testTaskInterruption() throws IOException, InterruptedException {
+        final ForkTask task = new ForkTask(5) {
+
+            @Override
+            public String getName() {
+                return "name";
+            }
+
+            @Override
+            protected void doRun() throws CommandRunException {
+                while (isRunning()) {
+                    String msg;
+                    try {
+                        msg = in.prompt(1000);
+                    } catch (final IOException e) {
+                        throw new CommandRunException(CommandRunExceptionType.INTERACTION,
+                                "Unable to prompt user");
+                    }
+                    if (msg != null) {
+                        out.println(msg);
+                    }
+                }
+            }
+        };
+
+        final Thread execThread = new Thread(task, "Task");
+        execThread.start();
+        execThread.join(100);
+
+        final AtomicBoolean interrupted = new AtomicBoolean(false);
+        final AtomicBoolean interrupted2 = new AtomicBoolean(false);
+        final InterruptionListener listener = new InterruptionListener() {
+
+            @Override
+            public void interrupted() {
+                interrupted.set(true);
+            }
+        };
+        final InterruptionListener listener2 = new InterruptionListener() {
+
+            @Override
+            public void interrupted() {
+                interrupted2.set(true);
+            }
+        };
+        task.addInterruptionListener(listener);
+        task.addInterruptionListener(listener2);
+        task.rmInterruptionListener(listener2);
+
+        assertFalse("Interruption should not be notified before actual interruption",
+                interrupted.get());
+        assertTrue("Task should be started", task.isStarted());
+        task.setRunning(false);
+        try (PipedConsoleOutput pco = new PipedConsoleOutput();
+                PipedConsoleInput pci = new PipedConsoleInput(null)) {
+            task.join(pco, pci, 2000);
+        }
+        execThread.join();
+        assertTrue("Interruption should be notified to listeners", interrupted.get());
+
+        assertFalse("Running state should be updated by task on its completion",
+                task.isRunning());
+        assertFalse("Interruption should not be notified to removed listeners",
+                interrupted2.get());
+
+        task.rmInterruptionListener(listener);
+    }
+
+    @Test
+    public void testFailingTask() throws InterruptedException {
+        final ForkTask task = new ForkTask(5) {
+
+            @Override
+            public String getName() {
+                return "name";
+            }
+
+            @Override
+            protected void doRun() throws CommandRunException {
+                throw new CommandRunException("Error");
+            }
+        };
+
+        final Thread execThread = new Thread(task, "Task");
+        execThread.start();
+        execThread.join();
+        assertNotNull("Exception should be forwarded", task.getException());
+    }
 }

gclc-process/src/test/java/net/bigeon/gclc/process/ProcessListTest.java

@@ -14,8 +14,8 @@ import org.junit.Test;
 
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 import net.bigeon.gclc.process.mocks.TaskMock;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
 
 /** @author Emmanuel Bigeon */
 public class ProcessListTest {

gclc-process/src/test/java/net/bigeon/gclc/process/io/ConnectingConsoleInputTest.java

@@ -1,25 +1,23 @@
 package net.bigeon.gclc.process.io;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.*;
 
 import java.io.IOException;
+import java.io.InterruptedIOException;
 import java.io.PipedInputStream;
 import java.io.PipedOutputStream;
 import java.nio.charset.StandardCharsets;
 import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.function.Supplier;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.tools.ConstantString;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.StreamConsoleInput;
-import net.bigeon.test.junitmt.ATestRunnable;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.StreamConsoleInput;
 import net.bigeon.test.junitmt.FunctionalTestRunnable;
-import net.bigeon.test.junitmt.TestFunction;
 import net.bigeon.test.junitmt.ThreadTest;
+import net.bigeon.test.junitmt.api.ATestRunnable;
+import net.bigeon.test.junitmt.api.TestFunction;
 
 public class ConnectingConsoleInputTest {
 
@@ -65,9 +63,8 @@ public class ConnectingConsoleInputTest {
     public void testCoveragePrompt() {
         final ConnectingConsoleInput in = new ConnectingConsoleInput();
         in.setPrompt("test");
-        assertEquals("Prompt should be set correctly", "test",
-                ((ConstantString) in.getPrompt()).apply());
-        final ConstantString prompt = new ConstantString("other");
+        assertEquals("Prompt should be set correctly", "test", in.getPrompt().get());
+        final Supplier<String> prompt = () -> "other";
         in.setPrompt(prompt);
         assertEquals("Prompt should be set correctly", prompt, in.getPrompt());
     }
@@ -77,30 +74,44 @@ public class ConnectingConsoleInputTest {
         final ConnectingConsoleInput in = new ConnectingConsoleInput();
         // Unconnected
         final AtomicBoolean ended = new AtomicBoolean(false);
-        final TestFunction one = new TestFunction() {
-
-            @Override
-            public void apply() throws Exception {
-                assertNull("Interrupted should return null", in.prompt("m1", -1));
-                assertNull("Interrupted should return null", in.prompt("m2", 5000));
+        final TestFunction one = () -> {
+            try {
+                final String res1 = in.prompt("m1", -1);
+                fail("interruption of infinite waiting prompt should cause error, but was "
+                        + res1);
+            } catch (final InterruptedIOException e1) {
+                // ok
+            }
+            try {
+                final String res2 = in.prompt("m2", 25000);
+                fail("interruption of finite waiting prompt should cause error, but was "
+                        + res2);
+            } catch (final InterruptedIOException e2) {
+                // ok
+            }
+            synchronized (ended) {
                 ended.set(true);
-                assertNull("Overtime should return null", in.prompt("m3", 200));
+                try {
+                    assertNull("Overtime should return null", in.prompt("m3", 200));
+                } catch (final InterruptedIOException e3) {
+                    fail("Unexpected interruption error in overtime");
+                }
             }
         };
         final ATestRunnable runnable = new FunctionalTestRunnable(one);
-        final Thread th = new Thread(runnable);
-        final Thread inter = new Thread(new Runnable() {
-
-            @Override
-            public void run() {
-                while (!ended.get()) {
-                    try {
-                        th.join(500);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
+        final Thread th = new Thread(runnable, "TestPromptSequence");
+        final Thread inter = new Thread(() -> {
+            while (!ended.get()) {
+                try {
+                    th.join(100);
+                } catch (final InterruptedException e) {
+                    // TODO Auto-generated catch block
+                    e.printStackTrace();
+                }
+                synchronized (ended) {
+                    if (!ended.get()) {
+                        in.interruptPrompt();
                     }
-                    in.interruptPrompt();
                 }
             }
         });
@@ -112,21 +123,21 @@ public class ConnectingConsoleInputTest {
 
         final PipedOutputStream os = new PipedOutputStream();
         final PipedInputStream pis = new PipedInputStream(os);
-        in.connect(new StreamConsoleInput(System.out, pis, StandardCharsets.UTF_8));
+        in.connect(new StreamConsoleInput(null, pis, StandardCharsets.UTF_8));
         final ATestRunnable runnable2 = new FunctionalTestRunnable(one);
         final Thread th2 = new Thread(runnable2);
-        final Thread inter2 = new Thread(new Runnable() {
-
-            @Override
-            public void run() {
-                while (!ended.get()) {
-                    try {
-                        th2.join(500);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
+        final Thread inter2 = new Thread(() -> {
+            while (!ended.get()) {
+                try {
+                    th2.join(100);
+                } catch (final InterruptedException e) {
+                    // TODO Auto-generated catch block
+                    e.printStackTrace();
+                }
+                synchronized (ended) {
+                    if (!ended.get()) {
+                        in.interruptPrompt();
                     }
-                    in.interruptPrompt();
                 }
             }
         });

gclc-process/src/test/java/net/bigeon/gclc/process/io/ConnectingConsoleOutputTest.java

@@ -8,7 +8,7 @@ import java.io.IOException;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 public class ConnectingConsoleOutputTest {
 

gclc-socket/pom.xml

@@ -2,12 +2,12 @@
 	<modelVersion>4.0.0</modelVersion>
 	<parent>
 		<groupId>net.bigeon.config</groupId>
-		<artifactId>ebigeon-config</artifactId>
-		<version>1.8.21</version>
+		<artifactId>ebigeon-public-conf</artifactId>
+		<version>1.0.12</version>
 	</parent>
 	<groupId>net.bigeon.gclc</groupId>
 	<artifactId>socket</artifactId>
-	<version>1.1.14-SNAPSHOT</version>
+	<version>1.1.19-SNAPSHOT</version>
 	<packaging>jar</packaging>
 	<name>GCLC Socket</name>
 	<description>Socket implementation of GCLC</description>
@@ -40,24 +40,23 @@
 	</scm>
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-		<project.scm.id>git.code.bigeon.net</project.scm.id>
 		<license.licenseName>cecill_2.1</license.licenseName>
 	</properties>
 	<dependencies>
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>gclc</artifactId>
-			<version>2.0.11</version>
+			<version>2.1.2</version>
 		</dependency>
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>smu</artifactId>
-			<version>1.0.6</version>
+			<version>1.1.2</version>
 		</dependency>
 		<dependency>
 			<groupId>net.bigeon.test</groupId>
 			<artifactId>junitmt</artifactId>
-			<version>1.0.0</version>
+			<version>1.0.4</version>
 			<scope>test</scope>
 		</dependency>
 	</dependencies>

gclc-socket/src/main/java/net/bigeon/gclc/socket/DisconnexionListener.java

@@ -0,0 +1,14 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.socket;
+
+/** Interface for listener of brutal disconnection from a pluggable
+ * input/output.
+ * 
+ * @author Emmanuel Bigeon */
+@FunctionalInterface
+public interface DisconnexionListener {
+    /** Indicate a brutal disconnection */
+    void disconnected();
+}

gclc-socket/src/main/java/net/bigeon/gclc/socket/PluggableConsoleInput.java

@@ -41,14 +41,16 @@ import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
+import java.io.InterruptedIOException;
 import java.io.PrintStream;
 import java.nio.charset.StandardCharsets;
+import java.util.HashSet;
+import java.util.Set;
+import java.util.function.Supplier;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.tools.ConstantString;
-import net.bigeon.gclc.tools.StringProvider;
 import net.bigeon.gclc.utils.ReadingRunnable;
 
 /** A console input where the stream can be plugged.
@@ -75,7 +77,7 @@ public final class PluggableConsoleInput implements ConsoleInput {
     /** If the element is closed. */
     private boolean             closed      = false;
     /** The default prompt. */
-    private StringProvider      prompt      = new ConstantString("> "); //$NON-NLS-1$
+    private Supplier<String>    prompt      = () -> "> ";     //$NON-NLS-1$
     /** If the input is plugged or buffering. */
     private boolean             connected   = false;
     /** The current connexion (if any). */
@@ -87,6 +89,8 @@ public final class PluggableConsoleInput implements ConsoleInput {
     /** The output for hints. */
     private PrintStream         output;
 
+    private final Set<DisconnexionListener> listeners = new HashSet<>();
+
     // Locks
     /** The lock for connexion and disconnexion of actual streams. */
     private final Object connexionLock = new Object();
@@ -119,6 +123,7 @@ public final class PluggableConsoleInput implements ConsoleInput {
             if (prompting) {
                 // print the hint, to indicate we are waiting for a user input.
                 out.print(hint);
+                out.println();
                 out.flush();
             }
             final InputStreamReader streamReader = new InputStreamReader(stream,
@@ -146,7 +151,7 @@ public final class PluggableConsoleInput implements ConsoleInput {
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#getPrompt() */
     @Override
-    public StringProvider getPrompt() {
+    public Supplier<String> getPrompt() {
         return prompt;
     }
 
@@ -175,14 +180,14 @@ public final class PluggableConsoleInput implements ConsoleInput {
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt() */
     @Override
     public String prompt() throws IOException {
-        return prompt(prompt.apply());
+        return prompt(prompt.get());
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(long) */
     @Override
     public String prompt(final long timeout) throws IOException {
-        return prompt(prompt.apply(), timeout);
+        return prompt(prompt.get(), timeout);
     }
 
     /* (non-Javadoc)
@@ -257,14 +262,14 @@ public final class PluggableConsoleInput implements ConsoleInput {
      * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
     @Override
     public void setPrompt(final String prompt) {
-        setPrompt(new ConstantString(prompt));
+        setPrompt(() -> prompt);
     }
 
     /* (non-Javadoc)
      * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(net.bigeon.gclc.tools.
      * StringProvider) */
     @Override
-    public void setPrompt(final StringProvider prompt) {
+    public void setPrompt(final Supplier<String> prompt) {
         this.prompt = prompt;
     }
 
@@ -279,10 +284,50 @@ public final class PluggableConsoleInput implements ConsoleInput {
             final long connexionTimeout) throws IOException, InterruptedException {
         synchronized (connexionLock) {
             if (connected) {
-                return connexion.getNextMessage(messageTimeout);
+                try {
+                    return connexion.getNextMessage(messageTimeout);
+                } catch (final InterruptedIOException e) {
+                    throw e;
+                } catch (final IOException e) {
+                    LOGGER.log(Level.INFO, "Communication was abrubptly interrupted", e);
+                    brutalDisconnection();
+                }
+            }
+            while (!connected) {
+                connexionLock.wait(connexionTimeout);
             }
-            connexionLock.wait(connexionTimeout);
         }
         return null;
     }
+
+    /**
+     * 
+     */
+    private void brutalDisconnection() {
+        // clean up the disconnection
+        disconnect();
+        // notify listeners
+        for (final DisconnexionListener listener : listeners) {
+            listener.disconnected();
+        }
+    }
+
+    /** Add a listener to the list of listeners.
+     * 
+     * @param e the listener
+     * @return if the listener was added
+     * @see java.util.Set#add(java.lang.Object) */
+    public boolean add(final DisconnexionListener e) {
+        return listeners.add(e);
+    }
+
+    /** Remove a listener from the list of listeners.
+     * 
+     * @param o the listener
+     * @return if the listener was removed
+     * @see java.util.Set#remove(java.lang.Object) */
+    public boolean remove(final DisconnexionListener o) {
+        return listeners.remove(o);
+    }
+
 }

gclc-socket/src/main/java/net/bigeon/gclc/socket/RemoteDisconnectCommand.java

@@ -4,60 +4,21 @@
  */
 package net.bigeon.gclc.socket;
 
-/*-
- * #%L
- * GCLC Socket
- * %%
- * Copyright (C) 2016 - 2018 Bigeon
- * %%
- * This software is governed by the CeCILL  license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/ or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- * #L%
- */
-import java.io.IOException;
-import java.text.MessageFormat;
-import java.util.Collection;
-
 import net.bigeon.gclc.command.Command;
 import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.exception.CommandRunExceptionType;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
 
 /** A {@link Command} to disconnect elements from a {@link ConnexionManager}.
  *
  * @author Emmanuel Bigeon
- * @param <T> the type of connected object */
+ * @param <T> the type of connected object
+ * @deprecated since 1.1.17, this has been moved to
+ *             {@link RemoteDisconnectCommand}. */
+@Deprecated
 public final class RemoteDisconnectCommand<T> extends Command {
 
-    /** The connexion manager. */
-    private final ConnexionManager<T> manager;
-    /** If all connexion should be disconnected when no argument have been
-     * specified. */
-    private final boolean             all;
+    private final net.bigeon.gclc.socket.cmd.RemoteDisconnectCommand<T> real;
 
     /** Create the disconnection command.
      *
@@ -68,8 +29,8 @@ public final class RemoteDisconnectCommand<T> extends Command {
     public RemoteDisconnectCommand(final String name, final ConnexionManager<T> manager,
                                    final boolean all) {
         super(name);
-        this.manager = manager;
-        this.all = all;
+        real = new net.bigeon.gclc.socket.cmd.RemoteDisconnectCommand<>(name, manager,
+                all);
     }
 
     /* (non-Javadoc)
@@ -78,52 +39,20 @@ public final class RemoteDisconnectCommand<T> extends Command {
     @Override
     public void execute(final ConsoleOutput out, final ConsoleInput in,
             final String... args) throws CommandRunException {
-        if (args.length == 0 && all) {
-            final Collection<String> coll = manager.getConnected();
-            for (final String string : coll) {
-                manager.disconnect(string);
-            }
-        }
-        for (final String string : args) {
-            if (manager.isConnected(string)) {
-                manager.disconnect(string);
-            } else {
-                print(out,
-                        MessageFormat.format("[WARNING] {0} is not connected", string)); //$NON-NLS-1$
-            }
-        }
-    }
-
-    /** Print a message if the output is defined.
-     *
-     * @param out the output
-     * @param string the message
-     * @throws CommandRunException if the output exists but cannot be printed to */
-    private static void print(final ConsoleOutput out, final String string)
-            throws CommandRunException {
-        if (out != null) {
-            try {
-                out.println(string);
-            } catch (final IOException e) {
-                throw new CommandRunException(CommandRunExceptionType.INTERACTION,
-                        "Unable to print to existing output", e); //$NON-NLS-1$
-            }
-        }
+        real.execute(out, in, args);
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.ICommand#tip() */
     @Override
     public String tip() {
-        return "Close a connexion."; //$NON-NLS-1$
+        return real.tip();
     }
 
     /* (non-Javadoc)
      * @see fr.bigeon.gclc.command.Command#usageDetail() */
     @Override
     protected String usageDetail() {
-        return MessageFormat.format(
-                "  If arguments are provided the corresponding connexions are closed, otherwise\n{0} are.",
-                all ? "all connexions" : "none");
+        return "";
     }
 }

gclc-socket/src/main/java/net/bigeon/gclc/socket/SocketConsoleApplicationShell.java

@@ -77,6 +77,8 @@ public final class SocketConsoleApplicationShell implements Runnable {
     /** The application. */
     private ConsoleApplication       app;
 
+    private final Object initLock = new Object();
+
     /** The server socket. */
     private ServerSocket      serverSocket;
     /** THe server address. */
@@ -123,7 +125,10 @@ public final class SocketConsoleApplicationShell implements Runnable {
         // Create the server
         try (ServerSocket actualServerSocket = new ServerSocket(port, 1, addr)) {
             serverSocket = actualServerSocket;
-            running = true;
+            synchronized (initLock) {
+                running = true;
+                initLock.notifyAll();
+            }
             // Create the streams
             runSokectServer();
         } catch (final IOException e) {
@@ -194,4 +199,23 @@ public final class SocketConsoleApplicationShell implements Runnable {
             LOGGER.log(Level.SEVERE, "Exception in closing socket server", e); //$NON-NLS-1$
         }
     }
+
+    /** A method to wait for the socket server initialization.
+     * 
+     * @param timeout the timeout */
+    public void waitStartUp(final long timeout) {
+        synchronized (initLock) {
+            final long tic = System.currentTimeMillis() + timeout;
+            long tac;
+            while (!running && (tac = System.currentTimeMillis()) < tic) {
+                final long idle = tic - tac;
+                try {
+                    initLock.wait(idle);
+                } catch (final InterruptedException e) {
+                    LOGGER.log(Level.INFO, "Thread interruption", e);
+                    Thread.currentThread().interrupt();
+                }
+            }
+        }
+    }
 }

gclc-socket/src/main/java/net/bigeon/gclc/socket/cmd/ConnexionListCommand.java

@@ -0,0 +1,98 @@
+/**
+ * gclc-socket:net.bigeon.gclc.socket.RemoteDisconnectCommand.java
+ * Created on: Nov 18, 2017
+ */
+package net.bigeon.gclc.socket.cmd;
+
+/*-
+ * #%L
+ * GCLC Socket
+ * %%
+ * Copyright (C) 2016 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ *
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ *
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ *
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+import java.io.IOException;
+import java.util.Collection;
+
+import net.bigeon.gclc.command.Command;
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.manager.ConsoleOutput;
+import net.bigeon.gclc.socket.ConnexionManager;
+
+/** A {@link Command} to disconnect elements from a {@link ConnexionManager}.
+ *
+ * @author Emmanuel Bigeon
+ * @param <T> the type of connected object */
+public final class ConnexionListCommand<T> extends Command {
+
+    /** The connexion manager. */
+    private final ConnexionManager<T> manager;
+
+    /** Create the connexion listing command.
+     *
+     * @param name the command name
+     * @param manager the manager */
+    public ConnexionListCommand(final String name, final ConnexionManager<T> manager) {
+        super(name);
+        this.manager = manager;
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
+    @Override
+    public void execute(final ConsoleOutput out, final ConsoleInput in,
+            final String... args) throws CommandRunException {
+        final Collection<String> coll = manager.getConnected();
+        try {
+            for (final String string : coll) {
+                out.println(string);
+            }
+        } catch (final IOException e) {
+            throw new CommandRunException(CommandRunExceptionType.INTERACTION,
+                    "User cannot be notified", e);
+        }
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.ICommand#tip() */
+    @Override
+    public String tip() {
+        return "List current connexions."; //$NON-NLS-1$
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.Command#usageDetail() */
+    @Override
+    protected String usageDetail() {
+        return "";
+    }
+}

gclc-socket/src/main/java/net/bigeon/gclc/socket/cmd/RemoteDisconnectCommand.java

@@ -0,0 +1,136 @@
+/**
+ * gclc-socket:net.bigeon.gclc.socket.RemoteDisconnectCommand.java
+ * Created on: Nov 18, 2017
+ */
+package net.bigeon.gclc.socket.cmd;
+
+/*-
+ * #%L
+ * GCLC Socket
+ * %%
+ * Copyright (C) 2016 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ *
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ *
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ *
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+import java.io.IOException;
+import java.text.MessageFormat;
+import java.util.Collection;
+
+import net.bigeon.gclc.command.Command;
+import net.bigeon.gclc.exception.CommandRunException;
+import net.bigeon.gclc.exception.CommandRunExceptionType;
+import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.manager.ConsoleOutput;
+import net.bigeon.gclc.socket.ConnexionManager;
+
+/** A {@link Command} to disconnect elements from a {@link ConnexionManager}.
+ *
+ * @author Emmanuel Bigeon
+ * @param <T> the type of connected object */
+public final class RemoteDisconnectCommand<T> extends Command {
+
+    /** The connexion manager. */
+    private final ConnexionManager<T> manager;
+    /** If all connexion should be disconnected when no argument have been
+     * specified. */
+    private final boolean             all;
+
+    /** Create the disconnection command.
+     *
+     * @param name the command name
+     * @param manager the manager
+     * @param all if all elements should be disconnected when no argument is
+     *        provided */
+    public RemoteDisconnectCommand(final String name, final ConnexionManager<T> manager,
+                                   final boolean all) {
+        super(name);
+        this.manager = manager;
+        this.all = all;
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
+     * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput, java.lang.String[]) */
+    @Override
+    public void execute(final ConsoleOutput out, final ConsoleInput in,
+            final String... args) throws CommandRunException {
+        if (args.length == 0 && all) {
+            final Collection<String> coll = manager.getConnected();
+            for (final String string : coll) {
+                manager.disconnect(string);
+            }
+        }
+        for (final String string : args) {
+            if (manager.isConnected(string)) {
+                manager.disconnect(string);
+            } else {
+                print(out,
+                        MessageFormat.format("[WARNING] {0} is not connected", string)); //$NON-NLS-1$
+            }
+        }
+    }
+
+    /** Print a message if the output is defined.
+     *
+     * @param out the output
+     * @param string the message
+     * @throws CommandRunException if the output exists but cannot be printed to */
+    private static void print(final ConsoleOutput out, final String string)
+            throws CommandRunException {
+        if (out != null) {
+            try {
+                out.println(string);
+            } catch (final IOException e) {
+                throw new CommandRunException(CommandRunExceptionType.INTERACTION,
+                        "Unable to print to existing output", e); //$NON-NLS-1$
+            }
+        }
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.ICommand#tip() */
+    @Override
+    public String tip() {
+        return "Close a connexion."; //$NON-NLS-1$
+    }
+
+    /* (non-Javadoc)
+     * @see fr.bigeon.gclc.command.Command#usageDetail() */
+    @Override
+    protected String usageDetail() {
+        final StringBuilder builder = new StringBuilder(
+                "  If arguments are provided the corresponding connexions are closed, otherwise\n");
+        if (all) {
+            builder.append("all connections");
+        } else {
+            builder.append("none");
+        }
+        builder.append(" are.");
+        return builder.toString();
+    }
+}

gclc-socket/src/main/java/net/bigeon/gclc/socket/cmd/package-info.java

@@ -0,0 +1,8 @@
+/**
+ * 
+ */
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+package net.bigeon.gclc.socket.cmd;

gclc-socket/src/test/java/net/bigeon/gclc/socket/PluggableConsoleInputTest.java

@@ -90,7 +90,7 @@ public class PluggableConsoleInputTest {
 
                 }
                 try {
-                    Object obj = new Object();
+                    final Object obj = new Object();
                     synchronized (obj) {
                             obj.wait(100);
                     }
@@ -115,7 +115,7 @@ public class PluggableConsoleInputTest {
             @Override
             public void run() {
                 try {
-                    Object obj = new Object();
+                    final Object obj = new Object();
                     synchronized (obj) {
                             obj.wait(200);
                     }
@@ -168,7 +168,7 @@ public class PluggableConsoleInputTest {
             });
             th.start();
 
-            Object obj = new Object();
+            final Object obj = new Object();
             synchronized (obj) {
                 while (!input.isPrompting()) {
                     obj.wait(10);
@@ -211,8 +211,8 @@ public class PluggableConsoleInputTest {
     @Test
     public final void testGetPrompt() {
         final PluggableConsoleInput input = new PluggableConsoleInput();
-        assertEquals("Default prompt invalid", "> ", input.getPrompt().apply());
+        assertEquals("Default prompt invalid", "> ", input.getPrompt().get());
         input.setPrompt("test");
-        assertEquals("Prompt setting failed", "test", input.getPrompt().apply());
+        assertEquals("Prompt setting failed", "test", input.getPrompt().get());
     }
 }

gclc-socket/src/test/java/net/bigeon/gclc/socket/SocketConsoleApplicationTest.java

@@ -43,6 +43,7 @@ import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStreamReader;
 import java.io.PrintWriter;
+import java.net.InetAddress;
 import java.net.Socket;
 import java.util.logging.Logger;
 
@@ -76,8 +77,8 @@ public class SocketConsoleApplicationTest {
     public void testIntegration() throws IOException, InterruptedException {
         Thread server;
         server = TestServer.getServer();
-        final String hostName = "127.0.0.1";
-        final int portNumber = 12345;
+        final InetAddress hostName = InetAddress.getLocalHost();
+        final int portNumber = TestServer.PORT;
 
         try (Socket kkSocket = new Socket(hostName, portNumber);
                 PrintWriter out = new PrintWriter(kkSocket.getOutputStream(), true);

gclc-socket/src/test/java/net/bigeon/gclc/socket/TestServer.java

@@ -46,7 +46,7 @@ import net.bigeon.gclc.ConsoleApplication;
 @SuppressWarnings({ "javadoc", "nls" })
 public class TestServer {
 
-    public static final int                      PORT = 3300;
+    public static final int                      PORT = 12343;
     private static SocketConsoleApplicationShell SHELL;
     private static ConnexionManager<Socket>      manager;
 
@@ -66,6 +66,7 @@ public class TestServer {
         if (server == null) {
             server = new Thread(getShell(), "gclcServer");
             server.start();
+            getShell().waitStartUp(500);
         }
         return server;
     }
@@ -76,20 +77,13 @@ public class TestServer {
             input.setPrompt("> \n");
             output = new PluggableConsoleOutput();
             manager = new DConnexionManager<>();
-            SHELL = new SocketConsoleApplicationShell(PORT,
-                    InetAddress.getByName("127.0.0.1"));
+            SHELL = new SocketConsoleApplicationShell(PORT, InetAddress.getLocalHost());
             final ConsoleApplication app = ConsoleTestApplication.create(output, input,
                     manager);
             SHELL.setInterface(new SocketConsoleInterface(input, output));
             SHELL.setConnexionManager(manager);
             SHELL.setApplication(app);
-            final Thread th = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    app.start();
-                }
-            });
+            final Thread th = new Thread(() -> app.start());
             th.start();
             try {
                 final Object waiting = new Object();

gclc-socket/src/test/java/net/bigeon/gclc/socket/cmd/RemoteDisconnectCommandTest.java

@@ -2,7 +2,7 @@
  * gclc-socket:net.bigeon.gclc.socket.RemoteDisconnectCommandTest.java
  * Created on: Nov 18, 2017
  */
-package net.bigeon.gclc.socket;
+package net.bigeon.gclc.socket.cmd;
 
 /*-
  * #%L
@@ -46,7 +46,8 @@ import java.io.IOException;
 import org.junit.Test;
 
 import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.socket.DConnexionManager;
 
 /**
  * <p>
@@ -56,7 +57,7 @@ import net.bigeon.gclc.utils.PipedConsoleOutput;
 public class RemoteDisconnectCommandTest {
 
     /** Test method for
-     * {@link net.bigeon.gclc.socket.RemoteDisconnectCommand#execute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[])}.
+     * {@link net.bigeon.gclc.socket.cmd.RemoteDisconnectCommand#execute(net.bigeon.gclc.manager.ConsoleOutput, net.bigeon.gclc.manager.ConsoleInput, java.lang.String[])}.
      *
      * @throws CommandRunException if the command unexpectedly failed.
      * @throws IOException if the output could not be written to */

gclc-swt/.gitignore

@@ -1,4 +0,0 @@
-/target/
-/.settings/
-/.classpath
-/.project

gclc-swt/pom.xml

@@ -2,12 +2,12 @@
 	<modelVersion>4.0.0</modelVersion>
 	<parent>
 		<groupId>net.bigeon.config</groupId>
-		<artifactId>swt-config</artifactId>
-		<version>1.8.11</version>
+		<artifactId>swt-public-conf</artifactId>
+		<version>1.0.2</version>
 	</parent>
 	<groupId>net.bigeon.gclc</groupId>
 	<artifactId>swt</artifactId>
-	<version>1.1.7-SNAPSHOT</version>
+	<version>1.2.1-SNAPSHOT</version>
 	<packaging>jar</packaging>
 	<name>GCLC swt</name>
 	<description>A swt window for console applications</description>
@@ -48,21 +48,26 @@
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>gclc</artifactId>
-			<version>2.0.11</version>
+			<version>2.1.2</version>
 		</dependency>
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>collections</artifactId>
-			<version>1.1.6</version>
+			<version>1.3.5</version>
+		</dependency>
+		<dependency>
+			<groupId>org.mockito</groupId>
+			<artifactId>mockito-core</artifactId>
+			<version>4.2.0</version>
+			<scope>test</scope>
 		</dependency>
 	</dependencies>
-
 	<reporting>
 		<plugins>
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-javadoc-plugin</artifactId>
-				<version>2.10.3</version>
+				<version>3.0.1</version>
 			</plugin>
 			<plugin>
 				<groupId>com.github.sevntu-checkstyle</groupId>

gclc-swt/src/main/java/net/bigeon/gclc/swt/SWTConsole.java

@@ -30,11 +30,13 @@
  * knowledge of the CeCILL license and that you accept its terms.
  */
 /**
- * gclc-swt:net.bigeon.gclc.swt.SWTConsole.java
+ * gclc-swt:net.bigeon.gclc.swt.SWTConsoleShell.java
  * Created on: Apr 18, 2015
  */
 package net.bigeon.gclc.swt;
 
+import java.io.BufferedReader;
+
 /*-
  * #%L
  * GCLC swt
@@ -68,411 +70,80 @@ package net.bigeon.gclc.swt;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import java.io.IOException;
-import java.util.logging.Level;
-import java.util.logging.Logger;
-
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.widgets.Composite;
-import org.eclipse.swt.widgets.Display;
 import org.eclipse.swt.widgets.Label;
 import org.eclipse.swt.widgets.Text;
 
-import net.bigeon.gclc.ConsoleApplication;
-import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.tools.ConstantString;
-import net.bigeon.gclc.tools.StringProvider;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.swt.api.ConsoleDelayIO;
+import net.bigeon.gclc.swt.api.ConsoleOutputDisplay;
+import net.bigeon.gclc.swt.io.ConsoleInputManager;
+import net.bigeon.gclc.swt.io.ConsoleOutputManager;
+import net.bigeon.gclc.swt.io.ConsolePromptManager;
 
-/** A SWT component to connect to gclc {@link ConsoleApplication}.
+/** A shell containing a {@link SWTConsoleView}
  * <p>
  *
  * @author Emmanuel Bigeon */
-public final class SWTConsole extends Composite
-        implements ConsoleDelayIO, ConsoleInput, ConsoleOutput {
-    private static final int    TWO                  = 2;
-    private static final String ERROR_SYNCHRO_PROMPT = "Error in synchronization of prompting";
-    /** The number of columns of the layout. */
-    private static final int    LAYOUT_NB_COLUMNS    = 2;
-    /** The cmd prefix in the output console. */
-    private static final String CMD_PREFIX           = "[CMD] ";                               //$NON-NLS-1$
-    /** The class logger. */
-    private static final Logger LOGGER               = Logger
-            .getLogger(SWTConsole.class.getName());
-    /** The empty string constant. */
-    private static final String EMPTY                = "";                                     //$NON-NLS-1$
+public final class SWTConsole extends Composite {
 
-    /** The console output text field. */
-    private final Text     consoleOutput;
-    /** The console input text field. */
-    private final Text     consoleInput;
-    /** The prompt label. */
-    private final Label    lblPromptlabel;
-    /** The prompt text. */
-    private StringProvider prompt     = new ConstantString("> "); //$NON-NLS-1$
-    /** The command entered by the user. */
-    private String         command    = null;
-    /** If the prompt should be active. */
-    private boolean        prompting  = false;
-    /** The object for thread synchronization with the prompt. */
-    private final Object   promptLock = new Object();
+    private ConsoleInputManager  inputManager;
+    private ConsoleOutputManager outputManager;
+    private ConsolePromptManager promptManager;
 
-    /** Create the composite.
+    /** Create the shell.
      *
-     * @param parent the prent composite
-     * @param style the composite style */
+     * @param parent the containing composite
+     * @param style the shell style */
     public SWTConsole(final Composite parent, final int style) {
         super(parent, style);
-
-        setLayout(new GridLayout(LAYOUT_NB_COLUMNS, false));
-
-        consoleOutput = new Text(this,
-                SWT.BORDER | SWT.READ_ONLY | SWT.WRAP | SWT.V_SCROLL | SWT.MULTI);
-        consoleOutput.setLayoutData(
-                new GridData(SWT.FILL, SWT.FILL, true, true, LAYOUT_NB_COLUMNS, 1));
-        consoleOutput.setRedraw(true);
-
-        lblPromptlabel = new Label(this, SWT.NONE);
-        lblPromptlabel.setText(prompt.apply());
-
-        consoleInput = new Text(this, SWT.BORDER);
-        consoleInput.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false, 1, 1));
-        consoleInput.addKeyListener(new HistoryTextKeyListener(this));
-
+        setLayout(new GridLayout(2, false));
+        createContents();
     }
 
-    /* (non-Javadoc)
-     * @see org.eclipse.swt.widgets.Composite#checkSubclass() */
-    @Override
-    protected void checkSubclass() {
-        // Disable the check that prevents subclassing of SWT components
+    /** Create contents of the shell. */
+    private void createContents() {
+        final Text output = new Text(this,
+                SWT.WRAP | SWT.READ_ONLY | SWT.MULTI | SWT.V_SCROLL);
+        output.setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true, 2, 1));
+        final Label prompt = new Label(this, SWT.NONE);
+        prompt.setLayoutData(new GridData(SWT.LEFT, SWT.CENTER, false, false));
+        final Text input = new Text(this, SWT.NONE);
+        input.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
+
+        inputManager = new ConsoleInputManager(input);
+        outputManager = new ConsoleOutputManager(output);
+        promptManager = new ConsolePromptManager(prompt);
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleManager#close() */
-    @Override
-    public void close() {
-        synchronized (promptLock) {
-            prompting = false;
-            promptLock.notifyAll();
-        }
-        if (consoleInput.isDisposed()) {
-            return;
-        }
-        consoleInput.setEnabled(false);
-        consoleOutput.setEnabled(false);
-    }
-
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.swt.ConsoleDelayIO#getInput() */
-    @Override
-    public String getInput() {
-        return consoleInput.getText();
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#getPrompt() */
-    @Override
-    public StringProvider getPrompt() {
-        return prompt;
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleManager#interruptPrompt() */
-    @Override
-    public void interruptPrompt() {
-        synchronized (promptLock) {
-            promptLock.notifyAll();
-        }
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleManager#isClosed() */
-    @Override
-    public boolean isClosed() {
-        return isDisposed();
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#print(java.lang.String) */
-    @Override
-    public void print(final String text) throws IOException {
-        if (isDisposed()) {
-            throw new IOException();
-        }
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                if (!consoleOutput.isDisposed()) {
-                    consoleOutput.append(text);
-                }
-            }
-        });
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#println() */
-    @Override
-    public void println() throws IOException {
-        if (isDisposed()) {
-            throw new IOException();
-        }
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                if (!consoleOutput.isDisposed()) {
-                    consoleOutput.append(System.lineSeparator());
-                }
-            }
-        });
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#println(java.lang.String) */
-    @Override
-    public void println(final String message) throws IOException {
-        if (isDisposed()) {
-            throw new IOException();
-        }
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                if (!consoleOutput.isDisposed()) {
-                    consoleOutput.append(message + System.lineSeparator());
-                }
-            }
-        });
-
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#prompt() */
-    @Override
-    public String prompt() throws IOException {
-        synchronized (promptLock) {
-            initPrompt(prompt.apply());
-            promptLock.notifyAll();
-            try {
-                while (prompting) {
-                    promptLock.wait();
-                }
-            } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
-                command = null;
-                Thread.currentThread().interrupt();
-            }
-        }
-        if (isDisposed()) {
-            throw new IOException("Input closed"); //$NON-NLS-1$
-        }
-        return command;
-    }
-
-    /** Initialize the prompting.
+    /** Connect the console parts to the shell.
      * 
-     * @param message the prompt message
-     * @throws IOException if the console is disposed */
-    private void initPrompt(final String message) throws IOException {
-        if (isDisposed()) {
-            throw new IOException();
-        }
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                if (!consoleOutput.isDisposed()) {
-                    lblPromptlabel.setText(message);
-                    // relayout
-                    SWTConsole.this.layout();
-                    consoleInput.setEnabled(true);
-                    consoleInput.setFocus();
-                }
-            }
-        });
-        prompting = true;
-        command = null;
+     * @param input the input
+     * @param output the output
+     * @param promptStream the stream where the prompts are forwarded. */
+    public void connect(final PipedConsoleInput input, final PipedConsoleOutput output,
+            final BufferedReader promptStream) {
+        inputManager.setManager(input);
+        outputManager.setManager(output);
+        promptManager.setStream(promptStream);
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(long) */
-    @Override
-    public String prompt(final long timeout) throws IOException {
-        synchronized (promptLock) {
-            initPrompt(prompt.apply());
-            promptLock.notifyAll();
-            try {
-                waitPromptResolution(timeout);
-            } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
-                command = null;
-                Thread.currentThread().interrupt();
-            }
-        }
-        return command;
+    /** @return the inputManager */
+    public ConsoleDelayIO getInputManager() {
+        return inputManager;
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#prompt(java.lang.String) */
-    @Override
-    public String prompt(final String message) throws IOException {
-        synchronized (promptLock) {
-            initPrompt(message);
-            promptLock.notifyAll();
-            try {
-                while (prompting) {
-                    promptLock.wait();
-                }
-                if (isDisposed()) {
-                    throw new IOException();
-                }
-            } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
-                command = null;
-                Thread.currentThread().interrupt();
-            } finally {
-                resetPrompt();
-            }
-        }
-        return command;
+    /** @return the outputManager */
+    public ConsoleOutputDisplay getOutputManager() {
+        return outputManager;
     }
 
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.manager.ConsoleInput#prompt(java.lang.String, long) */
-    @Override
-    public String prompt(final String message, final long timeout) throws IOException {
-        synchronized (promptLock) {
-            initPrompt(message);
-            promptLock.notifyAll();
-            try {
-                waitPromptResolution(timeout);
-            } catch (final InterruptedException e) {
-                LOGGER.log(Level.WARNING, ERROR_SYNCHRO_PROMPT, e);
-                command = null;
-                Thread.currentThread().interrupt();
-            } finally {
-                resetPrompt();
-            }
-        }
-        return command;
+    /** @return the promptManager */
+    public ConsolePromptManager getPromptManager() {
+        return promptManager;
     }
-
-    /** Wait the prompt resolution by user.
-     * 
-     * @param timeout the timeout for the wait
-     * @throws InterruptedException if the thread is interrupted
-     * @throws IOException if the console is disposed. */
-    private void waitPromptResolution(final long timeout)
-            throws InterruptedException, IOException {
-        final long start = System.currentTimeMillis();
-        long cur = start;
-        while (prompting && start + timeout > cur) {
-            promptLock.wait((cur - start - timeout) / TWO);
-            cur = System.currentTimeMillis();
-        }
-        if (isDisposed()) {
-            throw new IOException();
-        }
-    }
-
-    /** Reset the prompt message. */
-    private void resetPrompt() {
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                if (!consoleOutput.isDisposed()) {
-                    lblPromptlabel.setText(prompt.apply());
-                    // relayout
-                    SWTConsole.this.layout();
-                }
-            }
-        });
-    }
-
-    /* (non-Javadoc)
-     * @see org.eclipse.swt.widgets.Composite#setFocus() */
-    @Override
-    public boolean setFocus() {
-        return consoleInput.setFocus();
-    }
-
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.swt.ConsoleDelayIO#setInput(java.lang.String) */
-    @Override
-    public void setInput(final String input) {
-        consoleInput.setText(input);
-        consoleInput.setSelection(input.length());
-    }
-
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
-    @Override
-    public void setPrompt(final String prompt) {
-        setPrompt(new ConstantString(prompt));
-    }
-
-    /* (non-Javadoc)
-     * @see fr.bigeon.gclc.ConsoleManager#setPrompt(java.lang.String) */
-    @Override
-    public void setPrompt(final StringProvider prompt) {
-        this.prompt = prompt;
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                if (!consoleOutput.isDisposed()) {
-                    lblPromptlabel.setText(prompt.apply());
-                    // relayout
-                    SWTConsole.this.layout();
-                }
-            }
-        });
-    }
-
-    /** Set the input text.
-     * 
-     * @param string the text */
-    public void setText(final String string) {
-        consoleInput.setText(string);
-    }
-
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.swt.ConsoleDelayIO#validateInput() */
-    @Override
-    public void validateInput() {
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                consoleInput.setEnabled(false);
-            }
-        });
-        synchronized (promptLock) {
-            while (!prompting) {
-                try {
-                    promptLock.wait();
-                } catch (final InterruptedException e) {
-                    LOGGER.log(Level.SEVERE, "Interruption while waiting prompt", e); //$NON-NLS-1$
-                    Thread.currentThread().interrupt();
-                }
-            }
-            Display.getDefault().syncExec(new Runnable() {
-                @SuppressWarnings("synthetic-access")
-                @Override
-                public void run() {
-                    command = consoleInput.getText();
-                    prompting = false;
-                    consoleInput.setText(EMPTY);
-                    consoleOutput.append(CMD_PREFIX + command + System.lineSeparator());
-                }
-            });
-            promptLock.notifyAll();
-        }
-    }
-
 }

gclc-swt/src/main/java/net/bigeon/gclc/swt/SWTConsoleShell.java

@@ -1,118 +0,0 @@
-/*
- * GCLC swt, provide a swt window for console applications
- * Copyright (C) 2015-2017 E. Bigeon
- * mailto:emmanuel@bigeon.fr
- *
- * This software is governed by the CeCILL license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- */
-/**
- * gclc-swt:net.bigeon.gclc.swt.SWTConsoleShell.java
- * Created on: Apr 18, 2015
- */
-package net.bigeon.gclc.swt;
-
-/*-
- * #%L
- * GCLC swt
- * %%
- * Copyright (C) 2015 - 2018 Bigeon
- * %%
- * This software is governed by the CeCILL  license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/ or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- * #L%
- */
-import org.eclipse.swt.SWT;
-import org.eclipse.swt.layout.FillLayout;
-import org.eclipse.swt.widgets.Display;
-import org.eclipse.swt.widgets.Shell;
-
-/** A shell containing a {@link SWTConsole}
- * <p>
- *
- * @author Emmanuel Bigeon */
-public final class SWTConsoleShell extends Shell {
-
-    /** The console component */
-    private SWTConsole console;
-
-    /** Create the shell.
-     *
-     * @param display the display */
-    public SWTConsoleShell(final Display display, final int style) {
-        super(display, style);
-        setLayout(new FillLayout(SWT.HORIZONTAL));
-
-        createContents();
-    }
-
-    @Override
-    protected void checkSubclass() {
-        // Disable the check that prevents subclassing of SWT components
-    }
-
-    /** Create contents of the shell. */
-    private void createContents() {
-        console = new SWTConsole(this, SWT.NONE);
-        setText("Console Application"); //$NON-NLS-1$
-    }
-
-    /* (non-Javadoc)
-     * @see org.eclipse.swt.widgets.Shell#dispose() */
-    @Override
-    public void dispose() {
-        super.dispose();
-        console.close();
-    }
-
-    /** @return the input and output. */
-    public SWTConsole getManager() {
-        return console;
-    }
-}

gclc-swt/src/main/java/net/bigeon/gclc/swt/SWTConsoleView.java

@@ -68,66 +68,30 @@ package net.bigeon.gclc.swt;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import java.io.IOException;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.widgets.Composite;
-import org.eclipse.swt.widgets.Display;
 import org.eclipse.swt.widgets.Text;
 
 import net.bigeon.gclc.ConsoleApplication;
-import net.bigeon.gclc.utils.AOutputForwardRunnable;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.swt.api.ConsoleDelayIO;
+import net.bigeon.gclc.swt.api.ConsoleOutputDisplay;
+import net.bigeon.gclc.swt.io.ConsoleInputManager;
+import net.bigeon.gclc.swt.io.ConsoleOutputManager;
+import net.bigeon.gclc.swt.tools.HistoryTextKeyListener;
 
-/** A SWT component to connect to gclc {@link ConsoleApplication}
- * <p>
+/** A SWT component to connect to gclc {@link ConsoleApplication}.
  *
  * @author Emmanuel Bigeon */
-public final class SWTConsoleView extends Composite implements ConsoleDelayIO {
-    /** The local implementation of the forwarding runnable
-     *
-     * @author Emmanuel Bigeon */
-    private final class ToSWTConsoleForwardRunnable extends AOutputForwardRunnable {
-        /** The running status */
-        private boolean running = true;
-
-        /** @param manager the manager */
-        public ToSWTConsoleForwardRunnable(final PipedConsoleOutput manager) {
-            super(manager);
-        }
-
-        @Override
-        protected void forwardLine(final String m) {
-            appendConsoleOutput(System.lineSeparator() + m);
-        }
-
-        @Override
-        protected boolean isRunning() {
-            return running && !isDisposed();
-        }
-
-        /** @param running the running to set */
-        public void setRunning(final boolean running) {
-            this.running = running;
-        }
-    }
-
-    /** The class logger */
-    private static final Logger         LOGGER = Logger
-            .getLogger(SWTConsoleView.class.getName());
-    /** The console output text field */
-    private final Text                  consoleOutput;
-    /** The console input text field */
-    private final Text                  consoleInput;
-    /** The input. */
-    private PipedConsoleInput           input;
-    /** The forwarding runnable */
-    private ToSWTConsoleForwardRunnable forward;
+public final class SWTConsoleView extends Composite {
+    /** The input manager. */
+    private final ConsoleInputManager  inManager;
+    /** The output manager. */
+    private final ConsoleOutputManager outManager;
 
     /** Create the composite.
      *
@@ -138,52 +102,23 @@ public final class SWTConsoleView extends Composite implements ConsoleDelayIO {
 
         setLayout(new GridLayout(1, false));
 
-        consoleOutput = new Text(this,
+        final Text consoleOutput = new Text(this,
                 SWT.BORDER | SWT.READ_ONLY | SWT.WRAP | SWT.V_SCROLL | SWT.MULTI);
         consoleOutput.setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true, 1, 1));
         consoleOutput.setRedraw(true);
+        outManager = new ConsoleOutputManager(consoleOutput);
 
-        consoleInput = new Text(this, SWT.BORDER);
+        final Text consoleInput = new Text(this, SWT.BORDER);
         consoleInput.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false, 1, 1));
-        consoleInput.addKeyListener(new HistoryTextKeyListener(this));
-    }
-
-    /** @param next the next message */
-    public void appendConsoleOutput(final String next) {
-        Display.getDefault().syncExec(new Runnable() {
-            @SuppressWarnings("synthetic-access")
-            @Override
-            public void run() {
-                consoleOutput.append(next);
-            }
-        });
-    }
-
-    @Override
-    protected void checkSubclass() {
-        // Disable the check that prevents subclassing of SWT components
-    }
-
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.swt.ConsoleDelayIO#getInput() */
-    @Override
-    public String getInput() {
-        return consoleInput.getText();
+        inManager = new ConsoleInputManager(consoleInput);
+        consoleInput.addKeyListener(new HistoryTextKeyListener(inManager));
     }
 
     /* (non-Javadoc)
      * @see org.eclipse.swt.widgets.Composite#setFocus() */
     @Override
     public boolean setFocus() {
-        return consoleInput.setFocus();
-    }
-
-    /* (non-Javadoc)
-     * @see net.bigeon.gclc.swt.ConsoleDelayIO#setInput(java.lang.String) */
-    @Override
-    public void setInput(final String input) {
-        consoleInput.setText(input);
-        consoleInput.setSelection(input.length());
+        return inManager.getText().setFocus();
     }
 
     /** Set the input and output.
@@ -192,29 +127,21 @@ public final class SWTConsoleView extends Composite implements ConsoleDelayIO {
      * @param input the input */
     public void setManager(final PipedConsoleOutput manager,
             final PipedConsoleInput input) {
-        this.input = input;
-        if (forward != null) {
-            forward.setRunning(false);
-        }
-        forward = new ToSWTConsoleForwardRunnable(manager);
-        final Thread th = new Thread(forward, "gclcToSWT"); //$NON-NLS-1$
-        th.start();
+        outManager.setManager(manager);
+        inManager.setManager(input);
     }
 
-    /** @param string the text */
-    public void setText(final String string) {
-        consoleInput.setText(string);
-    }
-
-    /**
+    /** Get the manager of console input.
      * 
-     */
-    @Override
-    public void validateInput() {
-        try {
-            input.type(getInput());
-        } catch (final IOException e) {
-            LOGGER.log(Level.SEVERE, "Unable to input value to console", e); //$NON-NLS-1$
-        }
+     * @return the input manager */
+    public ConsoleDelayIO getInputManager() {
+        return inManager;
+    }
+
+    /** Get the manager of console output.
+     * 
+     * @return the output manager */
+    public ConsoleOutputDisplay getOutputManager() {
+        return outManager;
     }
 }

gclc-swt/src/main/java/net/bigeon/gclc/swt/api/ConsoleDelayIO.java

@@ -33,7 +33,9 @@
  * gclc-swt:net.bigeon.gclc.swt.ConsoleDelayIO.java
  * Created on: Nov 19, 2016
  */
-package net.bigeon.gclc.swt;
+package net.bigeon.gclc.swt.api;
+
+import java.io.IOException;
 
 /*-
  * #%L
@@ -86,6 +88,8 @@ public interface ConsoleDelayIO {
      * @param input the input to set */
     void setInput(String input);
 
-    /** Actually send the input as the prompt next input. */
-    void validateInput();
+    /** Actually send the input as the prompt next input.
+     * 
+     * @throws IOException if an error occurred */
+    void validateInput() throws IOException;
 }

gclc-swt/src/main/java/net/bigeon/gclc/swt/api/ConsoleOutputDisplay.java

@@ -0,0 +1,50 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.api;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+/** The common interface for console display.
+ * 
+ * @author Emmanuel Bigeon */
+public interface ConsoleOutputDisplay {
+
+    /** Append a line to the display.
+     * 
+     * @param m the line content */
+    void appendLine(String m);
+
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/api/package-info.java

@@ -0,0 +1,8 @@
+/**
+ * 
+ */
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+package net.bigeon.gclc.swt.api;

gclc-swt/src/main/java/net/bigeon/gclc/swt/io/ConsoleInputManager.java

@@ -0,0 +1,98 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.io;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import java.io.IOException;
+
+import org.eclipse.swt.widgets.Text;
+
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.swt.api.ConsoleDelayIO;
+
+/** The object managing the console input.
+ * 
+ * @author Emmanuel Bigeon */
+public final class ConsoleInputManager implements ConsoleDelayIO {
+
+    private final Text        text;
+    private PipedConsoleInput input;
+
+    /** Create a managing object.
+     * 
+     * @param text the text */
+    public ConsoleInputManager(final Text text) {
+        super();
+        this.text = text;
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.swt.ConsoleDelayIO#setInput(java.lang.String) */
+    @Override
+    public void setInput(final String string) {
+        text.setText(string);
+        text.setSelection(string.length());
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.swt.ConsoleDelayIO#getInput() */
+    @Override
+    public String getInput() {
+        return text.getText();
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.swt.ConsoleDelayIO#validateInput() */
+    @Override
+    public void validateInput() throws IOException {
+        input.type(text.getText());
+    }
+
+    /** Set the input to control.
+     * 
+     * @param input the input */
+    public void setManager(final PipedConsoleInput input) {
+        this.input = input;
+    }
+
+    /** Get the text component containing the currently set input.
+     * 
+     * @return the text */
+    public Text getText() {
+        return text;
+    }
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/io/ConsoleOutputManager.java

@@ -0,0 +1,107 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.io;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import org.eclipse.swt.widgets.Text;
+
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.swt.api.ConsoleOutputDisplay;
+import net.bigeon.gclc.swt.tools.TextAppendingRunnable;
+import net.bigeon.gclc.swt.tools.ToSWTConsoleForwardRunnable;
+
+/** The manager for console output to insert in a text.
+ * 
+ * @author Emmanuel Bigeon */
+public final class ConsoleOutputManager implements ConsoleOutputDisplay {
+    /** The SWT component displaying the output content. */
+    private final Text                  text;
+    /** The forwarding runnable. */
+    private ToSWTConsoleForwardRunnable forward;
+    /** The forwarding thread. */
+    private Thread                      forwardThread;
+
+    /** Create the manager.
+     * 
+     * @param text the text to display the output in. */
+    public ConsoleOutputManager(final Text text) {
+        super();
+        this.text = text;
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.swt.ConsoleOutputDisplay#appendLine(java.lang.String) */
+    @Override
+    public void appendLine(final String next) {
+        text.getDisplay().syncExec(new TextAppendingRunnable(text, next));
+    }
+
+    /** Set the output.
+     *
+     * @param output the output to set */
+    public void setManager(final PipedConsoleOutput output) {
+        if (forward != null) {
+            if (forward.getOuput() == output) {
+                return;
+            }
+            forward.setRunning(false);
+        }
+        if (output == null) {
+            forward = null;
+            forwardThread = null;
+            return;
+        }
+        forward = new ToSWTConsoleForwardRunnable(output, this, text);
+        forwardThread = new Thread(forward, "gclcToSWT"); //$NON-NLS-1$
+        forwardThread.start();
+    }
+
+    /** Get the current thread forwarding the console output to the SWT components.
+     * 
+     * @return the forwardThread */
+    public Thread getForwardThread() {
+        return forwardThread;
+    }
+
+    /** @return the output consumer */
+    public PipedConsoleOutput getManager() {
+        if (forward == null) {
+            return null;
+        }
+        return forward.getOuput();
+    }
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/io/ConsolePromptManager.java

@@ -0,0 +1,93 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.io;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import java.io.BufferedReader;
+
+import org.eclipse.swt.widgets.Label;
+
+import net.bigeon.gclc.swt.tools.PromptReadingRunnable;
+
+/** The manager for the console prompt updates.
+ * 
+ * @author Emmanuel Bigeon */
+public final class ConsolePromptManager {
+
+    /** The label to set the prompts at. */
+    private final Label label;
+
+    /** The current prompt reading runnable. */
+    private PromptReadingRunnable promptRead;
+
+    /** Create the manager.
+     * 
+     * @param label the label to update with prompts */
+    public ConsolePromptManager(final Label label) {
+        super();
+        this.label = label;
+    }
+
+    /** Set the prompt.
+     * <p>
+     * This method sets the prompt on the label as requested, without changing the
+     * actual prompt of the console input.
+     * 
+     * @param string the text */
+    public void setPrompt(final String string) {
+        label.setText(string);
+    }
+
+    /** Set the input to control.
+     * 
+     * @param promptStream the input */
+    public void setStream(final BufferedReader promptStream) {
+        if (promptRead != null) {
+            if (promptRead.getReader() == promptStream) {
+                return;
+            }
+            promptRead.setRunning(false);
+        }
+        if (promptStream == null) {
+            promptRead = null;
+            return;
+        }
+        promptRead = new PromptReadingRunnable(promptStream, label);
+        final Thread th = new Thread(promptRead, "Prompt To Label");
+        th.start();
+    }
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/io/package-info.java

@@ -0,0 +1,8 @@
+/**
+ * 
+ */
+/**
+ * @author Emmanuel Bigeon
+ *
+ */
+package net.bigeon.gclc.swt.io;

gclc-swt/src/main/java/net/bigeon/gclc/swt/tools/HistoryTextKeyListener.java

@@ -33,7 +33,11 @@
  * gclc-swt:net.bigeon.gclc.swt.HistoryTextKeyListener.java
  * Created on: Jun 9, 2016
  */
-package net.bigeon.gclc.swt;
+package net.bigeon.gclc.swt.tools;
+
+import java.io.IOException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /*-
  * #%L
@@ -72,8 +76,9 @@ import org.eclipse.swt.SWT;
 import org.eclipse.swt.events.KeyAdapter;
 import org.eclipse.swt.events.KeyEvent;
 
-import net.bigeon.collections.ArrayRibbon;
 import net.bigeon.collections.Ribbon;
+import net.bigeon.collections.ribbon.ArrayRibbon;
+import net.bigeon.gclc.swt.api.ConsoleDelayIO;
 
 /** A key listener to validate commands and manage the history of commands.
  *
@@ -83,7 +88,9 @@ public final class HistoryTextKeyListener extends KeyAdapter {
     /** The size of commands history. */
     private static final int     DEFAULT_HISTORY_SIZE = 10;
     /** The empty string constant. */
-    private static final String  EMPTY                = ""; //$NON-NLS-1$
+    private static final String  EMPTY                = "";    //$NON-NLS-1$
+    private static final Logger  LOGGER               = Logger
+            .getLogger(HistoryTextKeyListener.class.getName());
     /** The history ribbon. */
     private final Ribbon<String> commands;
     /** The current index in history search. */
@@ -118,7 +125,11 @@ public final class HistoryTextKeyListener extends KeyAdapter {
             if (!input.isEmpty()) {
                 commands.add(input);
             }
-            console.validateInput();
+            try {
+                console.validateInput();
+            } catch (final IOException e) {
+                LOGGER.log(Level.SEVERE, "Unable to write to console", e);
+            }
             currentIndex = -1;
         }
 

gclc-swt/src/main/java/net/bigeon/gclc/swt/tools/PromptReadingRunnable.java

@@ -0,0 +1,134 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.tools;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
+import org.eclipse.swt.widgets.Label;
+
+/** The runnable forwarding prompts from a buffered input to a label.
+ * 
+ * @author Emmanuel Bigeon */
+public class PromptReadingRunnable implements Runnable {
+
+    /** The logger for this class. */
+    private static final Logger LOGGER = Logger
+            .getLogger(PromptReadingRunnable.class.getName());
+
+    /** The reader providing the succession of prompts */
+    private final BufferedReader reader;
+    /** The label. */
+    private final Label          view;
+    /** The running status. */
+    private boolean              running = true;
+
+    /** The updating runnable implementation.
+     * 
+     * @author Emmanuel Bigeon */
+    private static class LabelUpdater implements Runnable {
+        /** The label to update. */
+        private final Label  label;
+        /** The text to set. */
+        private final String content;
+
+        /** Create the updating runnable.
+         * 
+         * @param label the label
+         * @param content the content */
+        public LabelUpdater(final Label label, final String content) {
+            super();
+            this.label = label;
+            this.content = content;
+        }
+
+        /* (non-Javadoc)
+         * @see java.lang.Runnable#run() */
+        @Override
+        public void run() {
+            label.setText(content);
+        }
+    }
+
+    /** Create the prompt flow updating runnable.
+     * 
+     * @param reader the reader
+     * @param view the view to update on lines. */
+    public PromptReadingRunnable(final BufferedReader reader, final Label view) {
+        super();
+        this.reader = reader;
+        this.view = view;
+    }
+
+    /* (non-Javadoc)
+     * @see java.lang.Runnable#run() */
+    @Override
+    public void run() {
+        String prompt;
+        try {
+            while (isRunning() && (prompt = reader.readLine()) != null) {
+                view.getDisplay().syncExec(new LabelUpdater(view, prompt));
+            }
+        } catch (final IOException e) {
+            LOGGER.log(Level.INFO, "Prompt cannot be read, or was prematuraly closed", e);
+        }
+    }
+
+    /** Get the running state.
+     * 
+     * @return the running */
+    public synchronized boolean isRunning() {
+        return running;
+    }
+
+    /** Set the running state.
+     * 
+     * @param running the running to set */
+    public synchronized void setRunning(final boolean running) {
+        this.running = running;
+    }
+
+    /** Get the prompt flow.
+     * 
+     * @return the reader */
+    public BufferedReader getReader() {
+        return reader;
+    }
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/tools/TextAppendingRunnable.java

@@ -0,0 +1,36 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.tools;
+
+import org.eclipse.swt.widgets.Text;
+
+/** A runnable appending text to the content of a {@link Text} component.
+ * 
+ * @author Emmanuel Bigeon */
+public class TextAppendingRunnable implements Runnable {
+    /** The text to append on a line (possibly new). */
+    private final String next;
+    /** The {@link Text} component. */
+    private final Text   text;
+
+    /** Create the appending runnable.
+     * 
+     * @param text the component to update
+     * @param next the text to append */
+    public TextAppendingRunnable(final Text text, final String next) {
+        this.text = text;
+        this.next = next;
+    }
+
+    /* (non-Javadoc)
+     * @see java.lang.Runnable#run() */
+    @Override
+    public void run() {
+        final String initialText = text.getText();
+        if (initialText != null && !initialText.isEmpty()) {
+            text.append(System.lineSeparator());
+        }
+        text.append(next);
+    }
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/tools/ToSWTConsoleForwardRunnable.java

@@ -0,0 +1,67 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt.tools;
+
+import org.eclipse.swt.widgets.Widget;
+
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.manager.forwarding.AOutputForwardRunnable;
+import net.bigeon.gclc.swt.api.ConsoleOutputDisplay;
+
+/** The local implementation of the forwarding runnable.
+ *
+ * @author Emmanuel Bigeon */
+public final class ToSWTConsoleForwardRunnable extends AOutputForwardRunnable {
+    /** The running status. */
+    private boolean                    running = true;
+    /** The console output. */
+    private final PipedConsoleOutput   out;
+    /** The console output display. */
+    private final ConsoleOutputDisplay display;
+    /** The actual SWT component. */
+    private final Widget               element;
+
+    /** Create the forwarding runnable.
+     * 
+     * @param manager the manager
+     * @param display the display
+     * @param element the composite */
+    public ToSWTConsoleForwardRunnable(final PipedConsoleOutput manager,
+                                       final ConsoleOutputDisplay display,
+                                       final Widget element) {
+        super(manager);
+        out = manager;
+        this.display = display;
+        this.element = element;
+    }
+
+    /* (non-Javadoc)
+     * @see
+     * net.bigeon.gclc.utils.AOutputForwardRunnable#forwardLine(java.lang.String) */
+    @Override
+    protected void forwardLine(final String m) {
+        display.appendLine(m);
+    }
+
+    /* (non-Javadoc)
+     * @see net.bigeon.gclc.utils.AOutputForwardRunnable#isRunning() */
+    @Override
+    protected boolean isRunning() {
+        return running && !element.isDisposed();
+    }
+
+    /** Set the running status.
+     * 
+     * @param running the running to set */
+    public void setRunning(final boolean running) {
+        this.running = running;
+    }
+
+    /** Get the output.
+     * 
+     * @return the currently forwarded output */
+    public PipedConsoleOutput getOuput() {
+        return out;
+    }
+}

gclc-swt/src/main/java/net/bigeon/gclc/swt/tools/package-info.java

@@ -0,0 +1,9 @@
+/**
+ * 
+ */
+/** Tool classes for the library. External code should not rely on classes in
+ * there as they can be removed without notice.
+ * 
+ * @author Emmanuel Bigeon */
+package net.bigeon.gclc.swt.tools;
+

gclc-swt/src/test/java/net/bigeon/gclc/swt/ConsoleInputManagerTest.java

@@ -0,0 +1,117 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.fail;
+import static org.mockito.Mockito.*;
+
+import java.io.IOException;
+
+import org.eclipse.swt.widgets.Text;
+import org.junit.Test;
+
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.swt.io.ConsoleInputManager;
+
+/** @author Emmanuel Bigeon */
+public class ConsoleInputManagerTest {
+
+    private final Text                text = mock(Text.class);
+    private final ConsoleInputManager cim  = new ConsoleInputManager(text);
+
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.io.ConsoleInputManager#setInput(java.lang.String)}. */
+    @Test
+    public void testSetText() {
+        cim.setInput("Text");
+        verify(text).setText("Text");
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.io.ConsoleInputManager#validateInput()}.
+     * 
+     * @throws IOException if an error occurred */
+    @Test
+    public void testValidateInput() throws IOException {
+        // Expected calls
+        when(text.getText()).thenReturn("abc");
+
+        try {
+            cim.validateInput();
+            fail("No input should cause exception for input validation");
+        } catch (final Exception e) {
+            // ok
+        }
+        final PipedConsoleInput input = mock(PipedConsoleInput.class);
+        cim.setManager(input);
+
+        cim.validateInput();
+        verify(input).type("abc");
+    }
+
+    @Test
+    public void testValidateInputWithError() throws IOException {
+        final PipedConsoleInput input = mock(PipedConsoleInput.class);
+        cim.setManager(input);
+        // Expected calls
+        when(text.getText()).thenReturn("abc");
+        doThrow(new IOException()).when(input).type("abc");
+
+        try {
+            cim.validateInput();
+            fail("IO exception should be forwarded");
+        } catch (final IOException e) {
+            // ok
+        }
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.io.ConsoleInputManager#getText()}. */
+    @Test
+    public void testGetText() {
+        assertEquals("Text component should be preserved", text, cim.getText());
+    }
+
+    @Test
+    public void testGetInput() {
+        when(text.getText()).thenReturn("abc");
+        assertEquals("Text component should be preserved", "abc", cim.getInput());
+    }
+
+}

gclc-swt/src/test/java/net/bigeon/gclc/swt/ConsoleOutputManagerTest.java

@@ -0,0 +1,103 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.*;
+
+import java.io.IOException;
+
+import org.eclipse.swt.widgets.Display;
+import org.eclipse.swt.widgets.Text;
+import org.junit.Test;
+import org.mockito.invocation.InvocationOnMock;
+import org.mockito.stubbing.Answer;
+
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.swt.io.ConsoleOutputManager;
+
+/** @author Emmanuel Bigeon */
+public class ConsoleOutputManagerTest {
+    private final Text                 text    = mock(Text.class);
+    private final ConsoleOutputManager com     = new ConsoleOutputManager(text);
+    private final Display              display = mock(Display.class);
+    {
+        when(text.getDisplay()).thenReturn(display);
+        doAnswer(new Answer<Object>() {
+            /* (non-Javadoc)
+             * @see
+             * org.mockito.stubbing.Answer#answer(org.mockito.invocation.InvocationOnMock) */
+            @Override
+            public Object answer(final InvocationOnMock invocation) throws Throwable {
+                final Runnable runnable = invocation.getArgument(0);
+                runnable.run();
+                return null;
+            }
+        }).when(display).syncExec(any(Runnable.class));
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.io.ConsoleOutputManager#appendLine(java.lang.String)}. */
+    @Test
+    public void testAppendConsoleOutput() {
+        when(text.getText()).thenReturn("", "abc",
+                "abc" + System.lineSeparator() + "def");
+        com.appendLine("abc");
+        verify(text).append("abc");
+        com.appendLine("def");
+        verify(text).append(System.lineSeparator());
+        verify(text).append("def");
+    }
+
+    @Test
+    public void testSetManager() throws IOException, InterruptedException {
+        final PipedConsoleOutput output = mock(PipedConsoleOutput.class);
+        com.setManager(output);
+        com.setManager(output);
+
+        try (PipedConsoleOutput output2 = new PipedConsoleOutput()) {
+            com.setManager(output2);
+            output2.println("line");
+            final Thread forwardThread = com.getForwardThread();
+            forwardThread.join(500);
+            com.setManager(null);
+            forwardThread.join();
+            verify(text).append("line");
+        }
+    }
+}

gclc-swt/src/test/java/net/bigeon/gclc/swt/ConsolePromptManagerTest.java

@@ -0,0 +1,164 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt;
+
+import static org.mockito.ArgumentMatchers.any;
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+import static org.mockito.Mockito.*;
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.util.concurrent.atomic.AtomicInteger;
+
+import org.eclipse.swt.widgets.Display;
+import org.eclipse.swt.widgets.Label;
+import org.junit.Test;
+import org.mockito.Mockito;
+import org.mockito.invocation.InvocationOnMock;
+import org.mockito.stubbing.Answer;
+
+import net.bigeon.gclc.swt.io.ConsolePromptManager;
+
+/** @author Emmanuel Bigeon */
+public class ConsolePromptManagerTest {
+
+    private final Label label = mock(Label.class);
+
+    private final ConsolePromptManager manager = new ConsolePromptManager(label);
+
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.io.ConsolePromptManager#setPrompt(java.lang.String)}. */
+    @Test
+    public void testSetPrompt() {
+        manager.setPrompt("abc");
+        verify(label).setText("abc");
+    }
+
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.io.ConsolePromptManager#setStream(java.io.BufferedReader)}.
+     * 
+     * @throws IOException if error */
+    @Test
+    public void testSetStream() throws IOException {
+        // Create the dispaly, in case...
+        final Display d = mock(Display.class);
+        when(label.getDisplay()).thenReturn(d);
+        Mockito.doAnswer(invocation -> {
+            ((Runnable) invocation.getArgument(0)).run();
+            return null;
+        }).when(d).asyncExec(any(Runnable.class));
+        Mockito.doAnswer(invocation -> {
+            ((Runnable) invocation.getArgument(0)).run();
+            return null;
+        }).when(d).syncExec(any(Runnable.class));
+        // Test.
+        BufferedReader output = mock(BufferedReader.class);
+        final Object lock = new Object();
+        final AtomicInteger calls = new AtomicInteger(0);
+        Mockito.when(output.readLine()).then(new Answer<String>() {
+            String[] ans = { "A line" };
+
+            @Override
+            public String answer(final InvocationOnMock invocation) throws Throwable {
+                synchronized (lock) {
+                    if (calls.get() >= ans.length) {
+                        calls.incrementAndGet();
+                        lock.notify();
+                        return null;
+                    }
+                    lock.notify();
+                    return ans[calls.getAndIncrement()];
+                }
+            }
+        });
+        manager.setStream(output);
+        manager.setStream(output);
+        synchronized (lock) {
+            while (calls.get() < 2) {
+                try {
+                    lock.wait(10);
+                } catch (final InterruptedException e) {
+                    // TODO Auto-generated catch block
+                    e.printStackTrace();
+                }
+            }
+        }
+        verify(output, times(2)).readLine();
+        output = mock(BufferedReader.class);
+        calls.set(0);
+        Mockito.when(output.readLine()).then(new Answer<String>() {
+            String[] ans = { "A line" };
+
+            @Override
+            public String answer(final InvocationOnMock invocation) throws Throwable {
+                synchronized (lock) {
+                    if (calls.get() >= ans.length) {
+                        calls.incrementAndGet();
+                        lock.notify();
+                        return null;
+                    }
+                    lock.notify();
+                    return ans[calls.getAndIncrement()];
+                }
+            }
+        });
+        manager.setStream(output);
+        while (calls.get() < 2) {
+            synchronized (lock) {
+                try {
+                    lock.wait(10);
+                } catch (final InterruptedException e) {
+                    // TODO Auto-generated catch block
+                    e.printStackTrace();
+                }
+            }
+        }
+        manager.setStream(null);
+        manager.setStream(output);
+        while (calls.get() < 3) {
+            synchronized (lock) {
+                try {
+                    lock.wait(10);
+                } catch (final InterruptedException e) {
+                    // TODO Auto-generated catch block
+                    e.printStackTrace();
+                }
+            }
+        }
+        verify(output, times(3)).readLine();
+        verify(label, times(2)).setText("A line");
+    }
+}

gclc-swt/src/test/java/net/bigeon/gclc/swt/HistoryTextKeyListenerTest.java

@@ -69,9 +69,17 @@ package net.bigeon.gclc.swt;
  * #L%
  */
 import static org.junit.Assert.assertEquals;
+import static org.mockito.Mockito.*;
+
+import java.io.IOException;
 
 import org.eclipse.swt.SWT;
+import org.eclipse.swt.events.KeyEvent;
 import org.junit.Test;
+import org.mockito.Mockito;
+
+import net.bigeon.gclc.swt.api.ConsoleDelayIO;
+import net.bigeon.gclc.swt.tools.HistoryTextKeyListener;
 
 /**
  * <p>
@@ -81,10 +89,10 @@ import org.junit.Test;
 public class HistoryTextKeyListenerTest {
 
     /** Test method for
-     * {@link net.bigeon.gclc.swt.HistoryTextKeyListener#keyPressed(org.eclipse.swt.events.KeyEvent)}. */
+     * {@link net.bigeon.gclc.swt.tools.HistoryTextKeyListener#keyPressed(org.eclipse.swt.events.KeyEvent)}. */
     @Test
     public final void testKeyPressedKeyEvent() {
-        ConsoleDelayIO io = new ConsoleDelayIO() {
+        final ConsoleDelayIO io = new ConsoleDelayIO() {
             private String input = "";
 
             @Override
@@ -93,7 +101,7 @@ public class HistoryTextKeyListenerTest {
             }
 
             @Override
-            public void setInput(String input) {
+            public void setInput(final String input) {
                 this.input = input;
             }
 
@@ -102,7 +110,7 @@ public class HistoryTextKeyListenerTest {
                 input = "";
             }
         };
-        HistoryTextKeyListener listener = new HistoryTextKeyListener(io);
+        final HistoryTextKeyListener listener = new HistoryTextKeyListener(io);
 
         // no effects
         assertEquals("", io.getInput());
@@ -139,4 +147,24 @@ public class HistoryTextKeyListenerTest {
         assertEquals("cmd arg2", io.getInput());
     }
 
+    @Test
+    public void testKeyPressedA() throws IOException {
+        final ConsoleDelayIO io = mock(ConsoleDelayIO.class);
+        final KeyEvent event = mock(KeyEvent.class);
+        event.keyCode = 'a';
+        final HistoryTextKeyListener listener = new HistoryTextKeyListener(io);
+        listener.keyPressed(event);
+        verifyNoInteractions(io);
+    }
+
+    @Test
+    public void testKeyPressedReturn() throws IOException {
+        final ConsoleDelayIO io = mock(ConsoleDelayIO.class);
+        Mockito.when(io.getInput()).thenReturn("Alpha");
+        final KeyEvent event = mock(KeyEvent.class);
+        event.keyCode = '\r';
+        final HistoryTextKeyListener listener = new HistoryTextKeyListener(io);
+        listener.keyPressed(event);
+        verify(io, times(1)).validateInput();
+    }
 }

gclc-swt/src/test/java/net/bigeon/gclc/swt/PromptReadingRunnableTest.java

@@ -0,0 +1,102 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.swt;
+
+/*-
+ * #%L
+ * GCLC swt
+ * %%
+ * Copyright (C) 2015 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.doAnswer;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
+import org.eclipse.swt.widgets.Display;
+import org.eclipse.swt.widgets.Label;
+import org.junit.Test;
+import org.mockito.Mockito;
+
+import net.bigeon.gclc.swt.tools.PromptReadingRunnable;
+
+/** @author Emmanuel Bigeon */
+public class PromptReadingRunnableTest {
+    private final Label   view    = mock(Label.class);
+    private final Display display = mock(Display.class);
+    {
+        when(view.getDisplay()).thenReturn(display);
+        doAnswer(invocation -> {
+            final Runnable runnable = invocation.getArgument(0);
+            runnable.run();
+            return null;
+        }).when(display).syncExec(any(Runnable.class));
+    }
+
+    /** Test method for {@link net.bigeon.gclc.swt.tools.PromptReadingRunnable#run()}.
+     * 
+     * @throws IOException if an error occurred */
+    @Test
+    public void testRun() throws IOException {
+        final BufferedReader reader = mock(BufferedReader.class);
+        when(reader.readLine()).thenReturn("abc", "def", null);
+
+        final PromptReadingRunnable runnable = new PromptReadingRunnable(reader, view);
+
+        runnable.run();
+        Mockito.verify(reader, Mockito.times(3)).readLine();
+    }
+
+    /** Test method for {@link net.bigeon.gclc.swt.tools.PromptReadingRunnable#run()}.
+     * 
+     * @throws IOException if an error occurred */
+    @Test
+    public void testRunWithIO() throws IOException {
+        final BufferedReader reader = mock(BufferedReader.class);
+        when(reader.readLine()).thenThrow(new IOException());
+
+        final PromptReadingRunnable runnable = new PromptReadingRunnable(reader, view);
+        final Logger logger = Logger.getLogger(PromptReadingRunnable.class.getName());
+        final Level back = logger.getLevel();
+        logger.setLevel(Level.OFF);
+
+        runnable.run();
+        logger.setLevel(back);
+        Mockito.verify(reader, Mockito.times(1)).readLine();
+    }
+
+}

gclc-swt/src/test/java/net/bigeon/gclc/swt/SWTConsoleShellTest.java

@@ -1,37 +1,5 @@
-/*
- * GCLC swt, provide a swt window for console applications
- * Copyright (C) 2015-2017 E. Bigeon
- * mailto:emmanuel@bigeon.fr
- *
- * This software is governed by the CeCILL license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- */
 /**
- * gclc-swt:net.bigeon.gclc.swt.SWTConsoleShellTest.java
- * Created on: Jun 8, 2016
+ * 
  */
 package net.bigeon.gclc.swt;
 
@@ -68,299 +36,44 @@ package net.bigeon.gclc.swt;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
 
-import java.io.IOException;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 
 import org.eclipse.swt.SWT;
-import org.eclipse.swt.widgets.Display;
+import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.Shell;
 import org.junit.Test;
 
-import net.bigeon.gclc.ConsoleApplication;
-import net.bigeon.gclc.command.Command;
-import net.bigeon.gclc.command.base.ExitCommand;
-import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.exception.InvalidCommandName;
-import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.manager.ConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.swt.io.ConsoleOutputManager;
 
-/**
- * <p>
- * TODO
- *
- * @author Emmanuel Bigeon */
-@SuppressWarnings({ "javadoc", "static-method", "nls", "deprecation" })
+/** @author Emmanuel Bigeon */
 public class SWTConsoleShellTest {
 
-    protected static final long  TWO_SECONDS = 2000;
-    private static final Display DISPLAY     = Display.getDefault();
-
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.SWTConsole#SWTConsole(Composite, int)}. */
     @Test
-    public void testGeneric() {
-        final SWTConsoleShell shell = new SWTConsoleShell(DISPLAY, SWT.SHELL_TRIM);
-        final SWTConsole swtConsole = shell.getManager();
-        try {
-            final ConsoleApplication appl = new ConsoleApplication(swtConsole, swtConsole,
-                    "Hello", "See you");
-            appl.add(new ExitCommand("exit", appl));
-            appl.add(new Command("long") {
-
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.
-                 * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-                 * java.lang.String[]) */
-                @Override
-                public void execute(final ConsoleOutput out, final ConsoleInput in,
-                        final String... args) throws CommandRunException {
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                }
-
-                @Override
-                public String tip() {
-                    return "a long running command";
-                }
-
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.Command#usageDetail() */
-                @Override
-                protected String usageDetail() {
-                    // TODO Auto-generated method stub
-                    // return null;
-                    throw new RuntimeException("Not implemented yet");
-                }
-            });
-// shell.pack();
-            shell.open();
-            shell.setVisible(false);
-            final Thread applThread = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    appl.start();
-                }
-            });
-            final Thread testThread = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("test"); //$NON-NLS-1$
-                            swtConsole.validateInput();
-                        }
-                    });
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            shell.dispose();
-                        }
-                    });
-                }
-            });
-            applThread.start();
-            testThread.start();
-            while (!shell.isDisposed()) {
-                if (!DISPLAY.readAndDispatch()) {
-                    DISPLAY.sleep();
-                }
-            }
-// DISPLAY.dispose();
-            assertTrue(swtConsole.isClosed());
-            Thread.sleep(TWO_SECONDS);
-            assertFalse(appl.isRunning());
-        } catch (final InvalidCommandName e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        } catch (final InterruptedException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        }
+    public void testSWTConsoleShell() {
+        final SWTConsole console = new SWTConsole(new Shell(), SWT.NONE);
+        assertNotNull("View should be initialized with managers",
+                console.getInputManager());
+        assertNotNull("View should be initialized with managers",
+                console.getOutputManager());
+        assertNotNull("View should be initialized with managers",
+                console.getPromptManager());
     }
 
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.SWTConsole#connect(PipedConsoleInput, PipedConsoleOutput, java.io.BufferedReader)}. */
     @Test
-    public void testConsoleClose() {
-        final SWTConsoleShell shell = new SWTConsoleShell(DISPLAY, SWT.SHELL_TRIM);
-        final SWTConsole swtConsole = shell.getManager();
-        swtConsole.close();
-        swtConsole.setPrompt(":");
-        try {
-            final ConsoleApplication appl = new ConsoleApplication(swtConsole, swtConsole,
-                    "Hello", "See you");
-            appl.add(new ExitCommand("exit", appl));
-            appl.add(new Command("long") {
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.
-                 * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-                 * java.lang.String[]) */
-                @Override
-                public void execute(final ConsoleOutput out, final ConsoleInput in,
-                        final String... args) throws CommandRunException {
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                }
-
-                @Override
-                public String tip() {
-                    return "a long running command";
-                }
-
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.Command#usageDetail() */
-                @Override
-                protected String usageDetail() {
-                    // TODO Auto-generated method stub
-                    // return null;
-                    throw new RuntimeException("Not implemented yet");
-                }
-            });
-            appl.add(new Command("test") {
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.
-                 * manager.ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-                 * java.lang.String[]) */
-                @Override
-                public void execute(final ConsoleOutput out, final ConsoleInput in,
-                        final String... args) throws CommandRunException {
-                    try {
-                        swtConsole.prompt("Test");
-                    } catch (final IOException e) {
-                        throw new CommandRunException("No input", e);
-                    }
-                }
-
-                @Override
-                public String tip() {
-                    return "a prompting running command";
-                }
-
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.Command#usageDetail() */
-                @Override
-                protected String usageDetail() {
-                    // TODO Auto-generated method stub
-                    // return null;
-                    throw new RuntimeException("Not implemented yet");
-                }
-            });
-// shell.pack();
-            shell.open();
-            shell.setVisible(false);
-            final Thread applThread = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    appl.start();
-                }
-            });
-            final Thread testThread = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("test"); //$NON-NLS-1$
-                            swtConsole.validateInput();
-                        }
-                    });
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("ok"); //$NON-NLS-1$
-                        }
-                    });
-                    swtConsole.validateInput();
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("long"); //$NON-NLS-1$
-                        }
-                    });
-                    swtConsole.validateInput();
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("test"); //$NON-NLS-1$
-                        }
-                    });
-                    swtConsole.validateInput();
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("test"); //$NON-NLS-1$
-                        }
-                    });
-                    swtConsole.validateInput();
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            shell.dispose();
-                        }
-                    });
-
-                }
-            });
-            applThread.start();
-            testThread.start();
-            while (!shell.isDisposed()) {
-                if (!DISPLAY.readAndDispatch()) {
-                    DISPLAY.sleep();
-                }
-            }
-            swtConsole.setPrompt(">");
-            try {
-                swtConsole.prompt();
-                fail("Prompting when closed should fail!");
-            } catch (final IOException e) {
-                assertNotNull(e);
-            }
-// DISPLAY.dispose();
-            assertTrue(swtConsole.isClosed());
-            Thread.sleep(TWO_SECONDS);
-            assertFalse(appl.isRunning());
-        } catch (final InvalidCommandName e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        } catch (final InterruptedException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        }
+    public void testConnect() {
+        final SWTConsole console = new SWTConsole(new Shell(), SWT.NONE);
+        // Disconnection should work.
+        console.connect(null, null, null);
+        assertNull("Console should disconnect",
+                ((ConsoleOutputManager) console.getOutputManager()).getManager());
     }
+
 }

gclc-swt/src/test/java/net/bigeon/gclc/swt/SWTConsoleViewTest.java

@@ -1,37 +1,5 @@
-/*
- * GCLC swt, provide a swt window for console applications
- * Copyright (C) 2015-2017 E. Bigeon
- * mailto:emmanuel@bigeon.fr
- *
- * This software is governed by the CeCILL license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- */
 /**
- * gclc-swt:net.bigeon.gclc.swt.SWTConsoleShellTest.java
- * Created on: Jun 8, 2016
+ * 
  */
 package net.bigeon.gclc.swt;
 
@@ -68,137 +36,47 @@ package net.bigeon.gclc.swt;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 
-import java.io.IOException;
-
 import org.eclipse.swt.SWT;
-import org.eclipse.swt.widgets.Display;
 import org.eclipse.swt.widgets.Shell;
 import org.junit.Test;
 
-import net.bigeon.gclc.ConsoleApplication;
-import net.bigeon.gclc.command.Command;
-import net.bigeon.gclc.command.base.ExitCommand;
-import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.exception.InvalidCommandName;
-import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+import net.bigeon.gclc.swt.io.ConsoleOutputManager;
 
-/**
- * <p>
- * TODO
- *
- * @author Emmanuel Bigeon */
-@SuppressWarnings({ "javadoc", "static-method", "nls", "deprecation" })
+/** @author Emmanuel Bigeon */
 public class SWTConsoleViewTest {
 
-    protected static final long  TWO_SECONDS = 2000;
-    private static final Display DISPLAY     = Display.getDefault();
-
+    /** Test method for {@link net.bigeon.gclc.swt.SWTConsoleView#setFocus()}. */
     @Test
-    public void testGeneric() {
-        final Shell shell = new Shell(DISPLAY);
-        final SWTConsoleView swtConsole = new SWTConsoleView(shell, SWT.NONE);
-        try (PipedConsoleOutput manager = new PipedConsoleOutput();
-                PipedConsoleInput input = new PipedConsoleInput(System.out)) {
-            swtConsole.setManager(manager, input);
-        } catch (final IOException e2) {
-            assertNull(e2);
-        }
-        try (PipedConsoleOutput manager = new PipedConsoleOutput();
-                PipedConsoleInput input = new PipedConsoleInput(System.out)) {
-            swtConsole.setManager(manager, input);
-            final ConsoleApplication appl = new ConsoleApplication(manager, input,
-                    "Hello", "See you");
-            appl.add(new ExitCommand("exit", appl));
-            appl.add(new Command("long") {
+    public void testSetFocus() {
+        final SWTConsoleView view = new SWTConsoleView(new Shell(), SWT.NONE);
+        assertFalse("Non visible component set focus should fail", view.setFocus());
+    }
 
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.ICommand#execute(fr.bigeon.gclc.manager.
-                 * ConsoleOutput, fr.bigeon.gclc.manager.ConsoleInput,
-                 * java.lang.String[]) */
-                @Override
-                public void execute(final ConsoleOutput out, final ConsoleInput in,
-                        final String... args) throws CommandRunException {
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                }
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.SWTConsoleView#SWTConsoleView(org.eclipse.swt.widgets.Composite, int)}. */
+    @Test
+    public void testSWTConsoleView() {
+        final SWTConsoleView view = new SWTConsoleView(new Shell(), SWT.NONE);
+        assertNotNull("View should be initialized with managers", view.getInputManager());
+        assertNotNull("View should be initialized with managers",
+                view.getOutputManager());
+    }
 
-                @Override
-                public String tip() {
-                    return "a long running command";
-                }
-
-                /* (non-Javadoc)
-                 * @see fr.bigeon.gclc.command.Command#usageDetail() */
-                @Override
-                protected String usageDetail() {
-                    // TODO Auto-generated method stub
-                    // return null;
-                    throw new RuntimeException("Not implemented yet");
-                }
-            });
-// shell.pack();
-            shell.open();
-            shell.setVisible(false);
-            final Thread applThread = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    appl.start();
-                }
-            });
-            final Thread testThread = new Thread(new Runnable() {
-
-                @Override
-                public void run() {
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            swtConsole.setText("test"); //$NON-NLS-1$
-                            swtConsole.validateInput();
-                        }
-                    });
-                    try {
-                        Thread.sleep(TWO_SECONDS);
-                    } catch (final InterruptedException e) {
-                        // TODO Auto-generated catch block
-                        e.printStackTrace();
-                    }
-                    Display.getDefault().syncExec(new Runnable() {
-                        @Override
-                        public void run() {
-                            shell.dispose();
-                        }
-                    });
-                }
-            });
-            applThread.start();
-            testThread.start();
-            while (!shell.isDisposed()) {
-                if (!DISPLAY.readAndDispatch()) {
-                    DISPLAY.sleep();
-                }
-            }
-        } catch (final InvalidCommandName e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        } catch (final IOException e1) {
-            // TODO Auto-generated catch block
-            e1.printStackTrace();
-        }
+    /** Test method for
+     * {@link net.bigeon.gclc.swt.SWTConsoleView#setManager(PipedConsoleOutput, PipedConsoleInput)}. */
+    @Test
+    public void testSetManager() {
+        final SWTConsoleView view = new SWTConsoleView(new Shell(), SWT.NONE);
+        // Disconnection should work.
+        view.setManager(null, null);
+        assertNull("Disconnection should replace the input.",
+                ((ConsoleOutputManager) view.getOutputManager()).getManager());
     }
 }

gclc-swt/src/test/resources/mockito-extensions/org.mockito.plugins.MockMaker

@@ -0,0 +1 @@
+mock-maker-inline

gclc-test/pom.xml

@@ -0,0 +1,79 @@
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+	<modelVersion>4.0.0</modelVersion>
+	<parent>
+		<groupId>net.bigeon.config</groupId>
+		<artifactId>ebigeon-config</artifactId>
+		<version>1.8.29</version>
+	</parent>
+	<groupId>net.bigeon.gclc</groupId>
+	<artifactId>test</artifactId>
+	<version>0.0.1-SNAPSHOT</version>
+	<packaging>jar</packaging>
+
+	<name>GCLC Tests</name>
+	<description>Test utilities for GCLC applications and extensions</description>
+	<inceptionYear>2018</inceptionYear>
+	<organization>
+		<name>Bigeon</name>
+		<url>https://bigeon.net</url>
+	</organization>
+	<licenses>
+		<license>
+			<distribution>manual</distribution>
+			<name>CeCILL 2.1</name>
+			<url>https://cecill.info/licences/Licence_CeCILL_V2.1-en.html</url>
+		</license>
+	</licenses>
+	<developers>
+		<developer>
+			<email>emmanuel@bigeon.fr</email>
+			<name>Emmanuel Bigeon</name>
+			<url>bigeon.net</url>
+			<roles>
+				<role>PM</role>
+			</roles>
+		</developer>
+	</developers>
+	<scm>
+		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
+		<tag>HEAD</tag>
+	</scm>
+	<properties>
+		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+		<license.licenseName>cecill_2.1</license.licenseName>
+	</properties>
+
+	<dependencies>
+		<dependency>
+			<groupId>net.bigeon</groupId>
+			<artifactId>gclc</artifactId>
+			<version>2.1.2</version>
+		</dependency>
+		<dependency>
+			<groupId>junit</groupId>
+			<artifactId>junit</artifactId>
+			<version>4.13.2</version>
+		</dependency>
+		<dependency>
+			<groupId>net.bigeon.test</groupId>
+			<artifactId>junitmt</artifactId>
+			<version>1.0.4</version>
+		</dependency>
+	</dependencies>
+	<reporting>
+		<plugins>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-javadoc-plugin</artifactId>
+				<version>3.0.1</version>
+			</plugin>
+			<plugin>
+				<groupId>com.github.sevntu-checkstyle</groupId>
+				<artifactId>dsm-maven-plugin</artifactId>
+				<version>2.2.0</version>
+			</plugin>
+		</plugins>
+	</reporting>
+</project>

gclc-test/src/main/java/net/bigeon/gclc/test/InputContract.java

@@ -0,0 +1,54 @@
+/**
+ * 
+ */
+package net.bigeon.gclc.test;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
+
+import java.io.IOException;
+import java.io.InterruptedIOException;
+import java.util.function.Supplier;
+
+import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.test.junitmt.FunctionalTestRunnable;
+import net.bigeon.test.junitmt.ThreadTest;
+
+/** @author Emmanuel Bigeon */
+public final class InputContract {
+
+    public void testInputContract(final Supplier<ConsoleInput> inputs)
+            throws IOException, InterruptedException {
+        // Test close contract
+        final ConsoleInput input = inputs.get();
+        assertFalse("An input should not initially be closed", input.isClosed());
+        input.close();
+        assertTrue("An input should close", input.isClosed());
+        input.close();
+        assertTrue("A closed input should stay closed", input.isClosed());
+        try {
+            input.prompt();
+            fail("Closed input prompting should fail");
+        } catch (final IOException e) {
+            // ok
+        }
+
+        // Test interruption contract
+        final ConsoleInput input2 = inputs.get();
+        final FunctionalTestRunnable prompting = new FunctionalTestRunnable(
+                () -> {
+                    try {
+                        input2.prompt();
+                        fail("Interrupted prompt should throw INterruptedIOException");
+                    } catch (final InterruptedIOException e) {
+                        // ok
+                    }
+                });
+        final Thread th = new Thread(prompting);
+        th.start();
+        th.join(200);
+        input2.interruptPrompt();
+        ThreadTest.assertRuns(th, prompting);
+    }
+}

gclc-test/src/test/java/net/bigeon/gclc/test/InputContractTest.java

@@ -0,0 +1,52 @@
+/** gclc-test: InputContractTest
+ * Created on Apr. 6, 2020
+ */
+package net.bigeon.gclc.test;
+
+import static org.junit.Assert.fail;
+
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.PipedInputStream;
+import java.io.PipedOutputStream;
+import java.io.PrintStream;
+import java.nio.charset.StandardCharsets;
+
+import org.junit.After;
+import org.junit.Test;
+
+import net.bigeon.gclc.manager.EmptyInput;
+import net.bigeon.gclc.manager.StreamConsoleInput;
+
+/** @author Emmanuel Bigeon */
+public class InputContractTest {
+
+    /** Test method for
+     * {@link net.bigeon.gclc.test.InputContract#testInputContract(java.util.function.Supplier)}. */
+    @Test
+    public void testTestInputContract() throws IOException, InterruptedException {
+        final InputContract contract = new InputContract();
+        try (InputStream in = new PipedInputStream(new PipedOutputStream());
+                PrintStream out = new PrintStream("temp.txt")) {
+            contract.testInputContract(
+                    () -> new StreamConsoleInput(out, in, StandardCharsets.UTF_8));
+        }
+
+        try {
+            contract.testInputContract(() -> EmptyInput.INSTANCE);
+            fail("Empty input does not fit the contract for inputs since it cannot be closed.");
+        } catch (final AssertionError e) {
+            // ok
+        }
+    }
+
+    @After
+    public void tearDown() {
+        final File f = new File("temp.txt");
+        if (f.exists()) {
+            f.delete();
+        }
+    }
+
+}

gclc.system/pom.xml

@@ -3,7 +3,7 @@
 	<parent>
 		<groupId>net.bigeon.config</groupId>
 		<artifactId>ebigeon-config</artifactId>
-		<version>1.8.21</version>
+		<version>1.8.29</version>
 	</parent>
 	<groupId>net.bigeon.gclc</groupId>
 	<artifactId>system</artifactId>
@@ -48,12 +48,12 @@
 		<dependency>
 			<groupId>net.bigeon</groupId>
 			<artifactId>gclc</artifactId>
-			<version>2.0.11</version>
+			<version>2.1.2</version>
 		</dependency>
 		<dependency>
 			<groupId>org.mockito</groupId>
 			<artifactId>mockito-core</artifactId>
-			<version>2.23.0</version>
+			<version>4.2.0</version>
 			<scope>test</scope>
 		</dependency>
 	</dependencies>
@@ -67,7 +67,7 @@
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-javadoc-plugin</artifactId>
-				<version>2.10.3</version>
+				<version>3.0.1</version>
 			</plugin>
 			<plugin>
 				<groupId>com.github.sevntu-checkstyle</groupId>

gclc.system/src/main/java/net/bigeon/gclc/system/ExecSystemCommand.java

@@ -101,7 +101,7 @@ public class ExecSystemCommand extends Command {
         try (BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(os))) {
             while (th.isAlive()) {
                 final String user = in.prompt();
-                // Avoid interruption being sent to process.
+                // Forward to process if not empty.
                 if (!user.isEmpty()) {
                     writer.write(user + EOL);
                 }

gclc.system/src/test/java/net/bigeon/gclc/system/ExecSystemCommandTest.java

@@ -42,8 +42,8 @@ import org.junit.Test;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.EmptyInput;
-import net.bigeon.gclc.utils.SinkOutput;
+import net.bigeon.gclc.manager.EmptyInput;
+import net.bigeon.gclc.manager.SinkOutput;
 
 public class ExecSystemCommandTest {
 
@@ -79,4 +79,26 @@ public class ExecSystemCommandTest {
         }
     }
 
+    /** Test the execution of the command.
+     * 
+     * @throws CommandRunException if the command fails */
+    @Test
+    public void testLongExecute() throws CommandRunException {
+        final ConsoleOutput out = SinkOutput.INSTANCE;
+        final ConsoleInput in = EmptyInput.INSTANCE;
+
+        final ExecSystemCommand cmd = new ExecSystemCommand();
+        if (System.getProperty("os.name").contains("indows")) {
+            cmd.execute(out, in, "cmd", "/C", "dir", "/s");
+        } else if (System.getProperty("os.name").contains("inux")) {
+            cmd.execute(out, in, "ls", "-R");
+        }
+        try {
+            cmd.execute(out, in, "inexistent");
+            fail("Able to execute inexistent command in system");
+        } catch (final CommandRunException e) {
+            // ok
+        }
+    }
+
 }

gclc.system/src/test/java/net/bigeon/gclc/system/ForwardingRunnableTest.java

@@ -13,8 +13,8 @@ import org.junit.Test;
 import org.mockito.Mockito;
 
 import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.utils.SinkOutput;
-import net.bigeon.gclc.utils.StreamConsoleOutput;
+import net.bigeon.gclc.manager.SinkOutput;
+import net.bigeon.gclc.manager.StreamConsoleOutput;
 
 /** @author Emmanuel Bigeon */
 public class ForwardingRunnableTest {

gclc/pom.xml

@@ -3,12 +3,12 @@
 	<modelVersion>4.0.0</modelVersion>
 	<parent>
 		<groupId>net.bigeon.config</groupId>
-		<artifactId>ebigeon-config</artifactId>
-		<version>1.8.21</version>
+		<artifactId>ebigeon-public-conf</artifactId>
+		<version>1.0.12</version>
 	</parent>
 	<groupId>net.bigeon</groupId>
 	<artifactId>gclc</artifactId>
-	<version>2.0.12</version>
+	<version>2.1.3-SNAPSHOT</version>
 	<packaging>jar</packaging>
 	<name>Generic Command Ligne console</name>
 	<description>A generic framework for console applications, with customized command input and output streams.</description>
@@ -37,7 +37,7 @@
 	</developers>
 	<scm>
 		<developerConnection>scm:git:gogs@git.code.bigeon.net:emmanuel/gclc.git</developerConnection>
-		<tag>gclc-2.0.12</tag>
+		<tag>HEAD</tag>
 	</scm>
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
@@ -49,7 +49,7 @@
 		<dependency>
 			<groupId>net.bigeon.test</groupId>
 			<artifactId>junitmt</artifactId>
-			<version>1.0.2</version>
+			<version>1.0.4</version>
 			<scope>test</scope>
 		</dependency>
 	</dependencies>

gclc/src/main/java/net/bigeon/gclc/ConsoleApplication.java

@@ -95,8 +95,8 @@ import net.bigeon.gclc.manager.ConsoleOutput;
  * A typical use case is the following:
  *
  * <pre>
- * {@link ConsoleOutput} out = new {@link net.bigeon.gclc.utils.StreamConsoleOutput StreamConsoleOutput}();
- * {@link ConsoleInput} in = new {@link net.bigeon.gclc.utils.StreamConsoleInput StreamConsoleInput}();
+ * {@link ConsoleOutput} out = new {@link net.bigeon.gclc.manager.StreamConsoleOutput StreamConsoleOutput}();
+ * {@link ConsoleInput} in = new {@link net.bigeon.gclc.manager.StreamConsoleInput StreamConsoleInput}();
  * {@link ConsoleApplication} app = new {@link ConsoleApplication}(out, in, "welcome", "see you latter")};
  * app.{@link ConsoleApplication#add(ICommand) add}("my_command", new {@link ICommand MyCommand()});
  * app.{@link ConsoleApplication#start() start()};
@@ -191,7 +191,7 @@ public final class ConsoleApplication implements ICommandProvider {
             args = GCLCConstants.splitCommand(cmd);
         } catch (final CommandParsingException e) {
             out.println("Command line cannot be parsed"); //$NON-NLS-1$
-            LOGGER.log(Level.FINE, "Invalid user command " + cmd, e); //$NON-NLS-1$
+            LOGGER.log(Level.FINE, e, () -> "Invalid user command " + cmd); //$NON-NLS-1$
             return;
         }
         if (!args.isEmpty()) {
@@ -199,7 +199,7 @@ public final class ConsoleApplication implements ICommandProvider {
                 executeSub(out, in, args.get(0),
                         Arrays.copyOfRange(args.toArray(new String[0]), 1, args.size()));
             } catch (final CommandRunException e) {
-                LOGGER.log(Level.FINE, "Command failed: " + cmd, e); //$NON-NLS-1$
+                LOGGER.log(Level.FINE, e, () -> "Command failed: " + cmd); //$NON-NLS-1$
                 out.println(Messages.getString("ConsoleApplication.cmd.failed", cmd)); //$NON-NLS-1$
                 out.println(e.getLocalizedMessage());
                 if (e.getType() == CommandRunExceptionType.USAGE) {

gclc/src/main/java/net/bigeon/gclc/GCLCConstants.java

@@ -146,10 +146,7 @@ public final class GCLCConstants {
                 continue;
             }
             if (c == ' ' && !inString) {
-                final String arg = cmd.substring(startIndex, index - 1);
-                if (!arg.isEmpty()) {
-                    args.add(removeEscaped(arg));
-                }
+                addArgument(args, cmd.substring(startIndex, index - 1));
                 startIndex = index;
             } else if (c == '"') {
                 if (inString) {
@@ -164,9 +161,18 @@ public final class GCLCConstants {
         }
         if (startIndex < cmd.length()) {
             final String arg = cmd.substring(startIndex, cmd.length());
-            args.add(arg);
+            args.add(removeEscaped(arg));
         }
         return args;
     }
 
+    /** Add an argument to the list of arguments if it is not an empty string.
+     * 
+     * @param args the list of argument
+     * @param arg the candidate argument */
+    private static void addArgument(final List<String> args, final String arg) {
+        if (!arg.isEmpty()) {
+            args.add(removeEscaped(arg));
+        }
+    }
 }

gclc/src/main/java/net/bigeon/gclc/command/Command.java

@@ -73,8 +73,11 @@ public abstract class Command implements ICommand {
     /** Create the command.
      *
      * @param name the command name */
-    public Command(final String name) {
+    protected Command(final String name) {
         super();
+        if (name == null) {
+            throw new IllegalArgumentException("The command name is mandatory");
+        }
         this.name = name;
     }
 

gclc/src/main/java/net/bigeon/gclc/command/CommandParameters.java

@@ -103,7 +103,7 @@ public final class CommandParameters {
      * @param key the key
      * @return if the key was specified */
     public boolean isActive(final String key) {
-        Boolean val = booleanArguments.get(key);
+        final Boolean val = booleanArguments.get(key);
         return val != null && val.booleanValue();
     }
 
@@ -198,9 +198,8 @@ public final class CommandParameters {
      * @param string the key
      * @param value the value */
     public void set(final String string, final boolean value) {
-        if (booleanArguments.containsKey(string)) {
-            booleanArguments.put(string, Boolean.valueOf(value));
-        }
+        booleanArguments.computeIfPresent(string,
+                (final String k, final Boolean v) -> Boolean.valueOf(value));
     }
 
     /** Set a string parameter value.
@@ -208,6 +207,9 @@ public final class CommandParameters {
      * @param string the key
      * @param value the value */
     public void set(final String string, final String value) {
+        // DO NOT USE computeIfPresent. This is a HashMap, not a ConcurrentHashMap and
+        // keys can be associated to null, but computeIfPresent will consider them as
+        // absent!
         if (stringArguments.containsKey(string)) {
             stringArguments.put(string, value);
         }

gclc/src/main/java/net/bigeon/gclc/command/ParametrizedCommand.java

@@ -94,7 +94,7 @@ public abstract class ParametrizedCommand extends Command {
      * parameters.
      *
      * @param name the name */
-    public ParametrizedCommand(final String name) {
+    protected ParametrizedCommand(final String name) {
         this(name, true);
     }
 
@@ -106,7 +106,7 @@ public abstract class ParametrizedCommand extends Command {
      *
      * @param name the name
      * @param strict if the arguments are restricted to the declared ones */
-    public ParametrizedCommand(final String name, final boolean strict) {
+    protected ParametrizedCommand(final String name, final boolean strict) {
         super(name);
         data = new ParametrizedCommandData(strict);
     }
@@ -116,8 +116,7 @@ public abstract class ParametrizedCommand extends Command {
      * @param flag the boolean flag
      * @throws InvalidParameterException if the parameter is already defined as a
      *         string parameter */
-    protected final void addBooleanParameter(final String flag)
-            throws InvalidParameterException {
+    protected final void addBooleanParameter(final String flag) {
         data.addBooleanParameter(flag);
     }
 
@@ -127,8 +126,7 @@ public abstract class ParametrizedCommand extends Command {
      * @param needed if the parameter's absence should cause an exception
      * @throws InvalidParameterException if the parameter is already defined as a
      *         boolean parameter */
-    protected final void addStringParameter(final String flag, final boolean needed)
-            throws InvalidParameterException {
+    protected final void addStringParameter(final String flag, final boolean needed) {
         data.addStringParameter(flag, needed);
     }
 

gclc/src/main/java/net/bigeon/gclc/command/ParametrizedCommandData.java

@@ -85,7 +85,7 @@ import java.util.concurrent.ConcurrentHashMap;
 import net.bigeon.gclc.exception.CommandParsingException;
 import net.bigeon.gclc.exception.InvalidParameterException;
 import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.utils.EmptyInput;
+import net.bigeon.gclc.manager.EmptyInput;
 
 /** An object to handle standardized command parameters.
  *
@@ -120,7 +120,7 @@ public final class ParametrizedCommandData {
      * @param flag the boolean flag
      * @throws InvalidParameterException if the parameter is already defined as a
      *         string parameter */
-    public void addBooleanParameter(final String flag) throws InvalidParameterException {
+    public void addBooleanParameter(final String flag) {
         if (params.containsKey(flag) && stringParams.containsKey(flag)) {
             throw new InvalidParameterException("Parameter is already defined as string"); //$NON-NLS-1$
         }
@@ -134,8 +134,7 @@ public final class ParametrizedCommandData {
      * @param needed if the parameter's absence should cause an exception
      * @throws InvalidParameterException if the parameter is already defined as a
      *         boolean parameter */
-    public void addStringParameter(final String flag, final boolean needed)
-            throws InvalidParameterException {
+    public void addStringParameter(final String flag, final boolean needed) {
         if (params.containsKey(flag)) {
             checkParam(flag, needed);
             return;
@@ -149,16 +148,17 @@ public final class ParametrizedCommandData {
      * @param param the string parameter
      * @param needed if the parameter is needed
      * @throws InvalidParameterException if the new definition is invalid */
-    private void checkParam(final String param, final boolean needed)
-            throws InvalidParameterException {
-        Boolean val = stringParams.get(param);
-        if (val != null) {
-            final Boolean need = Boolean.valueOf(needed || val.booleanValue());
-            stringParams.put(param, need);
+    private void checkParam(final String param, final boolean needed) {
+        final Boolean val = stringParams.computeIfPresent(param,
+                (final String k, final Boolean v) -> {
+                    final Boolean need = Boolean.valueOf(needed || v.booleanValue());
             params.put(param, need);
-            return;
+                    return need;
+                });
+        if (val == null) {
+            throw new InvalidParameterException(
+                    "Parameter is already defined as boolean"); //$NON-NLS-1$
         }
-        throw new InvalidParameterException("Parameter is already defined as boolean"); //$NON-NLS-1$
     }
 
     /** Retrieve the boolean parameters (aka flags).
@@ -217,7 +217,7 @@ public final class ParametrizedCommandData {
      * @param param the parameter name
      * @return if the parameter is needed */
     public boolean isNeeded(final String param) {
-        Boolean val = params.get(param);
+        final Boolean val = params.get(param);
         return val != null && val.booleanValue();
     }
 

gclc/src/main/java/net/bigeon/gclc/command/base/EchoCommand.java

@@ -34,6 +34,39 @@
  */
 package net.bigeon.gclc.command.base;
 
+/*-
+ * #%L
+ * Generic Command Ligne console
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
 import java.io.IOException;
 import java.text.MessageFormat;
 import java.util.Arrays;

gclc/src/main/java/net/bigeon/gclc/command/base/ScriptExecution.java

@@ -139,7 +139,7 @@ public final class ScriptExecution extends Command {
         try (InputStreamReader fReader = new InputStreamReader(
                 Files.newInputStream(Paths.get(scriptFile)), charset);
                 BufferedReader reader = new BufferedReader(fReader)) {
-            String[] emptyArray = new String[0];
+            final String[] emptyArray = new String[0];
             while ((cmd = reader.readLine()) != null) {
                 lineNo++;
                 final String cmdLine = readCommandLine(cmd, params);
@@ -237,10 +237,16 @@ public final class ScriptExecution extends Command {
      * @return the exception to actually throw */
     private static CommandRunException manageRunException(final CommandRunException e,
             final int lineNo) {
+        final StringBuilder builder = new StringBuilder(e.getLocalizedMessage());
+        int index = builder.indexOf("\n");
+        while (index > 0) {
+            builder.insert(index + 1, "  ");
+            index = builder.indexOf("\n", index + 1);
+        }
         return new CommandRunException(CommandRunExceptionType.EXECUTION,
                 MessageFormat.format(
-                        "The script could not complete due to command failure at line {0} ({1})", //$NON-NLS-1$
-                        Integer.valueOf(lineNo), e.getLocalizedMessage()),
+                        "The script could not complete due to command failure at line {0}\n  {1})", //$NON-NLS-1$
+                        Integer.valueOf(lineNo + 1), builder),
                 e);
     }
 

gclc/src/main/java/net/bigeon/gclc/exception/CommandParsingException.java

@@ -74,7 +74,7 @@ package net.bigeon.gclc.exception;
 /** An exception raised during command parsing.
  *
  * @author Emmanuel Bigeon */
-public class CommandParsingException extends Exception {
+public final class CommandParsingException extends Exception {
 
     /** svuid. */
     private static final long serialVersionUID = 1L;

gclc/src/main/java/net/bigeon/gclc/exception/InvalidCommandName.java

@@ -75,7 +75,7 @@ package net.bigeon.gclc.exception;
  * the command is already used.
  *
  * @author Emmanuel BIGEON */
-public class InvalidCommandName extends Exception {
+public final class InvalidCommandName extends Exception {
 
     /** the SVUID. */
     private static final long serialVersionUID = 1L;

gclc/src/main/java/net/bigeon/gclc/exception/InvalidParameterException.java

@@ -75,7 +75,7 @@ package net.bigeon.gclc.exception;
  * parameter definition.
  *
  * @author Emmanuel Bigeon */
-public class InvalidParameterException extends Exception {
+public final class InvalidParameterException extends RuntimeException {
 
     /** the SVUID. */
     private static final long serialVersionUID = 1L;

gclc/src/main/java/net/bigeon/gclc/i18n/Messages.java

@@ -105,8 +105,8 @@ public final class Messages {
         try {
             return MessageFormat.format(RESOURCE_BUNDLE.getString(key), args);
         } catch (final MissingResourceException e) {
-            LOGGER.log(Level.WARNING,
-                    "Unrecognized internationalization message key: " + key, e); //$NON-NLS-1$
+            LOGGER.log(Level.WARNING, e,
+                    () -> "Unrecognized internationalization message key: " + key); //$NON-NLS-1$
             return '!' + key + '!';
         }
     }

gclc/src/main/java/net/bigeon/gclc/manager/ConsoleInput.java

@@ -72,7 +72,6 @@ package net.bigeon.gclc.manager;
  * #L%
  */
 import java.io.IOException;
-import java.io.InterruptedIOException;
 import java.util.function.Supplier;
 
 /** A console application input.
@@ -94,8 +93,7 @@ public interface ConsoleInput extends AutoCloseable {
     /** Indicate to the input that is should interrompt the prompting, if possible.
      * <p>
      * The pending {@link #prompt()} or {@link #prompt(String)} operations should
-     * return immediatly. However the returned value can be anything (from the
-     * partial prompt content to an empty string or even a null pointer). */
+     * return immediately by throwing an java.io.InterruptedIOException. */
     void interruptPrompt();
 
     /** Test if the input is closed.
@@ -110,7 +108,7 @@ public interface ConsoleInput extends AutoCloseable {
      *
      * @return the user inputed string
      * @throws IOException if the manager is closed or could not read the prompt
-     * @throws InterruptedIOException if the prompt was interrupted */
+     * @throws java.io.InterruptedIOException if the prompt was interrupted */
     String prompt() throws IOException;
 
     /** Prompt the user, with an allotated time to answer.
@@ -118,7 +116,7 @@ public interface ConsoleInput extends AutoCloseable {
      * @param timeout the time to wait in milliseconds
      * @return the user inputed string, null if the timeout was reached
      * @throws IOException if the manager is closed or could not read the prompt
-     * @throws InterruptedIOException if the prompt was interrupted */
+     * @throws java.io.InterruptedIOException if the prompt was interrupted */
     String prompt(long timeout) throws IOException;
 
     /** Prompt the user, with a hint on what is prompted.
@@ -126,7 +124,7 @@ public interface ConsoleInput extends AutoCloseable {
      * @param message the message to prompt the user
      * @return the user inputed string
      * @throws IOException if the manager is closed or could not read the prompt
-     * @throws InterruptedIOException if the prompt was interrupted */
+     * @throws java.io.InterruptedIOException if the prompt was interrupted */
     String prompt(String message) throws IOException;
 
     /** Prompt the user, with a hint on what is prompted and an allotated time to
@@ -136,7 +134,7 @@ public interface ConsoleInput extends AutoCloseable {
      * @param message the message to prompt the user
      * @return the user inputed string, null if the timeout was reached
      * @throws IOException if the manager is closed or could not read the prompt
-     * @throws InterruptedIOException if the prompt was interrupted */
+     * @throws java.io.InterruptedIOException if the prompt was interrupted */
     String prompt(String message, long timeout) throws IOException;
 
     /** Set a prompting prefix.

gclc/src/main/java/net/bigeon/gclc/manager/EmptyInput.java

@@ -36,54 +36,16 @@
  * gclc:net.bigeon.gclc.manager.EmptyInput.java
  * Created on: Nov 13, 2017
  */
-package net.bigeon.gclc.utils;
+package net.bigeon.gclc.manager;
 
 import java.util.function.Supplier;
 
-import net.bigeon.gclc.manager.ConsoleInput;
-
-/*-
- * #%L
- * Generic Command Ligne console
- * %%
- * Copyright (C) 2014 - 2018 Bigeon
- * %%
- * This software is governed by the CeCILL  license under French law and
- * abiding by the rules of distribution of free software.  You can  use,
- * modify and/ or redistribute the software under the terms of the CeCILL
- * license as circulated by CEA, CNRS and INRIA at the following URL
- * "http://www.cecill.info".
- *
- * As a counterpart to the access to the source code and  rights to copy,
- * modify and redistribute granted by the license, users are provided only
- * with a limited warranty  and the software's author,  the holder of the
- * economic rights,  and the successive licensors  have only  limited
- * liability.
- *
- * In this respect, the user's attention is drawn to the risks associated
- * with loading,  using,  modifying and/or developing or reproducing the
- * software by the user in light of its specific status of free software,
- * that may mean  that it is complicated to manipulate,  and  that  also
- * therefore means  that it is reserved for developers  and  experienced
- * professionals having in-depth computer knowledge. Users are therefore
- * encouraged to load and test the software's suitability as regards their
- * requirements in conditions enabling the security of their systems and/or
- * data to be ensured and,  more generally, to use and operate it in the
- * same conditions as regards security.
- *
- * The fact that you are presently reading this means that you have had
- * knowledge of the CeCILL license and that you accept its terms.
- * #L%
- */
-
-import net.bigeon.gclc.tools.ConstantString;
-
 /** A console input that return empty to all prompting.
  *
  * @author Emmanuel Bigeon */
 public final class EmptyInput implements ConsoleInput {
 
-    private static final ConstantString CONSTANT_STRING = new ConstantString("");
+    private static final Supplier<String> CONSTANT_STRING = () -> "";
     /** The empty prompter. */
     public static final ConsoleInput INSTANCE = new EmptyInput();
 

gclc/src/main/java/net/bigeon/gclc/manager/PipedConsoleInput.java

@@ -36,7 +36,7 @@
  * gclc-test:net.bigeon.gclc.test.TestConsoleManager.java
  * Created on: Nov 18, 2016
  */
-package net.bigeon.gclc.utils;
+package net.bigeon.gclc.manager;
 
 /*-
  * #%L
@@ -78,7 +78,7 @@ import java.io.PrintStream;
 import java.nio.charset.StandardCharsets;
 import java.util.function.Supplier;
 
-import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.utils.WritingRunnable;
 
 /** This console input allows to enter commands and retrieve the output as an
  * input.

gclc/src/main/java/net/bigeon/gclc/manager/PipedConsoleOutput.java

@@ -36,7 +36,7 @@
  * gclc-test:net.bigeon.gclc.test.TestConsoleManager.java
  * Created on: Nov 18, 2016
  */
-package net.bigeon.gclc.utils;
+package net.bigeon.gclc.manager;
 
 /*-
  * #%L
@@ -79,7 +79,7 @@ import java.io.PipedOutputStream;
 import java.io.PrintStream;
 import java.nio.charset.StandardCharsets;
 
-import net.bigeon.gclc.manager.ConsoleOutput;
+import net.bigeon.gclc.utils.ReadingRunnable;
 
 /** This console output allows to retrieve the output as an input.
  * <p>
@@ -101,13 +101,14 @@ public final class PipedConsoleOutput implements ConsoleOutput {
     private final PrintStream         outPrint;
     /** The reading thread. */
     private final ReadingRunnable     reading;
+    private final PipedOutputStream   out;
 
     /** Create a manager that will write and read through piped stream.
      *
      * @throws IOException if the piping failed for streams */
     public PipedConsoleOutput() throws IOException {
         commandOutput = new PipedInputStream();
-        final PipedOutputStream out = new PipedOutputStream(commandOutput);
+        out = new PipedOutputStream(commandOutput);
         commandBuffOutput = new BufferedReader(
                 new InputStreamReader(commandOutput, StandardCharsets.UTF_8));
         outPrint = new PrintStream(out, true, UTF_8);
@@ -133,6 +134,7 @@ public final class PipedConsoleOutput implements ConsoleOutput {
         reading.setRunning(false);
         innerManager.close();
         outPrint.close();
+        out.close();
         commandBuffOutput.close();
         commandOutput.close();
     }

gclc/src/main/java/net/bigeon/gclc/manager/SinkOutput.java

@@ -36,9 +36,7 @@
  * gclc:net.bigeon.gclc.manager.SinkOutput.java
  * Created on: Nov 13, 2017
  */
-package net.bigeon.gclc.utils;
-
-import net.bigeon.gclc.manager.ConsoleOutput;
+package net.bigeon.gclc.manager;
 
 /*-
  * #%L

gclc/src/main/java/net/bigeon/gclc/manager/StreamConsoleInput.java

@@ -36,7 +36,7 @@
  * gclc:net.bigeon.gclc.system.SystemConsoleManager.java
  * Created on: Dec 19, 2014
  */
-package net.bigeon.gclc.utils;
+package net.bigeon.gclc.manager;
 
 /*-
  * #%L
@@ -79,8 +79,7 @@ import java.io.PrintStream;
 import java.nio.charset.Charset;
 import java.util.function.Supplier;
 
-import net.bigeon.gclc.manager.ConsoleInput;
-import net.bigeon.gclc.tools.ConstantString;
+import net.bigeon.gclc.utils.ReadingRunnable;
 
 /** A console using the input stream and print stream.
  * <p>
@@ -90,7 +89,7 @@ import net.bigeon.gclc.tools.ConstantString;
 public final class StreamConsoleInput implements ConsoleInput {
 
     /** The default prompt. */
-    public static final Supplier<String> DEFAULT_PROMPT = new ConstantString("> "); //$NON-NLS-1$
+    public static final Supplier<String> DEFAULT_PROMPT = () -> ("> "); //$NON-NLS-1$
 
     /** The command prompt. It can be changed. */
     private Supplier<String> prompt = DEFAULT_PROMPT;
@@ -213,7 +212,7 @@ public final class StreamConsoleInput implements ConsoleInput {
      * @see net.bigeon.gclc.manager.ConsoleInput#setPrompt(java.lang.String) */
     @Override
     public void setPrompt(final String prompt) {
-        this.prompt = new ConstantString(prompt);
+        this.prompt = () -> (prompt);
     }
 
     @Override

gclc/src/main/java/net/bigeon/gclc/manager/StreamConsoleOutput.java

@@ -36,7 +36,7 @@
  * gclc:net.bigeon.gclc.system.SystemConsoleManager.java
  * Created on: Dec 19, 2014
  */
-package net.bigeon.gclc.utils;
+package net.bigeon.gclc.manager;
 
 /*-
  * #%L
@@ -74,8 +74,6 @@ package net.bigeon.gclc.utils;
 import java.io.IOException;
 import java.io.PrintStream;
 
-import net.bigeon.gclc.manager.ConsoleOutput;
-
 /** A console using the input stream and print stream.
  * <p>
  * The default constructor will use the system standart input and output.

gclc/src/main/java/net/bigeon/gclc/tools/AOutputForwardRunnable.java

@@ -36,7 +36,7 @@
  * gclc:net.bigeon.gclc.tools.AOutputForwardRunnable.java
  * Created on: Dec 1, 2016
  */
-package net.bigeon.gclc.utils;
+package net.bigeon.gclc.tools;
 
 /*-
  * #%L
@@ -75,6 +75,8 @@ import java.io.IOException;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
+import net.bigeon.gclc.manager.PipedConsoleOutput;
+
 /** An incomplete implematation used to forward messages from a piped console.
  * <p>
  * This forwarding can be interrupted without closing the piped manager.
@@ -88,14 +90,14 @@ public abstract class AOutputForwardRunnable implements Runnable {
     /** The default timeout (one tenth of second). */
     private static final long        DEFAULT_TIMEOUT = 100;
     /** The manager. */
-    private final PipedConsoleOutput manager;
+    protected final PipedConsoleOutput manager;
     /** The timeout. */
-    private final long               timeout;
+    protected final long               timeout;
 
     /** Create a forwarding runnable.
      *
      * @param manager the manager */
-    public AOutputForwardRunnable(final PipedConsoleOutput manager) {
+    protected AOutputForwardRunnable(final PipedConsoleOutput manager) {
         super();
         this.manager = manager;
         timeout = DEFAULT_TIMEOUT;
@@ -114,7 +116,8 @@ public abstract class AOutputForwardRunnable implements Runnable {
      *
      * @param manager the manager
      * @param timeout the timeout between message requests. */
-    public AOutputForwardRunnable(final PipedConsoleOutput manager, final long timeout) {
+    protected AOutputForwardRunnable(final PipedConsoleOutput manager,
+                                     final long timeout) {
         super();
         this.manager = manager;
         this.timeout = timeout;
@@ -154,8 +157,11 @@ public abstract class AOutputForwardRunnable implements Runnable {
     /** a method to wait some time. */
     protected final void waitASec() {
         try {
+            final long tic = System.currentTimeMillis();
             synchronized (this) {
-                wait(timeout);
+                while (System.currentTimeMillis() - tic < timeout) {
+                    wait(timeout + tic - System.currentTimeMillis());
+                }
             }
         } catch (final InterruptedException e) {
             LOGGER.log(Level.SEVERE, "Interrupted wait", //$NON-NLS-1$

gclc/src/main/java/net/bigeon/gclc/tools/ConstantString.java

@@ -73,7 +73,9 @@ import java.util.function.Supplier;
 
 /** A supplier of string that hold a constant string.
  *
- * @author Emmanuel Bigeon */
+ * @author Emmanuel Bigeon
+ * @deprecated since 2.1.2, use a lambda expression instead */
+@Deprecated
 public class ConstantString implements Supplier<String> {
     private final String string;
 

gclc/src/main/java/net/bigeon/gclc/utils/ReadingRunnable.java

@@ -111,6 +111,7 @@ public final class ReadingRunnable implements Runnable {
     private final Map<String, Object> messageBlocker     = new ConcurrentHashMap<>();
     /** The lock. */
     private final Object              messageBlockerLock = new Object();
+    private boolean                   interrupting       = false;
     /** Create a reading runnable.
      *
      * @param reader the input to read from */
@@ -127,8 +128,11 @@ public final class ReadingRunnable implements Runnable {
      * @throws IOException if the runnable was stopped and no essage was found. */
     private void doWaitMessage(final long timeout) throws IOException {
         try {
+            final long tic = System.currentTimeMillis();
             synchronized (lock) {
-                lock.wait(timeout);
+                while (System.currentTimeMillis() - tic < timeout && messages.isEmpty()) {
+                    lock.wait(timeout);
+                }
             }
         } catch (final InterruptedException e) {
             LOGGER.log(Level.SEVERE, THREAD_INTERRUPTION_EXCEPTION, e);
@@ -146,7 +150,7 @@ public final class ReadingRunnable implements Runnable {
     public String getMessage() throws IOException {
         synchronized (lock) {
             if (!messages.isEmpty()) {
-                return messages.poll();
+                return pollMessage();
             }
             if (!running) {
                 throw new IOException(CLOSED_PIPE);
@@ -155,8 +159,7 @@ public final class ReadingRunnable implements Runnable {
             waitMessage(TIMEOUT);
             LOGGER.log(Level.FINEST, "Polled: {0}", messages.peek()); //$NON-NLS-1$
             waiting = false;
-            notifyMessage(messages.peek());
-            return messages.poll();
+            return pollMessage();
         }
     }
 
@@ -168,7 +171,7 @@ public final class ReadingRunnable implements Runnable {
     public String getNextMessage(final long timeout) throws IOException {
         synchronized (lock) {
             if (!messages.isEmpty()) {
-                return messages.poll();
+                return pollMessage();
             }
             if (!running) {
                 throw new IOException(CLOSED_PIPE);
@@ -179,20 +182,33 @@ public final class ReadingRunnable implements Runnable {
             if (messages.isEmpty()) {
                 return null;
             }
-            return messages.poll();
+            return pollMessage();
         }
     }
 
+    private String pollMessage() throws InterruptedIOException {
+        final String msg = messages.poll();
+        if (msg.isEmpty() && interrupting) {
+            throw new InterruptedIOException();
+        }
+        notifyMessage(msg);
+        return msg;
+    }
+    
+    /** Wait for a message to be delivered.
+     * 
+     * @param message the message
+     * @throws InterruptedException if the wait was interrupted */
     public void waitForDelivery(final String message) throws InterruptedException {
         Object mLock;
         synchronized (messageBlockerLock) {
-            if (!messageBlocker.containsKey(message)) {
-                messageBlocker.put(message, new Object());
-            }
+            messageBlocker.computeIfAbsent(message, (final String k) -> new Object());
             mLock = messageBlocker.get(message);
         }
         synchronized (mLock) {
-            mLock.wait();
+            while (messageBlocker.containsKey(message)) {
+                mLock.wait();
+            }
         }
     }
     
@@ -214,7 +230,8 @@ public final class ReadingRunnable implements Runnable {
     public void interrupt() {
         synchronized (lock) {
             if (waiting) {
-                messages.offer(""); //$NON-NLS-1$
+                messages.offer("");
+                interrupting = true;
                 lock.notifyAll();
             }
         }
@@ -237,9 +254,9 @@ public final class ReadingRunnable implements Runnable {
             final Object mLock = messageBlocker.get(message);
             if (mLock!=null) {
                 synchronized (mLock) {
+                    messageBlocker.remove(message);
                     mLock.notifyAll();
                 }
-                messageBlocker.remove(message);
             }
         }
     }

gclc/src/test/java/net/bigeon/gclc/CommandTestingApplication.java

@@ -75,8 +75,8 @@ import java.io.IOException;
 
 import net.bigeon.gclc.command.ICommand;
 import net.bigeon.gclc.exception.InvalidCommandName;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /**
  * <p>

gclc/src/test/java/net/bigeon/gclc/ConsoleApplicationTest.java

@@ -71,11 +71,7 @@ package net.bigeon.gclc;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 
 import java.io.BufferedReader;
 import java.io.IOException;
@@ -95,8 +91,8 @@ import net.bigeon.gclc.exception.InvalidCommandName;
 import net.bigeon.gclc.i18n.Messages;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /** Test class for ConsoleApplication
  *

gclc/src/test/java/net/bigeon/gclc/GCLCConstantsTest.java

@@ -72,7 +72,6 @@ package net.bigeon.gclc;
  * #L%
  */
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
 import java.util.List;
@@ -87,52 +86,77 @@ import net.bigeon.gclc.exception.CommandParsingException;
 @SuppressWarnings({ "nls", "static-method" })
 public class GCLCConstantsTest {
 
+    @Test
+    public void testSplitCommandSingle() throws CommandParsingException {
+        List<String> res;
+        res = GCLCConstants.splitCommand("aCommand");
+        assertEquals("single word command should have one element", 1, res.size());
+        assertEquals("Command should be preserved", "aCommand", res.get(0));
+    }
+
+    @Test
+    public void testSplitCommandSeveral() throws CommandParsingException {
+        List<String> res;
+        res = GCLCConstants.splitCommand("aCommand with some arguments");
+        assertEquals("Command size", 4, res.size());
+        assertEquals("Elements should be preserved", "aCommand", res.get(0));
+        assertEquals("Command should be preserved", "with", res.get(1));
+        assertEquals("Command should be preserved", "some", res.get(2));
+        assertEquals("Command should be preserved", "arguments", res.get(3));
+    }
+
     /** Test method for
      * {@link net.bigeon.gclc.GCLCConstants#splitCommand(java.lang.String)}.
      *
      * @throws CommandParsingException if an error occured */
     @Test
-    public void testSplitCommand() throws CommandParsingException {
+    public void testSplitCommandMultiSpaces() throws CommandParsingException {
         List<String> res;
-        res = GCLCConstants.splitCommand("aCommand");
-        assertTrue("single word command should have one element", res.size() == 1);
-        assertTrue("Command should be preserved", res.get(0).equals("aCommand"));
-
-        res = GCLCConstants.splitCommand("aCommand with some arguments");
-        assertEquals("Command size", 4, res.size());
-        assertEquals("Elements should be preserved", "aCommand", res.get(0));
-        assertTrue("Elements should be preserved", res.get(1).equals("with"));
-        assertTrue("Elements should be preserved", res.get(2).equals("some"));
-        assertTrue("Elements should be preserved", res.get(3).equals("arguments"));
         res = GCLCConstants.splitCommand("aCommand with  some arguments");
         assertEquals("Command size", 4, res.size());
-        assertTrue("Elements should be preserved", res.get(0).equals("aCommand"));
-        assertTrue("Elements should be preserved", res.get(1).equals("with"));
-        assertTrue("Elements should be preserved", res.get(2).equals("some"));
-        assertTrue("Elements should be preserved", res.get(3).equals("arguments"));
+        assertEquals("Command should be preserved", "aCommand", res.get(0));
+        assertEquals("Command should be preserved", "with", res.get(1));
+        assertEquals("Command should be preserved", "some", res.get(2));
+        assertEquals("Command should be preserved", "arguments", res.get(3));
+    }
+
+    public void testSplitCommandWithString() throws CommandParsingException {
+        List<String> res;
         res = GCLCConstants.splitCommand("aCommand \"with some\" arguments");
         assertEquals("Command size", 3, res.size());
-        assertTrue("Elements should be preserved", res.get(0).equals("aCommand"));
-        assertTrue("Elements should be preserved", res.get(1).equals("with some"));
-        assertTrue("Elements should be preserved", res.get(2).equals("arguments"));
-        res = GCLCConstants.splitCommand("aCommand with\\ some arguments");
-        assertEquals("Command size", 3, res.size());
-        assertTrue("Elements should be preserved", res.get(0).equals("aCommand"));
-        assertTrue("Elements should be preserved", res.get(1).equals("with some"));
-        assertTrue("Elements should be preserved", res.get(2).equals("arguments"));
-        res = GCLCConstants.splitCommand("aCommand wi\\\"th some arguments");
-        assertEquals("Command size", 4, res.size());
-        assertTrue("Elements should be preserved", res.get(0).equals("aCommand"));
-        assertTrue("Elements should be preserved", res.get(1).equals("wi\"th"));
-        assertTrue("Elements should be preserved", res.get(2).equals("some"));
-        assertTrue("Elements should be preserved", res.get(3).equals("arguments"));
+        assertEquals("Command should be preserved", "aCommand", res.get(0));
+        assertEquals("Command should be preserved", "with some", res.get(1));
+        assertEquals("Command should be preserved", "arguments", res.get(2));
 
         res = GCLCConstants.splitCommand("aCommand with \"some arguments\"");
         assertEquals("Command size", 3, res.size());
-        assertTrue("Elements should be preserved", res.get(0).equals("aCommand"));
-        assertTrue("Elements should be preserved", res.get(1).equals("with"));
-        assertTrue("Elements should be preserved", res.get(2).equals("some arguments"));
+        assertEquals("Command should be preserved", "aCommand", res.get(0));
+        assertEquals("Command should be preserved", "with", res.get(1));
+        assertEquals("Command should be preserved", "some arguments", res.get(2));
+    }
 
+    public void testSplitCommandWithEscape() throws CommandParsingException {
+        List<String> res;
+        res = GCLCConstants.splitCommand("aCommand with\\ some arguments");
+        assertEquals("Command size", 3, res.size());
+        assertEquals("Command should be preserved", "aCommand", res.get(0));
+        assertEquals("Command should be preserved", "with some", res.get(1));
+        assertEquals("Command should be preserved", "arguments", res.get(2));
+    }
+
+    public void testSplitCommandWithQuoteInString() throws CommandParsingException {
+        List<String> res;
+        res = GCLCConstants.splitCommand("aCommand wi\\\"th some arguments");
+        assertEquals("Command size", 4, res.size());
+        assertEquals("Command should be preserved", "aCommand", res.get(0));
+        assertEquals("Command should be preserved", "wi\"th", res.get(1));
+        assertEquals("Command should be preserved", "some", res.get(2));
+        assertEquals("Command should be preserved", "arguments", res.get(3));
+
+    }
+
+    public void testSplitCommandInvalidQuoting() throws CommandParsingException {
+        List<String> res;
         try {
             // Wrong lines?
             res = GCLCConstants.splitCommand("aCommand with \"some ar\"guments");

gclc/src/test/java/net/bigeon/gclc/command/CommandParametersTest.java

@@ -71,11 +71,7 @@ package net.bigeon.gclc.command;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 
 import java.util.HashSet;
 import java.util.Set;

gclc/src/test/java/net/bigeon/gclc/command/CommandTest.java

@@ -82,7 +82,7 @@ import org.junit.Test;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /**
  * <p>

gclc/src/test/java/net/bigeon/gclc/command/HelpExecutorTest.java

@@ -83,7 +83,7 @@ import net.bigeon.gclc.command.base.MockCommand;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /**
  * <p>

gclc/src/test/java/net/bigeon/gclc/command/ParametrizedCommandDataTest.java

@@ -1,6 +1,39 @@
 package net.bigeon.gclc.command;
 
-import static org.junit.Assert.*;
+/*-
+ * #%L
+ * Generic Command Ligne console
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+import static org.junit.Assert.fail;
 
 import org.junit.Test;
 

gclc/src/test/java/net/bigeon/gclc/command/ParametrizedCommandTest.java

@@ -71,11 +71,7 @@ package net.bigeon.gclc.command;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 
 import java.io.BufferedReader;
 import java.io.IOException;
@@ -91,12 +87,12 @@ import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.InvalidParameterException;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
-import net.bigeon.test.junitmt.ATestRunnable;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 import net.bigeon.test.junitmt.FunctionalTestRunnable;
-import net.bigeon.test.junitmt.TestFunction;
 import net.bigeon.test.junitmt.ThreadTest;
+import net.bigeon.test.junitmt.api.ATestRunnable;
+import net.bigeon.test.junitmt.api.TestFunction;
 
 /**
  * <p>
@@ -513,21 +509,18 @@ public class ParametrizedCommandTest {
             cmd.execute(out, in, "-" + str1, str2, "-" + addParam);
             cmd.execute(out, in, "-" + str1, str2, "-" + addParam, addParam);
 
-            final TestFunction function = new TestFunction() {
-
-                @Override
-                public void apply() throws Exception {
-                    assertEquals("value of " + str1 + "? ", buf.readLine());
-                    in.type("");
-                    assertEquals("value of " + str1 + "? (cannot be empty) ",
-                            buf.readLine());
-                    in.type("");
-                    assertEquals("value of " + str1 + "? (cannot be empty) ",
-                            buf.readLine());
-                    in.type(str2);
-                }
+            final TestFunction function = () -> {
+                assertEquals("value of " + str1 + "? ", buf.readLine());
+                in.type("");
+                assertEquals("value of " + str1 + "? (cannot be empty) ",
+                        buf.readLine());
+                in.type("");
+                assertEquals("value of " + str1 + "? (cannot be empty) ",
+                        buf.readLine());
+                in.type(str2);
             };
-            final ATestRunnable testConsole = new FunctionalTestRunnable(function);
+            final FunctionalTestRunnable testConsole = new FunctionalTestRunnable(
+                    function);
 
             final Thread th = new Thread(testConsole);
             th.start();
@@ -543,13 +536,9 @@ public class ParametrizedCommandTest {
                         new InputStreamReader(pis, StandardCharsets.UTF_8));
                 PipedConsoleInput in = new PipedConsoleInput(new PrintStream(pout))) {
 
-            final TestFunction function = new TestFunction() {
-
-                @Override
-                public void apply() throws Exception {
-                    assertEquals("value of " + str1 + "? ", buf.readLine());
-                    in.type(str2);
-                }
+            final TestFunction function = () -> {
+                assertEquals("value of " + str1 + "? ", buf.readLine());
+                in.type(str2);
             };
             final ATestRunnable testConsole = new FunctionalTestRunnable(function);
 
@@ -565,7 +554,6 @@ public class ParametrizedCommandTest {
             final PipedConsoleInput test = new PipedConsoleInput(null);
             test.close();
             out.close();
-            cmd.execute(out, test, "-" + str1, str2);
             cmd.execute(out, test, "-" + addParam);
             fail("Closed manager shall cause error");
         } catch (final CommandRunException e) {

gclc/src/test/java/net/bigeon/gclc/command/ScriptExecutionTest.java

@@ -85,8 +85,8 @@ import net.bigeon.gclc.ConsoleTestApplication;
 import net.bigeon.gclc.command.base.ScriptExecution;
 import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.CommandRunExceptionType;
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /**
  * <p>

gclc/src/test/java/net/bigeon/gclc/command/SubedCommandTest.java

@@ -71,10 +71,7 @@ package net.bigeon.gclc.command;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 
 import java.io.IOException;
 
@@ -85,7 +82,7 @@ import net.bigeon.gclc.exception.CommandRunException;
 import net.bigeon.gclc.exception.InvalidCommandName;
 import net.bigeon.gclc.manager.ConsoleInput;
 import net.bigeon.gclc.manager.ConsoleOutput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /**
  * <p>

gclc/src/test/java/net/bigeon/gclc/command/base/EchoCommandTest.java

@@ -1,5 +1,39 @@
 package net.bigeon.gclc.command.base;
 
+/*-
+ * #%L
+ * Generic Command Ligne console
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.fail;
@@ -9,7 +43,7 @@ import java.io.IOException;
 import org.junit.Test;
 
 import net.bigeon.gclc.exception.CommandRunException;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 public class EchoCommandTest {
 

gclc/src/test/java/net/bigeon/gclc/manager/ReadingRunnableTest.java

@@ -80,6 +80,7 @@ import java.io.BufferedWriter;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
+import java.io.InterruptedIOException;
 import java.io.OutputStreamWriter;
 import java.io.PipedInputStream;
 import java.io.PipedOutputStream;
@@ -90,6 +91,9 @@ import org.junit.Before;
 import org.junit.Test;
 
 import net.bigeon.gclc.utils.ReadingRunnable;
+import net.bigeon.test.junitmt.FunctionalTestRunnable;
+import net.bigeon.test.junitmt.ThreadTest;
+import net.bigeon.test.junitmt.api.TestFunction;
 
 /**
  * <p>
@@ -198,9 +202,11 @@ public class ReadingRunnableTest {
     public final void testGetPendingMessages() throws IOException, InterruptedException {
 
         final PipedOutputStream out = new PipedOutputStream();
-        final BufferedReader reader = new BufferedReader(new InputStreamReader(new PipedInputStream(out), StandardCharsets.UTF_8));
+        final BufferedReader reader = new BufferedReader(
+                new InputStreamReader(new PipedInputStream(out), StandardCharsets.UTF_8));
         final ReadingRunnable runnable = new ReadingRunnable(reader);
-        final BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(out, StandardCharsets.UTF_8));
+        final BufferedWriter writer = new BufferedWriter(
+                new OutputStreamWriter(out, StandardCharsets.UTF_8));
         writer.write("one");
         writer.newLine();
         writer.write("two");
@@ -222,4 +228,30 @@ public class ReadingRunnableTest {
             // ok
         }
     }
+
+    @Test
+    public void testInterruption() throws IOException, InterruptedException {
+        final PipedOutputStream out = new PipedOutputStream();
+        final BufferedReader reader = new BufferedReader(
+                new InputStreamReader(new PipedInputStream(out), StandardCharsets.UTF_8));
+        final ReadingRunnable runnable = new ReadingRunnable(reader);
+
+        final FunctionalTestRunnable test = new FunctionalTestRunnable(
+                new TestFunction() {
+                    @Override
+                    public void apply() throws IOException {
+                        try {
+                            runnable.getMessage();
+                            fail("Message interruption should cause an exception");
+                        } catch (final InterruptedIOException e) {
+                            // ok
+                        }
+                    }
+                });
+        final Thread th = new Thread(test, "Prompt wait");
+        th.start();
+        th.join(200);
+        runnable.interrupt();
+        ThreadTest.assertRuns(th, test);
+    }
 }

gclc/src/test/java/net/bigeon/gclc/manager/SystemConsoleManagerTest.java

@@ -71,10 +71,7 @@ package net.bigeon.gclc.manager;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 
 import java.io.IOException;
 import java.io.InputStream;
@@ -85,8 +82,6 @@ import java.nio.charset.Charset;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.utils.StreamConsoleInput;
-
 /**
  * <p>
  * TODO

gclc/src/test/java/net/bigeon/gclc/prompt/CLIPrompterTest.java

@@ -71,10 +71,7 @@ package net.bigeon.gclc.prompt;
  * knowledge of the CeCILL license and that you accept its terms.
  * #L%
  */
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.*;
 
 import java.io.BufferedReader;
 import java.io.IOException;
@@ -92,12 +89,12 @@ import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 
-import net.bigeon.gclc.utils.PipedConsoleInput;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
-import net.bigeon.test.junitmt.ATestRunnable;
+import net.bigeon.gclc.manager.PipedConsoleInput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 import net.bigeon.test.junitmt.FunctionalTestRunnable;
-import net.bigeon.test.junitmt.TestFunction;
 import net.bigeon.test.junitmt.ThreadTest;
+import net.bigeon.test.junitmt.api.ATestRunnable;
+import net.bigeon.test.junitmt.api.TestFunction;
 
 /**
  * <p>

gclc/src/test/java/net/bigeon/gclc/tools/AOutputForwardRunnableTest.java

@@ -77,8 +77,7 @@ import java.io.IOException;
 
 import org.junit.Test;
 
-import net.bigeon.gclc.utils.AOutputForwardRunnable;
-import net.bigeon.gclc.utils.PipedConsoleOutput;
+import net.bigeon.gclc.manager.PipedConsoleOutput;
 
 /**
  * <p>
@@ -143,7 +142,7 @@ public class AOutputForwardRunnableTest {
     }
 
     /** Test method for
-     * {@link net.bigeon.gclc.utils.AOutputForwardRunnable#run()}. */
+     * {@link net.bigeon.gclc.tools.AOutputForwardRunnable#run()}. */
     @Test
     public final void testRun() {
         try (PipedConsoleOutput manager = new PipedConsoleOutput()) {

gclc/src/test/java/net/bigeon/gclc/utils/EmptyInputTest.java

@@ -1,5 +1,39 @@
 package net.bigeon.gclc.utils;
 
+/*-
+ * #%L
+ * Generic Command Ligne console
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 
@@ -8,6 +42,7 @@ import java.io.IOException;
 import org.junit.Test;
 
 import net.bigeon.gclc.manager.ConsoleInput;
+import net.bigeon.gclc.manager.EmptyInput;
 import net.bigeon.gclc.tools.ConstantString;
 
 public class EmptyInputTest {

gclc/src/test/java/net/bigeon/gclc/utils/SinkOutputTest.java

@@ -1,5 +1,39 @@
 package net.bigeon.gclc.utils;
 
+/*-
+ * #%L
+ * Generic Command Ligne console
+ * %%
+ * Copyright (C) 2014 - 2018 Bigeon
+ * %%
+ * This software is governed by the CeCILL  license under French law and
+ * abiding by the rules of distribution of free software.  You can  use,
+ * modify and/ or redistribute the software under the terms of the CeCILL
+ * license as circulated by CEA, CNRS and INRIA at the following URL
+ * "http://www.cecill.info".
+ * 
+ * As a counterpart to the access to the source code and  rights to copy,
+ * modify and redistribute granted by the license, users are provided only
+ * with a limited warranty  and the software's author,  the holder of the
+ * economic rights,  and the successive licensors  have only  limited
+ * liability.
+ * 
+ * In this respect, the user's attention is drawn to the risks associated
+ * with loading,  using,  modifying and/or developing or reproducing the
+ * software by the user in light of its specific status of free software,
+ * that may mean  that it is complicated to manipulate,  and  that  also
+ * therefore means  that it is reserved for developers  and  experienced
+ * professionals having in-depth computer knowledge. Users are therefore
+ * encouraged to load and test the software's suitability as regards their
+ * requirements in conditions enabling the security of their systems and/or
+ * data to be ensured and,  more generally, to use and operate it in the
+ * same conditions as regards security.
+ * 
+ * The fact that you are presently reading this means that you have had
+ * knowledge of the CeCILL license and that you accept its terms.
+ * #L%
+ */
+
 import static org.junit.Assert.assertFalse;
 
 import java.io.IOException;
@@ -7,6 +41,7 @@ import java.io.IOException;
 import org.junit.Test;
 
 import net.bigeon.gclc.manager.ConsoleOutput;
+import net.bigeon.gclc.manager.SinkOutput;
 
 public class SinkOutputTest {