fix(harness): update tests for _run_entry full result passthrough

Two tests were asserting on keys from the old compact wrapper dict
(artifact_dir, tags at top level). Update them to match the new
behavior: capture artifact_dir via the fake's closure, and have the
fake return tags as real _run_single does.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: charlesbluca

nemo_retriever/tests/test_harness_run.py

@@ -322,7 +322,10 @@ def test_run_entry_session_artifact_dir_uses_run_name(monkeypatch, tmp_path: Pat
     )
     monkeypatch.setattr(harness_run, "load_harness_config", lambda **_: cfg)
 
+    captured_artifact_dir: list[Path] = []
+
     def _fake_run_single(_cfg: HarnessConfig, _artifact_dir: Path, run_id: str, tags: list[str] | None = None) -> dict:
+        captured_artifact_dir.append(_artifact_dir)
         assert tags == []
         return {
             "success": True,
@@ -333,15 +336,15 @@ def _fake_run_single(_cfg: HarnessConfig, _artifact_dir: Path, run_id: str, tags
 
     monkeypatch.setattr(harness_run, "_run_single", _fake_run_single)
 
-    result = harness_run._run_entry(
+    harness_run._run_entry(
         run_name="jp20_single",
         config_file=None,
         session_dir=tmp_path,
         dataset="jp20",
         preset="single_gpu",
     )
 
-    assert Path(result["artifact_dir"]).name == "jp20_single"
+    assert captured_artifact_dir[0].name == "jp20_single"
 
 
 def test_run_entry_returns_tags(monkeypatch, tmp_path: Path) -> None:
@@ -363,6 +366,7 @@ def _fake_run_single(_cfg: HarnessConfig, _artifact_dir: Path, run_id: str, tags
             "return_code": 0,
             "failure_reason": None,
             "summary_metrics": {"pages": 0, "ingest_secs": 1.0, "pages_per_sec_ingest": 0.0, "recall_5": None},
+            "tags": tags,
         }
 
     monkeypatch.setattr(harness_run, "_run_single", _fake_run_single)