More CTestResultParser robustification (PR #98).

Author: mpartel

tmc-plugin/src/fi/helsinki/cs/tmc/data/serialization/cresultparser/CTestResultParser.java

@@ -136,11 +136,12 @@ private void addValgrindOutput() throws FileNotFoundException {
             outputs[i] = "";
         }
 
-        Pattern errorPattern = Pattern.compile("ERROR SUMMARY: ([0-9]+)");
+        Pattern errorPattern = Pattern.compile("==[0-9]+== ERROR SUMMARY: ([0-9]+)");
 
         String line = scanner.nextLine();
         int firstPID = parsePID(line);
         parentOutput += "\n" + line;
+        boolean warningLogged = false;
         while (scanner.hasNextLine()) {
             line = scanner.nextLine();
             int pid = parsePID(line);
@@ -150,10 +151,18 @@ private void addValgrindOutput() throws FileNotFoundException {
             if (pid == firstPID) {
                 parentOutput += "\n" + line;
             } else {
-                outputs[findIndex(pid, pids)] += "\n" + line;
+                int outputIndex = findIndex(pid, pids);
+                if (outputIndex == -1) {
+                    if (!warningLogged) {
+                        log.warning("Valgrind output has more PIDs than the expected (# of test cases + 1).");
+                        warningLogged = true;
+                    }
+                    continue;
+                }
+                outputs[outputIndex] += "\n" + line;
                 Matcher m = errorPattern.matcher(line);
                 if (m.find()) {
-                    errors[findIndex(pid, pids)] = Integer.parseInt(m.group(1));
+                    errors[outputIndex] = Integer.parseInt(m.group(1));
                 }
             }
         }
@@ -162,7 +171,7 @@ private void addValgrindOutput() throws FileNotFoundException {
         for (int i = 0; i < outputs.length; i++) {
             if (errors[i] == 0) {
                 // Workaround for a bug where any valgrind output is considered a potential error.
-                outputs[i] = "";
+                outputs[i] = null;
             }
             tests.get(i).setValgrindTrace(outputs[i]);
         }
@@ -178,7 +187,7 @@ private int findIndex(int pid, int[] pids) {
                 return i;
             }
         }
-        return 0;
+        return -1;
     }
 
     private int parsePID(String line) {