npm-groovy-lint 14.6.0 → 14.6.1-beta202408252305.0

package/lib/rules/TrailingWhitespace.js

@@ -7,16 +7,16 @@ const rule = {
             const diff = errLine.length - errLine.trimEnd().length;
             return {
                 start: { line: errItem.line, character: errLine.length - diff },
-                end: { line: errItem.line, character: errLine.length }
+                end: { line: errItem.line, character: errLine.length },
             };
-        }
+        },
     },
     fix: {
         label: "Remove trailing whitespace",
         type: "function",
-        func: line => {
+        func: (line) => {
             return line.trimEnd();
-        }
+        },
     },
     tests: [
         {
@@ -25,9 +25,9 @@ const rule = {
 `,
             sourceAfter: `
     def str = "lelamanul"
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryDefInFieldDeclaration.js

@@ -1,19 +1,19 @@
 // Unnecessary def in field declaration (static def)
 
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, "def", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove def",
         type: "replaceString",
         before: "def ",
-        after: ""
+        after: "",
     },
     tests: [
         {
@@ -26,7 +26,7 @@ class lelamanul {
 class lelamanul {
     private str = "lelamanul"
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -38,9 +38,9 @@ class lelamanul {
 class lelamanul {
     public str = "lelamanul"
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryDefInMethodDeclaration.js

@@ -1,19 +1,19 @@
 // Unnecessary def in field declaration (static def)
 
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, "def", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove def",
         type: "replaceString",
         before: "def ",
-        after: ""
+        after: "",
     },
     tests: [
         {
@@ -26,9 +26,9 @@ static def getCommandLogAsObj(String command) {
 static getCommandLogAsObj(String command) {
     return Utils.fromJsonLogString(getCommandLog(command))
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryDefInVariableDeclaration.js

@@ -1,19 +1,19 @@
 // Unnecessary def in variable declaration
 
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, "def", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove def",
         type: "replaceString",
         before: "def ",
-        after: ""
+        after: "",
     },
     tests: [
         {
@@ -22,7 +22,7 @@ const rule = {
 `,
             sourceAfter: `
     private string1 = 'example'
-`
+`,
         },
         {
             sourceBefore: `
@@ -30,7 +30,7 @@ const rule = {
 `,
             sourceAfter: `
     final string5 = 'example'
-`
+`,
         },
         {
             sourceBefore: `
@@ -38,9 +38,9 @@ const rule = {
 `,
             sourceAfter: `
     String string6 = 'example'
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryDotClass.js

@@ -1,19 +1,19 @@
 // Unnecessary dot class
 
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, ".class", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove .class",
         type: "replaceString",
         before: ".class",
-        after: ""
+        after: "",
     },
     tests: [
         {
@@ -22,9 +22,9 @@ const rule = {
 `,
             sourceAfter: `
     def x = String
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryFinalOnPrivateMethod.js

@@ -1,19 +1,19 @@
 // Unnecessary private on final method
 
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, "final", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove private",
         type: "replaceString",
         before: "final ",
-        after: ""
+        after: "",
     },
     tests: [
         {
@@ -26,9 +26,9 @@ const rule = {
     private methodName() {
 
     }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryGString.js

@@ -1,6 +1,6 @@
 // Unnecessary Groovy String
 
-const { getVariable, getVariableRange, findRangeBetweenStrings } = require("../utils");
+import { getVariable, getVariableRange, findRangeBetweenStrings } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -8,8 +8,8 @@ const rule = {
     variables: [
         {
             name: "STRING",
-            regex: /The String '([\s\S]*)' can be wrapped in single quotes instead of double quotes/
-        }
+            regex: /The String '([\s\S]*)' can be wrapped in single quotes instead of double quotes/,
+        },
     ],
     range: {
         type: "function",
@@ -21,7 +21,7 @@ const rule = {
             } else {
                 return findRangeBetweenStrings(allLines, errItem, '"""', '"""');
             }
-        }
+        },
     },
     fix: {
         label: "Replace double quotes by single quotes",
@@ -40,7 +40,7 @@ const rule = {
                 allLines[range.end.line - 1] = allLines[range.end.line - 1].replace(`"""`, `'''`);
             }
             return allLines;
-        }
+        },
     },
     tests: [
         {
@@ -49,7 +49,7 @@ String str = "lelamanul"
 `,
             sourceAfter: `
 String str = 'lelamanul'
-`
+`,
         },
         {
             sourceBefore: `
@@ -57,7 +57,7 @@ String str = 'lelamanul' + "\\n"
 `,
             sourceAfter: `
 String str = 'lelamanul' + '\\n'
-`
+`,
         },
         {
             sourceBefore: `
@@ -65,7 +65,7 @@ String str = 'lelamanul' + "\\n\\r"
 `,
             sourceAfter: `
 String str = 'lelamanul' + '\\n\\r'
-`
+`,
         },
         {
             sourceBefore: `
@@ -73,7 +73,7 @@ String str = 'lelamanul' + "\\n\\r\\n" + "titi\\n" + "\\n\\r" + "lelamanul\\nwes
 `,
             sourceAfter: `
 String str = 'lelamanul' + '\\n\\r\\n' + 'titi\\n' + '\\n\\r' + 'lelamanul\\nwesh'
-`
+`,
         },
         {
             sourceBefore: `
@@ -85,7 +85,7 @@ I am a string
 def b = '''
 I am a string
 '''
-`
+`,
         },
         {
             sourceBefore: `
@@ -95,9 +95,9 @@ I am a string"""
             sourceAfter: `
 def b = '''
 I am a string'''
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryGroovyImport.js

@@ -1,6 +1,6 @@
 // Unnecessary Groovy Import
 
-const { getVariable } = require("../utils");
+import { getVariable } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -14,7 +14,7 @@ const rule = {
                 allLines.splice(lineNumber + 1, 1);
             }
             return allLines;
-        }
+        },
     },
 
     tests: [
@@ -33,9 +33,9 @@ import java.io.InputStream
 class ABC {
     InputStream input
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryPackageReference.js

@@ -1,18 +1,18 @@
 // Unnecessary package reference
-const { getVariable, getVariableRange } = require("../utils");
+import { getVariable, getVariableRange } from "../utils.js";
 
 const rule = {
     variables: [
         {
             name: "CLASS_WITH_PACKAGE",
-            regex: /The (.*) (.*) was explicitly imported, so specifying the package name is not necessary/
-        }
+            regex: /The (.*) (.*) was explicitly imported, so specifying the package name is not necessary/,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "CLASS_WITH_PACKAGE", errItem);
-        }
+        },
     },
     fix: {
         label: "Use short name",
@@ -21,7 +21,7 @@ const rule = {
             const packageName = getVariable(evaluatedVars, "CLASS_WITH_PACKAGE", { mandatory: true, htmlToString: true, line: line });
             const packageShortName = packageName.split(".").pop();
             return line.replace(packageName, packageShortName);
-        }
+        },
     },
     tests: [
         {
@@ -34,9 +34,9 @@ def body = groovy.json.JsonOutput.toJson(value)
 import groovy.json.JsonOutput
 
 def body = JsonOutput.toJson(value)
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryParenthesesForMethodCallWithClosure.js

@@ -1,26 +1,26 @@
 // Unnecessary Parenthesis for method call with closure
 
-const { getVariableRange } = require("../utils");
+import { getVariableRange } from "../utils.js";
 
 const rule = {
     variables: [
         {
             name: "METHOD",
             regex: /Violation in (.*). Parentheses in the '(.*)' method call are unnecessary and can be removed./,
-            regexPos: 2
-        }
+            regexPos: 2,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "METHOD", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove unnecessary parenthesis",
         type: "replaceString",
         before: "{{METHOD}}()",
-        after: "{{METHOD}}"
+        after: "{{METHOD}}",
     },
     tests: [
         {
@@ -33,9 +33,9 @@ dirFile.eachFile() {
 dirFile.eachFile {
     dirs << it
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryPublicModifier.js

@@ -1,13 +1,13 @@
 // Variable type required
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, "public", errItem);
-        }
-    }
+        },
+    },
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessarySemicolon.js

@@ -1,26 +1,23 @@
 // Unnecessary semi colon at the end of a line
 
-const { getLastStringRange } = require("../utils");
+import { getLastStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getLastStringRange(errLine, ";", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove unnecessary semicolon",
         type: "function",
-        func: line => {
+        func: (line) => {
             if ((line.match(/;/g) || []).length === 1) {
-                line = line
-                    .split(";")
-                    .join("")
-                    .trimRight();
+                line = line.split(";").join("").trimRight();
             }
             return line;
-        }
+        },
     },
     tests: [
         {
@@ -29,7 +26,7 @@ String str = 'lelamanul';
 `,
             sourceAfter: `
 String str = 'lelamanul'
-`
+`,
         },
         {
             sourceBefore: `
@@ -41,9 +38,9 @@ println("test");
 package my.company.server
 import java.lang.String
 println("test")
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnnecessaryToString.js

@@ -1,20 +1,20 @@
 // Unnecessary toString()
 
-const { getStringRange } = require("../utils");
+import { getStringRange } from "../utils.js";
 
 const rule = {
     range: {
         type: "function",
         func: (errLine, errItem) => {
             return getStringRange(errLine, ".toString()", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove unnecessary toString()",
         type: "function",
-        func: line => {
+        func: (line) => {
             return line.replace(".toString()", "");
-        }
+        },
     },
     tests: [
         {
@@ -25,9 +25,9 @@ String str2 = str.toString()
             sourceAfter: `
 String str = 'lelamanul'
 String str2 = str
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnusedImport.js

@@ -1,6 +1,6 @@
 // Unused import
 
-const { getVariable, getVariableRange } = require("../utils");
+import { getVariable, getVariableRange } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -9,14 +9,14 @@ const rule = {
         {
             name: "CLASSNAME",
             regex: /The \[(.*)\] import is never referenced/,
-            regexPos: 1
-        }
+            regexPos: 1,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "CLASSNAME", errItem);
-        }
+        },
     },
     fix: {
         label: "Remove unused import",
@@ -27,11 +27,11 @@ const rule = {
             if (allLines[lineNumber].includes(className)) {
                 allLines.splice(lineNumber, 1);
             } else {
-                const itemIndex = allLines.findIndex(line => line.includes(className));
+                const itemIndex = allLines.findIndex((line) => line.includes(className));
                 throw new Error(`FIX ERROR: UnusedImport was expecting ${className} at position ${lineNumber} but it is at position ${itemIndex} `);
             }
             return allLines;
-        }
+        },
     },
 
     tests: [
@@ -64,9 +64,9 @@ class Wesh {
 class Gros {
     JFileChooser inputX
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnusedMethodParameter.js

@@ -1,21 +1,21 @@
 // Unused method parameter
 
-const { getVariableRange } = require("../utils");
+import { getVariableRange } from "../utils.js";
 
 const rule = {
     variables: [
         {
             name: "PARAMNAME",
             regex: /Method parameter \[(.*)\] is never referenced in the method (.*) of class (.*)/,
-            regexPos: 1
-        }
+            regexPos: 1,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "PARAMNAME", errItem);
-        }
-    }
+        },
+    },
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/UnusedVariable.js

@@ -1,20 +1,20 @@
 // Unused variable
 
-const { getVariableRange } = require("../utils");
+import { getVariableRange } from "../utils.js";
 
 const rule = {
     variables: [
         {
             name: "VARNAME",
-            regex: /The variable \[(.*)\] in (.*) is not used/
-        }
+            regex: /The variable \[(.*)\] in (.*) is not used/,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "VARNAME", errItem);
-        }
-    }
+        },
+    },
 };
 
-module.exports = { rule };
+export { rule };