openapi-python-client 0.15.1__tar.gz → 0.16.0__tar.gz

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/CHANGELOG.md

@@ -13,6 +13,48 @@ Programmatic usage of this project (e.g., importing it as a Python module) and t
 
 The 0.x prefix used in versions for this project is to indicate that breaking changes are expected frequently (several times a year). Breaking changes will increment the minor number, all other changes will increment the patch number. You can track the progress toward 1.0 [here](https://github.com/openapi-generators/openapi-python-client/projects/2).
 
+## 0.16.0 (2023-12-07)
+
+### Breaking Changes
+
+#### Switch from Black to Ruff for formatting
+
+`black` is no longer a runtime dependency, so if you have them set in custom `post_hooks` in a config file, you'll need to make sure they're being installed manually. [`ruff`](https://docs.astral.sh/ruff) is now installed and used by default instead.
+
+#### Use Ruff instead of isort + autoflake at runtime
+
+`isort` and `autoflake` are no longer runtime dependencies, so if you have them set in custom `post_hooks` in a config file, you'll need to make sure they're being installed manually. [`ruff`](https://docs.astral.sh/ruff) is now installed and used by default instead.
+
+### Features
+
+#### Support all `text/*` content types in responses
+
+Within an API response, any content type which starts with `text/` will now be treated the same as `text/html` already was—they will return the `response.text` attribute from the [httpx Response](https://www.python-httpx.org/api/#response).
+
+Thanks to @fdintino for the initial implementation, and thanks for the discussions from @kairntech, @rubenfiszel, and @antoneladestito.
+
+Closes #797 and #821.
+
+#### Support `application/octet-stream` request bodies
+
+Endpoints that accept `application/octet-stream` request bodies are now supported using the same `File` type as octet-stream responses.
+
+Thanks to @kgutwin for the implementation and @rtaycher for the discussion!
+
+PR #899 closes #588
+
+### Fixes
+
+#### Remove useless `pass` statements from generated code
+
+## 0.15.2 (2023-09-16)
+
+### Features
+
+#### support httpx 0.25 (#854)
+
+#### Support content-type with attributes (#655, #809, #858). Thanks @sherbang!
+
 ## 0.15.1 (2023-08-12)
 
 ### Features

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: openapi-python-client
-Version: 0.15.1
+Version: 0.16.0
 Summary: Generate modern Python clients from OpenAPI
 Home-page: https://github.com/triaxtec/openapi-python-client
 License: MIT
@@ -16,20 +16,20 @@ Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: Software Development :: Code Generators
 Classifier: Typing :: Typed
 Requires-Dist: PyYAML (>=6.0,<7.0)
 Requires-Dist: attrs (>=21.3.0)
-Requires-Dist: autoflake (>=1.4,<3.0.0)
-Requires-Dist: black (>=23)
 Requires-Dist: colorama (>=0.4.3,<0.5.0) ; sys_platform == "win32"
-Requires-Dist: httpx (>=0.20.0,<0.25.0)
-Requires-Dist: isort (>=5.0.5,<6.0.0)
+Requires-Dist: httpx (>=0.20.0,<0.26.0)
 Requires-Dist: jinja2 (>=3.0.0,<4.0.0)
 Requires-Dist: pydantic (>=2.1.1,<3.0.0)
 Requires-Dist: python-dateutil (>=2.8.1,<3.0.0)
+Requires-Dist: ruff (>=0.1.2,<0.2.0)
 Requires-Dist: shellingham (>=1.3.2,<2.0.0)
 Requires-Dist: typer (>0.6,<0.10)
+Requires-Dist: typing-extensions (>=4.8.0,<5.0.0)
 Project-URL: Repository, https://github.com/triaxtec/openapi-python-client
 Description-Content-Type: text/markdown
 
@@ -37,7 +37,6 @@ Description-Content-Type: text/markdown
 [![codecov](https://codecov.io/gh/openapi-generators/openapi-python-client/branch/main/graph/badge.svg)](https://codecov.io/gh/triaxtec/openapi-python-client)
 [![MIT license](https://img.shields.io/badge/License-MIT-blue.svg)](https://lbesson.mit-license.org/)
 [![Generic badge](https://img.shields.io/badge/type_checked-mypy-informational.svg)](https://mypy.readthedocs.io/en/stable/introduction.html)
-[![Code style: black](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/ambv/black)
 [![PyPI version shields.io](https://img.shields.io/pypi/v/openapi-python-client.svg)](https://pypi.python.org/pypi/openapi-python-client/)
 [![Downloads](https://static.pepy.tech/personalized-badge/openapi-python-client?period=total&units=international_system&left_color=blue&right_color=green&left_text=Downloads)](https://pepy.tech/project/openapi-python-client)
 
@@ -56,13 +55,17 @@ This tool focuses on creating the best developer experience for Python developer
 
 1. Using all the latest and greatest Python features like type annotations and dataclasses.
 2. Having documentation and usage instructions specific to this one generator.
-1. Being written in Python with Jinja2 templates, making it easier to improve and extend for Python developers. It's also much easier to install and use if you already have Python.
+3. Being written in Python with Jinja2 templates, making it easier to improve and extend for Python developers. It's also much easier to install and use if you already have Python.
+
+## Sponsors
+
+<a href="https://www.devmark.ai/fern/?utm_source=openapi-python-client&utm_loc=readme&utm_type=logo" target="_blank" title="Fern | SDKs and API docs"><img src="https://raw.githubusercontent.com/openapi-generators/openapi-python-client/main/.github/sponsors/fern.png"></a>
 
 ## Installation
 
 I recommend you install with [pipx](https://pipxproject.github.io/pipx/) so you don't conflict with any other packages you might have: `pipx install openapi-python-client --include-deps`.
 
-> Note the `--include-deps` option which will also make `black`, `isort`, and `autoflake` available in your path so that `openapi-python-client` can use them to clean up the generated code.
+> Note the `--include-deps` option which will also make `black` and `ruff` available in your path so that `openapi-python-client` can use them to clean up the generated code.
 
 **If you use `pipx run` then the post-generation hooks will not be available unless you install them manually.**
 
@@ -186,9 +189,8 @@ In the config file, there's an easy way to tell `openapi-python-client` to run a
 
 ```yaml
 post_hooks:
-   - "autoflake -i -r --remove-all-unused-imports --remove-unused-variables --ignore-init-module-imports ."
-   - "isort ."
-   - "black ."
+   - "ruff check . --fix"
+   - "ruff format ."
 ```
 
 ### use_path_prefixes_for_title_model_names

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/README.md

@@ -2,7 +2,6 @@
 [![codecov](https://codecov.io/gh/openapi-generators/openapi-python-client/branch/main/graph/badge.svg)](https://codecov.io/gh/triaxtec/openapi-python-client)
 [![MIT license](https://img.shields.io/badge/License-MIT-blue.svg)](https://lbesson.mit-license.org/)
 [![Generic badge](https://img.shields.io/badge/type_checked-mypy-informational.svg)](https://mypy.readthedocs.io/en/stable/introduction.html)
-[![Code style: black](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/ambv/black)
 [![PyPI version shields.io](https://img.shields.io/pypi/v/openapi-python-client.svg)](https://pypi.python.org/pypi/openapi-python-client/)
 [![Downloads](https://static.pepy.tech/personalized-badge/openapi-python-client?period=total&units=international_system&left_color=blue&right_color=green&left_text=Downloads)](https://pepy.tech/project/openapi-python-client)
 
@@ -21,13 +20,17 @@ This tool focuses on creating the best developer experience for Python developer
 
 1. Using all the latest and greatest Python features like type annotations and dataclasses.
 2. Having documentation and usage instructions specific to this one generator.
-1. Being written in Python with Jinja2 templates, making it easier to improve and extend for Python developers. It's also much easier to install and use if you already have Python.
+3. Being written in Python with Jinja2 templates, making it easier to improve and extend for Python developers. It's also much easier to install and use if you already have Python.
+
+## Sponsors
+
+<a href="https://www.devmark.ai/fern/?utm_source=openapi-python-client&utm_loc=readme&utm_type=logo" target="_blank" title="Fern | SDKs and API docs"><img src="https://raw.githubusercontent.com/openapi-generators/openapi-python-client/main/.github/sponsors/fern.png"></a>
 
 ## Installation
 
 I recommend you install with [pipx](https://pipxproject.github.io/pipx/) so you don't conflict with any other packages you might have: `pipx install openapi-python-client --include-deps`.
 
-> Note the `--include-deps` option which will also make `black`, `isort`, and `autoflake` available in your path so that `openapi-python-client` can use them to clean up the generated code.
+> Note the `--include-deps` option which will also make `black` and `ruff` available in your path so that `openapi-python-client` can use them to clean up the generated code.
 
 **If you use `pipx run` then the post-generation hooks will not be available unless you install them manually.**
 
@@ -151,9 +154,8 @@ In the config file, there's an easy way to tell `openapi-python-client` to run a
 
 ```yaml
 post_hooks:
-   - "autoflake -i -r --remove-all-unused-imports --remove-unused-variables --ignore-init-module-imports ."
-   - "isort ."
-   - "black ."
+   - "ruff check . --fix"
+   - "ruff format ."
 ```
 
 ### use_path_prefixes_for_title_model_names

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/openapi_python_client/__init__.py

@@ -40,7 +40,7 @@ TEMPLATE_FILTERS = {
 }
 
 
-class Project:  # pylint: disable=too-many-instance-attributes
+class Project:
     """Represents a Python project (the top level file-tree) to generate"""
 
     def __init__(
@@ -151,7 +151,7 @@ class Project:  # pylint: disable=too-many-instance-attributes
             return
         try:
             cwd = self.package_dir if self.meta == MetaType.NONE else self.project_dir
-            subprocess.run(cmd, cwd=cwd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE, check=True)
+            subprocess.run(cmd, cwd=cwd, shell=True, capture_output=True, check=True)
         except CalledProcessError as err:
             self.errors.append(
                 GeneratorError(
@@ -253,7 +253,6 @@ class Project:  # pylint: disable=too-many-instance-attributes
         models_init_template = self.env.get_template("models_init.py.jinja")
         models_init.write_text(models_init_template.render(imports=imports, alls=alls), encoding=self.file_encoding)
 
-    # pylint: disable=too-many-locals
     def _build_api(self) -> None:
         # Generate Client
         client_path = self.package_dir / "client.py"
@@ -297,7 +296,7 @@ class Project:  # pylint: disable=too-many-instance-attributes
                 )
 
 
-def _get_project_for_url_or_path(  # pylint: disable=too-many-arguments
+def _get_project_for_url_or_path(
     url: Optional[str],
     path: Optional[Path],
     meta: MetaType,

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/openapi_python_client/cli.py

@@ -31,7 +31,8 @@ def _process_config(path: Optional[pathlib.Path]) -> Config:
 
 
 # noinspection PyUnusedLocal
-# pylint: disable=unused-argument
+
+
 @app.callback(name="openapi-python-client")
 def cli(
     version: bool = typer.Option(False, "--version", callback=_version_callback, help="Print the version and exit"),
@@ -110,7 +111,6 @@ _meta_option = typer.Option(
 CONFIG_OPTION = typer.Option(None, "--config", help="Path to the config file to use")
 
 
-# pylint: disable=too-many-arguments
 @app.command()
 def generate(
     url: Optional[str] = typer.Option(None, help="A URL to read the JSON from"),
@@ -149,7 +149,6 @@ def generate(
     handle_errors(errors, fail_on_warning)
 
 
-# pylint: disable=too-many-arguments
 @app.command()
 def update(
     url: Optional[str] = typer.Option(None, help="A URL to read the JSON from"),

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/openapi_python_client/config.py

@@ -29,9 +29,8 @@ class Config(BaseModel):
     package_version_override: Optional[str] = None
     use_path_prefixes_for_title_model_names: bool = True
     post_hooks: List[str] = [
-        "autoflake -i -r --remove-all-unused-imports --remove-unused-variables --ignore-init-module-imports .",
-        "isort .",
-        "black .",
+        "ruff check --fix .",
+        "ruff format .",
     ]
     field_prefix: str = "field_"
     http_timeout: int = 5

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/openapi_python_client/parser/openapi.py

@@ -3,7 +3,7 @@ from collections import OrderedDict
 from copy import deepcopy
 from dataclasses import dataclass, field
 from http import HTTPStatus
-from typing import Any, Dict, Iterator, List, Optional, Set, Tuple, Union
+from typing import Any, Dict, Iterator, List, Optional, Protocol, Set, Tuple, Union
 
 import attr
 from pydantic import ValidationError
@@ -46,7 +46,11 @@ class EndpointCollection:
 
     @staticmethod
     def from_data(
-        *, data: Dict[str, oai.PathItem], schemas: Schemas, parameters: Parameters, config: Config
+        *,
+        data: Dict[str, oai.PathItem],
+        schemas: Schemas,
+        parameters: Parameters,
+        config: Config,
     ) -> Tuple[Dict[utils.PythonIdentifier, "EndpointCollection"], Schemas, Parameters]:
         """Parse the openapi paths data to get EndpointCollections by tag"""
         endpoints_by_tag: Dict[utils.PythonIdentifier, EndpointCollection] = {}
@@ -72,7 +76,11 @@ class EndpointCollection:
                 # Add `PathItem` parameters
                 if not isinstance(endpoint, ParseError):
                     endpoint, schemas, parameters = Endpoint.add_parameters(
-                        endpoint=endpoint, data=path_data, schemas=schemas, parameters=parameters, config=config
+                        endpoint=endpoint,
+                        data=path_data,
+                        schemas=schemas,
+                        parameters=parameters,
+                        config=config,
                     )
                 if not isinstance(endpoint, ParseError):
                     endpoint = Endpoint.sort_parameters(endpoint=endpoint)
@@ -103,7 +111,15 @@ def generate_operation_id(*, path: str, method: str) -> str:
 models_relative_prefix: str = "..."
 
 
-# pylint: disable=too-many-instance-attributes
+class RequestBodyParser(Protocol):
+    __name__: str = "RequestBodyParser"
+
+    def __call__(
+        self, *, body: oai.RequestBody, schemas: Schemas, parent_name: str, config: Config
+    ) -> Tuple[Union[Property, PropertyError, None], Schemas]:
+        ...  # pragma: no cover
+
+
 @dataclass
 class Endpoint:
     """
@@ -126,6 +142,7 @@ class Endpoint:
     form_body: Optional[Property] = None
     json_body: Optional[Property] = None
     multipart_body: Optional[Property] = None
+    binary_body: Optional[Property] = None
     errors: List[ParseError] = field(default_factory=list)
     used_python_identifiers: Set[PythonIdentifier] = field(default_factory=set)
 
@@ -146,7 +163,13 @@ class Endpoint:
                 config=config,
             )
             if isinstance(prop, ModelProperty):
-                schemas = attr.evolve(schemas, classes_by_name={**schemas.classes_by_name, prop.class_info.name: prop})
+                schemas = attr.evolve(
+                    schemas,
+                    classes_by_name={
+                        **schemas.classes_by_name,
+                        prop.class_info.name: prop,
+                    },
+                )
             return prop, schemas
         return None, schemas
 
@@ -168,7 +191,13 @@ class Endpoint:
             )
             if isinstance(prop, ModelProperty):
                 prop = attr.evolve(prop, is_multipart_body=True)
-                schemas = attr.evolve(schemas, classes_by_name={**schemas.classes_by_name, prop.class_info.name: prop})
+                schemas = attr.evolve(
+                    schemas,
+                    classes_by_name={
+                        **schemas.classes_by_name,
+                        prop.class_info.name: prop,
+                    },
+                )
             return prop, schemas
         return None, schemas
 
@@ -179,9 +208,11 @@ class Endpoint:
         """Return json_body"""
         json_body = None
         for content_type, schema in body.content.items():
-            content_type = get_content_type(content_type)
+            parsed_content_type = get_content_type(content_type)
 
-            if content_type == "application/json" or content_type.endswith("+json"):
+            if parsed_content_type is not None and (
+                parsed_content_type == "application/json" or parsed_content_type.endswith("+json")
+            ):
                 json_body = schema
                 break
 
@@ -196,6 +227,30 @@ class Endpoint:
             )
         return None, schemas
 
+    @staticmethod
+    def parse_request_binary_body(
+        *, body: oai.RequestBody, schemas: Schemas, parent_name: str, config: Config
+    ) -> Tuple[Union[Property, PropertyError, None], Schemas]:
+        """Return binary_body"""
+        binary_body = None
+        for content_type, schema in body.content.items():
+            parsed_content_type = get_content_type(content_type)
+
+            if parsed_content_type == "application/octet-stream":
+                binary_body = schema
+                break
+
+        if binary_body is not None and binary_body.media_type_schema is not None:
+            return property_from_data(
+                name="binary_body",
+                required=True,
+                data=binary_body.media_type_schema,
+                schemas=schemas,
+                parent_name=parent_name,
+                config=config,
+            )
+        return None, schemas
+
     @staticmethod
     def _add_body(
         *,
@@ -209,59 +264,34 @@ class Endpoint:
         if data.requestBody is None or isinstance(data.requestBody, oai.Reference):
             return endpoint, schemas
 
-        form_body, schemas = Endpoint.parse_request_form_body(
-            body=data.requestBody, schemas=schemas, parent_name=endpoint.name, config=config
-        )
+        request_body_parsers: List[Tuple[str, RequestBodyParser]] = [
+            ("form_body", Endpoint.parse_request_form_body),
+            ("json_body", Endpoint.parse_request_json_body),
+            ("binary_body", Endpoint.parse_request_binary_body),
+            ("multipart_body", Endpoint.parse_multipart_body),
+        ]
 
-        if isinstance(form_body, ParseError):
-            return (
-                ParseError(
-                    header=f"Cannot parse form body of endpoint {endpoint.name}",
-                    detail=form_body.detail,
-                    data=form_body.data,
-                ),
-                schemas,
-            )
+        for property_name, parser in request_body_parsers:
+            body, schemas = parser(body=data.requestBody, schemas=schemas, parent_name=endpoint.name, config=config)
 
-        json_body, schemas = Endpoint.parse_request_json_body(
-            body=data.requestBody, schemas=schemas, parent_name=endpoint.name, config=config
-        )
-        if isinstance(json_body, ParseError):
-            return (
-                ParseError(
-                    header=f"Cannot parse JSON body of endpoint {endpoint.name}",
-                    detail=json_body.detail,
-                    data=json_body.data,
-                ),
-                schemas,
-            )
+            if isinstance(body, ParseError):
+                property_type = property_name
+                if property_type.endswith("_body"):
+                    property_type = property_type[:-5]
+                return (
+                    ParseError(
+                        header=f"Cannot parse {property_type} request body of endpoint {endpoint.name}",
+                        detail=body.detail,
+                        data=body.data,
+                    ),
+                    schemas,
+                )
 
-        multipart_body, schemas = Endpoint.parse_multipart_body(
-            body=data.requestBody, schemas=schemas, parent_name=endpoint.name, config=config
-        )
-        if isinstance(multipart_body, ParseError):
-            return (
-                ParseError(
-                    header=f"Cannot parse multipart body of endpoint {endpoint.name}",
-                    detail=multipart_body.detail,
-                    data=multipart_body.data,
-                ),
-                schemas,
-            )
+            if body is not None:
+                setattr(endpoint, property_name, body)
+                endpoint.relative_imports.update(body.get_imports(prefix=models_relative_prefix))
+                endpoint.relative_imports.update(body.get_lazy_imports(prefix=models_relative_prefix))
 
-        # No reasons to use lazy imports in endpoints, so add lazy imports to relative here.
-        if form_body is not None:
-            endpoint.form_body = form_body
-            endpoint.relative_imports.update(endpoint.form_body.get_imports(prefix=models_relative_prefix))
-            endpoint.relative_imports.update(endpoint.form_body.get_lazy_imports(prefix=models_relative_prefix))
-        if multipart_body is not None:
-            endpoint.multipart_body = multipart_body
-            endpoint.relative_imports.update(endpoint.multipart_body.get_imports(prefix=models_relative_prefix))
-            endpoint.relative_imports.update(endpoint.multipart_body.get_lazy_imports(prefix=models_relative_prefix))
-        if json_body is not None:
-            endpoint.json_body = json_body
-            endpoint.relative_imports.update(endpoint.json_body.get_imports(prefix=models_relative_prefix))
-            endpoint.relative_imports.update(endpoint.json_body.get_lazy_imports(prefix=models_relative_prefix))
         return endpoint, schemas
 
     @staticmethod
@@ -286,7 +316,11 @@ class Endpoint:
                 continue
 
             response, schemas = response_from_data(
-                status_code=status_code, data=response_data, schemas=schemas, parent_name=endpoint.name, config=config
+                status_code=status_code,
+                data=response_data,
+                schemas=schemas,
+                parent_name=endpoint.name,
+                config=config,
             )
             if isinstance(response, ParseError):
                 detail_suffix = "" if response.detail is None else f" ({response.detail})"
@@ -307,9 +341,8 @@ class Endpoint:
             endpoint.responses.append(response)
         return endpoint, schemas
 
-    # pylint: disable=too-many-return-statements
     @staticmethod
-    def add_parameters(
+    def add_parameters(  # noqa: PLR0911, PLR0912
         *,
         endpoint: "Endpoint",
         data: Union[oai.Operation, oai.PathItem],
@@ -338,7 +371,6 @@ class Endpoint:
             - https://swagger.io/docs/specification/describing-parameters/
             - https://swagger.io/docs/specification/paths-and-operations/
         """
-        # pylint: disable=too-many-branches, too-many-locals
         # There isn't much value in breaking down this function further other than to satisfy the linter.
 
         if data.parameters is None:
@@ -353,7 +385,15 @@ class Endpoint:
             oai.ParameterLocation.HEADER: endpoint.header_parameters,
             oai.ParameterLocation.COOKIE: endpoint.cookie_parameters,
             "RESERVED": {  # These can't be param names because codegen needs them as vars, the properties don't matter
-                "client": AnyProperty("client", True, False, None, PythonIdentifier("client", ""), None, None),
+                "client": AnyProperty(
+                    "client",
+                    True,
+                    False,
+                    None,
+                    PythonIdentifier("client", ""),
+                    None,
+                    None,
+                ),
                 "url": AnyProperty("url", True, False, None, PythonIdentifier("url", ""), None, None),
             },
         }
@@ -363,7 +403,7 @@ class Endpoint:
             param_or_error = parameter_from_reference(param=param, parameters=parameters)
             if isinstance(param_or_error, ParseError):
                 return param_or_error, schemas, parameters
-            param = param_or_error
+            param = param_or_error  # noqa: PLW2901
 
             if param.param_schema is None:
                 continue
@@ -396,7 +436,10 @@ class Endpoint:
 
             if isinstance(prop, ParseError):
                 return (
-                    ParseError(detail=f"cannot parse parameter of endpoint {endpoint.name}", data=prop.data),
+                    ParseError(
+                        detail=f"cannot parse parameter of endpoint {endpoint.name}",
+                        data=prop.data,
+                    ),
                     schemas,
                     parameters,
                 )
@@ -435,7 +478,8 @@ class Endpoint:
             if prop.python_name in endpoint.used_python_identifiers:
                 return (
                     ParseError(
-                        detail=f"Parameters with same Python identifier `{prop.python_name}` detected", data=data
+                        detail=f"Parameters with same Python identifier `{prop.python_name}` detected",
+                        data=data,
                     ),
                     schemas,
                     parameters,
@@ -468,7 +512,8 @@ class Endpoint:
         parameters_from_path = re.findall(_PATH_PARAM_REGEX, endpoint.path)
         try:
             sorted_params = sorted(
-                endpoint.path_parameters.values(), key=lambda param: parameters_from_path.index(param.name)
+                endpoint.path_parameters.values(),
+                key=lambda param: parameters_from_path.index(param.name),
             )
             endpoint.path_parameters = OrderedDict((param.name, param) for param in sorted_params)
         except ValueError:
@@ -509,7 +554,11 @@ class Endpoint:
         )
 
         result, schemas, parameters = Endpoint.add_parameters(
-            endpoint=endpoint, data=data, schemas=schemas, parameters=parameters, config=config
+            endpoint=endpoint,
+            data=data,
+            schemas=schemas,
+            parameters=parameters,
+            config=config,
         )
         if isinstance(result, ParseError):
             return result, schemas, parameters
@@ -537,6 +586,8 @@ class Endpoint:
             yield self.multipart_body
         if self.json_body:
             yield self.json_body
+        if self.binary_body:
+            yield self.binary_body
 
     def list_all_parameters(self) -> List[Property]:
         """Return a List of all the parameters of this endpoint"""
@@ -573,7 +624,9 @@ class GeneratorData:
             schemas = build_schemas(components=openapi.components.schemas, schemas=schemas, config=config)
         if openapi.components and openapi.components.parameters:
             parameters = build_parameters(
-                components=openapi.components.parameters, parameters=parameters, config=config
+                components=openapi.components.parameters,
+                parameters=parameters,
+                config=config,
             )
         endpoint_collections_by_tag, schemas, parameters = EndpointCollection.from_data(
             data=openapi.paths, schemas=schemas, parameters=parameters, config=config

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/openapi_python_client/parser/properties/__init__.py

@@ -16,9 +16,8 @@ from typing import Any, ClassVar, Dict, Generic, Iterable, List, Optional, Set,
 
 from attrs import define, evolve
 
-from ... import Config
+from ... import Config, utils
 from ... import schema as oai
-from ... import utils
 from ..errors import ParameterError, ParseError, PropertyError, ValidationError
 from .converter import convert, convert_chain
 from .enum_property import EnumProperty
@@ -188,7 +187,6 @@ class ListProperty(Property, Generic[InnerProp]):
     inner_property: InnerProp
     template: ClassVar[str] = "list_property.py.jinja"
 
-    # pylint: disable=unused-argument
     def get_base_type_string(self, *, quoted: bool = False) -> str:
         return f"List[{self.inner_property.get_type_string(quoted=not self.inner_property.is_base_type)}]"
 
@@ -236,7 +234,6 @@ class UnionProperty(Property):
             return inner_types.pop()
         return f"Union[{', '.join(sorted(inner_types))}]"
 
-    # pylint: disable=unused-argument
     def get_base_type_string(self, *, quoted: bool = False) -> str:
         return self._get_type_string_from_inner_type_strings(self._get_inner_type_strings(json=False))
 
@@ -570,7 +567,6 @@ def build_list_property(
     )
 
 
-# pylint: disable=too-many-arguments
 def _property_from_ref(
     name: str,
     required: bool,
@@ -605,8 +601,7 @@ def _property_from_ref(
     return prop, schemas
 
 
-# pylint: disable=too-many-arguments,too-many-return-statements
-def _property_from_data(
+def _property_from_data(  # noqa: PLR0911
     name: str,
     required: bool,
     data: Union[oai.Reference, oai.Schema],

{openapi_python_client-0.15.1 → openapi_python_client-0.16.0}/openapi_python_client/parser/properties/enum_property.py

@@ -30,7 +30,6 @@ class EnumProperty(Property):
         oai.ParameterLocation.HEADER,
     }
 
-    # pylint: disable=unused-argument
     def get_base_type_string(self, *, quoted: bool = False) -> str:
         return self.class_info.name