bvandeusen/FabledScribe
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 50 Wiki Activity

test: fix obsolete create_task kind=plan passthrough test
CI & Build / Python lint (push) Successful in 2s
Details
CI & Build / TypeScript typecheck (push) Successful in 33s
Details
CI & Build / Python tests (push) Successful in 48s
Details
CI & Build / Build & push image (push) Successful in 1m8s
Details

Browse Source 
test_create_task_passes_kind asserted create_task forwards kind=plan; the
hard-retire guard now rejects that. Exercise passthrough with kind=issue
instead. (Service-level create_note still accepts task_kind=plan by design —
the guard lives at the user-facing tool/route layer, not the primitive.)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Bryan Van Deusen
2026-06-14 12:34:30 -04:00
parent f7742173aa
commit e8d6de287b
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_mcp_tool_tasks_kind.py
+4 -3
View File
@@ -20,11 +20,12 @@ def _fake_note(task_kind="work"):
@pytest.mark.asyncio @pytest.mark.asyncio
async def test_create_task_passes_kind(): async def test_create_task_passes_kind():
mock = AsyncMock(return_value=_fake_note(task_kind="plan")) # kind=plan is retired (plans are milestones); 'issue' exercises passthrough.
mock = AsyncMock(return_value=_fake_note(task_kind="issue"))
with patch("scribe.mcp.tools.tasks.notes_svc.create_note", mock): with patch("scribe.mcp.tools.tasks.notes_svc.create_note", mock):
from scribe.mcp.tools.tasks import create_task from scribe.mcp.tools.tasks import create_task
await create_task(title="P", kind="plan") await create_task(title="P", kind="issue")
assert mock.call_args.kwargs["task_kind"] == "plan" assert mock.call_args.kwargs["task_kind"] == "issue"
@pytest.mark.asyncio @pytest.mark.asyncio