skip log event for no suppressed violations

Author: namrata111f

packages/code-analyzer-core/src/code-analyzer.ts

@@ -412,12 +412,8 @@ export class CodeAnalyzer {
         // Note: Inline suppressions are now applied per-engine in runEngineAndValidateResults() before EngineResultsEvent is emitted
 
         // Log aggregate suppression count if any violations were suppressed
-        if (this.config.getSuppressionsEnabled()) {
-            if (this.totalSuppressedViolations > 0) {
-                this.emitLogEvent(LogLevel.Info, getMessage('SuppressedViolationsCount', this.totalSuppressedViolations));
-            } else {
-                this.emitLogEvent(LogLevel.Info, getMessage('NoViolationsSuppressed'));
-            }
+        if (this.config.getSuppressionsEnabled() && this.totalSuppressedViolations > 0) {
+            this.emitLogEvent(LogLevel.Info, getMessage('SuppressedViolationsCount', this.totalSuppressedViolations));
         }
 
         return runResults;

packages/code-analyzer-core/src/messages.ts

@@ -240,10 +240,7 @@ const MESSAGE_CATALOG : MessageCatalog = {
         `Since the engine '%s' emitted an error, the following temporary working folder will not be removed: %s`,
 
     SuppressedViolationsCount:
-        `%d violation(s) were suppressed by inline suppression markers.`,
-
-    NoViolationsSuppressed:
-        `No violations were suppressed by inline suppression markers.`
+        `%d violation(s) were suppressed by inline suppression markers.`
 }
 
 /**

packages/code-analyzer-core/test/per-engine-suppressions.test.ts

@@ -163,7 +163,7 @@ describe('Per-Engine Suppression Processing', () => {
         expect(suppressionLog).toBeDefined();
     });
 
-    it('should log "No violations were suppressed" when no suppressions apply', async () => {
+    it('should not log suppression message when no suppressions apply', async () => {
         // Setup
         const config = CodeAnalyzerConfig.fromObject({
             suppressions: { disable_suppressions: false },
@@ -201,9 +201,7 @@ describe('Per-Engine Suppression Processing', () => {
         const ruleSelection = await codeAnalyzer.selectRules(['stubEngine1'], { workspace });
         await codeAnalyzer.run(ruleSelection, { workspace });
 
-        // Verify "No violations were suppressed" log was emitted
-        expect(logEvents.length).toBeGreaterThan(0);
-        const noSuppressionLog = logEvents.find(e => e.message.includes('No violations were suppressed'));
-        expect(noSuppressionLog).toBeDefined();
+        // Verify no suppression log was emitted (since no violations were suppressed)
+        expect(logEvents.length).toBe(0);
     });
 });