test for sign-off phase name

Author: douglowe

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

@@ -76,6 +76,13 @@ five-safes-crate:SignOffPhaseProperties
             }
         """
     ] ;
+    sh:property [
+        sh:description "Check if the Sign Off phase has a name" ;
+        sh:path schema:name ;
+        sh:minCount 1 ;
+        sh:severity sh:Warning ;
+        sh:message "The Sign-Off Phase SHOULD have a human-readable name" ;
+    ] ;
     sh:property [
         sh:description "Check if the Sign Off phase has an endTime" ;
         sh:path schema:endTime ;

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

@@ -115,6 +115,37 @@ def test_5src_signoff_phase_no_endtime():
     )
 
 
+def test_5src_signoff_phase_no_name():
+    """
+    Test a Five Safes Crate where the Sign-Off phase has no name.
+    """
+    sparql = (
+        SPARQL_PREFIXES
+        + """
+        DELETE {
+            ?signoff schema:name ?name .
+        }
+        WHERE {
+           ?signoff a schema:AssessAction ;
+               schema:additionalType <https://w3id.org/shp#SignOff> ;
+               schema:name ?name .
+        }
+        """
+    )
+
+    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 a human-readable name"
+        ],
+        profile_identifier="five-safes-crate",
+        rocrate_entity_mod_sparql=sparql,
+    )
+
+
 def test_5src_signoff_phase_no_actionstatus():
     """
     Test a Five Safes Crate where the Sign-Off phase has no actionStatus.