ed/manual_slop
Private
Public Access
0
0
Fork 0
Code Activity

conductor(plan): Mark Task 1.4 complete (offloading Result unwrap tests)

Browse Source 
Task 1.4: 2 new tests in tests/test_app_controller_offloading.py cover the
Result unwrap happy path and the error path with logging.debug assertion.

Refs: 4b07e934
This commit is contained in:
ed
2026-06-18 19:33:37 -04:00
parent 4b07e9341c
commit 5d00581234
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
conductor/tracks/result_migration_app_controller_20260618/plan.md
+2
View File
@@ -62,6 +62,8 @@ Focus: register the sub-track, then immediately fix the 2 known regressions (tes
- **COMMIT:** `fix(app_controller): _offload_entry_payload unwraps Result from session_logger (regression fix)`
- **GIT NOTE:** Closes the regression in `test_tool_ask_approval`; the `session_logger.log_tool_call` was partially migrated to return `Result` but the call site was not updated. The convention's "AND over OR" pattern handles it here.
- [x] **Task 1.4: Add test for the unwrap path** [4b07e934]
### Task 1.4: Add test for the unwrap path
- **WHERE:** `tests/test_app_controller_offloading.py` (existing file; add 2 new tests)
- **WHAT:** Add 2 tests: