fix: skipping parsing command line in case of empty string or white s…

…pace [hotfix]
STAMP-project · Mar 1, 2019 · a9b14f1 · a9b14f1
1 parent 95bd281
commit a9b14f1
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/src/main/java/eu/stamp_project/testrunner/EntryPoint.java b/src/main/java/eu/stamp_project/testrunner/EntryPoint.java
@@ -183,11 +183,13 @@ public static TestResult runTests(String classpath, String[] fullQualifiedNameOf
                         ParserOptions.FLAG_fullQualifiedNameOfTestClassToRun,
                         String.join(ConstantsHelper.PATH_SEPARATOR, fullQualifiedNameOfTestClasses),
                         methodNames.length == 0 ? ""
-                                : ParserOptions.FLAG_testMethodNamesToRun + ConstantsHelper.WHITE_SPACE
-                                        + String.join(ConstantsHelper.PATH_SEPARATOR, methodNames),
+                                : (ParserOptions.FLAG_testMethodNamesToRun + ConstantsHelper.WHITE_SPACE
+                                        + String.join(ConstantsHelper.PATH_SEPARATOR, methodNames)),
                         EntryPoint.blackList.isEmpty() ? ""
                                 : (ParserOptions.FLAG_blackList + ConstantsHelper.WHITE_SPACE
-                                        + String.join(ConstantsHelper.PATH_SEPARATOR, EntryPoint.blackList)) });
+                                        + String.join(ConstantsHelper.PATH_SEPARATOR, EntryPoint.blackList))
+                }
+        );
         return EntryPoint.runTests(javaCommand);
     }
 
@@ -264,10 +266,11 @@ public static Coverage runCoverage(String classpath, String targetProjectClasses
                         EntryPoint.JACOCO_RUNNER_QUALIFIED_NAME, ParserOptions.FLAG_pathToCompiledClassesOfTheProject,
                         targetProjectClasses, ParserOptions.FLAG_fullQualifiedNameOfTestClassToRun,
                         String.join(ConstantsHelper.PATH_SEPARATOR, fullQualifiedNameOfTestClasses),
-                        methodNames.length == 0 ? "" : ParserOptions.FLAG_testMethodNamesToRun,
-                        String.join(ConstantsHelper.PATH_SEPARATOR, methodNames),
-                        EntryPoint.blackList.isEmpty() ? ""
-                                : (ParserOptions.FLAG_blackList + ConstantsHelper.WHITE_SPACE
+                        methodNames.length == 0 ? "" :
+                                (ParserOptions.FLAG_testMethodNamesToRun + ConstantsHelper.WHITE_SPACE +
+                                        String.join(ConstantsHelper.PATH_SEPARATOR, methodNames)),
+                        EntryPoint.blackList.isEmpty() ? "" :
+                                (ParserOptions.FLAG_blackList + ConstantsHelper.WHITE_SPACE
                                         + String.join(ConstantsHelper.PATH_SEPARATOR, EntryPoint.blackList)),
                         EntryPoint.jUnit5Mode ? ParserOptions.FLAG_isJUnit5 : "" });
         return EntryPoint.runCoverage(javaCommand);
@@ -351,8 +354,8 @@ public static CoveragePerTestMethod runCoveragePerTestMethods(String classpath,
                 EntryPoint.JACOCO_RUNNER_PER_TEST_QUALIFIED_NAME, ParserOptions.FLAG_pathToCompiledClassesOfTheProject,
                 targetProjectClasses, ParserOptions.FLAG_fullQualifiedNameOfTestClassToRun,
                 String.join(ConstantsHelper.PATH_SEPARATOR, fullQualifiedNameOfTestClasses),
-                methodNames.length == 0 ? "" : ParserOptions.FLAG_testMethodNamesToRun,
-                String.join(ConstantsHelper.PATH_SEPARATOR, methodNames),
+                methodNames.length == 0 ? "" : ParserOptions.FLAG_testMethodNamesToRun + ConstantsHelper.WHITE_SPACE +
+                        String.join(ConstantsHelper.PATH_SEPARATOR, methodNames),
                 EntryPoint.blackList.isEmpty() ? ""
                         : (ParserOptions.FLAG_blackList + ConstantsHelper.WHITE_SPACE
                                 + String.join(ConstantsHelper.PATH_SEPARATOR, EntryPoint.blackList)),
@@ -429,7 +432,7 @@ private static void runGivenCommandLine(String commandLine) throws TimeoutExcept
 
     /**
      * This method redirect Process IO to given destination using pipes
-     * 
+     *
      * @param src
      *            process input stream
      * @param dest

diff --git a/src/main/java/eu/stamp_project/testrunner/runner/ParserOptions.java b/src/main/java/eu/stamp_project/testrunner/runner/ParserOptions.java
@@ -40,6 +40,9 @@ public static ParserOptions parse(String[] args) {
                 case FLAG_isJUnit5:
                     parserOptions.isJUnit5 = true;
                     break;
+                case " ":
+                case "":
+                    break;
                 default:
                     System.err.println(String.format("[ERROR]: %s is not a supported command line options", args[i]));
                     usage();

diff --git a/src/main/resources/runner-classes/eu/stamp_project/testrunner/runner/ParserOptions.class b/src/main/resources/runner-classes/eu/stamp_project/testrunner/runner/ParserOptions.class