fix(core): 🐛 ensure skip_checks is defined before use

Author: kikkomep

rocrate_validator/models.py

@@ -1671,10 +1671,20 @@ def __initialise__(cls, validation_settings: ValidationSettings):
                     if severity < severity_validation:
                         continue
                     # count the checks
-                    requirement_checks = [_ for _ in requirement.get_checks_by_level(LevelCollection.get(severity.name))
-                                          if _.identifier not in validation_settings.skip_checks and
-                                          (not _.overridden or
-                                          _.requirement.profile.identifier == target_profile_identifier)]
+                    requirement_checks = [
+                        _
+                        for _ in requirement.get_checks_by_level(
+                            LevelCollection.get(severity.name)
+                        )
+                        if (
+                            not validation_settings.skip_checks
+                            or _.identifier not in validation_settings.skip_checks
+                        )
+                        and (
+                            not _.overridden
+                            or _.requirement.profile.identifier == target_profile_identifier
+                        )
+                    ]
                     num_checks = len(requirement_checks)
                     requirement_checks_count += num_checks
                     if num_checks > 0:

rocrate_validator/requirements/shacl/checks.py

@@ -200,11 +200,12 @@ def __do_execute_check__(self, shacl_context: SHACLValidationContext):
             assert shape is not None, "Unable to map the violation to a shape"
             requirementCheck = SHACLCheck.get_instance(shape)
             assert requirementCheck is not None, "The requirement check cannot be None"
-            if requirementCheck.identifier not in shacl_context.settings.skip_checks:
+            if (not shacl_context.settings.skip_checks or
+                    requirementCheck.identifier not in shacl_context.settings.skip_checks):
                 failed_requirements_checks.add(requirementCheck)
                 violations = failed_requirements_checks_violations.get(requirementCheck.identifier, None)
                 if violations is None:
-                    failed_requirements_checks_violations[requirementCheck.identifier] = violations = []
+                    failed_requirements_checks_violations[requirementCheck.identifier] = (violations := [])
                 violations.append(violation)
         # sort the failed checks by identifier and severity
         # to ensure a consistent order of the issues