test: add tests for CircuitBreaker service #6

New issue

Open

opened 2026-02-21 00:59:16 +00:00 by Clotho · 0 comments

Clotho commented

2026-02-21 00:59:16 +00:00

Member

Summary

The CircuitBreaker service (src/Mcp/Services/CircuitBreaker.php) has no unit tests.

What needs testing

Closed state: requests pass through normally
Open state: CircuitOpenException thrown when circuit is open
Half-open state: probe requests allowed, circuit re-closes on success
Failure threshold: circuit opens after N consecutive failures
Recovery timeout: circuit transitions to half-open after timeout
Success reset: consecutive successes close the circuit

Files

src/Mcp/Services/CircuitBreaker.php
src/Mcp/Exceptions/CircuitOpenException.php
src/Mcp/Tests/Unit/CircuitBreakerTest.php (to be created)

## Summary The `CircuitBreaker` service (src/Mcp/Services/CircuitBreaker.php) has no unit tests. ## What needs testing - Closed state: requests pass through normally - Open state: `CircuitOpenException` thrown when circuit is open - Half-open state: probe requests allowed, circuit re-closes on success - Failure threshold: circuit opens after N consecutive failures - Recovery timeout: circuit transitions to half-open after timeout - Success reset: consecutive successes close the circuit ## Files - `src/Mcp/Services/CircuitBreaker.php` - `src/Mcp/Exceptions/CircuitOpenException.php` - `src/Mcp/Tests/Unit/CircuitBreakerTest.php` (to be created)