remove call to _convert_request

packages/autorest.python/autorest/codegen/serializers/builder_serializer.py

@@ -846,11 +846,6 @@ def _create_request_builder_call(
 
     def _postprocess_http_request(self, builder: OperationType, template_url: Optional[str] = None) -> List[str]:
         retval: List[str] = []
-        if not self.code_model.options["version_tolerant"]:
-            pass_files = ""
-            if builder.parameters.has_body and builder.parameters.body_parameter.client_name == "files":
-                pass_files = ", _files"
-            retval.append(f"_request = _convert_request(_request{pass_files})")
         if builder.parameters.path:
             retval.extend(self.serialize_path(builder))
         url_to_format = "_request.url"

packages/autorest.python/samples/specification/management/generated/azure/mgmt/sample/aio/operations/_http_success_operations.py

@@ -74,7 +74,6 @@ async def head200(self, **kwargs: Any) -> bool:
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False
@@ -119,7 +118,6 @@ async def head204(self, **kwargs: Any) -> bool:
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False
@@ -164,7 +162,6 @@ async def head404(self, **kwargs: Any) -> bool:
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False

packages/autorest.python/samples/specification/management/generated/azure/mgmt/sample/operations/_http_success_operations.py

@@ -98,7 +98,6 @@ def head200(self, **kwargs: Any) -> bool:
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False
@@ -143,7 +142,6 @@ def head204(self, **kwargs: Any) -> bool:
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False
@@ -188,7 +186,6 @@ def head404(self, **kwargs: Any) -> bool:
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False

packages/autorest.python/samples/specification/multiapi/generated/azure/multiapi/sample/__init__.py

@@ -7,12 +7,10 @@
 # --------------------------------------------------------------------------
 
 from ._multiapi_service_client import MultiapiServiceClient
-
-__all__ = ["MultiapiServiceClient"]
+__all__ = ['MultiapiServiceClient']
 
 try:
     from ._patch import patch_sdk  # type: ignore
-
     patch_sdk()
 except ImportError:
     pass

packages/autorest.python/samples/specification/multiapi/generated/azure/multiapi/sample/_configuration.py

@@ -19,7 +19,6 @@
     # pylint: disable=unused-import,ungrouped-imports
     from azure.core.credentials import TokenCredential
 
-
 class MultiapiServiceClientConfiguration:
     """Configuration for MultiapiServiceClient.
 
@@ -30,27 +29,32 @@ class MultiapiServiceClientConfiguration:
     :type credential: ~azure.core.credentials.TokenCredential
     """
 
-    def __init__(self, credential: "TokenCredential", **kwargs: Any):
+    def __init__(
+        self,
+        credential: "TokenCredential",
+        **kwargs: Any
+    ):
         if credential is None:
             raise ValueError("Parameter 'credential' must not be None.")
 
         self.credential = credential
-        self.credential_scopes = kwargs.pop("credential_scopes", ["https://management.azure.com/.default"])
-        kwargs.setdefault("sdk_moniker", "azure-multiapi-sample/{}".format(VERSION))
+        self.credential_scopes = kwargs.pop('credential_scopes', ['https://management.azure.com/.default'])
+        kwargs.setdefault('sdk_moniker', 'azure-multiapi-sample/{}'.format(VERSION))
         self.polling_interval = kwargs.get("polling_interval", 30)
         self._configure(**kwargs)
 
-    def _configure(self, **kwargs: Any):
-        self.user_agent_policy = kwargs.get("user_agent_policy") or policies.UserAgentPolicy(**kwargs)
-        self.headers_policy = kwargs.get("headers_policy") or policies.HeadersPolicy(**kwargs)
-        self.proxy_policy = kwargs.get("proxy_policy") or policies.ProxyPolicy(**kwargs)
-        self.logging_policy = kwargs.get("logging_policy") or policies.NetworkTraceLoggingPolicy(**kwargs)
-        self.http_logging_policy = kwargs.get("http_logging_policy") or ARMHttpLoggingPolicy(**kwargs)
-        self.retry_policy = kwargs.get("retry_policy") or policies.RetryPolicy(**kwargs)
-        self.custom_hook_policy = kwargs.get("custom_hook_policy") or policies.CustomHookPolicy(**kwargs)
-        self.redirect_policy = kwargs.get("redirect_policy") or policies.RedirectPolicy(**kwargs)
-        self.authentication_policy = kwargs.get("authentication_policy")
+    def _configure(
+        self,
+        **kwargs: Any
+    ):
+        self.user_agent_policy = kwargs.get('user_agent_policy') or policies.UserAgentPolicy(**kwargs)
+        self.headers_policy = kwargs.get('headers_policy') or policies.HeadersPolicy(**kwargs)
+        self.proxy_policy = kwargs.get('proxy_policy') or policies.ProxyPolicy(**kwargs)
+        self.logging_policy = kwargs.get('logging_policy') or policies.NetworkTraceLoggingPolicy(**kwargs)
+        self.http_logging_policy = kwargs.get('http_logging_policy') or ARMHttpLoggingPolicy(**kwargs)
+        self.retry_policy = kwargs.get('retry_policy') or policies.RetryPolicy(**kwargs)
+        self.custom_hook_policy = kwargs.get('custom_hook_policy') or policies.CustomHookPolicy(**kwargs)
+        self.redirect_policy = kwargs.get('redirect_policy') or policies.RedirectPolicy(**kwargs)
+        self.authentication_policy = kwargs.get('authentication_policy')
         if self.credential and not self.authentication_policy:
-            self.authentication_policy = ARMChallengeAuthenticationPolicy(
-                self.credential, *self.credential_scopes, **kwargs
-            )
+            self.authentication_policy = ARMChallengeAuthenticationPolicy(self.credential, *self.credential_scopes, **kwargs)

packages/autorest.python/samples/specification/multiapi/generated/azure/multiapi/sample/_multiapi_service_client.py

@@ -25,15 +25,13 @@
     # pylint: disable=unused-import,ungrouped-imports
     from azure.core.credentials import TokenCredential
 
-
 class _SDKClient(object):
     def __init__(self, *args, **kwargs):
         """This is a fake class to support current implemetation of MultiApiClientMixin."
         Will be removed in final version of multiapi azure-core based client
         """
         pass
 
-
 class MultiapiServiceClient(MultiapiServiceClientOperationsMixin, MultiApiClientMixin, _SDKClient):
     """Service client for multiapi client testing.
 
@@ -56,30 +54,28 @@ class MultiapiServiceClient(MultiapiServiceClientOperationsMixin, MultiApiClient
     :keyword int polling_interval: Default waiting time between two polls for LRO operations if no Retry-After header is present.
     """
 
-    DEFAULT_API_VERSION = "3.0.0"
+    DEFAULT_API_VERSION = '3.0.0'
     _PROFILE_TAG = "azure.multiapi.sample.MultiapiServiceClient"
-    LATEST_PROFILE = ProfileDefinition(
-        {
-            _PROFILE_TAG: {
-                None: DEFAULT_API_VERSION,
-                "begin_test_lro": "1.0.0",
-                "begin_test_lro_and_paging": "1.0.0",
-                "test_one": "2.0.0",
-            }
-        },
-        _PROFILE_TAG + " latest",
+    LATEST_PROFILE = ProfileDefinition({
+        _PROFILE_TAG: {
+            None: DEFAULT_API_VERSION,
+            'begin_test_lro': '1.0.0',
+            'begin_test_lro_and_paging': '1.0.0',
+            'test_one': '2.0.0',
+        }},
+        _PROFILE_TAG + " latest"
     )
 
     def __init__(
         self,
         credential: "TokenCredential",
-        api_version: Optional[str] = None,
+        api_version: Optional[str]=None,
         base_url: str = "http://localhost:3000",
-        profile: KnownProfiles = KnownProfiles.default,
+        profile: KnownProfiles=KnownProfiles.default,
         **kwargs: Any
     ):
         if api_version:
-            kwargs.setdefault("api_version", api_version)
+            kwargs.setdefault('api_version', api_version)
         self._config = MultiapiServiceClientConfiguration(credential, **kwargs)
         _policies = kwargs.pop("policies", None)
         if _policies is None:
@@ -100,7 +96,10 @@ def __init__(
                 self._config.http_logging_policy,
             ]
         self._client = ARMPipelineClient(base_url=base_url, policies=_policies, **kwargs)
-        super(MultiapiServiceClient, self).__init__(api_version=api_version, profile=profile)
+        super(MultiapiServiceClient, self).__init__(
+            api_version=api_version,
+            profile=profile
+        )
 
     @classmethod
     def _models_dict(cls, api_version):
@@ -110,79 +109,62 @@ def _models_dict(cls, api_version):
     def models(cls, api_version=DEFAULT_API_VERSION):
         """Module depends on the API version:
 
-        * 1.0.0: :mod:`v1.models<azure.multiapi.sample.v1.models>`
-        * 2.0.0: :mod:`v2.models<azure.multiapi.sample.v2.models>`
-        * 3.0.0: :mod:`v3.models<azure.multiapi.sample.v3.models>`
+           * 1.0.0: :mod:`v1.models<azure.multiapi.sample.v1.models>`
+           * 2.0.0: :mod:`v2.models<azure.multiapi.sample.v2.models>`
+           * 3.0.0: :mod:`v3.models<azure.multiapi.sample.v3.models>`
         """
-        if api_version == "1.0.0":
+        if api_version == '1.0.0':
             from .v1 import models
-
             return models
-        elif api_version == "2.0.0":
+        elif api_version == '2.0.0':
             from .v2 import models
-
             return models
-        elif api_version == "3.0.0":
+        elif api_version == '3.0.0':
             from .v3 import models
-
             return models
         raise ValueError("API version {} is not available".format(api_version))
 
     @property
     def operation_group_one(self):
         """Instance depends on the API version:
 
-        * 1.0.0: :class:`OperationGroupOneOperations<azure.multiapi.sample.v1.operations.OperationGroupOneOperations>`
-        * 2.0.0: :class:`OperationGroupOneOperations<azure.multiapi.sample.v2.operations.OperationGroupOneOperations>`
-        * 3.0.0: :class:`OperationGroupOneOperations<azure.multiapi.sample.v3.operations.OperationGroupOneOperations>`
+           * 1.0.0: :class:`OperationGroupOneOperations<azure.multiapi.sample.v1.operations.OperationGroupOneOperations>`
+           * 2.0.0: :class:`OperationGroupOneOperations<azure.multiapi.sample.v2.operations.OperationGroupOneOperations>`
+           * 3.0.0: :class:`OperationGroupOneOperations<azure.multiapi.sample.v3.operations.OperationGroupOneOperations>`
         """
-        api_version = self._get_api_version("operation_group_one")
-        if api_version == "1.0.0":
+        api_version = self._get_api_version('operation_group_one')
+        if api_version == '1.0.0':
             from .v1.operations import OperationGroupOneOperations as OperationClass
-        elif api_version == "2.0.0":
+        elif api_version == '2.0.0':
             from .v2.operations import OperationGroupOneOperations as OperationClass
-        elif api_version == "3.0.0":
+        elif api_version == '3.0.0':
             from .v3.operations import OperationGroupOneOperations as OperationClass
         else:
             raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version))
         self._config.api_version = api_version
