Refactor RunCommandTest to cover new approach

No need for RunActionTest anymore

Author: pesse

src/test/java/org/utplsql/cli/PicocliRunCommandTest.java

@@ -16,13 +16,7 @@
 public class PicocliRunCommandTest {
 
     private RunCommandConfig parseForConfig( String... args ) throws Exception {
-        Object obj = new UtplsqlPicocliCommand();
-        CommandLine cline = new CommandLine(obj);
-        cline.setTrimQuotes(true);
-        List<CommandLine> parsed = cline.parse(args);
-
-        RunPicocliCommand runCmd = parsed.get(1).getCommand();
-        return runCmd.getRunCommandConfig();
+        return TestHelper.parseRunConfig(args);
     }
 
     @Test

src/test/java/org/utplsql/cli/RunActionTest.java

@@ -81,7 +81,7 @@ void reporterOptions_TwoReporters() {
         assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER.name(), reporterOptions1.getReporterName());
         assertNull(reporterOptions1.getOutputFileName());
         assertFalse(reporterOptions1.outputToFile());
-        assertTrue(reporterOptions1.outputToScreen());
+        assertFalse(reporterOptions1.outputToScreen());
 
         ReporterOptions reporterOptions2 = reporterOptionsList.get(1);
         assertEquals(CoreReporters.UT_COVERAGE_HTML_REPORTER.name(), reporterOptions2.getReporterName());
@@ -91,7 +91,7 @@ void reporterOptions_TwoReporters() {
     }
 
     @Test
-    void randomOrder_default() {
+    void randomOrder_default() throws Exception {
         IRunCommand runCmd = TestHelper.createRunCommand(TestHelper.getConnectionString());
 
         TestRunnerOptions options = runCmd.newTestRunner(new ArrayList<>()).getOptions();
@@ -102,7 +102,7 @@ void randomOrder_default() {
     @Test
     void randomOrder_withoutSeed() {
         IRunCommand runCmd = TestHelper.createRunCommand(TestHelper.getConnectionString(),
-                "-random");
+                "-r");
 
         TestRunnerOptions options = runCmd.newTestRunner(new ArrayList<>()).getOptions();
         assertThat(options.randomTestOrder, equalTo(true));