handle prefixes whe nconverting to sparql paths

buildingmotif/dataclasses/validation.py

@@ -14,7 +14,7 @@
 
 from buildingmotif import get_building_motif
 from buildingmotif.dataclasses.shape_collection import ShapeCollection
-from buildingmotif.namespaces import CONSTRAINT, PARAM, RDF, SH, A
+from buildingmotif.namespaces import CONSTRAINT, PARAM, RDF, SH, A, bind_prefixes
 from buildingmotif.utils import (
     _gensym,
     _guarantee_unique_template_name,
@@ -42,6 +42,9 @@ class GraphDiff:
     validation_result: Graph
     graph: Graph
 
+    def __post_init__(self):
+        bind_prefixes(self.graph)
+
     def resolve(self, lib: "Library") -> List["Template"]:
         """Produces a list of templates to resolve this GraphDiff.
 
@@ -184,7 +187,7 @@ def from_validation_report(cls, report: Graph) -> List["PathClassCount"]:
     def reason(self) -> str:
         """Human-readable explanation of this GraphDiff."""
         # interpret a SHACL property path as a sparql property path
-        path = self.graph.qname(shacl_path_to_sparql_path(self.graph, self.path))
+        path = shacl_path_to_sparql_path(self.graph, self.path, prefixes=dict(self.graph.namespaces()))
 
         classname = self.graph.qname(self.classname)
         if self.maxc is None and self.minc is not None:
@@ -374,7 +377,7 @@ def from_validation_report(cls, report: Graph) -> List["RequiredPath"]:
 
     def reason(self) -> str:
         """Human-readable explanation of this GraphDiff."""
-        path = self.graph.qname(shacl_path_to_sparql_path(self.graph, self.path))
+        path = shacl_path_to_sparql_path(self.graph, self.path, prefixes=dict(self.graph.namespaces()))
         if self.maxc is None and self.minc is not None:
             return f"{self.focus} needs at least {self.minc} uses of path {path}"
         if self.minc is None and self.maxc is not None:

tests/unit/api/test_model.py

@@ -277,7 +277,7 @@ def test_validate_model(client, building_motif, shacl_engine):
     )
 
     # Assert
-    assert results.status_code == 200
+    assert results.status_code == 200, results.data
 
     assert results.get_json().keys() == {"message", "reasons", "valid"}
     assert isinstance(results.get_json()["message"], str)
@@ -304,7 +304,7 @@ def test_validate_model(client, building_motif, shacl_engine):
     )
 
     # Assert
-    assert results.status_code == 200
+    assert results.status_code == 200, results.data
 
     assert results.get_json().keys() == {"message", "reasons", "valid"}
     assert isinstance(results.get_json()["message"], str)