CHANGE @W-17053004@ Polished run command output (#1669)

Author: jfeingold35

messages/action-summary-viewer.md

@@ -21,3 +21,19 @@ Found %d rule(s) from %d engine(s):
 # rules-action.rules-item
 
 %d %s rule(s) found.
+
+# run-action.found-no-violations
+
+Found 0 violations.
+
+# run-action.violations-total
+
+Found %d violation(s) across %d file(s):
+
+# run-action.violations-item
+
+%d %s severity violation(s) found.
+
+# run-action.outfiles-total
+
+Results written to:

messages/progress-event-listener.md

@@ -11,7 +11,7 @@ done.
 Executing rules
 
 # execution-spinner.progress-summary
-%d of %d engines still executing after %ds.
+%d of %d engines finished after %ds.
 
 # execution-spinner.engine-status
  - %s at %d% completion.

messages/results-viewer.md

@@ -1,11 +1,11 @@
-# summary.detail.found-results
-
-Found %d violation(s) across %d file(s):
-
 # summary.detail.violation-header
 
 %d. %s
 
+# summary.table.results-relative-to
+
+Violation file paths relative to '%s'.
+
 # summary.table.found-results
 
 Found %d violation(s) across %d file(s) relative to '%s':

messages/run-summary-viewer.md

@@ -62,7 +62,7 @@ export default class ConfigCommand extends SfCommand<void> implements Displayabl
 
 	protected createDependencies(outputFile?: string): ConfigDependencies {
 		const uxDisplay: UxDisplay = new UxDisplay(this, this.spinner);
-		const modelGeneratorFunction = (relevantEngines: Set<string>, userContext: ConfigContext, defaultContext: ConfigContext) => {
+		const modelGeneratorFunction = /* istanbul ignore next - Model tested separately */ (relevantEngines: Set<string>, userContext: ConfigContext, defaultContext: ConfigContext) => {
 			return AnnotatedConfigModel.fromSelection(relevantEngines, userContext, defaultContext);
 		};
 		const dependencies: ConfigDependencies = {

src/commands/code-analyzer/config.ts

@@ -6,7 +6,7 @@ import {CodeAnalyzerConfigFactoryImpl} from '../../lib/factories/CodeAnalyzerCon
 import {EnginePluginsFactoryImpl} from '../../lib/factories/EnginePluginsFactory';
 import {CompositeResultsWriter} from '../../lib/writers/ResultsWriter';
 import {ResultsDetailDisplayer, ResultsTableDisplayer} from '../../lib/viewers/ResultsViewer';
-import {RunSummaryDisplayer} from '../../lib/viewers/RunSummaryViewer';
+import {RunActionSummaryViewer} from '../../lib/viewers/ActionSummaryViewer';
 import {BundleName, getMessage, getMessages} from '../../lib/messages';
 import {LogEventDisplayer} from '../../lib/listeners/LogEventListener';
 import {EngineRunProgressSpinner, RuleSelectionProgressSpinner} from '../../lib/listeners/ProgressEventListener';
@@ -108,7 +108,7 @@ export default class RunCommand extends SfCommand<void> implements Displayable {
 			resultsViewer: view === View.TABLE
 				? new ResultsTableDisplayer(uxDisplay)
 				: new ResultsDetailDisplayer(uxDisplay),
-			runSummaryViewer: new RunSummaryDisplayer(uxDisplay)
+			actionSummaryViewer: new RunActionSummaryViewer(uxDisplay)
 		};
 	}
 }

src/commands/code-analyzer/run.ts

@@ -13,7 +13,7 @@ import {EnginePluginsFactory} from '../factories/EnginePluginsFactory';
 import {createPathStarts} from '../utils/PathStartUtil';
 import {createWorkspace} from '../utils/WorkspaceUtil';
 import {ResultsViewer} from '../viewers/ResultsViewer';
-import {RunSummaryViewer} from '../viewers/RunSummaryViewer';
+import {RunActionSummaryViewer} from '../viewers/ActionSummaryViewer';
 import {ResultsWriter} from '../writers/ResultsWriter';
 import {LogFileWriter} from '../writers/LogWriter';
 import {LogEventListener, LogEventLogger} from '../listeners/LogEventListener';
@@ -27,7 +27,7 @@ export type RunDependencies = {
 	progressListeners: ProgressEventListener[];
 	writer: ResultsWriter;
 	resultsViewer: ResultsViewer;
-	runSummaryViewer: RunSummaryViewer;
+	actionSummaryViewer: RunActionSummaryViewer;
 }
 
 export type RunInput = {
@@ -80,7 +80,7 @@ export class RunAction {
 		this.dependencies.logEventListeners.forEach(listener => listener.stopListening());
 		this.dependencies.writer.write(results);
 		this.dependencies.resultsViewer.view(results);
-		this.dependencies.runSummaryViewer.view(results, logWriter.getLogDestination(), input['output-file']);
+		this.dependencies.actionSummaryViewer.view(results, logWriter.getLogDestination(), input['output-file']);
 
 		const thresholdValue = input['severity-threshold'];
 		if (thresholdValue) {

src/lib/actions/RunAction.ts

@@ -207,7 +207,7 @@ export class EngineRunProgressSpinner extends ProgressSpinner implements Progres
 			engineLines.push(getMessage(BundleName.ProgressEventListener, 'execution-spinner.engine-status', [name, progress]));
 		}
 		return [
-			getMessage(BundleName.ProgressEventListener, 'execution-spinner.progress-summary', [unfinishedEngines, totalEngines, secondsRunning]),
+			getMessage(BundleName.ProgressEventListener, 'execution-spinner.progress-summary', [totalEngines - unfinishedEngines, totalEngines, secondsRunning]),
 			...engineLines
 		].join('\n');
 	}

src/lib/listeners/ProgressEventListener.ts

@@ -50,6 +50,6 @@ export function getMessage(bundle: BundleName, messageKey: string, tokens?: Toke
 }
 
 export function getMessages(bundle: BundleName, messageKey: string, tokens?: Tokens): string[] {
-	INSTANCE = INSTANCE || new MessageCatalog();
+	INSTANCE = INSTANCE || /* istanbul ignore next */ new MessageCatalog();
 	return INSTANCE.getMessages(bundle, messageKey, tokens);
 }

src/lib/messages.ts

@@ -1,5 +1,5 @@
 import {Display} from '../Display';
-import {RuleSelection} from '@salesforce/code-analyzer-core';
+import {RuleSelection, RunResults, SeverityLevel, Violation} from '@salesforce/code-analyzer-core';
 import {toStyledHeader, indent} from '../utils/StylingUtil';
 import {BundleName, getMessage} from '../messages';
 
@@ -75,6 +75,64 @@ export class RulesActionSummaryViewer extends AbstractActionSummaryViewer {
 			this.display.displayLog(indent(getMessage(BundleName.ActionSummaryViewer, 'rules-action.rules-item', [ruleCountForEngine, engineName])));
 		}
 	}
+}
+
+export class RunActionSummaryViewer extends AbstractActionSummaryViewer {
+	public constructor(display: Display) {
+		super(display);
+	}
+
+	public view(results: RunResults, logFile: string, outfiles: string[]): void {
+		// Start with separator to cleanly break from anything that's already been logged.
+		this.displayLineSeparator();
+		this.displaySummaryHeader();
+		this.displayLineSeparator();
+
+		if (results.getViolationCount() === 0) {
+			this.display.displayLog(getMessage(BundleName.ActionSummaryViewer, 'run-action.found-no-violations'));
+		} else {
+			this.displayResultsSummary(results);
+		}
+		this.displayLineSeparator();
 
+		if (outfiles.length > 0) {
+			this.displayOutfiles(outfiles);
+			this.displayLineSeparator();
+		}
 
+		this.displayLogFileInfo(logFile);
+	}
+
+	private displayResultsSummary(results: RunResults): void {
+		this.display.displayLog(getMessage(BundleName.ActionSummaryViewer, 'run-action.violations-total', [results.getViolationCount(), this.countUniqueFiles(results.getViolations())]));
+		for (const sev of Object.values(SeverityLevel)) {
+			// Some of the keys will be numbers, since the enum is numerical. Skip those.
+			if (typeof sev !== 'string') {
+				continue;
+			}
+			const sevCount = results.getViolationCountOfSeverity(SeverityLevel[sev] as SeverityLevel);
+			if (sevCount > 0) {
+				this.display.displayLog(indent(getMessage(BundleName.ActionSummaryViewer, 'run-action.violations-item', [sevCount, sev])));
+			}
+		}
+	}
+
+	private countUniqueFiles(violations: Violation[]): number {
+		const fileSet: Set<string> = new Set();
+		violations.forEach(v => {
+			const primaryLocation = v.getCodeLocations()[v.getPrimaryLocationIndex()];
+			const file = primaryLocation.getFile();
+			if (file) {
+				fileSet.add(file);
+			}
+		});
+		return fileSet.size;
+	}
+
+	private displayOutfiles(outfiles: string[]): void {
+		this.display.displayLog(getMessage(BundleName.ActionSummaryViewer, 'run-action.outfiles-total'));
+		for (const outfile of outfiles) {
+			this.display.displayLog(indent(outfile));
+		}
+	}
 }