add check for signoff agent

Author: douglowe

rocrate_validator/profiles/five-safes-crate/should/4_sign_off.ttl

@@ -89,6 +89,13 @@ five-safes-crate:SignOffPhaseProperties
         sh:minCount 1 ;
         sh:severity sh:Warning ;
         sh:message "The Sign-Off Phase SHOULD have an actionStatus" ;
+    ] ;
+    sh:property [
+        sh:description "Check if the Sign Off phase has an agent" ;
+        sh:path schema:agent ;
+        sh:minCount 1 ;
+        sh:severity sh:Warning ;
+        sh:message "The Sign-Off Phase SHOULD have an agent" ;
     ] .
 
 

tests/integration/profiles/five-safes-crate/test_5src_signoff_phase.py

@@ -143,4 +143,35 @@ def test_5src_signoff_phase_no_actionstatus():
         ],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
+    )
+
+
+def test_5src_signoff_phase_no_agent():
+    """
+    Test a Five Safes Crate where the Sign-Off phase has no agent.
+    """
+    sparql = (
+        SPARQL_PREFIXES
+        + """
+        DELETE {
+            ?signoff schema:agent ?agent .
+        }
+        WHERE {
+           ?signoff a schema:AssessAction ;
+               schema:additionalType <https://w3id.org/shp#SignOff> ;
+               schema:agent ?agent .
+        }
+        """
+    )
+
+    do_entity_test(
+        rocrate_path=ValidROC().five_safes_crate_result,
+        requirement_severity=Severity.RECOMMENDED,
+        expected_validation_result=False,
+        expected_triggered_requirements=["SignOffPhaseProperties"],
+        expected_triggered_issues=[
+            "The Sign-Off Phase SHOULD have an agent"
+        ],
+        profile_identifier="five-safes-crate",
+        rocrate_entity_mod_sparql=sparql,
     )