npm-groovy-lint 8.2.0 → 9.3.0

package/lib/example/SampleFile.groovy

@@ -2,14 +2,6 @@
 ///////////////////////////////////////////////// COMMON LIBRARY /////////////////////////////////////////
 //////////////////////////////////////////////////////////////////////////////////////////////////////////
 
-@Grapes([
-  @Grab(group='org.codehaus.groovy.modules.http-builder', module='http-builder', version='0.7.1'),
-  @Grab(group='com.google.guava', module='guava', version='19.0'),
-  @Grab(group='org.apache.commons', module='commons-lang3', version='3.7')
-])
-
-import com.google.common.collect.Lists
-
 import groovy.io.FileType
 import groovy.json.JsonSlurper
 import groovy.time.TimeCategory 

package/lib/example/SampleFileSmall.groovy

@@ -1,12 +1,6 @@
 import groovy.io.FileType
 import groovy.json.*
 import groovy.time.TimeCategory
-import static groovyx2.gpars.GParsPool.withPool
-
-        def script = new GroovyScriptEngine( '.' ).with{
-                loadScriptByName( 'Utils.groovy' ) ;
-   }
-this.metaClass.mixin script
 
 def returnCode = 0
          Exception eThrow = null ;

package/lib/example/SampleFileSmallFixed.txt

@@ -1,11 +1,6 @@
 import groovy.json.*
 import java.security.SecureRandom
 
