Add a note.

ericsnowcurrently · ericsnowcurrently · commit 5700c5010a9d · 2021-02-11T13:52:28.000-07:00
diff --git a/src/client/testing/common/services/unitTestDiagnosticService.ts b/src/client/testing/common/services/unitTestDiagnosticService.ts
@@ -34,6 +34,9 @@ export class UnitTestDiagnosticService implements ITestDiagnosticService {
     }
     public getMessagePrefix(status: NonPassingTestStatus): string | undefined {
         const msgType = this.MessageTypes.get(status);
+        // If `msgType` is `undefined` then it means we've added a new
+        // failing test status but forgot to support it here (or it means
+        // elsewhere we asserted a bogus value, like `undefined`).
         return msgType !== undefined ? this.MessagePrefixes.get(msgType) : undefined;
     }
     public getSeverity(unitTestSeverity: NonPassingTestSeverity): DiagnosticSeverity | undefined {

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,9 @@ export class UnitTestDiagnosticService implements ITestDiagnosticService {`
`34`	`34`	`}`
`35`	`35`	`public getMessagePrefix(status: NonPassingTestStatus): string \| undefined {`
`36`	`36`	`const msgType = this.MessageTypes.get(status);`
	`37`	+ // If `msgType` is `undefined` then it means we've added a new
	`38`	`+ // failing test status but forgot to support it here (or it means`
	`39`	+ // elsewhere we asserted a bogus value, like `undefined`).
`37`	`40`	`return msgType !== undefined ? this.MessagePrefixes.get(msgType) : undefined;`
`38`	`41`	`}`
`39`	`42`	`public getSeverity(unitTestSeverity: NonPassingTestSeverity): DiagnosticSeverity \| undefined {`