Add tests for NoTrailingWhitespace rule multiple empty lines detection

- Added test data files with multiple consecutive empty lines
- Added test data file with empty lines at EOF
- Added test data file with single empty lines (valid case)
- Added test cases to verify multiple empty lines are flagged
- Added test case to verify single empty lines are allowed
- Updated existing test to include new violations from test data

Author: aruntyagiTutu

packages/code-analyzer-regex-engine/test/engine.test.ts

@@ -268,7 +268,69 @@ describe('Tests for runRules', () => {
                     endColumn: 21
                 }]
             },
+            {
+                ruleName: "NoTrailingWhitespace",
+                message: getMessage('TrailingWhitespaceRuleMessage'),
+                primaryLocationIndex: 0,
+                codeLocations: [{
+                    file: path.resolve(__dirname, "test-data", "apexClassWhitespace", "4_multipleEmptyLines", "EmptyLinesAtEof.cls"),
+                    startLine: 7,
+                    startColumn: 1,
+                    endLine: 8,
+                    endColumn: 1
+                }]
+            },
+            {
+                ruleName: "NoTrailingWhitespace",
+                message: getMessage('TrailingWhitespaceRuleMessage'),
+                primaryLocationIndex: 0,
+                codeLocations: [{
+                    file: path.resolve(__dirname, "test-data", "apexClassWhitespace", "4_multipleEmptyLines", "MultipleEmptyLines.cls"),
+                    startLine: 6,
+                    startColumn: 1,
+                    endLine: 7,
+                    endColumn: 1
+                }]
+            },
+
+        ];
+
+        expect(runResults.violations).toHaveLength(expectedViolations.length);
+        for (const expectedViolation of expectedViolations) {
+            expect(runResults.violations).toContainEqual(expectedViolation);
+        }
+    });
 
+    it("NoTrailingWhitespace rule should flag multiple consecutive empty lines", async () => {
+        const runOptions: RunOptions = createRunOptions(
+            new Workspace('id', [path.resolve(__dirname, "test-data", "apexClassWhitespace", "4_multipleEmptyLines")]));
+        const runResults: EngineRunResults = await engine.runRules(["NoTrailingWhitespace"], runOptions);
+
+        const expectedViolations: Violation[] = [
+            {
+                ruleName: "NoTrailingWhitespace",
+                message: getMessage('TrailingWhitespaceRuleMessage'),
+                primaryLocationIndex: 0,
+                codeLocations: [{
+                    file: path.resolve(__dirname, "test-data", "apexClassWhitespace", "4_multipleEmptyLines", "MultipleEmptyLines.cls"),
+                    startLine: 6,
+                    startColumn: 1,
+                    endLine: 7,
+                    endColumn: 1
+                }]
+            },
+            {
+                ruleName: "NoTrailingWhitespace",
+                message: getMessage('TrailingWhitespaceRuleMessage'),
+                primaryLocationIndex: 0,
+                codeLocations: [{
+                    file: path.resolve(__dirname, "test-data", "apexClassWhitespace", "4_multipleEmptyLines", "EmptyLinesAtEof.cls"),
+                    startLine: 7,
+                    startColumn: 1,
+                    endLine: 8,
+                    endColumn: 1
+                }]
+            }
         ];
 
         expect(runResults.violations).toHaveLength(expectedViolations.length);
@@ -277,6 +339,14 @@ describe('Tests for runRules', () => {
         }
     });
 
+    it("NoTrailingWhitespace rule should NOT flag single empty lines between code", async () => {
+        const runOptions: RunOptions = createRunOptions(
+            new Workspace('id', [path.resolve(__dirname, "test-data", "apexClassWhitespace", "5_singleEmptyLinesValid")]));
+        const runResults: EngineRunResults = await engine.runRules(["NoTrailingWhitespace"], runOptions);
+
+        expect(runResults.violations).toHaveLength(0);
+    });
+
     it("Ensure runRules when called on a directory of Apex classes with getHeapSize in a loop, it properly emits violations", async () => {
         const runOptions: RunOptions = createRunOptions(
             new Workspace('id', [path.resolve(__dirname, "test-data", "apexClassGetLimitsInLoop")]));

packages/code-analyzer-regex-engine/test/test-data/apexClassWhitespace/4_multipleEmptyLines/EmptyLinesAtEof.cls

@@ -0,0 +1,7 @@
+public class EmptyLinesAtEof {
+    public void method1() {
+        System.debug('test');
+    }
+}
+
+

packages/code-analyzer-regex-engine/test/test-data/apexClassWhitespace/4_multipleEmptyLines/MultipleEmptyLines.cls

@@ -0,0 +1,10 @@
+public class MultipleEmptyLines {
+    public void method1() {
+        System.debug('test1');
+    }
+
+
+    public void method2() {
+        System.debug('test2');
+    }
+}

packages/code-analyzer-regex-engine/test/test-data/apexClassWhitespace/5_singleEmptyLinesValid/SingleEmptyLines.cls

@@ -0,0 +1,13 @@
+public class SingleEmptyLines {
+    public void method1() {
+        System.debug('test1');
+    }
+
+    public void method2() {
+        System.debug('test2');
+    }
+
+    public void method3() {
+        System.debug('test3');
+    }
+}