changed sh:or to sh:xone

Author: EttoreM

rocrate_validator/profiles/five-safes-crate/should/5_profile_conformance.ttl

@@ -64,14 +64,13 @@ five-safes-crate:RootDatasetLicenseWhenPublished
     sh:description "If the root dataset is published (has schema:publisher), it SHOULD declare a license." ;
     sh:targetClass ro-crate:RootDataEntity ;
     sh:severity sh:Warning ;
-    sh:message "Profile Conformance: Published crates SHOULD include a license." ;
-    sh:or (
-        # license not required if no publisher:
+    sh:message "A crate SHOULD include a license if and only if it has a publisher." ;
+    sh:xone (
+        # license SHOULD be present if and only if the publisher is specified:
         [ sh:not [ sh:property [
             sh:path schema:publisher ;
             sh:minCount 1 ;
         ]]]
-        # license required if publisher present:
         [ sh:property [
             sh:path schema:license ;
             sh:minCount 1 ;

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

@@ -183,7 +183,39 @@ def test_5src_root_data_entity_has_publisher_but_not_license():
         expected_validation_result=False,
         expected_triggered_requirements=["License present on published crates"],
         expected_triggered_issues=[
-            "Profile Conformance: Published crates SHOULD include a license."
+            "A crate SHOULD include a license if and only if it has a publisher."
+        ],
+        profile_identifier="five-safes-crate",
+        rocrate_entity_mod_sparql=sparql,
+    )
+
+
+def test_5src_root_data_entity_has_license_but_not_publisher():
+    """
+    Test a Five Safes Crate where the RootDataEntity has publisher but not license.
+    """
+    sparql = (
+        SPARQL_PREFIXES
+        + """
+        DELETE {
+            ?rootdataentity schema:publisher ?publisher .
+        }
+        WHERE {
+            ?metadatafile a schema:CreativeWork ;
+                          schema:about ?rootdataentity .
+            ?rootdataentity schema:publisher ?publisher ;
+                            schema:license ?license .
+        }
+        """
+    )
+
+    do_entity_test(
+        rocrate_path=ValidROC().five_safes_crate_result,
+        requirement_severity=Severity.RECOMMENDED,
+        expected_validation_result=False,
+        expected_triggered_requirements=["License present on published crates"],
+        expected_triggered_issues=[
+            "A crate SHOULD include a license if and only if it has a publisher."
         ],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,