refactor(datafabric): address review nits (split bind test, single-string ontology prompt)

Author: sankalp-uipath

src/uipath_langchain/agent/tools/datafabric_tool/datafabric_prompt_builder.py

@@ -153,17 +153,13 @@ def format_sql_context(ctx: SQLContext, ontology_names: list[str] | None = None)
 
     if ontology_names:
         names = ", ".join(f"`{n}`" for n in ontology_names)
-        lines.append("## Available Ontology (authoritative semantic schema)")
-        lines.append("")
         lines.append(
+            "## Available Ontology (authoritative semantic schema)\n\n"
             f"This agent has a semantic ontology attached for these entities: "
             f"{names}. It is the authoritative source for the exact column names, "
             "value formats (date formats, codes, zero-padding), allowed values, "
             "and the relationships between entities — richer and more reliable "
-            "than the field list below, which omits value formats and semantics."
-        )
-        lines.append("")
-        lines.append(
+            "than the field list below, which omits value formats and semantics.\n\n"
             "**Before writing any SQL, call the `fetch_ontology` tool once** to "
             "load it, then base your column names, filter values, and joins on "
             "what it says. The entity tables below are a quick reference only; "

tests/agent/tools/test_datafabric_ontology_subgraph.py

@@ -57,14 +57,16 @@ def _tc(name, args=None, cid="c1"):
     return {"name": name, "args": args or {}, "id": cid, "type": "tool_call"}
 
 
-def test_fetch_ontology_bound_only_when_ontologies(make_graph):
+def test_fetch_ontology_bound_when_ontologies_present(make_graph):
+    with_onto = make_graph([("library", None)])
+    assert "fetch_ontology" in with_onto._tools_by_name
+
+
+def test_fetch_ontology_not_bound_when_ontologies_absent(make_graph):
     without = make_graph(None)
     assert "execute_sql" in without._tools_by_name
     assert "fetch_ontology" not in without._tools_by_name
 
-    with_onto = make_graph([("library", None)])
-    assert "fetch_ontology" in with_onto._tools_by_name
-
 
 def test_fetch_ontology_not_bound_when_flag_off(make_graph):
     # The feature flag decides which graph is built: even with ontologies