From 1f87548c0686d823e5f1490a2053b70bd9f5022e Mon Sep 17 00:00:00 2001
From: Alex Anderson <anderson@medicmobile.org>
Date: Tue, 26 Apr 2016 23:38:14 +0200
Subject: [PATCH] Print reports to stdout

---
 android-check-plugin/build.gradle             |  2 +-
 .../android/check/common/CommonCheck.groovy   | 12 +++++++
 .../android/check/common/CommonConfig.groovy  | 12 +++++++
 .../checkstyle/checkstyle.stdout.xsl          | 35 +++++++++++++++++++
 .../resources/findbugs/findbugs.stdout.xsl    | 34 ++++++++++++++++++
 .../src/main/resources/pmd/pmd.stdout.xsl     | 35 +++++++++++++++++++
 6 files changed, 129 insertions(+), 1 deletion(-)
 create mode 100644 android-check-plugin/src/main/resources/checkstyle/checkstyle.stdout.xsl
 create mode 100644 android-check-plugin/src/main/resources/findbugs/findbugs.stdout.xsl
 create mode 100644 android-check-plugin/src/main/resources/pmd/pmd.stdout.xsl

diff --git a/android-check-plugin/build.gradle b/android-check-plugin/build.gradle
index 47ea37e..9e25ed2 100644
--- a/android-check-plugin/build.gradle
+++ b/android-check-plugin/build.gradle
@@ -19,7 +19,7 @@ dependencies {
 }
 
 group = 'com.noveogroup.android'
-version = '1.2.3'
+version = '1.3.0-SNAPSHOT'
 description = 'Static code analysis plugin for Android project.'
 
 task wrapper(type: Wrapper) {
diff --git a/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonCheck.groovy b/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonCheck.groovy
index 4784666..2c179c0 100644
--- a/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonCheck.groovy
+++ b/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonCheck.groovy
@@ -60,6 +60,16 @@ abstract class CommonCheck<Config extends CommonConfig> {
         }
     }
 
