test(executions): cover all report_outcome kwargs

Before this change, test_report_with_evidence exercised only three
of the ten optional evidence kwargs on report_outcome (external_id,
result_url, result_type). The remaining seven — result_ref,
summary, artifacts, metadata, result, error — were supported by
the SDK but had no test confirming they serialize correctly into
the POST body.

Most urgent of those is result_ref, which shipped in 0.1.4 via
PR #20 with no coverage at all. A future refactor dropping the
kwarg or breaking its serialization would pass CI silently.

Adds six tests:
- test_report_with_result_ref
- test_report_with_summary
- test_report_with_artifacts
- test_report_with_metadata
- test_report_with_all_evidence_fields — every optional kwarg in
  one call; guards against accidental field drops
- test_report_omits_none_kwargs — ensures unspecified kwargs do
  NOT appear in the body (important because server distinguishes
  "not provided" from "explicitly null" for evidence merge
  semantics)

Tests: 18/18 pass locally (was 12/12).

Author: Gk

tests/test_executions_resource.py

@@ -50,6 +50,122 @@ def test_report_with_evidence(self):
         assert body["result_type"] == "tweet"
         assert body["success"] is True
 
+    def test_report_with_result_ref(self):
+        # result_ref was added in 0.1.4 (PR #20); confirm it reaches the body.
+        mock_client = MagicMock()
+        resource = ExecutionsResource(mock_client)
+
+        resource.report_outcome(
+            "exec_123",
+            success=True,
+            result_ref="batch-id:7842",
+        )
+
+        body = mock_client._post.call_args.kwargs["json"]
+        assert body["result_ref"] == "batch-id:7842"
+        assert body["success"] is True
+
+    def test_report_with_summary(self):
+        mock_client = MagicMock()
+        resource = ExecutionsResource(mock_client)
+
+        resource.report_outcome(
+            "exec_123",
+            success=True,
+            summary="Generated 47 qualified leads",
+        )
+
+        body = mock_client._post.call_args.kwargs["json"]
+        assert body["summary"] == "Generated 47 qualified leads"
+
+    def test_report_with_artifacts(self):
+        mock_client = MagicMock()
+        resource = ExecutionsResource(mock_client)
+
+        artifacts = [
+            {"name": "leads.csv", "url": "https://storage/leads.csv"},
+            {"name": "report.pdf", "url": "https://storage/report.pdf"},
+        ]
+        resource.report_outcome(
+            "exec_123",
+            success=True,
+            artifacts=artifacts,
+        )
+
+        body = mock_client._post.call_args.kwargs["json"]
+        assert body["artifacts"] == artifacts
+
+    def test_report_with_metadata(self):
+        mock_client = MagicMock()
+        resource = ExecutionsResource(mock_client)
+
+        resource.report_outcome(
+            "exec_123",
+            success=True,
+            metadata={"agent": "lead-finder-v3", "duration_ms": 1420},
+        )
+
+        body = mock_client._post.call_args.kwargs["json"]
+        assert body["metadata"] == {"agent": "lead-finder-v3", "duration_ms": 1420}
+
+    def test_report_with_all_evidence_fields(self):
+        # Single request that exercises every optional kwarg the API
+        # currently accepts. Guards against a future refactor silently
+        # dropping one of them.
+        mock_client = MagicMock()
+        resource = ExecutionsResource(mock_client)
+
+        resource.report_outcome(
+            "exec_123",
+            success=True,
+            result="ok",
+            metadata={"foo": "bar"},
+            external_id="ext-1",
+            result_url="https://example.com/1",
+            result_ref="ref-1",
+            result_type="report",
+            summary="done",
+            artifacts=[{"name": "a.json", "url": "https://example.com/a"}],
+        )
+
+        body = mock_client._post.call_args.kwargs["json"]
+        assert body == {
+            "success": True,
+            "result": "ok",
+            "metadata": {"foo": "bar"},
+            "external_id": "ext-1",
+            "result_url": "https://example.com/1",
+            "result_ref": "ref-1",
+            "result_type": "report",
+            "summary": "done",
+            "artifacts": [{"name": "a.json", "url": "https://example.com/a"}],
+        }
+
+    def test_report_omits_none_kwargs(self):
+        # Optional kwargs left at their None default must NOT appear
+        # in the POST body. Important because the server distinguishes
+        # "field not provided" from "field explicitly set to null"
+        # for evidence merging semantics.
+        mock_client = MagicMock()
+        resource = ExecutionsResource(mock_client)
+
+        resource.report_outcome("exec_123", success=True)
+
+        body = mock_client._post.call_args.kwargs["json"]
+        assert body == {"success": True}
+        for key in (
+            "result",
+            "error",
+            "metadata",
+            "external_id",
+            "result_url",
+            "result_ref",
+            "result_type",
+            "summary",
+            "artifacts",
+        ):
+            assert key not in body
+
 
 class TestContextManager:
     def test_clean_exit_reports_success(self):