improve logging in shortcuts.py and turn to pytest in test_shortcut.py

Author: vyuroshchin

docs/index.rst

@@ -20,7 +20,7 @@ Installation
 
 .. md-tab-set::
 
-   .. md-tab-item:: Pip + PyPI (recommented)
+   .. md-tab-item:: Pip + PyPI (recommended)
 
       .. code-block:: console
 

openapi_schema_validator/shortcuts.py

@@ -18,8 +18,8 @@ def validate(
     schema_dict = cast(dict[str, Any], schema)
     cls.check_schema(schema_dict)
     validator = cls(schema_dict, *args, **kwargs)
-    error = best_match(
-        validator.evolve(schema=schema_dict).iter_errors(instance)
-    )
-    if error is not None:
+    errors = list(validator.evolve(schema=schema_dict).iter_errors(instance))
+    if errors:
+        error = best_match(errors)
+        error.message = f"Validation failed: {error.message}"
         raise error

openapi_schema_validator/validators.py

@@ -1,6 +1,5 @@
 from typing import Any
 from typing import cast
-
 from jsonschema import _keywords
 from jsonschema import _legacy_keywords
 from jsonschema.validators import Draft202012Validator

tests/unit/test_shortcut.py

@@ -1,21 +1,33 @@
-from unittest import TestCase
-
+import pytest
 from openapi_schema_validator import validate
 
 
-class ValidateTest(TestCase):
-    def test_validate_does_not_mutate_schema_adding_nullable_key(self):
-        schema = {
-            "type": "object",
-            "properties": {
-                "email": {"type": "string"},
-                "enabled": {
-                    "type": "boolean",
-                },
+@pytest.fixture(scope="function")
+def schema():
+    return {
+        "type": "object",
+        "properties": {
+            "email": {"type": "string"},
+            "enabled": {
+                "type": "boolean",
             },
-            "example": {"enabled": False, "email": "foo@bar.com"},
-        }
+        },
+        "example": {"enabled": False, "email": "foo@bar.com"},
+    }
+
+
+def test_validate_does_not_add_nullable_to_schema(schema):
+    """
+    Verify that calling validate does not add the 'nullable' key to the schema
+    """
+    validate({"email": "foo@bar.com"}, schema)
+    assert "nullable" not in schema["properties"]["email"].keys()
 
-        validate({"email": "foo@bar.com"}, schema)
 
-        self.assertTrue("nullable" not in schema["properties"]["email"].keys())
+def test_validate_does_not_mutate_schema(schema):
+    """
+    Verify that calling validate does not mutate the schema
+    """
+    original_schema = schema.copy()
+    validate({"email": "foo@bar.com"}, schema)
+    assert schema == original_schema