claude-code/docs/test-plans/10-fix-weak-tests.md

Plan 10 — 修复 WEAK 评分测试文件

优先级：高 | 8 个文件 | 预估新增/修改 ~60 个测试用例

本计划修复 testing-spec.md 中评定为 WEAK 的 8 个测试文件的断言缺陷和覆盖缺口。

---

10.1 src/utils/__tests__/format.test.ts

问题：formatNumber、formatTokens、formatRelativeTime 使用 toContain 代替精确匹配，无法检测格式回归。

修改清单

formatNumber — toContain → toBe

// 当前（弱）
expect(formatNumber(1321)).toContain("k");
expect(formatNumber(1500000)).toContain("m");

// 修复为
expect(formatNumber(1321)).toBe("1.3k");
expect(formatNumber(1500000)).toBe("1.5m");

注意：Intl.NumberFormat 输出可能因 locale 不同。若 CI locale 不一致，改用 toMatch(/^\d+(\.\d)?[km]$/) 正则匹配。

formatTokens — 补精确断言

expect(formatTokens(1000)).toBe("1k");
expect(formatTokens(1500)).toBe("1.5k");

formatRelativeTime — toContain → toBe

// 当前（弱）
expect(formatRelativeTime(diff, now)).toContain("30");
expect(formatRelativeTime(diff, now)).toContain("ago");

// 修复为
expect(formatRelativeTime(diff, now)).toBe("30s ago");

新增：formatDuration 进位边界

用例	输入	期望
59.5s 进位	59500ms	至少含 1m
59m59s 进位	3599000ms	至少含 1h
sub-millisecond	0.5ms	"<1ms" 或 "0ms"

新增：未测试函数

函数	最少用例
formatRelativeTimeAgo	2（过去 / 未来）
formatLogMetadata	1（基本调用不抛错）
formatResetTime	2（有值 / null）
formatResetText	1（基本调用）

---

10.2 src/tools/shared/__tests__/gitOperationTracking.test.ts

问题：detectGitOperation 内部调用 getCommitCounter()、getPrCounter()、logEvent()，测试产生分析副作用。

修改清单

添加 analytics mock

在文件顶部添加 mock.module：

import { mock, afterAll, afterEach, beforeEach } from "bun:test";

mock.module("src/services/analytics/index.ts", () => ({
  logEvent: mock(() => {}),
}));

mock.module("src/bootstrap/state.ts", () => ({
  getCommitCounter: mock(() => ({ increment: mock(() => {}) })),
  getPrCounter: mock(() => ({ increment: mock(() => {}) })),
}));

需验证 detectGitOperation 的实际导入路径，按需调整 mock 目标。

新增：缺失的 GH PR actions

用例	输入	期望
gh pr edit	'gh pr edit 123 --title "fix"'	result.pr.number === 123
gh pr close	'gh pr close 456'	result.pr.number === 456
gh pr ready	'gh pr ready 789'	result.pr.number === 789
gh pr comment	'gh pr comment 123 --body "done"'	result.pr.number === 123

新增：parseGitCommitId 边界

用例	输入	期望
完整 40 字符 SHA	'[abcdef0123456789abcdef0123456789abcdef01] ...'	返回完整 40 字符
畸形括号输出	'create mode 100644 file.txt'	返回 null

---

10.3 src/utils/permissions/__tests__/PermissionMode.test.ts

问题：isExternalPermissionMode 在非 ant 环境永远返回 true，false 路径从未执行；mode 覆盖不完整。

修改清单

补全 mode 覆盖

函数	缺失的 mode
permissionModeTitle	bypassPermissions, dontAsk
permissionModeShortTitle	dontAsk, acceptEdits
getModeColor	dontAsk, acceptEdits, plan
permissionModeFromString	acceptEdits, bypassPermissions
toExternalPermissionMode	acceptEdits, bypassPermissions

修复 isExternalPermissionMode

// 当前：只测了非 ant 环境（永远 true）
// 需要新增 ant 环境测试
describe("when USER_TYPE is 'ant'", () => {
  beforeEach(() => {
    process.env.USER_TYPE = "ant";
  });
  afterEach(() => {
    delete process.env.USER_TYPE;
  });

  test("returns false for 'auto' in ant context", () => {
    expect(isExternalPermissionMode("auto")).toBe(false);
  });

  test("returns false for 'bubble' in ant context", () => {
    expect(isExternalPermissionMode("bubble")).toBe(false);
  });

  test("returns true for non-ant modes in ant context", () => {
    expect(isExternalPermissionMode("plan")).toBe(true);
  });
});

新增：permissionModeSchema

用例	输入	期望
有效 mode	'plan'	success: true
无效 mode	'invalid'	success: false

---

10.4 src/utils/permissions/__tests__/dangerousPatterns.test.ts

问题：纯数据 smoke test，无行为验证。

修改清单

新增：重复值检查

test("CROSS_PLATFORM_CODE_EXEC has no duplicates", () => {
  const set = new Set(CROSS_PLATFORM_CODE_EXEC);
  expect(set.size).toBe(CROSS_PLATFORM_CODE_EXEC.length);
});

test("DANGEROUS_BASH_PATTERNS has no duplicates", () => {
  const set = new Set(DANGEROUS_BASH_PATTERNS);
  expect(set.size).toBe(DANGEROUS_BASH_PATTERNS.length);
});

