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

package/lib/rules/BracesForForLoop.js

@@ -1,6 +1,6 @@
 // Braces for for loop
 
-const { getVariable, moveOpeningBracket, findRangeBetweenStrings } = require("../utils");
+import { getVariable, moveOpeningBracket, findRangeBetweenStrings } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -9,7 +9,7 @@ const rule = {
         type: "function",
         func: (_errLine, errItem, _evaluatedVars, allLines) => {
             return findRangeBetweenStrings(allLines, errItem, "for", "{");
-        }
+        },
     },
     fix: {
         label: "Move opening brace on the same line",
@@ -20,7 +20,7 @@ const rule = {
                 return moveOpeningBracket(allLines, variables);
             }
             return allLines;
-        }
+        },
     },
     tests: [
         {
@@ -34,7 +34,7 @@ for (int i = 0; i < toto.length ; i++)
 for (int i = 0; i < toto.length ; i++) {
     def a = 1
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -46,7 +46,7 @@ for (int i = 0; i < toto.length ; i++)
 for (int i = 0; i < toto.length ; i++) {
     def a = 1
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -58,9 +58,9 @@ for (int i = 0; i < toto.length ; i++) {
 for (int i = 0; i < toto.length ; i++) {
     def a = 1
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/BracesForIfElse.js

@@ -1,6 +1,6 @@
 // Braces for if else
 
-const { moveOpeningBracket, findRangeBetweenStrings } = require("../utils");
+import { moveOpeningBracket, findRangeBetweenStrings } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -9,14 +9,14 @@ const rule = {
         type: "function",
         func: (_errLine, errItem, _evaluatedVars, allLines) => {
             return findRangeBetweenStrings(allLines, errItem, "if", "{");
-        }
+        },
     },
     fix: {
         label: "Move opening brace on the same line",
         type: "function",
         func: (allLines, variables) => {
             return moveOpeningBracket(allLines, variables);
-        }
+        },
     },
     tests: [
         {
@@ -30,7 +30,7 @@ if (true)
 if (true) {
     def a = 1
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -42,7 +42,7 @@ if (true)
 if (true) {
     def a = 1
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -59,9 +59,9 @@ if (true &&
     true ) {
     def a = 1
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/BracesForMethod.js

@@ -1,6 +1,6 @@
 // Braces for method
 
-const { moveOpeningBracket, findRangeBetweenStrings, getVariable } = require("../utils");
+import { moveOpeningBracket, findRangeBetweenStrings, getVariable } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -8,22 +8,22 @@ const rule = {
     variables: [
         {
             name: "METHOD_NAME",
-            regex: /Opening brace for the method (.*) should start on the same line/
-        }
+            regex: /Opening brace for the method (.*) should start on the same line/,
+        },
     ],
     range: {
         type: "function",
         func: (_errLine, errItem, evaluatedVars, allLines) => {
             const methodName = getVariable(evaluatedVars, "METHOD_NAME", { mandatory: true });
             return findRangeBetweenStrings(allLines, errItem, methodName, "{");
-        }
+        },
     },
     fix: {
         label: "Move opening brace on the same line",
         type: "function",
         func: (allLines, variables) => {
             return moveOpeningBracket(allLines, variables);
-        }
+        },
     },
     tests: [
         {
@@ -37,7 +37,7 @@ def someMethodName (String toto)
 def someMethodName (String toto) {
     def a = 1
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -49,7 +49,7 @@ def someMethodName (String toto)
 def someMethodName (String toto) {
     def a = 1
 }
-`
+`,
         },
         {
             sourceBefore: `
@@ -84,7 +84,7 @@ class VersionHandler implements Serializable {
     }
 
 }
-`
+`,
         } /*
         NV: Disable fix while not detected by CodeNarc
         ,
@@ -104,8 +104,8 @@ def someMethodName (String toto,
     def a = 1
 }
 `
-        } */
-    ]
+        } */,
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/BracesForTryCatchFinally.js

@@ -1,6 +1,6 @@
 // Braces for try catch finally
 
-const { moveOpeningBracket, findRangeBetweenStrings } = require("../utils");
+import { moveOpeningBracket, findRangeBetweenStrings } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -9,14 +9,14 @@ const rule = {
         type: "function",
         func: (_errLine, errItem, _evaluatedVars, allLines) => {
             return findRangeBetweenStrings(allLines, errItem, "try", "{");
-        }
+        },
     },
     fix: {
         label: "Move opening brace on the same line",
         type: "function",
         func: (allLines, variables) => {
             return moveOpeningBracket(allLines, variables);
-        }
+        },
     },
     tests: [
         {
@@ -38,9 +38,9 @@ try {
 } finally {
     def c = 6
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/CatchException.js

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

package/lib/rules/ClassEndsWithBlankLine.js

@@ -1,6 +1,6 @@
 // Unused import
 
-const { getVariable } = require("../utils");
+import { getVariable } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -14,7 +14,7 @@ const rule = {
                 allLines.splice(lineNumber, 0, "");
             }
             return allLines;
-        }
+        },
     },
 
     tests: [
@@ -37,9 +37,9 @@ class NullResultsProcessor implements ResultsProcessor {
     }
 
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/ClassStartsWithBlankLine.js

@@ -1,6 +1,6 @@
 // Unused import
 
-const { getVariable } = require("../utils");
+import { getVariable } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -14,7 +14,7 @@ const rule = {
                 allLines.splice(lineNumber, 0, "");
             }
             return allLines;
-        }
+        },
     },
 
     tests: [
@@ -37,9 +37,9 @@ class NullResultsProcessor implements ResultsProcessor {
     }
 
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/ClosingBraceNotAlone.js

@@ -8,13 +8,13 @@ const rule = {
             const closingBracePos = errLine.lastIndexOf("}");
             return {
                 start: { line: errItem.line, character: closingBracePos },
-                end: { line: errItem.line, character: closingBracePos + 1 }
+                end: { line: errItem.line, character: closingBracePos + 1 },
             };
-        }
+        },
     },
     fix: {
         type: "function",
-        func: allLines => {
+        func: (allLines) => {
             const newFileLines = [];
             let prevLine = "";
             for (const line of allLines) {
@@ -30,7 +30,7 @@ const rule = {
                 }
             }
             return newFileLines;
-        }
+        },
     },
     tests: [
         {
@@ -42,9 +42,9 @@ if (a == 1) {
 if (a == 1) {
     whatever()
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/ConsecutiveBlankLines.js

@@ -1,6 +1,6 @@
 // Consecutive blank lines
 
-const { getVariable } = require("../utils");
+import { getVariable } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -23,7 +23,7 @@ const rule = {
                 pos++;
             }
             return newFileLines;
-        }
+        },
     },
     tests: [
         {
@@ -37,7 +37,7 @@ x = 1
 if (a == 2){
 
 x = 1
-}`
+}`,
         },
         {
             sourceBefore: `
@@ -52,9 +52,9 @@ x = 1
 if (a == 2){
 
 x = 1
-}`
-        }
-    ]
+}`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/DuplicateImport.js

@@ -1,6 +1,6 @@
 // Duplicate import
 
-const { getVariable } = require("../utils");
+import { getVariable } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -16,7 +16,7 @@ const rule = {
                 throw new Error(`FIX ERROR: Duplicate Import not found at at position ${lineNumber}`);
             }
             return allLines;
-        }
+        },
     },
 
     tests: [
@@ -58,9 +58,9 @@ class Wesh {
 class Gros {
     JFileChooser inputX
 }
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/DuplicateNumberLiteral.js

@@ -1,21 +1,21 @@
 // Duplicate number literal
 
-const { getVariableRange } = require("../utils");
+import { getVariableRange } from "../utils.js";
 
 const rule = {
     variables: [
         {
             name: "NUMBER_LITERAL",
             regex: /Duplicate Number Literal: (.*)/,
-            regexPos: 1
-        }
+            regexPos: 1,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "NUMBER_LITERAL", errItem);
-        }
-    }
+        },
+    },
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/DuplicateStringLiteral.js

@@ -1,21 +1,21 @@
 // Duplicate string literal
 
-const { getVariableRange } = require("../utils");
+import { getVariableRange } from "../utils.js";
 
 const rule = {
     variables: [
         {
             name: "STRING_LITERAL",
             regex: /Duplicate String Literal: (.*)/,
-            regexPos: 1
-        }
+            regexPos: 1,
+        },
     ],
     range: {
         type: "function",
         func: (errLine, errItem, evaluatedVars) => {
             return getVariableRange(errLine, evaluatedVars, "STRING_LITERAL", errItem);
-        }
-    }
+        },
+    },
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/ElseBlockBraces.js

@@ -1,5 +1,5 @@
 // Missing else braces
-const { containsOtherThan, getStringRangeMultiline, getVariable, isValidCodeLine } = require("../utils");
+import { containsOtherThan, getStringRangeMultiline, getVariable, isValidCodeLine } from "../utils.js";
 
 const rule = {
     scope: "file",
@@ -10,7 +10,7 @@ const rule = {
         type: "function",
         func: (_errLine, errItem, _evaluatedVars, allLines) => {
             return getStringRangeMultiline(allLines, "else", errItem, "if");
-        }
+        },
     },
     fix: {
         label: "Add else statement braces",
@@ -59,7 +59,7 @@ const rule = {
                 pos++;
             }
             return allLines;
-        }
+        },
     },
     tests: [
         {
@@ -78,7 +78,7 @@ else {
     whateverElse()
 }
 `,
-            codeNarcCallsNumber: 2
+            codeNarcCallsNumber: 2,
         },
         {
             sourceBefore: `
@@ -96,7 +96,7 @@ else {
     doRetrieve = true
 }
 `,
-            codeNarcCallsNumber: 2
+            codeNarcCallsNumber: 2,
         },
         {
             sourceBefore: `
@@ -133,7 +133,7 @@ private doReplaceInFile(String fileName, def searchRegex, String replacementValu
     }
 }
 `,
-            codeNarcCallsNumber: 2
+            codeNarcCallsNumber: 2,
         },
         {
             sourceBefore: `
@@ -152,9 +152,9 @@ static getFileExtension(fileName) {
     else return '' 
 }
 `,
-            codeNarcCallsNumber: 1
-        }
-    ]
+            codeNarcCallsNumber: 1,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/ExplicitArrayListInstantiation.js

