docs: update tests README.md with final structure

- Document final test organization with accurate test counts
- Update to reflect integration tests moved to npm/ directory
- Include detailed file listings and test counts
- Clarify backward compatibility and migration benefits
- Final counts: 37 native + 39 npm + 7 unit = 83 total tests

tests/README.md

@@ -2,40 +2,85 @@
 
 ## Organization
 
-- `native/` - Traditional installation (curl|bash, irm|iex)
-  - `unix/` - Unix/Linux/macOS native tests
+- `native/` - Native installation tests only (curl|bash, irm|iex)
+  - `unix/` - Unix/Linux/macOS native tests (37 tests)
   - `windows/` - Windows PowerShell tests
-- `npm/` - npm package tests
-  - `postinstall.test.js` - Postinstall behavior tests
+- `npm/` - npm package tests only (39 tests)
+  - `postinstall.test.js` - Postinstall behavior tests (Section 10)
   - `cli.test.js` - CLI argument parsing tests
   - `cross-platform.test.js` - Cross-platform compatibility tests
-- `unit/` - Node.js unit tests
-- `integration/` - Integration tests
-- `shared/` - Shared utilities and test data
+  - `special-commands.test.js` - npm package integration tests
+- `shared/` - Shared utilities, fixtures, and unit tests
+  - `helpers.sh` - Bash test utilities and functions
+  - `test-data.js` - Test data for npm tests
+  - `fixtures/` - Test configuration files
+  - `unit/` - Unit tests for helper functions (7 tests)
 
 ## Running Tests
 
-- All tests: `npm test`
-- npm package tests only: `npm run test:npm`
-- Native installation tests only: `npm run test:native`
-- Unit tests only: `npm run test:unit`
-- Integration tests only: `npm run test:integration`
-- Master test suite (backward compatible): `npm run test:edge-cases`
+- **All tests**: `npm test` (83 tests total)
+- **npm package only**: `npm run test:npm` (39 tests)
+- **Native installation only**: `npm run test:native` (37 tests)
+- **Unit tests only**: `npm run test:unit` (7 tests)
+- **Master orchestrator**: `npm run test:edge-cases` (backward compatible)
 
 ## Test Structure
 
-### Native Tests
+### Native Tests (`native/`)
 Test the traditional installation methods where CCS is installed via:
-- Unix/Linux/macOS: `curl | bash`
-- Windows: `irm | iex` (PowerShell)
+- Unix/Linux/macOS: `curl | bash` → tests `lib/ccs`
+- Windows: `irm | iex` (PowerShell) → tests `lib/ccs.ps1`
 
-These tests use bash/PowerShell scripts and test the `lib/ccs` and `lib/ccs.ps1` files.
+These tests use bash/PowerShell scripts and cover Sections 1-9 from the original edge-cases.sh.
 
-### npm Tests
+**Files**:
+- `native/unix/edge-cases.sh` - 37 native Unix tests
+- `native/windows/edge-cases.ps1` - Windows PowerShell tests
+- `native/unix/install.sh` - Unix installation tests
+- `native/windows/install.ps1` - Windows installation tests
+
+### npm Tests (`npm/`)
 Test the npm package installation where CCS is installed via:
-- npm: `npm install -g @kaitranntt/ccs`
+- npm: `npm install -g @kaitranntt/ccs` → tests `bin/ccs.js`
 
-These tests use Node.js/mocha framework and test the `bin/ccs.js` file.
+These tests use Node.js/mocha framework and include Section 10 (postinstall) plus CLI tests.
 
-### Shared Utilities
-Common test code, data, and helper functions are stored in `shared/` to avoid duplication across test suites.
+**Files**:
+- `npm/postinstall.test.js` - 6 postinstall behavior tests (Section 10)
+- `npm/cli.test.js` - 15 CLI argument parsing tests
+- `npm/cross-platform.test.js` - 13 cross-platform compatibility tests
+- `npm/special-commands.test.js` - 5 integration tests for npm package
+
+### Shared Resources (`shared/`)
+Common test code, data, and helper functions shared across test suites to avoid duplication.
+
+**Files**:
+- `shared/helpers.sh` - Bash test utilities and functions
+- `shared/test-data.js` - Test data for npm tests
+- `shared/fixtures/` - Test configuration files
+- `shared/unit/` - 7 unit tests for helper functions
+
+## Test Counts
+
+| Test Type | Count | Location |
+|-----------|-------|----------|
+| Native Unix | 37 | `native/unix/` |
+| npm Package | 39 | `npm/` |
+| Unit Tests | 7 | `shared/unit/` |
+| **Total** | **83** | **All suites** |
+
+## Backward Compatibility
+
+All existing commands still work:
+- `bash tests/edge-cases.sh` - Master orchestrator (runs all tests)
+- `npm test` - Now runs comprehensive test suite
+- No breaking changes to existing workflows
+
+## Migration Notes
+
+This restructure solves the original problem where Section 10 (npm postinstall tests) was buried in the native `edge-cases.sh` file. Now:
+- ✅ Clear separation: npm tests in `npm/`, native in `native/`
+- ✅ Targeted execution: `npm run test:npm` vs `npm run test:native`
+- ✅ Better organization: Obvious where to add new tests
+- ✅ DRY principle: Shared utilities in `shared/`
+- ✅ Increased coverage: From 41 to 83 tests