FIX: @W-19420771@: Workaround PMD Suppress whitespace issue (#274)

Author: stephen-carter-at-sf

src/lib/pmd/pmd-suppressions-code-action-provider.ts

@@ -90,7 +90,7 @@ function generateClassLevelSuppression(document: vscode.TextDocument, diag: Code
         const existingSuppressionTags = suppressionMatch[1].split(',').map(rule => rule.trim());
         if (!existingSuppressionTags.includes(suppressionTag)) {
             // If the rule is not present, add it to the existing @SuppressWarnings
-            const updatedSuppressionTagsList = [...existingSuppressionTags, suppressionTag].join(', ');
+            const updatedSuppressionTagsList = [...existingSuppressionTags, suppressionTag].join(',');
             const updatedSuppression = `@SuppressWarnings('${updatedSuppressionTagsList}')`;
             const suppressionStartPosition = document.positionAt(classText.indexOf(suppressionMatch[0]));
             const suppressionEndPosition = document.positionAt(classText.indexOf(suppressionMatch[0]) + suppressionMatch[0].length);

src/test/unit/lib/pmd/pmd-suppressions-code-action-provider.test.ts

@@ -116,7 +116,7 @@ describe('PMDSupressionsCodeActionProvider Tests', () => {
             const classEdits: vscode.TextEdit[] = codeActions[1].edit.get(sampleApexUri);
             expect(classEdits).toHaveLength(1);
             expect(classEdits[0].range).toEqual(new vscode.Range(0, 0, 0, 40));
-            expect(classEdits[0].newText).toEqual("@SuppressWarnings('PMD.EmptyCatchBlock, PMD.ApexDoc')");
+            expect(classEdits[0].newText).toEqual("@SuppressWarnings('PMD.EmptyCatchBlock,PMD.ApexDoc')");
             expect(codeActions[1].command.command).toEqual("sfca.removeDiagnosticsOnSelectedFile"); // TODO: This is wrong. It should only clear the PMD diagnostics within the class instead of all diagnostics within the file
         });