@dommaker/harness 1.2.3 → 1.4.0

package/src/cli/commands/__tests__/check-skip-output.test.ts

@@ -5,7 +5,7 @@
  */
 
 import { check } from '../check';
-import { constraintChecker } from '../../../core/constraints/checker';
+import { captureIO, type CapturingIO } from '../../command-contract';
 
 // Mock fs
 jest.mock('fs', () => ({
@@ -26,12 +26,16 @@ jest.mock('child_process', () => ({
   execSync: jest.fn(() => Buffer.from('')),
 }));
 
-// Mock constraintChecker
+// Mock checker：命令 per-run 构造 checker（harness#88），构造替身即控 checkConstraints 返回值
+const mockChecker = { checkConstraints: jest.fn() };
 jest.mock('../../../core/constraints/checker', () => ({
-  constraintChecker: {
-    setTraceRecorder: jest.fn(),
-    checkConstraints: jest.fn(),
-  },
+  ConstraintChecker: jest.fn(function () {
+    return mockChecker;
+  }),
+}));
+
+jest.mock('../../../monitoring/traces', () => ({
+  getTraceCollector: jest.fn(() => ({ record: jest.fn() })),
 }));
 
 // Mock effective-constraints（ADR-0001：check 经 getMergedConstraintsConfig 走生效集链路）
@@ -55,8 +59,6 @@ jest.mock('chalk', () => ({
   red: jest.fn((str: string) => str),
 }));
 
-const mockChecker = constraintChecker as jest.Mocked<typeof constraintChecker>;
-
 function fakeConstraint(id: string, level: 'iron_law' | 'guideline') {
   return {
     kind: 'check' as const,
@@ -70,17 +72,12 @@ function fakeConstraint(id: string, level: 'iron_law' | 'guideline') {
 }
 
 describe('check 命令 skip 输出', () => {
-  let consoleSpy: jest.SpyInstance;
+  let io: CapturingIO;
 
   beforeEach(() => {
-    jest.clearAllMocks();
-    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
-    process.exitCode = 0;
-  });
 
-  afterEach(() => {
-    consoleSpy.mockRestore();
-    process.exitCode = 0;
+    io = captureIO();
+    jest.clearAllMocks();
   });
 
   it('skipped 单独列示，不计入铁律通过条数', async () => {
@@ -94,15 +91,15 @@ describe('check 命令 skip 输出', () => {
       warningCount: 0,
     });
 
-    await check({ preset: 'default', staged: false });
+    await check({ preset: 'default', staged: false }, io);
 
     // 通过条数只计实际评估的 1 条
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('铁律: 全部通过 (1 条)'));
+    expect(io.outText()).toContain('铁律: 全部通过 (1 条)');
     // skipped 单独列示
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('跳过评估: 1 条'));
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('- law_skip'));
+    expect(io.outText()).toContain('跳过评估: 1 条');
+    expect(io.outText()).toContain('- law_skip');
     // skip 不影响整体通过
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('约束检查通过'));
+    expect(io.outText()).toContain('约束检查通过');
   });
 
   it('指导原则通过计数排除 skipped', async () => {
@@ -116,9 +113,9 @@ describe('check 命令 skip 输出', () => {
       warningCount: 0,
     });
 
-    await check({ preset: 'default', staged: false });
+    await check({ preset: 'default', staged: false }, io);
 
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('指导原则: 1/1 通过'));
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('跳过评估: 1 条'));
+    expect(io.outText()).toContain('指导原则: 1/1 通过');
+    expect(io.outText()).toContain('跳过评估: 1 条');
   });
 });