Jean-Francois Denise created WFCORE-2985: -------------------------------------------- Summary: ProcessWrapper.waitForClose shouldn't close the output stream Key: WFCORE-2985 URL: https://issues.jboss.org/browse/WFCORE-2985 Project: WildFly Core Issue Type: Bug Components: CLI, Test Suite Reporter: Jean-Francois Denise Assignee: Jean-Francois Denise When the output stream is closed, if aesh is reading on the other side of the stream inside the CLI process, this could make the CLI process to exit. That is occurring systematically with aesh-readline. ProcessWrapper.waitForClose is called in the following cases: - ctrlCAndWaitForClose: The test expects that Ctrl-C will make the process to exit. - pushLineAndWaitForClose: The test expects that the passed command will make the process to exit (e.g.: quit) - executeNonInteractive: The test expects that the CLI process will exit by itself, no aesh console started (file or command options). In this case, closing the output stream shouldn't hide any issue. But it shouldn't be required either. If closing the output stream makes the CLI process to exit it could hide potential CLI issues. I suggest that we remove the output stream.close once we have switched to use aesh-readline and see on all platforms how tests are going to behave. We could have exotic platforms that don't make the process to exit properly if output is not closed. We will have to handle them specificaly. -- This message was sent by Atlassian JIRA (v7.2.3#72005)