Add check for error message for packaging.test.ts

This will add a check to ensure that `showAndLogExceptionWithTelemetry`
is not called when downloading packs. This expectation is placed before
the check for `showAndLogInformationMessage` so that when the test
fails, the error message will be shown.

Author: koesie10

extensions/ql-vscode/test/vscode-tests/cli-integration/packaging.test.ts

@@ -61,6 +61,7 @@ describe("Packaging commands", () => {
     );
 
     await handleDownloadPacks(cli, progress);
+    expect(showAndLogExceptionWithTelemetrySpy).not.toHaveBeenCalled();
     expect(showAndLogInformationMessageSpy).toHaveBeenCalledWith(
       expect.stringContaining("Finished downloading packs."),
     );
@@ -73,6 +74,7 @@ describe("Packaging commands", () => {
     inputBoxSpy.mockResolvedValue("codeql/csharp-solorigate-queries");
 
     await handleDownloadPacks(cli, progress);
+    expect(showAndLogExceptionWithTelemetrySpy).not.toHaveBeenCalled();
     expect(showAndLogInformationMessageSpy).toHaveBeenCalledWith(
       expect.stringContaining("Finished downloading packs."),
     );