work for #29

Author: monperrus

pom.xml

@@ -203,7 +203,7 @@
             <plugin>
                 <groupId>org.jacoco</groupId>
                 <artifactId>jacoco-maven-plugin</artifactId>
-                <version>0.8.4</version>
+                <version>0.8.12</version>
                 <executions>
                     <execution>
                         <goals>

src/main/java/fr/inria/spirals/npefix/main/all/Launcher.java

@@ -109,7 +109,7 @@ public void instrument() {
             compiler.compile();
         } catch (Throwable e) {
             e.printStackTrace();
-            throw e;
+            //throw e;
         }
         logger.debug("End code instrumentation");
     }

src/main/java/fr/inria/spirals/npefix/main/run/Main.java

@@ -54,7 +54,7 @@ public static void main(String[] args) throws Exception {
 		} finally {
 
 		}
-		System.exit(0);
+		//System.exit(0);// don't do that, it does not run in a test
 	}
 
 	private RepairStrategy getRepairStrategy() throws Exception {
@@ -101,6 +101,9 @@ private NPEOutput multipleRuns(Launcher  npefix, List<String> npeTests, Selector
 		DecisionServer decisionServer = new DecisionServer(selector);
 		decisionServer.startServer();
 		List<String> testMethods = npefix.getTests(npeTests.toArray(new String[0]));
+		if (testMethods.isEmpty()) {
+			throw new RuntimeException("No test found");
+		}
 
 		NPEOutput output = new NPEOutput();
 
@@ -224,7 +227,7 @@ private void initJSAP() throws JSAPException {
 		testOpt.setShortFlag('t');
 		testOpt.setList(true);
 		testOpt.setStringParser(JSAP.STRING_PARSER);
-		testOpt.setHelp("Define the tests of the project (both failing and passing), fully-qualified, separated with ':' (even if the classpath contains other tests, only those are considered.");
+		testOpt.setHelp("Define the test classes of the project to take into account (both failing and passing), fully-qualified class names, separated with ':' (even if the classpath contains other tests, only those are considered.");
 		jsap.registerParameter(testOpt);
 
 		FlaggedOption complianceLevelOpt = new FlaggedOption("complianceLevel");

src/test/java/fr/inria/spirals/npefix/main/all/End2EndTest.java

@@ -0,0 +1,25 @@
+package fr.inria.spirals.npefix.main.all;
+
+import org.junit.Ignore;
+import org.junit.Test;
+
+public class End2EndTest {
+	@Test
+	@Ignore
+	public void test() throws Exception {
+		// test end 2 end for helping @Peanu11 in https://github.com/SpoonLabs/npefix/issues/29
+
+		// execute command ls
+		Runtime.getRuntime().exec("rm -rf npefix-src");
+
+		fr.inria.spirals.npefix.main.run.Main.main(new String[] { "-s"
+				, "../npe-dataset/pdfbox_2965/src/"
+				, "-c"
+				, "../npe-dataset/pdfbox_2965/target/classes/:../npe-dataset/pdfbox_2965/target/test-classes/:/home/martin/.m2/repository/org/apache/pdfbox/fontbox/2.0.0-RC1/fontbox-2.0.0-RC1.jar:/home/martin/.m2/repository/commons-logging/commons-logging/1.2/commons-logging-1.2.jar:/home/martin/.m2/repository/org/bouncycastle/bcpkix-jdk15on/1.50/bcpkix-jdk15on-1.50.jar:/home/martin/.m2/repository/org/bouncycastle/bcprov-jdk15on/1.50/bcprov-jdk15on-1.50.jar:/home/martin/.m2/repository/junit/junit/4.12/junit-4.12.jar:/home/martin/.m2/repository/org/hamcrest/hamcrest-core/1.3/hamcrest-core-1.3.jar"
+				, "-t"
+				, "org.apache.pdfbox.pdmodel.interactive.form.PDAcroFormTest"
+		});
+
+
+	}
+}