Update inconsistent support message

extensions/ql-vscode/src/model-editor/consistency-check.ts

@@ -4,11 +4,7 @@ import { BaseLogger } from "../common/logging";
 
 interface Notifier {
   missingMethod(signature: string): void;
-  inconsistentSupported(
+  inconsistentSupported(signature: string, expectedSupported: boolean): void;
-    signature: string,
-    expectedSupported: boolean,
-    actualSupported: boolean,
-  ): void;
 }
 
 export function checkConsistency(
@@ -48,11 +44,7 @@ function checkMethodConsistency(
   );
 
   if (method.supported !== expectSupported) {
-    notifier.inconsistentSupported(
+    notifier.inconsistentSupported(method.signature, expectSupported);
-      method.signature,
-      expectSupported,
-      method.supported,
-    );
   }
 }
 
@@ -65,13 +57,13 @@ export class DefaultNotifier implements Notifier {
     );
   }
 
-  inconsistentSupported(
+  inconsistentSupported(signature: string, expectedSupported: boolean) {
-    signature: string,
+    const expectedMessage = expectedSupported
-    expectedSupported: boolean,
+      ? `Expected method to be supported, but it is not.`
-    actualSupported: boolean,
+      : `Expected method to not be supported, but it is.`;
-  ) {
+
     void this.logger.log(
-      `Model editor query consistency check: Inconsistent supported flag for method ${signature}. Expected supported: ${expectedSupported}, actual supported: ${actualSupported}.`,
+      `Model editor query consistency check: Inconsistent supported flag for method ${signature}. ${expectedMessage}`,
     );
   }
 }

extensions/ql-vscode/test/unit-tests/model-editor/consistency-check.test.ts

@@ -52,7 +52,6 @@ describe("checkConsistency", () => {
     expect(notifier.inconsistentSupported).toHaveBeenCalledWith(
       "Microsoft.CodeAnalysis.CSharp.SyntaxFactory.SeparatedList`1(System.Collections.Generic.IEnumerable<TNode>)",
       true,
-      false,
     );
     expect(notifier.missingMethod).not.toHaveBeenCalled();
   });