+    protected void reformatReport(Project project, File styleFile,
+                                  File xmlReportFile) {
+        def temp = File.createTempFile('android-check', '.txt')
+        project.ant.xslt(in: xmlReportFile, out: temp) {
+            style { string(styleFile.text) }
+        }
+        System.out << temp.text
+        System.out.flush()
+    }
+
     void apply(Project target) {
         target.task(
                 [group      : 'verification',
@@ -72,6 +82,7 @@ abstract class CommonCheck<Config extends CommonConfig> {
             boolean abortOnError = config.resolveAbortOnError(extension.abortOnError)
             File configFile = config.resolveConfigFile(taskCode)
             File styleFile = config.resolveStyleFile(taskCode)
+            File stdoutStyleFile = config.resolveStdoutStyleFile(taskCode)
             File xmlReportFile = config.resolveXmlReportFile(taskCode)
             File htmlReportFile = config.resolveHtmlReportFile(taskCode)
             List<File> sources = config.getAndroidSources()
@@ -83,6 +94,7 @@ abstract class CommonCheck<Config extends CommonConfig> {
                 performCheck(target, sources, configFile, xmlReportFile)
                 htmlReportFile.parentFile.mkdirs()
                 reformatReport(target, styleFile, xmlReportFile, htmlReportFile)
+                reformatReport(target, stdoutStyleFile, xmlReportFile)
 
                 int errorCount = getErrorCount(xmlReportFile)
                 if (errorCount) {
diff --git a/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonConfig.groovy b/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonConfig.groovy
index 86acc0d..03ccca5 100644
--- a/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonConfig.groovy
+++ b/android-check-plugin/src/main/groovy/com/noveogroup/android/check/common/CommonConfig.groovy
@@ -138,6 +138,10 @@ class CommonConfig {
         return Utils.getResource(project, "$code/${code}.xsl")
     }
 
+    private String resolveStdoutStyle(String code) {
+        return Utils.getResource(project, "$code/${code}.stdout.xsl")
+    }
+
     File resolveStyleFile(String code) {
         File file = new File(project.buildDir, "tmp/android-check/${code}.xsl")
         file.parentFile.mkdirs()
@@ -146,6 +150,14 @@ class CommonConfig {
         return file
     }
 
+    File resolveStdoutStyleFile(String code) {
+        File file = new File(project.buildDir, "tmp/android-check/${code}.stdout.xsl")
+        file.parentFile.mkdirs()
+        file.delete()
+        file << resolveStdoutStyle(code)
+        return file
+    }
+
     private File resolveReportFile(String extension, File reportFile, File reportDirectory, String code) {
         if (reportFile) {
             return reportFile
diff --git a/android-check-plugin/src/main/resources/checkstyle/checkstyle.stdout.xsl b/android-check-plugin/src/main/resources/checkstyle/checkstyle.stdout.xsl
new file mode 100644
index 0000000..b2293e1
--- /dev/null
+++ b/android-check-plugin/src/main/resources/checkstyle/checkstyle.stdout.xsl
@@ -0,0 +1,35 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+
+<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0">
+
+    <xsl:output indent="yes" method="text" />
+    <xsl:decimal-format decimal-separator="." grouping-separator="," />
+
+    <xsl:key name="files" match="file" use="@name" />
+
+    <xsl:template match="checkstyle">
+    Checkstyle: <xsl:apply-templates mode="summary" select="." />
+        Files: <xsl:apply-templates select="file[@name and generate-id(.) = generate-id(key('files', @name))]" />
+
+    </xsl:template>
+
+    <xsl:template match="file">
+        <xsl:variable name="errorCount" select="count(error)" />
+            - <xsl:value-of select="@name" />
+                Errors: <xsl:value-of select="$errorCount" />
+                <xsl:for-each select="key('files', @name)/error">
+                    <xsl:sort data-type="number" order="ascending" select="@line" />
+                    Error Description: <xsl:value-of select="@message" />
+                        Line: <xsl:value-of select="@line" />
+                </xsl:for-each>
+    </xsl:template>
+
+    <xsl:template match="checkstyle" mode="summary">
+        <xsl:variable name="fileCount" select="count(file[@name and generate-id(.) = generate-id(key('files', @name))])" />
+        <xsl:variable name="errorCount" select="count(file/error)" />
+        Summary:
+            Files: <xsl:value-of select="$fileCount" />
+            Errors: <xsl:value-of select="$errorCount" />
+    </xsl:template>
+
+</xsl:stylesheet>
diff --git a/android-check-plugin/src/main/resources/findbugs/findbugs.stdout.xsl b/android-check-plugin/src/main/resources/findbugs/findbugs.stdout.xsl
new file mode 100644
index 0000000..21a5289
--- /dev/null
+++ b/android-check-plugin/src/main/resources/findbugs/findbugs.stdout.xsl
@@ -0,0 +1,34 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+
+<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0">
+
+    <xsl:output indent="yes" method="text" />
+    <xsl:decimal-format decimal-separator="." grouping-separator="," />
+
+    <xsl:template match="BugCollection">
+    Findbugs: <xsl:apply-templates mode="summary" select="." />
+        Files: <xsl:apply-templates select="FindBugsSummary/FileStats[@path and @bugCount]" />
+
+    </xsl:template>
+
+    <xsl:template match="FileStats">
+        <xsl:variable name="path" select="@path" />
+            - <xsl:value-of select="$path" />
+                Errors: <xsl:value-of select="@bugCount" />
+            <xsl:for-each select="//BugInstance[SourceLine/@sourcepath = $path]">
+                <xsl:sort data-type="number" order="ascending" select="SourceLine/@start" />
+                    Error Description: <xsl:value-of select="@type" />
+                        Line: <xsl:value-of select="LongMessage/text()" />
+                        <xsl:value-of select="SourceLine/@start" /> - <xsl:value-of select="SourceLine/@end" />
+            </xsl:for-each>
+    </xsl:template>
+
+    <xsl:template match="BugCollection" mode="summary">
+        <xsl:variable name="fileCount" select="count(FindBugsSummary/FileStats)" />
+        <xsl:variable name="errorCount" select="count(BugInstance)" />
+        Summary:
+            Files: <xsl:value-of select="$fileCount" />
+            Errors: <xsl:value-of select="$errorCount" />
+    </xsl:template>
+
+</xsl:stylesheet>
diff --git a/android-check-plugin/src/main/resources/pmd/pmd.stdout.xsl b/android-check-plugin/src/main/resources/pmd/pmd.stdout.xsl
new file mode 100644
index 0000000..cc7280a
--- /dev/null
+++ b/android-check-plugin/src/main/resources/pmd/pmd.stdout.xsl
@@ -0,0 +1,35 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+
+<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0">
+
+    <xsl:output indent="yes" method="text" />
+    <xsl:decimal-format decimal-separator="." grouping-separator="," />
+
+    <xsl:key name="files" match="file" use="@name" />
+
+    <xsl:template match="pmd">
+    PMD: <xsl:apply-templates mode="summary" select="." />
+        Files: <xsl:apply-templates select="file[@name and generate-id(.) = generate-id(key('files', @name))]" />
+
+    </xsl:template>
+
+    <xsl:template match="file">
+        <xsl:variable name="violationCount" select="count(violation)" />
+        - <xsl:value-of select="@name" />
+            Violations: <xsl:value-of select="$violationCount" />
+            <xsl:for-each select="key('files', @name)/violation">
+                <xsl:sort data-type="number" order="ascending" select="@beginline" />
+                Violation: <xsl:value-of select="normalize-space(node())" />
+                    Location: <xsl:value-of select="@beginline" />:<xsl:value-of select="@begincolumn" /> - <xsl:value-of select="@endline" />:<xsl:value-of select="@endcolumn" />
+            </xsl:for-each>
+    </xsl:template>
+
+    <xsl:template match="pmd" mode="summary">
+        <xsl:variable name="fileCount" select="count(file[@name and generate-id(.) = generate-id(key('files', @name))])" />
+        <xsl:variable name="violationCount" select="count(file/violation)" />
+        Summary:
+            Files: <xsl:value-of select="$fileCount" />
+            Violations: <xsl:value-of select="$violationCount" />
+    </xsl:template>
+
+</xsl:stylesheet>