refactor(tests): convert PlanTemplateServiceTest to Pest functional syntax

Convert PHPUnit class-based tests to Pest functional syntax with:
- 47 test cases organised into 9 describe blocks
- Proper beforeEach/afterEach hooks for test setup/teardown
- Covers: template listing, retrieval, preview, variable substitution,
  plan creation, validation, categories, context generation, edge cases
- Uses expect() assertions and method chaining for clarity

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Snider 2026-01-29 18:56:51 +00:00
parent 489c8136e1
commit 9e513af049
2 changed files with 453 additions and 336 deletions

View file

@ -70,9 +70,9 @@ Production-quality task list for the AI agent orchestration package.
- 78 test cases for security-critical IP whitelisting - 78 test cases for security-critical IP whitelisting
- [x] **TEST-004: Add PlanTemplateService tests** (FIXED 2026-01-29) - [x] **TEST-004: Add PlanTemplateService tests** (FIXED 2026-01-29)
- Created `tests/Feature/PlanTemplateServiceTest.php` - Created `tests/Feature/PlanTemplateServiceTest.php` using Pest functional syntax
- Covers: template loading, variable substitution, plan creation, validation - Covers: template listing, retrieval, preview, variable substitution, plan creation, validation, categories, context generation
- 35+ test cases with temporary template file handling - 47 test cases organised into 9 describe blocks with proper beforeEach/afterEach setup
- [x] **TEST-005: Add AI provider service tests** (FIXED 2026-01-29) - [x] **TEST-005: Add AI provider service tests** (FIXED 2026-01-29)
- Created `tests/Unit/ClaudeServiceTest.php` - Anthropic Claude API tests - Created `tests/Unit/ClaudeServiceTest.php` - Anthropic Claude API tests

File diff suppressed because it is too large Load diff