Decouple test helpers from production src

Author: brandonkachen

cli/src/__tests__/utils/env.test.ts

@@ -1,6 +1,6 @@
 import { describe, test, expect, afterEach } from 'bun:test'
 
-import { createTestCliEnv } from '../../testing/env'
+import { createTestCliEnv } from '../../../testing/env'
 import { getCliEnv } from '../../utils/env'
 
 describe('cli/utils/env', () => {

cli/src/testing/env.ts cli/testing/env.tscli/src/testing/env.ts renamed to cli/testing/env.ts

@@ -4,7 +4,7 @@
 
 import { createTestBaseEnv } from '@codebuff/common/testing/env-process'
 
-import type { CliEnv } from '../types/env'
+import type { CliEnv } from '../src/types/env'
 
 /**
  * Create a test CliEnv with optional overrides.
@@ -45,4 +45,3 @@ export const createTestCliEnv = (overrides: Partial<CliEnv> = {}): CliEnv => ({
   CODEBUFF_SCROLL_MULTIPLIER: undefined,
   ...overrides,
 })
-

cli/tsconfig.json

@@ -17,5 +17,14 @@
     }
   },
   "include": ["src/**/*"],
-  "exclude": ["node_modules", "dist"]
+  "exclude": [
+    "node_modules",
+    "dist",
+    "src/**/__tests__/**",
+    "src/**/__mocks__/**",
+    "src/**/*.test.ts",
+    "src/**/*.test.tsx",
+    "src/**/*.spec.ts",
+    "src/**/*.spec.tsx"
+  ]
 }

common/package.json

@@ -12,10 +12,40 @@
       "default": "./src/*.ts"
     },
     "./testing/fixtures": {
-      "bun": "./src/testing/fixtures/index.ts",
-      "import": "./src/testing/fixtures/index.ts",
-      "types": "./src/testing/fixtures/index.ts",
-      "default": "./src/testing/fixtures/index.ts"
+      "bun": "./testing/fixtures/index.ts",
+      "import": "./testing/fixtures/index.ts",
+      "types": "./testing/fixtures/index.ts",
+      "default": "./testing/fixtures/index.ts"
+    },
+    "./testing/fixtures/*": {
+      "bun": "./testing/fixtures/*.ts",
+      "import": "./testing/fixtures/*.ts",
+      "types": "./testing/fixtures/*.ts",
+      "default": "./testing/fixtures/*.ts"
+    },
+    "./testing/env-process": {
+      "bun": "./testing/env-process.ts",
+      "import": "./testing/env-process.ts",
+      "types": "./testing/env-process.ts",
+      "default": "./testing/env-process.ts"
+    },
+    "./testing/env-ci": {
+      "bun": "./testing/env-ci.ts",
+      "import": "./testing/env-ci.ts",
+      "types": "./testing/env-ci.ts",
+      "default": "./testing/env-ci.ts"
+    },
+    "./testing/mock-modules": {
+      "bun": "./testing/mock-modules.ts",
+      "import": "./testing/mock-modules.ts",
+      "types": "./testing/mock-modules.ts",
+      "default": "./testing/mock-modules.ts"
+    },
+    "./testing/impl/*": {
+      "bun": "./testing/impl/*.ts",
+      "import": "./testing/impl/*.ts",
+      "types": "./testing/impl/*.ts",
+      "default": "./testing/impl/*.ts"
     }
   },
   "scripts": {

common/src/__tests__/env-ci.test.ts

@@ -1,7 +1,7 @@
 import { describe, test, expect, afterEach } from 'bun:test'
 
 import { getCiEnv, ciEnv, isCI } from '../env-ci'
-import { createTestCiEnv } from '../testing/env-ci'
+import { createTestCiEnv } from '../../testing/env-ci'
 
 describe('env-ci', () => {
   describe('getCiEnv', () => {

common/src/__tests__/env-process.test.ts

@@ -1,7 +1,7 @@
 import { describe, test, expect, afterEach } from 'bun:test'
 
 import { getProcessEnv, processEnv } from '../env-process'
-import { createTestProcessEnv } from '../testing/env-process'
+import { createTestProcessEnv } from '../../testing/env-process'
 
 describe('env-process', () => {
   describe('getProcessEnv', () => {

common/src/testing/env-ci.ts common/testing/env-ci.tscommon/src/testing/env-ci.ts renamed to common/testing/env-ci.ts

@@ -2,7 +2,7 @@
  * Test-only CiEnv fixtures.
  */
 
-import type { CiEnv } from '../types/contracts/env'
+import type { CiEnv } from '../src/types/contracts/env'
 
 /**
  * Create a test CiEnv with optional overrides.
@@ -16,4 +16,3 @@ export const createTestCiEnv = (overrides: Partial<CiEnv> = {}): CiEnv => ({
   CODEBUFF_API_KEY: 'test-api-key',
   ...overrides,
 })
-

common/src/testing/env-process.ts common/testing/env-process.tscommon/src/testing/env-process.ts renamed to common/testing/env-process.ts

@@ -2,7 +2,7 @@
  * Test-only ProcessEnv fixtures.
  */
 
-import type { BaseEnv, ProcessEnv } from '../types/contracts/env'
+import type { BaseEnv, ProcessEnv } from '../src/types/contracts/env'
 
 /**
  * Create test defaults for BaseEnv.
@@ -84,4 +84,3 @@ export const createTestProcessEnv = (
   OVERRIDE_ARCH: undefined,
   ...overrides,
 })
-

…on/src/testing/fixtures/agent-runtime.ts common/testing/fixtures/agent-runtime.tscommon/src/testing/fixtures/agent-runtime.ts renamed to common/testing/fixtures/agent-runtime.ts common/src/testing/fixtures/agent-runtime.ts renamed to common/testing/fixtures/agent-runtime.ts

@@ -7,13 +7,13 @@
 
 import { spyOn } from 'bun:test'
 
-import type { AgentTemplate } from '../../types/agent-template'
+import type { AgentTemplate } from '../../src/types/agent-template'
 import type {
   AgentRuntimeDeps,
   AgentRuntimeScopedDeps,
-} from '../../types/contracts/agent-runtime'
-import type { ClientEnv, CiEnv } from '../../types/contracts/env'
-import type { Logger } from '../../types/contracts/logger'
+} from '../../src/types/contracts/agent-runtime'
+import type { ClientEnv, CiEnv } from '../../src/types/contracts/env'
+import type { Logger } from '../../src/types/contracts/logger'
 
 export const testLogger: Logger = {
   debug: () => {},

common/src/testing/fixtures/billing.ts common/testing/fixtures/billing.tscommon/src/testing/fixtures/billing.ts renamed to common/testing/fixtures/billing.ts

@@ -5,7 +5,7 @@
  * These helpers create properly-typed mocks without requiring ugly `as unknown as` casts.
  */
 
-import type { Logger } from '../../types/contracts/logger'
+import type { Logger } from '../../src/types/contracts/logger'
 
 // Re-export the test logger for convenience
 export { testLogger } from './agent-runtime'