tests/test_mcp_tool_rulebooks.py

@@ -115,7 +115,46 @@ async def test_update_rule_only_sends_non_default_fields():
         await update_rule(rule_id=1, statement="new statement")
     args, kwargs = mock.call_args
     assert args == (1, 7)
-    assert kwargs == {"statement": "new statement"}
+    # `clear` is always forwarded — an empty tuple is "clear nothing", which is
+    # a value, not an absent argument. Everything the caller left at its
+    # default stays out: that is the property this test pins.
+    assert kwargs == {"statement": "new statement", "clear": ()}
+
+
+@pytest.mark.asyncio
+async def test_update_rule_forwards_the_fields_named_for_clearing():
+    """Naming a field is the only way to empty it through this door.
+
+    "" means "leave unchanged" here, so a caller has no value that means
+    "remove it" — which is what makes an explicit list necessary and what
+    stops a partial update from wiping the fields it did not mention.
+    """
+    rule = fake_rule(id=100, title="r", statement="s", topic_id=10)
+    mock = AsyncMock(return_value=rule)
+    with patch("scribe.mcp.tools.rulebooks.rulebooks_svc.update_rule", mock), _plain_detail():
+        from scribe.mcp.tools.rulebooks import update_rule
+        await update_rule(rule_id=1, clear_fields=["verify_with"])
+    _args, kwargs = mock.call_args
+    assert kwargs == {"clear": ["verify_with"]}
+
+
+@pytest.mark.asyncio
+async def test_update_rule_sends_the_check_fields_when_given():
+    rule = fake_rule(id=100, title="r", statement="s", topic_id=10)
+    mock = AsyncMock(return_value=rule)
+    with patch("scribe.mcp.tools.rulebooks.rulebooks_svc.update_rule", mock), _plain_detail():
+        from scribe.mcp.tools.rulebooks import update_rule
+        await update_rule(
+            rule_id=1,
+            verify_with="cat CI-runner/renovate/config.js",
+            expires_when="approval is turned off",
+        )
+    _args, kwargs = mock.call_args
+    assert kwargs == {
+        "verify_with": "cat CI-runner/renovate/config.js",
+        "expires_when": "approval is turned off",
+        "clear": (),
+    }
 
 
 @pytest.mark.asyncio