Ugly hack to populate IT and Overall coverage

Author: mjdetullio

src/main/java/org/sonar/plugins/objectivec/cobertura/CoberturaReportParser.java

@@ -26,10 +26,12 @@
 import org.codehaus.staxmate.in.SMInputCursor;
 import org.sonar.api.batch.SensorContext;
 import org.sonar.api.batch.fs.FileSystem;
+import org.sonar.api.measures.CoreMetrics;
 import org.sonar.api.measures.CoverageMeasuresBuilder;
 import org.sonar.api.measures.Measure;
 import org.sonar.api.resources.Project;
 import org.sonar.api.resources.Resource;
+import org.sonar.api.utils.KeyValueFormat;
 import org.sonar.api.utils.ParsingUtils;
 import org.sonar.api.utils.StaxParser;
 import org.sonar.api.utils.XmlParserException;
@@ -39,6 +41,7 @@
 import java.text.ParseException;
 import java.util.Locale;
 import java.util.Map;
+import java.util.SortedMap;
 
 final class CoberturaReportParser {
     private final FileSystem fileSystem;
@@ -82,8 +85,11 @@ private void collectPackageMeasures(SMInputCursor pack) throws XMLStreamExceptio
                 String filePath = entry.getKey();
                 Resource resource = org.sonar.api.resources.File.fromIOFile(new File(fileSystem.baseDir(), filePath), project);
                 if (resourceExists(resource)) {
-                    for (Measure measure : entry.getValue().createMeasures()) {
+                    CoverageMeasuresBuilder builder = entry.getValue();
+                    for (Measure measure : builder.createMeasures()) {
                         context.saveMeasure(resource, measure);
+                        context.saveMeasure(resource, zeroForIt(builder, measure));
+                        context.saveMeasure(resource, convertForOverall(measure));
                     }
                 }
             }
@@ -126,4 +132,48 @@ private static void collectFileData(SMInputCursor clazz,
             }
         }
     }
+
+    private static Measure zeroForIt(CoverageMeasuresBuilder builder, Measure measure) {
+        Measure itMeasure = null;
+        if (CoreMetrics.LINES_TO_COVER.equals(measure.getMetric())) {
+            itMeasure = new Measure(CoreMetrics.IT_LINES_TO_COVER, measure.getValue());
+        } else if (CoreMetrics.UNCOVERED_LINES.equals(measure.getMetric())) {
+            itMeasure = new Measure(CoreMetrics.IT_UNCOVERED_LINES, (double) builder.getLinesToCover());
+        } else if (CoreMetrics.COVERAGE_LINE_HITS_DATA.equals(measure.getMetric())) {
+            SortedMap<Integer, Integer> map = Maps.newTreeMap();
+            for (Integer line : builder.getHitsByLine().keySet()) {
+                map.put(line, 0);
+            }
+            itMeasure = new Measure(CoreMetrics.IT_COVERAGE_LINE_HITS_DATA, KeyValueFormat.format(map));
+        } else if (CoreMetrics.CONDITIONS_TO_COVER.equals(measure.getMetric())) {
+            itMeasure = new Measure(CoreMetrics.IT_CONDITIONS_TO_COVER, measure.getValue()); // should be 0
+        } else if (CoreMetrics.UNCOVERED_CONDITIONS.equals(measure.getMetric())) {
+            itMeasure = new Measure(CoreMetrics.IT_UNCOVERED_CONDITIONS, measure.getValue()); // should be 0
+        } else if (CoreMetrics.COVERED_CONDITIONS_BY_LINE.equals(measure.getMetric())) {
+            itMeasure = new Measure(CoreMetrics.IT_COVERED_CONDITIONS_BY_LINE, measure.getData()); // should be 0
+        } else if (CoreMetrics.CONDITIONS_BY_LINE.equals(measure.getMetric())) {
+            itMeasure = new Measure(CoreMetrics.IT_CONDITIONS_BY_LINE, measure.getData()); // should be 0
+        }
+        return itMeasure;
+    }
+
+    private static Measure convertForOverall(Measure measure) {
+        Measure overallMeasure = null;
+        if (CoreMetrics.LINES_TO_COVER.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_LINES_TO_COVER, measure.getValue());
+        } else if (CoreMetrics.UNCOVERED_LINES.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_UNCOVERED_LINES, measure.getValue());
+        } else if (CoreMetrics.COVERAGE_LINE_HITS_DATA.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_COVERAGE_LINE_HITS_DATA, measure.getData());
+        } else if (CoreMetrics.CONDITIONS_TO_COVER.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_CONDITIONS_TO_COVER, measure.getValue());
+        } else if (CoreMetrics.UNCOVERED_CONDITIONS.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_UNCOVERED_CONDITIONS, measure.getValue());
+        } else if (CoreMetrics.COVERED_CONDITIONS_BY_LINE.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_COVERED_CONDITIONS_BY_LINE, measure.getData());
+        } else if (CoreMetrics.CONDITIONS_BY_LINE.equals(measure.getMetric())) {
+            overallMeasure = new Measure(CoreMetrics.OVERALL_CONDITIONS_BY_LINE, measure.getData());
+        }
+        return overallMeasure;
+    }
 }

src/main/java/org/sonar/plugins/objectivec/cobertura/CoberturaSensor.java

@@ -28,6 +28,7 @@
 import org.sonar.api.batch.SensorContext;
 import org.sonar.api.batch.fs.FileSystem;
 import org.sonar.api.config.Settings;
+import org.sonar.api.measures.MeasuresFilter;
 import org.sonar.api.resources.Project;
 import org.sonar.api.scan.filesystem.PathResolver;