Add OpenAPI 3.2 tags

Author: p1c2u

openapi_spec_validator/validation/keywords.py

@@ -642,6 +642,18 @@ def __call__(self, components: SchemaPath) -> Iterator[ValidationError]:
             yield from self.schemas_validator(schemas)
 
 
+class OpenAPIV32TagsValidator(KeywordValidator):
+    def __call__(self, tags: SchemaPath) -> Iterator[ValidationError]:
+        seen: set[str] = set()
+        for tag in tags:
+            tag_name = (tag / "name").read_str()
+            if tag_name in seen:
+                yield OpenAPIValidationError(
+                    f"Duplicate tag name '{tag_name}'"
+                )
+            seen.add(tag_name)
+
+
 class RootValidator(KeywordValidator):
     @property
     def paths_validator(self) -> PathsValidator:
@@ -658,3 +670,15 @@ def __call__(self, spec: SchemaPath) -> Iterator[ValidationError]:
         if "components" in spec:
             components = spec / "components"
             yield from self.components_validator(components)
+
+
+class OpenAPIV32RootValidator(RootValidator):
+    @property
+    def tags_validator(self) -> OpenAPIV32TagsValidator:
+        return cast(OpenAPIV32TagsValidator, self.registry["tags"])
+
+    def __call__(self, spec: SchemaPath) -> Iterator[ValidationError]:
+        if "tags" in spec:
+            tags = spec / "tags"
+            yield from self.tags_validator(tags)
+        yield from super().__call__(spec)

openapi_spec_validator/validation/validators.py

@@ -154,7 +154,7 @@ class OpenAPIV31SpecValidator(SpecValidator):
 class OpenAPIV32SpecValidator(SpecValidator):
     schema_validator = openapi_v32_schema_validator
     keyword_validators = {
-        "__root__": keywords.RootValidator,
+        "__root__": keywords.OpenAPIV32RootValidator,
         "components": keywords.ComponentsValidator,
         "content": keywords.ContentValidator,
         "default": keywords.OpenAPIV32ValueValidator,
@@ -168,5 +168,6 @@ class OpenAPIV32SpecValidator(SpecValidator):
         "responses": keywords.ResponsesValidator,
         "schema": keywords.OpenAPIV32SchemaValidator,
         "schemas": keywords.SchemasValidator,
+        "tags": keywords.OpenAPIV32TagsValidator,
     }
     root_keywords = ["paths", "components"]

tests/integration/validation/test_validators.py

@@ -199,6 +199,64 @@ def test_additional_operations_are_semantically_validated(self):
         assert len(errors) == 1
         assert "Path parameter 'item_id'" in errors[0].message
 
+    def test_top_level_duplicate_tags_are_invalid(self):
+        spec = {
+            "openapi": "3.2.0",
+            "info": {
+                "title": "Tag API",
+                "version": "1.0.0",
+            },
+            "tags": [
+                {
+                    "name": "pets",
+                },
+                {
+                    "name": "pets",
+                },
+            ],
+            "paths": {
+                "/pets": {
+                    "get": {
+                        "responses": {
+                            "200": {
+                                "description": "ok",
+                            },
+                        },
+                    },
+                },
+            },
+        }
+
+        errors = list(OpenAPIV32SpecValidator(spec).iter_errors())
+
+        assert len(errors) == 1
+        assert errors[0].message == "Duplicate tag name 'pets'"
+
+    def test_operation_tags_without_root_declaration_are_valid(self):
+        spec = {
+            "openapi": "3.2.0",
+            "info": {
+                "title": "Tag API",
+                "version": "1.0.0",
+            },
+            "paths": {
+                "/pets": {
+                    "get": {
+                        "tags": ["pets", "animals"],
+                        "responses": {
+                            "200": {
+                                "description": "ok",
+                            },
+                        },
+                    },
+                },
+            },
+        }
+
+        errors = list(OpenAPIV32SpecValidator(spec).iter_errors())
+
+        assert not errors
+
 
 def test_oas31_query_operation_is_not_semantically_traversed():
     spec = {