test: add edge case tests for output channel state, batch parsing, and search pattern

- setPatchloomLog/getPatchloomLog round-trip module state
- parseBatchOperationCount rejects non-array operations values
- buildSearchQuickAction preserves spaces in multi-word patterns

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

Author: SebTardif

test/unit/batchApply.test.ts

@@ -40,3 +40,9 @@ test("parseBatchOperationCount returns 0 for missing operations", () => {
 test("parseBatchOperationCount returns 0 for empty operations array", () => {
   assert.equal(parseBatchOperationCount('{"operations": []}'), 0);
 });
+
+test("parseBatchOperationCount returns 0 when operations is not an array", () => {
+  assert.equal(parseBatchOperationCount('{"operations": "not-an-array"}'), 0);
+  assert.equal(parseBatchOperationCount('{"operations": 42}'), 0);
+  assert.equal(parseBatchOperationCount('{"operations": null}'), 0);
+});

test/unit/outputChannel.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
 import test from "node:test";
-import { createPatchloomLog } from "../../src/logging/outputChannel.js";
+import { createPatchloomLog, getPatchloomLog, setPatchloomLog } from "../../src/logging/outputChannel.js";
 
 test("createPatchloomLog lazily creates channel on first log call", () => {
   let channelCreated = false;
@@ -103,6 +103,26 @@ test("dispose disposes channel and subsequent log creates a new one", () => {
   assert.equal(createCount, 2);
 });
 
+test("setPatchloomLog and getPatchloomLog round-trip module state", () => {
+  const original = getPatchloomLog();
+  try {
+    assert.equal(getPatchloomLog(), undefined);
+
+    const log = createPatchloomLog(() => ({
+      appendLine() {},
+      show() {},
+      dispose() {}
+    }));
+    setPatchloomLog(log);
+    assert.equal(getPatchloomLog(), log);
+
+    setPatchloomLog(undefined);
+    assert.equal(getPatchloomLog(), undefined);
+  } finally {
+    setPatchloomLog(original);
+  }
+});
+
 test("logResult handles multiline stderr", () => {
   const lines: string[] = [];
   const log = createPatchloomLog(() => ({

test/unit/quickActions.test.ts

@@ -144,6 +144,12 @@ test("buildCreateQuickAction builds a create command", () => {
   assert.deepEqual(action.targetArgIndices, [1]);
 });
 
+test("buildSearchQuickAction preserves spaces in pattern as a single arg", () => {
+  const action = buildSearchQuickAction("/workspace/demo", "hello world");
+
+  assert.deepEqual(action.args, ["search", "hello world", "/workspace/demo"]);
+});
+
 test("buildDocGetQuickAction builds a doc get command", () => {
   const action = buildDocGetQuickAction("/workspace/demo/package.json", "scripts.test");