npm-groovy-lint 10.0.2 → 10.0.3

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## UNRELEASED
 
+## [10.0.3] 2022-08-15
+
+- Do not output results summary in console logs when output is json or sarif
+- Add test methods for SARIF called by CLI
+
 ## [10.0.2] 2022-08-15
 
 - Fix error when absolute files sent as positional arguments on a linux system ([#232](https://github.com/nvuillam/npm-groovy-lint/issues/232))

package/lib/groovy-lint.js

@@ -459,12 +459,12 @@ class NpmGroovyLint {
             this.options.failon && this.options.failon !== "none"
                 ? this.options.failon
                 : this.options.failonerror
-                ? "error"
-                : this.options.failonwarning
-                ? "warning"
-                : this.options.failoninfo
-                ? "info"
-                : "none";
+                    ? "error"
+                    : this.options.failonwarning
+                        ? "warning"
+                        : this.options.failoninfo
+                            ? "info"
+                            : "none";
         if (failureLevel === "none") {
             return;
         }
@@ -475,21 +475,27 @@ class NpmGroovyLint {
 
         // Fail on error
         if (failureLevel === "error" && errorNb > 0) {
-            console.error(`Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found`);
+            if (!["json", "sarif"].includes(this.outputType)) {
+                console.error(`Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found`);
+            }
             this.status = 1;
         }
         // Fail on warning
         else if (failureLevel === "warning" && (errorNb > 0 || warningNb > 0)) {
-            console.error(
-                `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found`
-            );
+            if (!["json", "sarif"].includes(this.outputType)) {
+                console.error(
+                    `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found`
+                );
+            }
             this.status = 1;
         }
         // Fail on info
         else if (failureLevel === "info" && (errorNb > 0 || warningNb > 0 || infoNb > 0)) {
-            console.error(
-                `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found \n ${this.lintResult.summary.totalFoundInfoNumber} info(s) have been found`
-            );
+            if (!["json", "sarif"].includes(this.outputType)) {
+                console.error(
+                    `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found \n ${this.lintResult.summary.totalFoundInfoNumber} info(s) have been found`
+                );
+            }
             this.status = 1;
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-groovy-lint",
-  "version": "10.0.2",
+  "version": "10.0.3",
   "description": "Lint, format and auto-fix your Groovy / Jenkinsfile / Gradle files",
   "main": "index.js",
   "scripts": {