@@ -3,9 +3,9 @@ const rule = {
     fix: {
         label: "Replace ArrayList declaration by []",
         type: "function",
-        func: line => {
+        func: (line) => {
             return line.replace(/new ArrayList(.*)\)/gi, "[]");
-        }
+        },
     },
     tests: [
         {
@@ -14,7 +14,7 @@ const rule = {
 `,
             sourceAfter: `
     List<String> commandArray = []
-`
+`,
         },
         {
             sourceBefore: `
@@ -22,9 +22,9 @@ List<String> commandArray=new ArrayList<String> ()
 `,
             sourceAfter: `
 List<String> commandArray=[]
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/ExplicitLinkedListInstantiation.js

@@ -3,9 +3,9 @@ const rule = {
     fix: {
         label: "Replace LinkedList declaration by [] as Queue",
         type: "function",
-        func: line => {
+        func: (line) => {
             return line.replace(/new LinkedList(.*)\)/gi, "[] as Queue");
-        }
+        },
     },
     tests: [
         {
@@ -14,7 +14,7 @@ const rule = {
 `,
             sourceAfter: `
     List<String> printList = [] as Queue
-`
+`,
         },
         {
             sourceBefore: `
@@ -22,9 +22,9 @@ List<String> printList=new LinkedList<String>()
 `,
             sourceAfter: `
 List<String> printList=[] as Queue
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/FileEndsWithoutNewline.js

@@ -1,5 +1,5 @@
 // File ends without new line
-const os = require("os");
+import * as os from "os";
 
 const rule = {
     scope: "file",
@@ -7,9 +7,9 @@ const rule = {
     fix: {
         label: "Add new line at the end of file",
         type: "function",
-        func: allLines => {
+        func: (allLines) => {
             return (allLines.join(os.EOL) + os.EOL).split(os.EOL);
-        }
+        },
     },
     tests: [
         {
@@ -19,9 +19,9 @@ def b = 2`,
             sourceAfter: `
 def a = 1
 def b = 2
-`
-        }
-    ]
+`,
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };

package/lib/rules/GStringExpressionWithinString.js

@@ -9,15 +9,15 @@ def someString = '\${SCRIPT,template=\\"mail_template_robot_results.groovy\\"}'
             expectedRange: {
                 start: {
                     line: 2,
-                    character: 0
+                    character: 0,
                 },
                 end: {
                     line: 2,
-                    character: 76
-                }
-            }
-        }
-    ]
+                    character: 76,
+                },
+            },
+        },
+    ],
 };
 
-module.exports = { rule };
+export { rule };