新增：全量成员断言（用 Set 确保精确）

test("CROSS_PLATFORM_CODE_EXEC contains expected interpreters", () => {
  const expected = ["node", "python", "python3", "ruby", "perl", "php",
    "bun", "deno", "npx", "tsx"];
  const set = new Set(CROSS_PLATFORM_CODE_EXEC);
  for (const entry of expected) {
    expect(set.has(entry)).toBe(true);
  }
});

新增：空字符串不匹配

test("empty string does not match any pattern", () => {
  for (const pattern of DANGEROUS_BASH_PATTERNS) {
    expect("".startsWith(pattern)).toBe(false);
  }
});

---

10.5 src/utils/__tests__/zodToJsonSchema.test.ts

问题：object 属性仅 toBeDefined 未验证类型结构；optional 字段未验证 absence。

修改清单

修复 object schema 测试

// 当前（弱）
expect(schema.properties!.name).toBeDefined();
expect(schema.properties!.age).toBeDefined();

// 修复为
expect(schema.properties!.name).toEqual({ type: "string" });
expect(schema.properties!.age).toEqual({ type: "number" });

修复 optional 字段测试

test("optional field is not in required array", () => {
  const schema = zodToJsonSchema(z.object({
    required: z.string(),
    optional: z.string().optional(),
  }));
  expect(schema.required).toEqual(["required"]);
  expect(schema.required).not.toContain("optional");
});

新增：缺失的 schema 类型

用例	输入	期望
z.literal("foo")	z.literal("foo")	{ const: "foo" }
z.null()	z.null()	{ type: "null" }
z.union()	z.union([z.string(), z.number()])	{ anyOf: [...] }
z.record()	z.record(z.string(), z.number())	{ type: "object", additionalProperties: { type: "number" } }
z.tuple()	z.tuple([z.string(), z.number()])	{ type: "array", items: [...], additionalItems: false }
嵌套 object	z.object({ a: z.object({ b: z.string() }) })	验证嵌套属性结构

---

10.6 src/utils/__tests__/envValidation.test.ts

问题：validateBoundedIntEnvVar lower bound=100 时 value=1 返回 status: "valid"，疑似源码 bug。

修改清单

验证 lower bound 行为

// 当前测试
test("value of 1 with lower bound 100", () => {
  const result = validateBoundedIntEnvVar("1", { defaultValue: 100, upperLimit: 1000, lowerLimit: 100 });
  // 如果源码有 bug，这里应该暴露
  expect(result.effective).toBeGreaterThanOrEqual(100);
  expect(result.status).toBe(result.effective !== 100 ? "capped" : "valid");
});

新增边界用例

用例	value	lowerLimit	期望
低于 lower bound	"50"	100	effective: 100, status: "capped"
等于 lower bound	"100"	100	effective: 100, status: "valid"
浮点截断	"50.7"	100	effective: 100（parseInt 截断后 cap）
空白字符	" 500 "	1	effective: 500, status: "valid"
defaultValue 为 0	"0"	0	需确认 parsed <= 0 逻辑

行动：先确认 validateBoundedIntEnvVar 源码中 lower bound 的实际执行路径。如果确实不生效，需先修源码再补测试。

---

10.7 src/utils/__tests__/file.test.ts

问题：addLineNumbers 仅 toContain，未验证完整格式。

修改清单

修复 addLineNumbers 断言

// 当前（弱）
expect(result).toContain("1");
expect(result).toContain("hello");

// 修复为（需确定 isCompactLinePrefixEnabled 行为）
// 假设 compact=false，格式为 "     1→hello"
test("formats single line with tab prefix", () => {
  // 先确认环境，如果 compact 模式不确定，用正则
  expect(result).toMatch(/^\s*\d+[→\t]hello$/m);
});

新增：stripLineNumberPrefix 边界

用例	输入	期望
纯数字行	"123"	""
无内容前缀	"→"	""
compact 格式 "1\thello"	"1\thello"	"hello"

新增：pathsEqual 边界

用例	a	b	期望
尾部斜杠差异	"/a/b"	"/a/b/"	false
.. 段	"/a/../b"	"/b"	视实现而定

---

10.8 src/utils/__tests__/notebook.test.ts

问题：mapNotebookCellsToToolResult 内容检查用 toContain，未验证 XML 格式。

修改清单

修复 content 断言

// 当前（弱）
expect(result).toContain("cell-0");
expect(result).toContain("print('hello')");

// 修复为
expect(result).toContain('<cell id="cell-0">');
expect(result).toContain("</cell>");

新增：parseCellId 边界

用例	输入	期望
负数	"cell--1"	null
前导零	"cell-007"	7
极大数	"cell-999999999"	999999999

新增：mapNotebookCellsToToolResult 边界

用例	输入	期望
空 data 数组	{ cells: [] }	空字符串或空结果
无 cell_id	{ cell_type: "code", source: "x" }	fallback 到 cell-${index}
error output	{ output_type: "error", ename: "Error", evalue: "msg" }	包含 error 信息

---

验收标准

• bun test 全部通过
• 8 个文件评分从 WEAK 提升至 ACCEPTABLE 或 GOOD
• toContain 仅用于警告文本等确实不确定精确值的场景
• envValidation bug 确认并修复（或确认非 bug 并更新测试）