test: cover 5 low-coverage modules (closes ROADMAP L2578 executor → all repos ≥80%)

[cipher813]

Summary

• Closes ROADMAP L2578 entirely — executor was the last repo below 80%
• Executor overall coverage: 76.30% → 80.03% ✅
• Test suite: 684 → 755 passing (+71 tests)
• All new tests use mocked IBKRClient / boto3 / subprocess / urllib — no IB Gateway, no S3, no real AWS credentials

Coverage delta

Module	Before	After	Notes
executor/config_loader.py	46.2%	100%	5 tests; path-resolution + no-silent-fallback guarantee
executor/price_monitor.py	22.4%	100%	21 tests; PriceMonitor lifecycle + tick aggregation + live/delayed field fallback
executor/bracket_orders.py	7.8%	98%	8 tests; market-BUY + trailing stop happy/partial/timeout/qualify-failure
executor/emergency_shutdown.py	0%	97%	12 tests; paper-account-only hard-exit + cascading failure resilience
executor/trade_logger.py	59.1%	100%	23 tests; full schema + idempotent migrations + entry-lookup helpers + S3 backup

ROADMAP L2578 status after this PR

All 6 repos at or above the 80% target:

Repo	Before this arc	After
Backtester	77.7%	80.3% (PR #198)
Predictor	74.9%	80.1% (PR #150)
Executor	76.3%	80.0% (this PR)
Data	82%	already ≥80%
Research	80%	already ≥80%
Dashboard	86%	already ≥80%

Notes

• executor/liquidate_all.py was excluded. It uses bare imports (from ibkr import IBKRClient, from trade_logger import ...) that only resolve when the file is run as a script from the executor/ directory; import executor.liquidate_all fails with ModuleNotFoundError. Would require a small refactor to package-qualified imports before becoming unit-testable. Not in scope for this coverage push.
• executor/bracket_orders.py:117 is an unreachable else branch (the upstream terminal_states set already covers every value the prior elif checks against). Pinned at 98% rather than 100%.

Test plan

• pytest -q — 755 passed
• pytest --cov=. --cov-report=term-missing — overall 80.03%
• No production-code changes
• No S3 / IB Gateway / network / AWS-credential dependencies in any new test
• No new dependencies in requirements.txt

🤖 Generated with Claude Code