Amended formatting

Author: EttoreM

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -87,9 +87,7 @@ def test_result_output_does_not_have_allowed_type():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["Output"],
-        expected_triggered_issues=[
-            "Result SHOULD have a `@type` among an allowed set of values."
-        ],
+        expected_triggered_issues=["Result SHOULD have a `@type` among an allowed set of values."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -267,9 +267,7 @@ def test_5src_workflow_object_has_no_end_time_if_ended():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["WorkflowExecution"],
-        expected_triggered_issues=[
-            "The workflow execution object SHOULD have an endTime property if it has ended."
-        ],
+        expected_triggered_issues=["The workflow execution object SHOULD have an endTime property if it has ended."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -75,9 +75,7 @@ def test_5src_validation_check_name_not_a_string():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["ValidationCheck"],
-        expected_triggered_issues=[
-            "ValidationCheck MUST have a human readable name string."
-        ],
+        expected_triggered_issues=["ValidationCheck MUST have a human readable name string."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -200,9 +198,7 @@ def test_5src_root_data_entity_does_not_mention_validation_check_entity():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["RootDataEntity"],
-        expected_triggered_issues=[
-            "RootDataEntity SHOULD mention a ValidationCheck object."
-        ],
+        expected_triggered_issues=["RootDataEntity SHOULD mention a ValidationCheck object."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -229,9 +225,7 @@ def test_5src_validation_check_object_does_not_point_to_root_data_entity():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["ValidationCheck"],
-        expected_triggered_issues=[
-            "`ValidationCheck` --> `object` SHOULD point to the root of the RO-Crate"
-        ],
+        expected_triggered_issues=["`ValidationCheck` --> `object` SHOULD point to the root of the RO-Crate"],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -283,9 +277,7 @@ def test_5src_Validation_check_does_not_have_action_status_property():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["ValidationCheck"],
-        expected_triggered_issues=[
-            "ValidationCheck SHOULD have actionStatus property."
-        ],
+        expected_triggered_issues=["ValidationCheck SHOULD have actionStatus property."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -45,9 +45,7 @@ def test_5src_download_action_does_not_have_name():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["DownloadAction"],
-        expected_triggered_issues=[
-            "DownloadAction MUST have a human readable name string."
-        ],
+        expected_triggered_issues=["DownloadAction MUST have a human readable name string."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -74,9 +72,7 @@ def test_5src_download_action_name_not_a_string():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["DownloadAction"],
-        expected_triggered_issues=[
-            "DownloadAction MUST have a human readable name string."
-        ],
+        expected_triggered_issues=["DownloadAction MUST have a human readable name string."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -206,10 +202,7 @@ def test_5src_downloaded_workflow_distribution_is_not_the_same_as_download_actio
         expected_validation_result=False,
         expected_triggered_requirements=["Downloaded Workflow"],
         expected_triggered_issues=[
-            (
-                "DownloadedWorkflow --> `distribution` MUST reference "
-                "the same entity as `DownloadAction` --> `object`."
-            )
+            ("DownloadedWorkflow --> `distribution` MUST reference the same entity as `DownloadAction` --> `object`.")
         ],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
@@ -295,9 +288,7 @@ def test_5src_root_data_entity_does_not_mention_download_action_entity():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["RootDataEntity"],
-        expected_triggered_issues=[
-            "RootDataEntity SHOULD mention DownloadAction if this exists."
-        ],
+        expected_triggered_issues=["RootDataEntity SHOULD mention DownloadAction if this exists."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -352,9 +343,7 @@ def test_5src_download_action_does_not_have_action_status_property():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
         expected_triggered_requirements=["DownloadAction"],
-        expected_triggered_issues=[
-            "`DownloadAction` SHOULD have `actionStatus` property."
-        ],
+        expected_triggered_issues=["`DownloadAction` SHOULD have `actionStatus` property."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -16,8 +16,8 @@
 import logging
 
 from rocrate_validator.models import Severity
-from tests.ro_crates import ValidROC, Invalid5sROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.ro_crates import Invalid5sROC, ValidROC
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -47,9 +47,7 @@ def test_5src_conforms_to_old_version():
         rocrate_path=ValidROC().five_safes_crate_request,
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
-        expected_triggered_requirements=[
-            "RO-Crate conforms to 1.2 or later minor version"
-        ],
+        expected_triggered_requirements=["RO-Crate conforms to 1.2 or later minor version"],
         expected_triggered_issues=[
             "The RO-Crate metadata file descriptor MUST have a `conformsTo` property with "
             "RO-Crate specification version 1.2 or later minor version"

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -44,12 +44,9 @@ def test_5src_assess_action_not_referenced_from_rde():
         rocrate_path=ValidROC().five_safes_crate_result,
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
-        expected_triggered_requirements=[
-            "All AssessActions are mentioned from Root Data Entity"
-        ],
+        expected_triggered_requirements=["All AssessActions are mentioned from Root Data Entity"],
         expected_triggered_issues=[
-            "All AssessAction entities in the crate MUST be referenced from "
-            "the Root Dataset via `mentions`."
+            "All AssessAction entities in the crate MUST be referenced from the Root Dataset via `mentions`."
         ],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -54,9 +54,7 @@ def test_5src_agent_memberOf_not_project():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["Requesting Agent"],
-        expected_triggered_issues=[
-            "The 'memberOf' property of an agent MUST be of type Project."
-        ],
+        expected_triggered_issues=["The 'memberOf' property of an agent MUST be of type Project."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -131,9 +129,7 @@ def test_5src_agent_memberOf_missing_warning():
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,  # or True if warnings are not treated as failures
         expected_triggered_requirements=["Requesting Agent"],
-        expected_triggered_issues=[
-            "The Requesting Agent SHOULD have a `memberOf` property."
-        ],
+        expected_triggered_issues=["The Requesting Agent SHOULD have a `memberOf` property."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -56,9 +56,7 @@ def test_5src_responsible_project_funding_not_grant():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["Responsible Project"],
-        expected_triggered_issues=[
-            "The property 'funding' of the Responsible Project MUST be of type Grant."
-        ],
+        expected_triggered_issues=["The property 'funding' of the Responsible Project MUST be of type Grant."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -136,9 +134,7 @@ def test_5src_responsible_project_member_and_agent_affiliation_no_intersection()
         rocrate_path=ValidROC().five_safes_crate_request,
         requirement_severity=Severity.RECOMMENDED,
         expected_validation_result=False,
-        expected_triggered_requirements=[
-            "Organizations (members of Responsible Project)"
-        ],
+        expected_triggered_requirements=["Organizations (members of Responsible Project)"],
         expected_triggered_issues=[
             (
                 "At least one of the organisations that are members of the responsible project SHOULD "
@@ -178,9 +174,7 @@ def test_5src_responsible_project_missing_funding_property():
         requirement_severity=Severity.OPTIONAL,
         expected_validation_result=False,
         expected_triggered_requirements=["Responsible Project"],
-        expected_triggered_issues=[
-            "The Responsible Project does not have the property `funding`."
-        ],
+        expected_triggered_issues=["The Responsible Project does not have the property `funding`."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )
@@ -211,9 +205,7 @@ def test_5src_responsible_project_missing_member_property():
         requirement_severity=Severity.OPTIONAL,
         expected_validation_result=False,
         expected_triggered_requirements=["Responsible Project"],
-        expected_triggered_issues=[
-            "The Responsible Project does not have the property `member`."
-        ],
+        expected_triggered_issues=["The Responsible Project does not have the property `member`."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -47,9 +47,7 @@ def test_5src_root_data_entity_without_source_organization():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["RootDataEntity"],
-        expected_triggered_issues=[
-            """The Root Data Entity MUST have a `sourceOrganization` property."""
-        ],
+        expected_triggered_issues=["""The Root Data Entity MUST have a `sourceOrganization` property."""],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )

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

@@ -17,7 +17,7 @@
 
 from rocrate_validator.models import Severity
 from tests.ro_crates import ValidROC
-from tests.shared import do_entity_test, SPARQL_PREFIXES
+from tests.shared import SPARQL_PREFIXES, do_entity_test
 
 # set up logging
 logger = logging.getLogger(__name__)
@@ -50,9 +50,7 @@ def test_createaction_does_not_have_agent():
         requirement_severity=Severity.REQUIRED,
         expected_validation_result=False,
         expected_triggered_requirements=["CreateAction"],
-        expected_triggered_issues=[
-            "CreateAction MUST have at least one agent that is a contextual entity."
-        ],
+        expected_triggered_issues=["CreateAction MUST have at least one agent that is a contextual entity."],
         profile_identifier="five-safes-crate",
         rocrate_entity_mod_sparql=sparql,
     )