Remove support for defaults in lists/dicts. Closes #170

Author: dbanty

CHANGELOG.md

@@ -4,6 +4,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 0.6.0 - Unreleased
+### Breaking Changes
+- Reorganized api calls in generated clients. `async_api` will no longer be generated. Each path operation will now 
+    have it's own module under its tag. For example, if there was a generated function `api.my_tag.my_function()` it is 
+    replaced with `api.my_tag.my_function.sync()`. The async version can be called with `asyncio()` instead of `sync()`.
+- Removed support for mutable default values (e.g. dicts, lists). They may be added back in a future version given enough 
+    demand, but the existing implementation was not up to this project's standards.
+
 
 ## 0.5.4 - Unreleased
 ### Additions

end_to_end_tests/golden-master/my_test_api_client/api/tests/test_defaults_tests_test_defaults_post.py

@@ -1,5 +1,5 @@
 import datetime
-from dataclasses import asdict, field
+from dataclasses import asdict
 from typing import Any, Dict, List, Optional, Union, cast
 
 import httpx
@@ -20,9 +20,7 @@ def _get_kwargs(
     float_prop: Optional[float] = 3.14,
     int_prop: Optional[int] = 7,
     boolean_prop: Optional[bool] = False,
-    list_prop: Optional[List[AnEnum]] = field(
-        default_factory=lambda: cast(Optional[List[AnEnum]], [AnEnum.FIRST_VALUE, AnEnum.SECOND_VALUE])
-    ),
+    list_prop: Optional[List[AnEnum]] = None,
     union_prop: Optional[Union[Optional[float], Optional[str]]] = "not a float",
     enum_prop: Optional[AnEnum] = None,
 ) -> Dict[str, Any]:
@@ -105,9 +103,7 @@ def sync(
     float_prop: Optional[float] = 3.14,
     int_prop: Optional[int] = 7,
     boolean_prop: Optional[bool] = False,
-    list_prop: Optional[List[AnEnum]] = field(
-        default_factory=lambda: cast(Optional[List[AnEnum]], [AnEnum.FIRST_VALUE, AnEnum.SECOND_VALUE])
-    ),
+    list_prop: Optional[List[AnEnum]] = None,
     union_prop: Optional[Union[Optional[float], Optional[str]]] = "not a float",
     enum_prop: Optional[AnEnum] = None,
 ) -> Union[
@@ -144,9 +140,7 @@ async def asyncio(
     float_prop: Optional[float] = 3.14,
     int_prop: Optional[int] = 7,
     boolean_prop: Optional[bool] = False,
-    list_prop: Optional[List[AnEnum]] = field(
-        default_factory=lambda: cast(Optional[List[AnEnum]], [AnEnum.FIRST_VALUE, AnEnum.SECOND_VALUE])
-    ),
+    list_prop: Optional[List[AnEnum]] = None,
     union_prop: Optional[Union[Optional[float], Optional[str]]] = "not a float",
     enum_prop: Optional[AnEnum] = None,
 ) -> Union[

end_to_end_tests/golden-master/my_test_api_client/models/a_model.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 import datetime
-from dataclasses import dataclass, field
+from dataclasses import dataclass
 from typing import Any, Dict, List, Optional, Union, cast
 
 from .an_enum import AnEnum
@@ -16,9 +16,7 @@ class AModel:
     some_dict: Dict[Any, Any]
     a_camel_date_time: Union[datetime.datetime, datetime.date]
     a_date: datetime.date
-    nested_list_of_enums: Optional[List[List[DifferentEnum]]] = field(
-        default_factory=lambda: cast(Optional[List[List[DifferentEnum]]], [])
-    )
+    nested_list_of_enums: Optional[List[List[DifferentEnum]]] = None
 
     def to_dict(self) -> Dict[str, Any]:
         an_enum_value = self.an_enum_value.value

openapi_python_client/parser/properties.py

@@ -233,20 +233,10 @@ def get_imports(self, *, prefix: str) -> Set[str]:
         imports = super().get_imports(prefix=prefix)
         imports.update(self.inner_property.get_imports(prefix=prefix))
         imports.add("from typing import List")
-        if self.default is not None:
-            imports.add("from dataclasses import field")
-            imports.add("from typing import cast")
         return imports
 
-    def _validate_default(self, default: Any) -> str:
-        if not isinstance(default, list):
-            raise ValidationError()
-
-        default = list(map(self.inner_property._validate_default, default))
-        if isinstance(self.inner_property, RefProperty):  # Fix enums to use the actual value
-            default = str(default).replace("'", "")
-
-        return f"field(default_factory=lambda: cast({self.get_type_string()}, {default}))"
+    def _validate_default(self, default: Any) -> None:
+        return None
 
 
 @dataclass
@@ -432,10 +422,8 @@ def get_imports(self, *, prefix: str) -> Set[str]:
             imports.add("from typing import cast")
         return imports
 
-    def _validate_default(self, default: Any) -> str:
-        if isinstance(default, dict):
-            return repr(default)
-        raise ValidationError
+    def _validate_default(self, default: Any) -> None:
+        return None
 
 
 def _string_based_property(