chore: Regen integration test client and add to regen task

Author: dbanty

CONTRIBUTING.md

@@ -21,13 +21,14 @@
 
 2. When in a Poetry shell (`poetry shell`) run `task check` in order to run most of the same checks CI runs. This will auto-reformat the code, check type annotations, run unit tests, check code coverage, and lint the code.
 
-### Rework end to end tests 
+### Rework end-to-end tests 
 
-3. If you're writing a new feature, try to add it to the end to end test.
+3. If you're writing a new feature, try to add it to the end-to-end test.
    1. If adding support for a new OpenAPI feature, add it somewhere in `end_to_end_tests/openapi.json`
-   2. Regenerate the "golden records" with `task regen`. This client is generated from the OpenAPI document used for end to end testing.
+   2. Regenerate the "golden records" with `task regen`. This client is generated from the OpenAPI document used for end-to-end testing.
    3. Check the changes to `end_to_end_tests/golden-record` to confirm only what you intended to change did change and that the changes look correct.
-4. Run the end to end tests with `task e2e`. This will generate clients against `end_to_end_tests/openapi.json` and compare them with the golden record. The tests will fail if **anything is different**. The end to end tests are not included in `task check` as they take longer to run and don't provide very useful feedback in the event of failure. If an e2e test does fail, the easiest way to check what's wrong is to run `task regen` and check the diffs. You can also use `task re` which will run `regen` and `e2e` in that order.
+4. **If you added a test above OR modified the templates**: Run the end-to-end tests with `task e2e`. This will generate clients against `end_to_end_tests/openapi.json` and compare them with the golden record. The tests will fail if **anything is different**. The end-to-end tests are not included in `task check` as they take longer to run and don't provide very useful feedback in the event of failure. If an e2e test does fail, the easiest way to check what's wrong is to run `task regen` and check the diffs. You can also use `task re` which will run `regen` and `e2e` in that order.
+
 
 ## Creating a Pull Request
 

integration_tests/open-api-test-server-client/open_api_test_server_client/api/body/post_body_multipart.py

@@ -22,6 +22,7 @@ def _get_kwargs(
     multipart_multipart_data = multipart_data.to_multipart()
 
     return {
+        "method": "post",
         "url": url,
         "headers": headers,
         "cookies": cookies,
@@ -56,12 +57,20 @@ def sync_detailed(
     client: Client,
     multipart_data: PostBodyMultipartMultipartData,
 ) -> Response[Union[PostBodyMultipartResponse200, PublicError]]:
+    """
+    Args:
+        multipart_data (PostBodyMultipartMultipartData):
+
+    Returns:
+        Response[Union[PostBodyMultipartResponse200, PublicError]]
+    """
+
     kwargs = _get_kwargs(
         client=client,
         multipart_data=multipart_data,
     )
 
-    response = httpx.post(
+    response = httpx.request(
         verify=client.verify_ssl,
         **kwargs,
     )
@@ -74,7 +83,13 @@ def sync(
     client: Client,
     multipart_data: PostBodyMultipartMultipartData,
 ) -> Optional[Union[PostBodyMultipartResponse200, PublicError]]:
-    """ """
+    """
+    Args:
+        multipart_data (PostBodyMultipartMultipartData):
+
+    Returns:
+        Response[Union[PostBodyMultipartResponse200, PublicError]]
+    """
 
     return sync_detailed(
         client=client,
@@ -87,13 +102,21 @@ async def asyncio_detailed(
     client: Client,
     multipart_data: PostBodyMultipartMultipartData,
 ) -> Response[Union[PostBodyMultipartResponse200, PublicError]]:
+    """
+    Args:
+        multipart_data (PostBodyMultipartMultipartData):
+
+    Returns:
+        Response[Union[PostBodyMultipartResponse200, PublicError]]
+    """
+
     kwargs = _get_kwargs(
         client=client,
         multipart_data=multipart_data,
     )
 
     async with httpx.AsyncClient(verify=client.verify_ssl) as _client:
-        response = await _client.post(**kwargs)
+        response = await _client.request(**kwargs)
 
     return _build_response(response=response)
 
@@ -103,7 +126,13 @@ async def asyncio(
     client: Client,
     multipart_data: PostBodyMultipartMultipartData,
 ) -> Optional[Union[PostBodyMultipartResponse200, PublicError]]:
-    """ """
+    """
+    Args:
+        multipart_data (PostBodyMultipartMultipartData):
+
+    Returns:
+        Response[Union[PostBodyMultipartResponse200, PublicError]]
+    """
 
     return (
         await asyncio_detailed(

integration_tests/open-api-test-server-client/open_api_test_server_client/models/post_body_multipart_multipart_data.py

@@ -10,7 +10,13 @@
 
 @attr.s(auto_attribs=True)
 class PostBodyMultipartMultipartData:
-    """ """
+    """
+    Attributes:
+        a_string (str):
+        file (File): For the sake of this test, include a file name and content type. The payload should also be valid
+            UTF-8.
+        description (Union[Unset, str]):
+    """
 
     a_string: str
     file: File

integration_tests/open-api-test-server-client/open_api_test_server_client/models/post_body_multipart_response_200.py

@@ -7,7 +7,14 @@
 
 @attr.s(auto_attribs=True)
 class PostBodyMultipartResponse200:
-    """ """
+    """
+    Attributes:
+        a_string (str): Echo of the 'a_string' input parameter from the form.
+        file_data (str): Echo of content of the 'file' input parameter from the form.
+        description (str): Echo of the 'description' input parameter from the form.
+        file_name (str): The name of the file uploaded.
+        file_content_type (str): The content type of the file uploaded.
+    """
 
     a_string: str
     file_data: str

integration_tests/open-api-test-server-client/open_api_test_server_client/models/problem.py

@@ -9,7 +9,11 @@
 
 @attr.s(auto_attribs=True)
 class Problem:
-    """ """
+    """
+    Attributes:
+        parameter_name (Union[Unset, str]):
+        description (Union[Unset, str]):
+    """
 
     parameter_name: Union[Unset, str] = UNSET
     description: Union[Unset, str] = UNSET

integration_tests/open-api-test-server-client/open_api_test_server_client/models/public_error.py

@@ -10,7 +10,13 @@
 
 @attr.s(auto_attribs=True)
 class PublicError:
-    """ """
+    """
+    Attributes:
+        errors (Union[Unset, List[str]]):
+        extra_parameters (Union[Unset, List[str]]):
+        invalid_parameters (Union[Unset, List[Problem]]):
+        missing_parameters (Union[Unset, List[str]]):
+    """
 
     errors: Union[Unset, List[str]] = UNSET
     extra_parameters: Union[Unset, List[str]] = UNSET

pyproject.toml

@@ -65,7 +65,12 @@ isort .\
  && TASKIPY=true pytest --cov openapi_python_client tests --cov-report=term-missing --basetemp=tests/tmp\
  && rm -r tests/tmp\
 """
-regen = "python -m end_to_end_tests.regen_golden_record"
+regen = """
+python -m end_to_end_tests.regen_golden_record\
+&& pushd integration_tests\
+&& openapi-python-client update --url https://raw.githubusercontent.com/openapi-generators/openapi-test-server/main/openapi.json\
+&& popd
+"""
 e2e = "pytest openapi_python_client end_to_end_tests/test_end_to_end.py"
 re = """
 task regen\