Fix flaky VSCode extension E2E tests due to silently ignored timeout option (#19309)

The E2E test suite intermittently failed because `timeoutMs` passed to
`until()` was silently dropped — `retryWhile` looked for `timeout`, not
`timeoutMs` — capping all `until()` waits at the 10s default regardless
of what callers specified. This caused `deploypane` tests (which
intended a 30s webview-ready timeout) to flake whenever CI was slow.

## Changes

- **`src/test/utils/time.ts`**: Rename `timeout` → `timeoutMs` in
`retryWhile`'s options type so it matches the `until()` API:
  ```ts
  // Before — `timeoutMs` from callers was silently ignored
  retryOptions?: Readonly<{ interval?: number; timeout?: number }>

  // After — consistent naming across retryWhile and until
  retryOptions?: Readonly<{ interval?: number; timeoutMs?: number }>
  ```

- **`deploypane.test.ts`**: Add `jest.setTimeout(60000)` — now that
`timeoutMs: 30000` actually works, the global 20s jest timeout would be
exceeded (2s language server wait + up to 30s webview-ready poll).

- **`visualizer.test.ts`**: Add explicit `timeoutMs: 20000` to all three
`until()` calls (previously relied on the 10s default) and
`jest.setTimeout(30000)` to match.

- **`testReporter.ts`**: Fix the `Tests:` summary line, which was
incorrectly reporting test-suite counts (`numPassedTestSuites` /
`numTotalTestSuites`) instead of individual test counts
(`numPassedTests` / `numTotalTests`).

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>

Author: Copilot

src/vscode-bicep/src/test/e2e/deploypane.test.ts

@@ -13,6 +13,9 @@ import { resolveExamplePath } from "./examples";
 
 const extensionLogPath = path.join(__dirname, `../../../${e2eLogName}`);
 
+// Each test opens a document (2s sleep) and waits up to 30s for the webview to be ready.
+jest.setTimeout(60000);
+
 describe("deploypane", (): void => {
   afterEach(executeCloseAllEditors);
 

src/vscode-bicep/src/test/e2e/testReporter.ts

@@ -31,6 +31,6 @@ export default class TestReporter implements Pick<Reporter, "onTestResult" | "on
   onRunComplete(_contexts: Set<TestContext>, results: AggregatedResult): void {
     console.log("");
     console.log(`Test Suites: ${results.numPassedTestSuites} passed, ${results.numTotalTestSuites} total`);
-    console.log(`Tests:       ${results.numPassedTestSuites} passed, ${results.numTotalTestSuites} total`);
+    console.log(`Tests:       ${results.numPassedTests} passed, ${results.numTotalTests} total`);
   }
 }

src/vscode-bicep/src/test/e2e/visualizer.test.ts

@@ -18,6 +18,9 @@ import { resolveExamplePath } from "./examples";
 
 const extensionLogPath = path.join(__dirname, `../../../${e2eLogName}`);
 
+// Each test opens a document (2s sleep) and waits up to 20s for the visualizer to be ready.
+jest.setTimeout(30000);
+
 describe("visualizer", (): void => {
   afterEach(executeCloseAllEditors);
 
@@ -32,6 +35,7 @@ describe("visualizer", (): void => {
     const viewColumn = await executeShowVisualizerCommand(document.uri);
     await until(() => visualizerIsReady(document.uri), {
       interval: 100,
+      timeoutMs: 20000,
     });
     if (!visualizerIsReady(document.uri)) {
       throw new Error(`Expected visualizer to be ready for ${document.uri.toString()}`);
@@ -50,6 +54,7 @@ describe("visualizer", (): void => {
     const viewColumn = await executeShowVisualizerToSideCommand(document.uri);
     await until(() => visualizerIsReady(document.uri), {
       interval: 100,
+      timeoutMs: 20000,
     });
     if (!visualizerIsReady(document.uri)) {
       throw new Error(`Expected visualizer to be ready for ${document.uri.toString()}`);
@@ -68,6 +73,7 @@ describe("visualizer", (): void => {
 
     await until(() => visualizerIsReady(document.uri), {
       interval: 100,
+      timeoutMs: 20000,
     });
 
     if (!visualizerIsReady(document.uri)) {

src/vscode-bicep/src/test/utils/time.ts

@@ -8,13 +8,13 @@ export async function retryWhile<T>(
   predicate: (result: T) => boolean,
   retryOptions?: Readonly<{
     interval?: number;
-    timeout?: number;
+    timeoutMs?: number;
   }>,
 ): Promise<T> {
   let result = await func();
 
   const interval = retryOptions?.interval ?? 2000;
-  let count = (retryOptions?.timeout ?? 10000) / interval;
+  let count = (retryOptions?.timeoutMs ?? 10000) / interval;
 
   while (predicate(result)) {
     if (count-- <= 0) {