-def script = new GroovyScriptEngine( '.' ).with {
-    loadScriptByName( 'Utils.groovy' )
-}
-this.metaClass.mixin script
-
 def returnCode = 0
 Exception eThrow = null
 try {

package/lib/example/SampleFileSmallFormatted.txt

@@ -1,12 +1,6 @@
 import groovy.io.FileType
 import groovy.json.*
 import groovy.time.TimeCategory
-import static groovyx2.gpars.GParsPool.withPool
-
-def script = new GroovyScriptEngine( '.' ).with {
-    loadScriptByName( 'Utils.groovy' )
-}
-this.metaClass.mixin script
 
 def returnCode = 0
 Exception eThrow = null

package/lib/groovy-lint-fix.js

@@ -1,5 +1,3 @@
-#! /usr/bin/env node
-
 // Imports
 const fse = require("fs-extra");
 const cliProgress = require("cli-progress");

package/lib/groovy-lint.js

@@ -1,5 +1,3 @@
-#! /usr/bin/env node
-
 // Imports
 const debug = require("debug")("npm-groovy-lint");
 const os = require("os");

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "npm-groovy-lint",
-  "version": "8.2.0",
+  "version": "9.3.0",
   "description": "Lint, format and auto-fix your Groovy / Jenkinsfile / Gradle files",
   "main": "index.js",
   "scripts": {
     "lint:fix": "eslint **/*.js --fix && prettier --write \"./lib/**/*.{js,jsx,json}\" --tab-width 4 --print-width 150",
-    "groovy:run-server-from-source": "npm run dev:kill-server && groovy -cp \"lib/java/CodeNarc-2.0.0.jar;lib/java/groovy/lib/groovy-3.0.5.jar;lib/java/groovy/lib/groovy-templates-3.0.5.jar;lib/java/groovy/lib/groovy-xml-3.0.5.jar;lib/java/groovy/lib/groovy-json-3.0.5.jar;lib/java/groovy/lib/groovy-ant-3.0.5.jar;lib/java/groovy/lib/ant-1.10.8.jar;lib/java/groovy/lib/ant-launcher-1.10.8.jar;lib/java/slf4j-api-1.7.9.jar;lib/java/log4j-slf4j-impl-2.13.0.jar;lib/java/log4j-api-2.13.0.jar;lib/java/log4j-core-2.13.0.jar;lib/java/gmetrics-1.1.jar\" groovy/src/main/com/nvuillam/CodeNarcServer.groovy --server",
+    "groovy:run-server-from-source": "npm run dev:kill-server && groovy -cp \"lib/java/CodeNarc-2.2.0.jar;lib/java/groovy/lib/groovy-3.0.9.jar;lib/java/groovy/lib/groovy-templates-3.0.9.jar;lib/java/groovy/lib/groovy-xml-3.0.9.jar;lib/java/groovy/lib/groovy-json-3.0.9.jar;lib/java/groovy/lib/groovy-ant-3.0.9.jar;lib/java/groovy/lib/ant-1.10.11.jar;lib/java/groovy/lib/ant-launcher-1.10.11.jar;lib/java/slf4j-api-1.7.9.jar;lib/java/log4j-slf4j-impl-2.17.1.jar;lib/java/log4j-api-2.17.1.jar;lib/java/log4j-core-2.17.1.jar;lib/java/gmetrics-1.1.jar\" groovy/src/main/com/nvuillam/CodeNarcServer.groovy --server",
     "groovy:build": "npm run dev:kill-server && groovyc -cp \"./lib/java*\" --encoding utf-8 ./groovy/src/main/com/nvuillam/CodeNarcServer.groovy -d ./tmp && cd ./tmp && jar -cvfm ./../lib/java/CodeNarcServer.jar ./../MANIFEST.txt ./com/nvuillam/*.class && cd ..",
-    "test": "npm run dev:kill-server && mocha \"lib/test/**/*.test.js\"",
+    "test": "npm run dev:kill-server && mocha \"test/**/*.test.js\"",
     "test:coverage": "nyc npm run test",
-    "test:debug": "npm run dev:kill-server && mocha --reporter spec --inspect-brk \"lib/test/**/*.test.js\"",
+    "test:debug": "npm run dev:kill-server && mocha --reporter spec --inspect-brk \"test/**/*.test.js\"",
     "build": "node script-build-config-all.js && cp -f README.md docs/index.md && cp -f CHANGELOG.md docs/CHANGELOG.md",
     "dev:kill-server": "npm-groovy-lint --killserver",
     "dev:lint-install-local": "npm run dev:kill-server && npm run lint:fix && npm link --force",
@@ -44,9 +44,10 @@
     "npm-groovy-lint": "lib/index.js"
   },
   "dependencies": {
-    "amplitude": "^5.1.2",
+    "amplitude": "^5.1.6",
     "ansi-colors": "^4.1.1",
     "axios": "^0.21.1",
+    "chalk": "^4.1.2",
     "cli-progress": "^3.6.0",
     "debug": "^4.1.1",
     "decode-html": "^2.0.0",
@@ -63,16 +64,17 @@
     "uuid": "^8.2.0"
   },
   "devDependencies": {
-    "babel-eslint": "^10.0.3",
+    "@babel/core": "^7.16.0",
+    "@babel/eslint-parser": "^7.16.3",
     "diff": "^4.0.2",
-    "eslint": "^6.8.0",
-    "eslint-config-standard": "^14.1.0",
-    "eslint-plugin-import": "^2.20.1",
-    "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-promise": "^4.2.1",
-    "eslint-plugin-standard": "^4.0.1",
+    "eslint": "^8.2.0",
+    "eslint-config-standard": "^16.0.3",
+    "eslint-plugin-import": "^2.25.3",
+    "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-promise": "^5.1.1",
+    "eslint-plugin-standard": "^5.0.0",
     "mocha": "^7.0.1",
-    "nyc": "^15.0.0-beta.3",
+    "nyc": "^15.1.0",
     "prettier": "1.19.1",
     "rimraf": "^3.0.2",
     "which": "^2.0.2"
@@ -82,7 +84,7 @@
   },
   "mocha": {
     "require": [
-      "lib/test/helpers/init.js"
+      "test/helpers/init.js"
     ],
     "watch-extensions": [
       "js"

package/lib/test/errors.test.js

@@ -1,92 +0,0 @@
-#! /usr/bin/env node
-"use strict";
-const NpmGroovyLint = require("../groovy-lint.js");
-let assert = require("assert");
-const fse = require("fs-extra");
-
-const { SAMPLE_FILE_SMALL_PATH } = require("./helpers/common");
-
-describe("Errors", function() {
-    it("(API:source) should trigger a parse options error", async () => {
-        const prevFileContent = fse.readFileSync("./lib/example/SampleFile.groovy").toString();
-        const npmGroovyLintConfig = {
-            source: prevFileContent,
-            someUnknownParam: "lelamanul"
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-
-        assert(linter.status === 2, `Linter status is 2 (${linter.status} returned)`);
-        assert(linter.error.msg.includes("Invalid option"), "Invalid option detected");
-    });
-
-    it("(API:source) should trigger a loglevel failon consistency error", async () => {
-        const prevFileContent = fse.readFileSync("./lib/example/SampleFile.groovy").toString();
-        const npmGroovyLintConfig = {
-            source: prevFileContent,
-            failon: "warning",
-            loglevel: "error"
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-
-        assert(linter.status === 2, `Linter status is 2 (${linter.status} returned)`);
-        assert(linter.error.msg.includes("failon option"), "failon + loglevel options consistency error detected");
-    });
-
-    it("(API:source) should trigger a codenarc error", async () => {
-        const npmGroovyLintConfig = {
-            path: "/not/existing/path",
-            output: "txt",
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-
-        assert(linter.status === 2, `Linter status is 2 (${linter.status} returned)`);
-        assert(
-            linter.error && linter.error.msg.includes("java.io.FileNotFoundException"),
-            `FileNotFoundException returned by CodeNarc (error: ${linter.error.msg})`
-        );
-    });
-
-    it("(API:source) should trigger a codenarc error (--noserver)", async () => {
-        const npmGroovyLintConfig = {
-            path: "/not/existing/path",
-            noserver: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-
-        assert(linter.status === 2, `Linter status is 2 (${linter.status} returned)`);
-        assert(
-            linter.error && linter.error.msg.includes("java.io.FileNotFoundException"),
-            `FileNotFoundException returned by CodeNarc (error: ${linter.error.msg})`
-        );
-    });
-
-    it("(API:source) should trigger a fix function error", async () => {
-        const prevFileContent = fse.readFileSync(SAMPLE_FILE_SMALL_PATH).toString();
-        const npmGroovyLintConfig = {
-            source: prevFileContent,
-            fix: true,
-            fixrules: "TriggerTestError",
-            insight: false,
-            output: "txt",
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-    });
-
-    it("(API:source) should detect and display a parse error", async () => {
-        const npmGroovyLintConfig = {
-            failon: "error",
-            path: "./lib/example/test",
-            files: "**/groovy-bad.groovy",
-            insight: false,
-            output: "txt"
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-
-        assert(linter.status === 1, `Linter status is 1 (${linter.status} returned)`);
-        assert(linter.outputString.includes("NglParseError"), "Parse error has been detected");
-    });
-});

package/lib/test/helpers/common.js

@@ -1,84 +0,0 @@
-#! /usr/bin/env node
-let assert = require("assert");
-const os = require("os");
-const fse = require("fs-extra");
-const jsdiff = require("diff");
-
-const NPM_GROOVY_LINT = "npm-groovy-lint ";
-const startPath = "./lib/example/";
-const SAMPLE_FILE_BIG = "SampleFile.groovy";
-const SAMPLE_FILE_BIG_PATH = startPath + SAMPLE_FILE_BIG;
-const SAMPLE_FILE_SMALL = "SampleFileSmall.groovy";
-const SAMPLE_FILE_SMALL_FORMAT = "SampleFileSmallFormatted.txt";
-const SAMPLE_FILE_SMALL_FIX = "SampleFileSmallFixed.txt";
-const SAMPLE_FILE_SMALL_PATH = startPath + SAMPLE_FILE_SMALL;
-const SAMPLE_FILE_PARSE_ERROR = "WithParseError.groovy";
-const SAMPLE_FILE_PARSE_ERROR_PATH = startPath + SAMPLE_FILE_PARSE_ERROR;
-const SAMPLE_RULESET_1 = "RuleSet-1.groovy";
-const SAMPLE_RULESET_1_PATH = startPath + SAMPLE_RULESET_1;
-const SAMPLE_RULESET_2 = "RuleSet-2.groovy";
-const SAMPLE_RULESET_2_PATH = startPath + SAMPLE_RULESET_2;
-
-// Reset codeNarcCallsCounter before each test
-const beforeEachTestCase = function() {
-    globalThis.codeNarcCallsCounter = 0;
-    globalThis.codeNarcCalls = [];
-};
-
-// Check counter of calls to CodeNarc
-const checkCodeNarcCallsCounter = expectedNb => {
-    assert(
-        expectedNb === globalThis.codeNarcCallsCounter,
-        `Number of calls to codeNarc is wrong: ${expectedNb} expected but ${
-            globalThis.codeNarcCallsCounter
-        } returned.\nCodeNarc calls: \n${JSON.stringify(globalThis.codeNarcCalls, null, 2)}`
-    );
-};
-
-// Copy files in temp directory to not update the package files
-async function copyFilesInTmpDir() {
-    const rootTmpDir =
-        process.platform.includes("linux") || process.platform.includes("darwin") // Linux or mac
-            ? "./tmptest"
-            : os.tmpdir(); // Windows / other
-    const tmpDir = rootTmpDir + "/" + ("tmpGroovyLintTest_" + Math.random()).replace(".", "");
-    await fse.ensureDir(tmpDir, { mode: "0777" });
-    await fse.copy("./lib/example", tmpDir);
-    console.info("GroovyLint: Copied ./lib/example into " + tmpDir);
-    return tmpDir;
-}
-
-// Get diff between 2 strings
-function getDiff(expected, afterUpdate, beforeUpdate) {
-    const diff = jsdiff.diffChars(expected, afterUpdate);
-    const effectiveDiffs = diff.filter(item => item.added || item.removed);
-    if (effectiveDiffs.length > 0) {
-        console.error("BeforeFix: \n" + beforeUpdate);
-        console.error("AfterFix: \n" + afterUpdate);
-        console.error("Expected: \n" + expected);
-    } else {
-        console.info("BeforeFix: \n" + beforeUpdate);
-        console.info("Verified: \n" + afterUpdate);
-    }
-    return effectiveDiffs;
-}
-
-module.exports = {
-    NPM_GROOVY_LINT,
-    SAMPLE_FILE_BIG,
-    SAMPLE_FILE_SMALL,
-    SAMPLE_FILE_SMALL_FORMAT,
-    SAMPLE_FILE_SMALL_FIX,
-    SAMPLE_FILE_BIG_PATH,
-    SAMPLE_FILE_SMALL_PATH,
-    SAMPLE_FILE_PARSE_ERROR,
-    SAMPLE_FILE_PARSE_ERROR_PATH,
-    SAMPLE_RULESET_1,
-    SAMPLE_RULESET_1_PATH,
-    SAMPLE_RULESET_2,
-    SAMPLE_RULESET_2_PATH,
-    beforeEachTestCase,
-    checkCodeNarcCallsCounter,
-    copyFilesInTmpDir,
-    getDiff
-};

package/lib/test/helpers/init.js

@@ -1,29 +0,0 @@
-#! /usr/bin/env node
-"use strict";
-
-//const NpmGroovyLint = require("../../groovy-lint.js");
-
-console.log("npm run test initialized");
-// Activate debug log if we are in debug mode
-const debug = typeof v8debug === "object" || /--debug|--inspect|--inspect-brk/.test(process.execArgv.join(" "));
-if (debug) {
-    require("debug").enable("npm-groovy-lint");
-    require("debug").enable("java-caller");
-}
-// Reinitialize java-caller cache
-globalThis.NPM_JAVA_CALLER_IS_INITIALIZED = false;
-globalThis.NPM_GROOVY_LINT_TEST = true;
-/*
-// Kill server if already running
-(async () => {
-    try {
-        await new NpmGroovyLint({
-            killserver: true,
-            insight: false,
-            verbose: true
-        }).run();
-    } catch (err) {
-        console.error("Error while killing running server");
-    }
-})();
-*/

package/lib/test/lint-api.test.js

@@ -1,221 +0,0 @@
-#! /usr/bin/env node
-"use strict";
-const NpmGroovyLint = require("../groovy-lint.js");
-let assert = require("assert");
-const c = require("ansi-colors");
-const fse = require("fs-extra");
-const path = require("path");
-const {
-    beforeEachTestCase,
-    checkCodeNarcCallsCounter,
-    SAMPLE_FILE_PARSE_ERROR_PATH,
-    SAMPLE_FILE_SMALL,
-    SAMPLE_FILE_SMALL_PATH,
-    SAMPLE_RULESET_1_PATH,
-    SAMPLE_RULESET_2_PATH
-} = require("./helpers/common");
-
-describe("Lint with API", () => {
-    beforeEach(beforeEachTestCase);
-
-    it("(API:file) should generate text console output ans stats", async () => {
-        const linter = await new NpmGroovyLint([process.execPath, "", "--path", '"lib/example"', "--files", "**/" + SAMPLE_FILE_SMALL, "--verbose"], {
-            verbose: true
-        }).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.outputString.includes("warning"), "Output string contains warning");
-        assert(linter.lintResult.summary.totalFoundWarningNumber > 0, "Warnings found");
-        assert(linter.lintResult.summary.totalFoundInfoNumber > 0, "Infos found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:file) should generate text console output with loglevel=warning", async () => {
-        const linter = await new NpmGroovyLint(
-            [process.execPath, "", "--path", '"lib/example"', "--files", "**/" + SAMPLE_FILE_SMALL, "--loglevel", "warning", "--verbose"],
-            {
-                verbose: true
-            }
-        ).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.outputString.includes("warning"), "Output string contains warning");
-        assert(!linter.outputString.includes("info"), "Output string should not contain info");
-        assert(linter.lintResult.summary.totalFoundWarningNumber > 0, "Warnings found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:file) should generate json output with rules", async () => {
-        const linter = await new NpmGroovyLint(
-            [
-                process.execPath,
-                "",
-                "--path",
-                '"lib/example"',
-                "--files",
-                "**/" + SAMPLE_FILE_SMALL,
-                "--output",
-                "json",
-                "--no-insight",
-                "--loglevel",
-                "warning"
-            ],
-            {}
-        ).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.outputString.includes(`"totalFoundWarningNumber":`), "Property totalFoundWarningNumber is in result");
-        assert(linter.lintResult.summary.totalFoundWarningNumber > 0, "Warnings found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:file) should generate codenarc HTML file report", async () => {
-        const reportFileName = path.resolve("./tmp/ReportTestCodenarc.html");
-        const linter = await new NpmGroovyLint(
-            [process.execPath, "", "--path", "./lib/example", "--files", "**/" + SAMPLE_FILE_SMALL, "--no-insight", "--output", reportFileName],
-            {}
-        ).run();
-
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(fse.existsSync(reportFileName), `CodeNarc HTML report generated at ${reportFileName}`);
-        fse.removeSync(reportFileName);
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:file) should generate codenarc XML file report", async () => {
-        const reportFileName = path.resolve("./tmp/ReportTestCodenarc.xml");
-        const linter = await new NpmGroovyLint(
-            [process.execPath, "", "--path", "lib/example", "--files", "**/" + SAMPLE_FILE_SMALL, "--no-insight", "--output", reportFileName],
-            {}
-        ).run();
-
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(fse.existsSync(reportFileName), `CodeNarc XML report generated at ${path.resolve(reportFileName)}`);
-        fse.removeSync(reportFileName);
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:file) should use --codenarcargs to generate XML report", async () => {
-        const reportFileName = path.resolve("./tmp/ReportTestCodenarc.xml");
-        const linter = await new NpmGroovyLint(
-            [
-                process.execPath,
-                "",
-                "--codenarcargs",
-                `-basedir="${path.resolve("lib/example")}"`,
-                '-title="TestTitleCodenarc"',
-                "-maxPriority1Violations=0",
-                `-report="xml:${reportFileName}"`
-            ],
-            {}
-        ).run();
-
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(fse.existsSync(reportFileName), `XML HTML report generated at ${path.resolve(reportFileName)}`);
-        fse.removeSync(reportFileName);
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:file) should run on a Jenkinsfile", async () => {
-        const linter = await new NpmGroovyLint(
-            [process.execPath, "", "--path", '"lib/example"', "-f", "**/Jenkinsfile", "-c", "recommended-jenkinsfile", "--no-insight", "--verbose"],
-            {
-                verbose: true
-            }
-        ).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.outputString.includes("warning"), "Output string contains warning");
-        assert(linter.lintResult.summary.totalFoundWarningNumber > 0, "Warnings found");
-        assert(linter.lintResult.summary.totalFoundInfoNumber > 0, "Infos found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:files) should ignore fake_node_modules pattern", async () => {
-        const lintedFilesNb = 10;
-        const npmGroovyLintConfig = {
-            files: "**/*.groovy",
-            ignorepattern: "**/fake_node_modules/**",
-            output: "txt",
-            insight: false,
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-        assert(!linter.outputString.includes(`ToIgnore.groovy`), "ToIgnore.groovy has been ignored");
-        assert(
-            linter.outputString.includes(`npm-groovy-lint results in ${c.bold(lintedFilesNb)} linted files`),
-            `Number of linted files is displayed in summary: ${c.bold(lintedFilesNb)}`
-        );
-    });
-
-    it("(API:source) should run with source only (no parsing)", async () => {
-        const npmGroovyLintConfig = {
-            source: fse.readFileSync(SAMPLE_FILE_SMALL_PATH).toString(),
-            sourcefilepath: SAMPLE_FILE_SMALL_PATH,
-            insight: false,
-            output: "txt",
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.lintResult.files[0].errors.length > 0, "Errors have been found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:source) should run with CodeNarc ruleset file", async () => {
-        const npmGroovyLintConfig = {
-            source: fse.readFileSync(SAMPLE_FILE_SMALL_PATH).toString(),
-            sourcefilepath: SAMPLE_FILE_SMALL_PATH,
-            rulesets: `${SAMPLE_RULESET_1_PATH},${SAMPLE_RULESET_2_PATH}`,
-            output: "txt",
-            insight: false,
-            parse: true,
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.lintResult.files[0].errors.length > 0, "Errors have been found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:source) should run with source only (parse success)", async () => {
-        const npmGroovyLintConfig = {
-            source: fse.readFileSync(SAMPLE_FILE_SMALL_PATH).toString(),
-            sourcefilepath: SAMPLE_FILE_SMALL_PATH,
-            output: "txt",
-            insight: false,
-            parse: true,
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.lintResult.files[0].errors.length > 0, "Errors have been found");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:source) should run with source only (parse error)", async () => {
-        const npmGroovyLintConfig = {
-            source: fse.readFileSync(SAMPLE_FILE_PARSE_ERROR_PATH).toString(),
-            sourcefilepath: SAMPLE_FILE_PARSE_ERROR_PATH,
-            output: "txt",
-            insight: false,
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.lintResult.files[0].errors.length > 0, "Errors have been found");
-        assert(linter.outputString.includes("NglParseError"), "Parse error has been detected");
-        checkCodeNarcCallsCounter(1);
-    });
-
-    it("(API:source) should run without CodeNarc Server", async () => {
-        const npmGroovyLintConfig = {
-            source: fse.readFileSync(SAMPLE_FILE_SMALL_PATH).toString(),
-            sourcefilepath: SAMPLE_FILE_SMALL_PATH,
-            noserver: true,
-            output: "none",
-            insight: false,
-            verbose: true
-        };
-        const linter = await new NpmGroovyLint(npmGroovyLintConfig, {}).run();
-        assert(linter.status === 0, `Linter status is 0 (${linter.status} returned)`);
-        assert(linter.lintResult.files[0].errors.length > 0, "Errors have been found");
-        checkCodeNarcCallsCounter(1);
-    });
-});

package/lib/test/lint-build-fix.test.js

@@ -1,43 +0,0 @@
-#! /usr/bin/env node
-"use strict";
-let assert = require("assert");
-const util = require("util");
-const fse = require("fs-extra");
-const rimraf = require("rimraf");
-const childProcess = require("child_process");
-const exec = util.promisify(childProcess.exec);
-const { beforeEachTestCase, copyFilesInTmpDir, SAMPLE_FILE_SMALL, SAMPLE_FILE_SMALL_FIX, NPM_GROOVY_LINT } = require("./helpers/common");
-
-describe("Lint & fix with EXE", function() {
-    beforeEach(beforeEachTestCase);
-
-    it("(API:file) should lint and fix a file in one shot", async function() {
-        const tmpDir = await copyFilesInTmpDir();
-        const prevFileContent = fse.readFileSync(tmpDir + "/" + SAMPLE_FILE_SMALL).toString();
-        const params = [
-            "--output",
-            '"npm-groovy-fix-log.json"',
-            "--path",
-            '"' + tmpDir + '"',
-            "--files",
-            "**/" + SAMPLE_FILE_SMALL,
-            "--fix",
-            "--no-insight",
-            "--verbose"
-        ];
-        const { stdout, stderr } = await exec(NPM_GROOVY_LINT + params.join(" "));
-        if (stderr) {
-            console.error(stderr);
-        }
-        assert(stdout, "stdout is set");
-
-        assert(fse.existsSync("npm-groovy-fix-log.json"), "Output json file has been produced");
-
-        const newFileContent = fse.readFileSync(tmpDir + "/" + SAMPLE_FILE_SMALL).toString();
-        assert(prevFileContent !== newFileContent, "Groovy file has been updated");
-        const expectedFileContent = fse.readFileSync(tmpDir + "/" + SAMPLE_FILE_SMALL_FIX).toString();
-        assert.strictEqual(newFileContent, expectedFileContent, "Formatted file is corresponding to expected result");
-        fse.removeSync("npm-groovy-fix-log.json");
-        rimraf.sync(tmpDir);
-    }).timeout(120000);
-});