fix: add conversion to dict type for attributes and relations in Resource check_all_values validator

Author: impocode

jsonapi_pydantic/constants.py

@@ -0,0 +1,13 @@
+from dataclasses import dataclass
+
+
+@dataclass
+class JsonApiVersion:
+    v1_0 = "1.0"
+    v1_1 = "1.1"
+
+
+JSON_API_VERSION = JsonApiVersion
+JSON_API_MEDIA_TYPE = "application/vnd.api+json"
+
+__all__ = ["JSON_API_MEDIA_TYPE"]

jsonapi_pydantic/v1_0/__init__.py

@@ -1,6 +1,7 @@
 from jsonapi_pydantic.v1_0.error import *
 from jsonapi_pydantic.v1_0.jsonapi import *
 from jsonapi_pydantic.v1_0.links import *
+from jsonapi_pydantic.v1_0.meta import *
 from jsonapi_pydantic.v1_0.resource import *
 from jsonapi_pydantic.v1_0.resource_identifier import *
 from jsonapi_pydantic.v1_0.toplevel import *

jsonapi_pydantic/v1_0/resource/resource.py

@@ -25,16 +25,24 @@ class Resource(BaseModel):
     def check_all_values(cls, values: dict) -> dict:
         # More about these restrictions: https://jsonapi.org/format/#document-resource-object-fields
         attributes, relationships = values.get("attributes"), values.get("relationships")
+        try:
+            attributes, relationships = dict(attributes), dict(relationships)
+        except ValueError:
+            raise ValueError("Attributes and relationships must be json objects.")
+
         if attributes and (attributes.get("id") or attributes.get("type")):
             raise ValueError("Attributes can not have keys named id or type.")
+
         if relationships and (relationships.get("id") or relationships.get("type")):
             raise ValueError("Relationships can not have keys named id or type.")
+
         if attributes and relationships:
             for key in attributes:
                 if relationships.get(key):
                     raise ValueError(
                         f"A resource can not have an attribute and relationship with the same name. Name: {key}."
                     )
+
         return values