Move integration tests into test/integ (#328)

This unifies the package & integ tests under one directory, also co-located with test artifacts, resulting a cleaner directory structure.

Co-authored-by: Mackenzie Zastrow <[email protected]>

Author: zastrowm

AGENTS.md

@@ -90,7 +90,7 @@ sdk-typescript/
 │   │   └── README.md             # Notebook tool documentation
 │   └── README.md                 # Vended tools overview
 │
-├── tests_integ/                  # Integration tests (separate from source)
+├── test/integ/                  # Integration tests (separate from source)
 │   ├── bedrock.test.ts           # Bedrock integration tests (requires AWS credentials)
 │   ├── hooks.test.ts             # Hooks integration tests
 │   └── registry.test.ts          # ToolRegistry integration tests
@@ -140,7 +140,7 @@ sdk-typescript/
 - **`src/tools/`**: Tool definitions and types for agent tool use
 - **`src/types/`**: Core type definitions used across the SDK
 - **`vended_tools/`**: Optional vended tools (not part of core SDK, independently importable)
-- **`tests_integ/`**: Integration tests (tests public API and external integrations)
+- **`test/integ/`**: Integration tests (tests public API and external integrations)
 - **`.github/workflows/`**: CI/CD automation and quality gates
 - **`.project/`**: Task management and project tracking
 
@@ -284,7 +284,7 @@ export async function* mainFunction() {
 
 **For integration tests**:
 ```
-tests_integ/
+test/integ/
 └── feature.test.ts        # Tests public API
 ```
 
@@ -669,18 +669,18 @@ src/subdir/
 
 ### Integration Test Location
 
-**Rule**: Integration tests are separate in `tests_integ/`
+**Rule**: Integration tests are separate in `test/integ/`
 
 ```
-tests_integ/
+test/integ/
 ├── api.test.ts                 # Tests public API
 └── environment.test.ts         # Tests environment compatibility
 ```
 
 ### Test File Naming
 
 - Unit tests: `{sourceFileName}.test.ts` in `src/**/__tests__/**`
-- Integration tests: `{feature}.test.ts` in `tests_integ/`
+- Integration tests: `{feature}.test.ts` in `test/integ/`
 
 ### Test Coverage
 

CONTRIBUTING.md

@@ -75,7 +75,7 @@ npm run test:watch
 npm run test:integ
 
 # Run integ tests for a single file
-npm run test:integ -- tests_integ/openai.test.ts
+npm run test:integ -- test/integ/openai.test.ts
 
 # Run browser tests (Chromium)
 npm run test:browser
@@ -91,7 +91,7 @@ npm run test:all:coverage
 
 - **80%+ Coverage**: All code should have at least 80% test coverage
 - **Unit Tests**: Test individual functions in `src/**/__tests__/**` directories
-- **Integration Tests**: Test complete workflows in `tests_integ/` directory
+- **Integration Tests**: Test complete workflows in `test/integ/` directory
 - **TSDoc Coverage**: All exported functions must have complete documentation
 
 For detailed testing patterns and examples, see [AGENTS.md - Testing Patterns](AGENTS.md#testing-patterns).

eslint.config.js

@@ -22,13 +22,13 @@ export default [
   }),
   // Apply UT rules to the integ tests
   unitTestRules({
-    files: ['tests_integ/**/*.ts'],
-    tsconfig: './tests_integ/tsconfig.json',
+    files: ['test/integ/**/*.ts'],
+    tsconfig: './test/integ/tsconfig.json',
   }),
   // Then stricter integ test rules
   integTestRules({
-    files: ['tests_integ/**/*.ts'],
-    tsconfig: './tests_integ/tsconfig.json',
+    files: ['test/integ/**/*.ts'],
+    tsconfig: './test/integ/tsconfig.json',
   }),
 ]
 

package.json

@@ -55,11 +55,11 @@
     "test:all": "vitest run --project unit-node --project unit-browser",
     "test:all:coverage": "vitest run --coverage --project unit-node --project unit-browser",
     "test:package": "cd test/packages/esm-module && npm install && node esm.js && cd ../cjs-module && npm install && node cjs.js",
-    "lint": "eslint src tests_integ",
-    "lint:fix": "eslint src tests_integ --fix",
-    "format": "prettier --write src tests_integ",
-    "format:check": "prettier --check src tests_integ",
-    "type-check": "tsc --noEmit --project src/tsconfig.json && tsc --noEmit --project tests_integ/tsconfig.json",
+    "lint": "eslint src test/integ",
+    "lint:fix": "eslint src test/integ --fix",
+    "format": "prettier --write src test/integ",
+    "format:check": "prettier --check src test/integ",
+    "type-check": "tsc --noEmit --project src/tsconfig.json && tsc --noEmit --project test/integ/tsconfig.json",
     "type-check:watch": "tsc --noEmit --watch",
     "prepare": "npm run build && husky"
   },

…integ/__fixtures__/model-test-helpers.ts …integ/__fixtures__/model-test-helpers.tstests_integ/__fixtures__/model-test-helpers.ts renamed to test/integ/__fixtures__/model-test-helpers.ts tests_integ/__fixtures__/model-test-helpers.ts renamed to test/integ/__fixtures__/model-test-helpers.ts

@@ -3,7 +3,7 @@ import { join } from 'node:path'
 
 /**
  * Helper to load fixture files from Vite URL imports.
- * Vite ?url imports return paths like '/tests_integ/__resources__/file.png' in test environment.
+ * Vite ?url imports return paths like '/test/integ/__resources__/file.png' in test environment.
  *
  * @param url - The URL from a Vite ?url import
  * @returns The file contents as a Uint8Array