amati 0.2.1__py3-none-any.whl → 0.2.2__py3-none-any.whl

amati/amati.py

@@ -16,7 +16,7 @@ sys.path.insert(0, str(Path(__file__).parent.parent))
 from amati._error_handler import handle_errors
 from amati._resolve_forward_references import resolve_forward_references
 from amati.file_handler import load_file
-from amati.logging import Log, LogMixin
+from amati.logging import Log, Logger
 
 type JSONPrimitive = str | int | float | bool | None
 type JSONArray = list["JSONValue"]
@@ -113,9 +113,9 @@ def run(
 
     logs: list[Log] = []
 
-    with LogMixin.context():
+    with Logger.context():
         result, errors = dispatch(data)
-        logs.extend(LogMixin.logs)
+        logs.extend(Logger.logs)
 
     if errors or logs:
 
@@ -158,19 +158,38 @@ def run(
     if result and consistency_check:
         return check(data, result)
 
+    return True
 
-def discover(discover_dir: str = ".") -> list[Path]:
+
+def discover(spec: str, discover_dir: str = ".") -> list[Path]:
     """
     Finds OpenAPI Specification files to validate
 
     Args:
+        spec: The path to a specific OpenAPI specification file.
         discover_dir: The directory to search through.
     Returns:
-        A list of paths to validate.
+        A list of specifications to validate.
     """
 
     specs: list[Path] = []
 
+    # If a spec is provided, check if it exists and erorr if not
+    if spec:
+        spec_path = Path(spec)
+
+        if not spec_path.exists():
+            raise FileNotFoundError(f"File {spec} does not exist.")
+
+        if not spec_path.is_file():
+            raise IsADirectoryError(f"{spec} is a directory, not a file.")
+
+        specs.append(spec_path)
+
+        # End early if we're not also trying to find files
+        if not discover_dir:
+            return specs
+
     if Path("openapi.json").exists():
         specs.append(Path("openapi.json"))
 
@@ -258,16 +277,20 @@ if __name__ == "__main__":
     )
 
     args = parser.parse_args()
+    
+    print('Starting amati...')
 
-    if args.spec:
-        specifications: list[Path] = [Path(args.spec)]
-    else:
-        specifications = discover(args.discover)
+    specifications = discover(args.spec, args.discover)
+    print(specifications)
 
     for specification in specifications:
-        if successful_check := run(
+        successful_check = run(
             specification, args.consistency_check, args.local, args.html_report
-        ):
-            print("Consistency check successful for {specification}")
-        else:
-            print("Consistency check failed for {specification}")
+        )
+
+        if args.consistency_check and successful_check:
+            print(f"Consistency check successful for {specification}")
+        elif args.consistency_check:
+            print(f"Consistency check failed for {specification}")
+            
+    print('completed.')

amati/logging.py

@@ -18,7 +18,7 @@ class Log(TypedDict):
     url: NotRequired[str]
 
 
-class LogMixin:
+class Logger:
     """
     A mixin class that provides logging functionality.
 

amati/model_validators.py

@@ -10,7 +10,7 @@ from pydantic._internal._decorators import (
     PydanticDescriptorProxy,
 )
 
-from amati.logging import LogMixin
+from amati.logging import Logger
 from amati.validators.generic import GenericObject
 
 
@@ -107,7 +107,7 @@ def at_least_one_of(
         The validator that ensures at least one public field is non-empty.
 
     Example:
-        >>> LogMixin.logs = []
+        >>> Logger.logs = []
         >>>
         >>> class User(GenericObject):
         ...     name: str = ""
@@ -116,8 +116,8 @@ def at_least_one_of(
         ...     _reference_uri = "https://example.com"
         ...
         >>> user = User()
-        >>> assert len(LogMixin.logs) == 1
-        >>> LogMixin.logs = []
+        >>> assert len(Logger.logs) == 1
+        >>> Logger.logs = []
 
         >>> class User(GenericObject):
         ...     name: str = ""
@@ -128,11 +128,11 @@ def at_least_one_of(
         ...
         >>>
         >>> user = User(name="John")  # Works fine
-        >>> assert not LogMixin.logs
+        >>> assert not Logger.logs
         >>> user = User()
-        >>> assert len(LogMixin.logs) == 1
+        >>> assert len(Logger.logs) == 1
         >>> user = User(age=30)
-        >>> assert len(LogMixin.logs) == 2
+        >>> assert len(Logger.logs) == 2
 
 
     Note:
@@ -157,7 +157,7 @@ def at_least_one_of(
         public_fields = ", ".join(f"{name}" for name in candidates.keys())
 
         msg = f"{public_fields} do not have values, expected at least one."
-        LogMixin.log(
+        Logger.log(
             {
                 "msg": msg,
                 "type": "value_error",
@@ -191,7 +191,7 @@ def only_one_of(
         The validator that ensures at one public field is non-empty.
 
     Example:
-        >>> LogMixin.logs = []
+        >>> Logger.logs = []
         >>>
         >>> class User(GenericObject):
         ...     email: str = ""
@@ -201,10 +201,10 @@ def only_one_of(
         ...
         >>> user = User(email="test@example.com")  # Works fine
         >>> user = User(name="123-456-7890")  # Works fine
-        >>> assert not LogMixin.logs
+        >>> assert not Logger.logs
         >>> user = User(email="a@b.com", name="123")
-        >>> assert LogMixin.logs
-        >>> LogMixin.logs = []
+        >>> assert Logger.logs
+        >>> Logger.logs = []
 
         >>> class User(GenericObject):
         ...     name: str = ""
@@ -216,11 +216,11 @@ def only_one_of(
         >>> user = User(name="Bob")  # Works fine
         >>> user = User(email="test@example.com")  # Works fine
         >>> user = User(name="Bob", age=30)  # Works fine
-        >>> assert not LogMixin.logs
+        >>> assert not Logger.logs
         >>> user = User(name="Bob", email="a@b.com")
-        >>> assert len(LogMixin.logs) == 1
+        >>> assert len(Logger.logs) == 1
         >>> user = User(age=30)
-        >>> assert len(LogMixin.logs) == 2
+        >>> assert len(Logger.logs) == 2
 
     Note:
         Only public fields (not starting with '_') are checked. Private fields
@@ -249,7 +249,7 @@ def only_one_of(
                 field_string = "none"
             msg = f"Expected at most one field to have a value, {field_string} did"
 
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": msg,
                     "type": type_ or "value_error",
@@ -282,7 +282,7 @@ def all_of(
         The validator that ensures at most one public field is non-empty.
 
     Example:
-        >>> LogMixin.logs = []
+        >>> Logger.logs = []
         >>>
         >>> class User(GenericObject):
         ...     email: str = ""
@@ -291,11 +291,11 @@ def all_of(
         ...     _reference_uri = "https://example.com"
         ...
         >>> user = User(email="a@b.com", name="123") # Works fine
-        >>> assert not LogMixin.logs
+        >>> assert not Logger.logs
         >>> user = User(email="test@example.com")
-        >>> assert len(LogMixin.logs) == 1
+        >>> assert len(Logger.logs) == 1
         >>> user = User(name="123-456-7890")
-        >>> assert len(LogMixin.logs) == 2
+        >>> assert len(Logger.logs) == 2
 
         >>> class User(GenericObject):
         ...     name: str = ""
@@ -304,17 +304,17 @@ def all_of(
         ...     _all_of = all_of(["name", "email"])
         ...     _reference_uri = "https://example.com"
         ...
-        >>> LogMixin.logs = []
+        >>> Logger.logs = []
         >>> user = User(name="Bob", email="a@b.com") # Works fine
-        >>> assert not LogMixin.logs
+        >>> assert not Logger.logs
         >>> user = User(name="Bob")
-        >>> assert len(LogMixin.logs) == 1
+        >>> assert len(Logger.logs) == 1
         >>> user = User(email="test@example.com")
-        >>> assert len(LogMixin.logs) == 2
+        >>> assert len(Logger.logs) == 2
         >>> user = User(age=30)
-        >>> assert len(LogMixin.logs) == 3
+        >>> assert len(Logger.logs) == 3
         >>> user = User(name="Bob", age=30)
-        >>> assert len(LogMixin.logs) == 4
+        >>> assert len(Logger.logs) == 4
 
     Note:
         Only public fields (not starting with '_') are checked. Private fields
@@ -339,7 +339,7 @@ def all_of(
         if falsy:
             msg = f"Expected at all fields to have a value, {", ".join(falsy)} did not"
 
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": msg,
                     "type": "value_error",
@@ -378,7 +378,7 @@ def if_then(
         ValueError: If a condition and consequence are not present
 
     Example:
-        >>> LogMixin.logs = []
+        >>> Logger.logs = []
         >>>
         >>> class User(GenericObject):
         ...     role: str = ""
@@ -390,11 +390,11 @@ def if_then(
         ...     _reference_uri = "https://example.com"
         ...
         >>> user = User(role="admin", can_edit=True)  # Works fine
-        >>> assert not LogMixin.logs
+        >>> assert not Logger.logs
         >>> user = User(role="admin", can_edit=False)  # Fails validation
-        >>> assert len(LogMixin.logs) == 1
+        >>> assert len(Logger.logs) == 1
         >>> user = User(role="user", can_edit=False)  # Works fine
-        >>> assert len(LogMixin.logs) == 1
+        >>> assert len(Logger.logs) == 1
     """
 
     @model_validator(mode="after")
@@ -431,7 +431,7 @@ def if_then(
             if value == actual:
                 continue
 
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": f"Expected {field} to be {"in " if iterable else ""}"
                     f"{value} found {actual}",

amati/validators/generic.py

@@ -20,7 +20,7 @@ from typing import (
 from pydantic import BaseModel, ConfigDict, PrivateAttr
 from pydantic_core._pydantic_core import PydanticUndefined
 
-from amati.logging import LogMixin
+from amati.logging import Logger
 
 
 class GenericObject(BaseModel):
@@ -47,7 +47,7 @@ class GenericObject(BaseModel):
                 and field not in self.get_field_aliases()
             ):
                 message = f"{field} is not a valid field for {self.__repr_name__()}."
-                LogMixin.log(
+                Logger.log(
                     {
                         "msg": message,
                         "type": "value_error",
@@ -79,7 +79,7 @@ class GenericObject(BaseModel):
 
         for field in excess_fields:
             message = f"{field} is not a valid field for {self.__repr_name__()}."
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": message,
                     "type": "value_error",

amati/validators/oas304.py

@@ -42,7 +42,7 @@ from amati.fields import (
 from amati.fields.commonmark import CommonMark
 from amati.fields.json import JSON
 from amati.fields.oas import OpenAPI, RuntimeExpression
-from amati.logging import LogMixin
+from amati.logging import Logger
 from amati.validators.generic import GenericObject, allow_extra_fields
 
 type JSONPrimitive = str | int | float | bool | None
@@ -179,7 +179,7 @@ class ServerVariableObject(GenericObject):
             return self
 
         if self.default not in self.enum:
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": f"The default value {self.default} is not in the enum list {self.enum}",  # pylint: disable=line-too-long
                     "type": "warning",
@@ -647,7 +647,7 @@ class XMLObject(GenericObject):
         """
         if value.type == URIType.RELATIVE:
             message = "XML namespace {value} cannot be a relative URI"
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": message,
                     "type": "value_error",
@@ -726,7 +726,7 @@ class SchemaObject(GenericObject):
             # This will validate the structure conforms to JSON Schema
             validator_cls(meta_schema).validate(schema_dict)  # type: ignore
         except JSONVSchemeValidationError as e:
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": f"Invalid JSON Schema: {e.message}",
                     "type": "value_error",

amati/validators/oas311.py

@@ -37,7 +37,7 @@ from amati.fields.commonmark import CommonMark
 from amati.fields.json import JSON
 from amati.fields.oas import OpenAPI
 from amati.fields.spdx_licences import VALID_LICENCES
-from amati.logging import LogMixin
+from amati.logging import Logger
 from amati.validators.generic import GenericObject, allow_extra_fields
 from amati.validators.oas304 import (
     CallbackObject,
@@ -105,7 +105,7 @@ class LicenceObject(GenericObject):
             try:
                 SPDXURL(self.url)
             except AmatiValueError:
-                LogMixin.log(
+                Logger.log(
                     {
                         "msg": f"{str(self.url)} is not a valid SPDX URL",
                         "type": "warning",
@@ -122,7 +122,7 @@ class LicenceObject(GenericObject):
             and self.identifier
             and str(self.url) not in VALID_LICENCES[self.identifier]
         ):
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": f"{self.url} is not associated with the identifier {self.identifier}",  # pylint: disable=line-too-long
                     "type": "warning",
@@ -214,7 +214,7 @@ class ServerVariableObject(GenericObject):
             return self
 
         if self.default not in self.enum:
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": f"The default value {self.default} is not in the enum list {self.enum}",  # pylint: disable=line-too-long
                     "type": "value_error",
@@ -396,7 +396,7 @@ class SchemaObject(GenericObject):
             # This will validate the structure conforms to JSON Schema
             validator_cls(meta_schema).validate(schema_dict)  # type: ignore
         except JSONVSchemeValidationError as e:
-            LogMixin.log(
+            Logger.log(
                 {
                     "msg": f"Invalid JSON Schema: {e.message}",
                     "type": "value_error",

{amati-0.2.1.dist-info → amati-0.2.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: amati
-Version: 0.2.1
+Version: 0.2.2
 Summary: Validates that a .yaml or .json file conforms to the OpenAPI Specifications 3.x.
 Project-URL: Homepage, https://github.com/ben-alexander/amati
 Project-URL: Issues, https://github.com/ben-alexander/amati/issues

{amati-0.2.1.dist-info → amati-0.2.2.dist-info}/RECORD

@@ -1,11 +1,11 @@
 amati/__init__.py,sha256=bsPoaYcFmejsqQN6eDxcuGKXg8ZXgk1NoD6Hujwhkkw,428
 amati/_error_handler.py,sha256=s_Nhnq8hz7xQP_yeCCJ8Hwzs5xyrxlisd1XFYYECtSY,1244
 amati/_resolve_forward_references.py,sha256=iOibCv_XuIUpe7qbRBtPzd7H5rvL-aldUzlXV_9FJaw,6511
-amati/amati.py,sha256=sgWQrlvvusK8EgdcsYkCZWUNiv0TAHV8jRivUCBByQA,7698
+amati/amati.py,sha256=y-dHwu137R18hC-vWmOMmmFvagOJ0rpIsMwFUG4QOpw,8367
 amati/exceptions.py,sha256=X_RDBVjcFn_gXPFkhKuREmm5nDCdGs4rBy6sZsXBJDo,609
 amati/file_handler.py,sha256=h95t7TDxDA_qr4rIe2bddCjh2Xw5Ht27yLK7zCQzT68,5085
-amati/logging.py,sha256=X37gLui5Y94X1mlJIzkt3fQDwPcHlZiP3QBnM5Ipric,1339
-amati/model_validators.py,sha256=zKK9G-xaERc-e9AmWBLzCIGKkmpi0nMMx94N_nd8fs8,14979
+amati/logging.py,sha256=srg-T3z4mDFlP-RpcQhz685omnL24nXTthrTEIfKNTg,1337
+amati/model_validators.py,sha256=ffOmhhGRGPKZsr9TPW1q_57PoLfSq7zYNTq86t3BvJc,14915
 amati/data/http-status-codes.json,sha256=xEGBlE7eCXaQFG2gNSeK1aCs7A8XxOe6JcXwAKxwWBU,10903
 amati/data/iso9110.json,sha256=YLv6V8dPrgagjAnHUWKMmade5xboV_eJB9smnUfwgDY,1692
 amati/data/media-types.json,sha256=tSRZTUqIQeiW5cmw5aDAWNqXLGcvX4gwwEkDL5H6lDg,47108
@@ -27,11 +27,11 @@ amati/grammars/oas.py,sha256=vTMnJIxeiTC3gJATOo0RRZDlRsQOWkY6jG1VRzazLjM,1666
 amati/grammars/rfc6901.py,sha256=ChpKnoPDTsaNMvJT_UCpY8xwbl9zWZcD36MjRsD8IpQ,656
 amati/grammars/rfc7159.py,sha256=eksX1m7WRzQ9iGeANG1BgmsXbAhV6gQVRkPpLqPjq6U,2218
 amati/validators/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-amati/validators/generic.py,sha256=AYIHIuEHpf1p-lCynBp-YGaayIZOylzNgGi533ykVbc,3983
-amati/validators/oas304.py,sha256=84sAc37azkgrOt29scjRtYtJGypnFZd5UkV9VGLsX8c,32472
-amati/validators/oas311.py,sha256=FVMOjNXWlZhq6fgah0NGGPlgdcSZFgVxVbEwWnk4Sn4,17712
-amati-0.2.1.dist-info/METADATA,sha256=D2pXSaxSmc8pRwCdpqJNeGozaUZjKdsXvJ1BBODaF4Y,6425
-amati-0.2.1.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-amati-0.2.1.dist-info/entry_points.txt,sha256=sacBb6g0f0ZJtNjNYx93_Xe4y5xzawvklCFVXup9ru0,37
-amati-0.2.1.dist-info/licenses/LICENSE,sha256=WAA01ZXeNs1bwpNWKR6aVucjtYjYm_iQIUYkCAENjqM,1070
-amati-0.2.1.dist-info/RECORD,,
+amati/validators/generic.py,sha256=huFG9_Fbk5ih8y9S7fl8vRGl5Fr1158yb08967oUjw4,3977
+amati/validators/oas304.py,sha256=KUthNRVLRCjcLvN6IQuqLAUmui0WfKc8llRFt0qJaf4,32464
+amati/validators/oas311.py,sha256=XX8nUq0mGrbe5FR9C0_mTmIzMnWG09XbGwqpt8GHY8g,17702
+amati-0.2.2.dist-info/METADATA,sha256=jFpRbk4OdLgFrxuedrk9VjM0J-kqmNxfI7c93SzHWQA,6425
+amati-0.2.2.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+amati-0.2.2.dist-info/entry_points.txt,sha256=sacBb6g0f0ZJtNjNYx93_Xe4y5xzawvklCFVXup9ru0,37
+amati-0.2.2.dist-info/licenses/LICENSE,sha256=WAA01ZXeNs1bwpNWKR6aVucjtYjYm_iQIUYkCAENjqM,1070
+amati-0.2.2.dist-info/RECORD,,