-        return OperationClass(
-            self._client,
-            self._config,
-            Serializer(self._models_dict(api_version)),
-            Deserializer(self._models_dict(api_version)),
-            api_version,
-        )
+        return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version)), api_version)
 
     @property
     def operation_group_two(self):
         """Instance depends on the API version:
 
-        * 2.0.0: :class:`OperationGroupTwoOperations<azure.multiapi.sample.v2.operations.OperationGroupTwoOperations>`
-        * 3.0.0: :class:`OperationGroupTwoOperations<azure.multiapi.sample.v3.operations.OperationGroupTwoOperations>`
+           * 2.0.0: :class:`OperationGroupTwoOperations<azure.multiapi.sample.v2.operations.OperationGroupTwoOperations>`
+           * 3.0.0: :class:`OperationGroupTwoOperations<azure.multiapi.sample.v3.operations.OperationGroupTwoOperations>`
         """
-        api_version = self._get_api_version("operation_group_two")
-        if api_version == "2.0.0":
+        api_version = self._get_api_version('operation_group_two')
+        if api_version == '2.0.0':
             from .v2.operations import OperationGroupTwoOperations as OperationClass
-        elif api_version == "3.0.0":
+        elif api_version == '3.0.0':
             from .v3.operations import OperationGroupTwoOperations as OperationClass
         else:
             raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version))
         self._config.api_version = api_version
-        return OperationClass(
-            self._client,
-            self._config,
-            Serializer(self._models_dict(api_version)),
-            Deserializer(self._models_dict(api_version)),
-            api_version,
-        )
+        return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version)), api_version)
 
     def close(self):
         self._client.close()
-
     def __enter__(self):
         self._client.__enter__()
         return self
-
     def __exit__(self, *exc_details):
         self._client.__exit__(*exc_details)