diff --git a/ChangeLog.md b/ChangeLog.md index 4cea224eb2d..9bfa044e9b6 100644 --- a/ChangeLog.md +++ b/ChangeLog.md @@ -5,6 +5,10 @@ Autorest Core version: 3.0.6306 Modelerfour version: 4.15.416 +**Breaking Changes** + +- Raise `ValueError` instead of `NotImplementedError` if API version code doesn't exist #764 + ### 2020-08-31 - 5.2.0-preview.1 Autorest Core version: 3.0.6306 diff --git a/autorest/multiapi/templates/multiapi_operations_mixin.py.jinja2 b/autorest/multiapi/templates/multiapi_operations_mixin.py.jinja2 index 6119830cac0..f52bd624ecc 100644 --- a/autorest/multiapi/templates/multiapi_operations_mixin.py.jinja2 +++ b/autorest/multiapi/templates/multiapi_operations_mixin.py.jinja2 @@ -27,7 +27,7 @@ class {{ client_name }}OperationsMixin(object): from {{ ".." if async_mode else "." }}{{ api }}{{ ".aio" if async_mode else "" }}.operations import {{ client_name }}OperationsMixin as OperationClass {% endfor %} else: - raise NotImplementedError("API version {} does not have operation '{{ operation_name }}'".format(api_version)) + raise ValueError("API version {} does not have operation '{{ operation_name }}'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/autorest/multiapi/templates/multiapi_service_client.py.jinja2 b/autorest/multiapi/templates/multiapi_service_client.py.jinja2 index abf6a3ac09a..a5a1a10f3e1 100644 --- a/autorest/multiapi/templates/multiapi_service_client.py.jinja2 +++ b/autorest/multiapi/templates/multiapi_service_client.py.jinja2 @@ -102,7 +102,7 @@ class {{ client_name }}({% if mixin_operations %}{{ client_name }}OperationsMixi base_url = {{ custom_base_url }} {% endfor %} else: - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) {% else %} if not base_url: base_url = {{ base_url }} @@ -132,7 +132,7 @@ class {{ client_name }}({% if mixin_operations %}{{ client_name }}OperationsMixi from {{ relative_path }}{{ mod_api_version }} import models return models {% endfor %} - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) {% for operation_group, available_apis in operations|dictsort %} @property @@ -150,7 +150,7 @@ class {{ client_name }}({% if mixin_operations %}{{ client_name }}OperationsMixi from {{ relative_path }}{{ api[0] }}.{{ "aio." if async_mode else "" }}operations import {{ api[1] }} as OperationClass {% endfor %} else: - raise NotImplementedError("API version {} does not have operation group '{{ operation_group }}'".format(api_version)) + raise ValueError("API version {} does not have operation group '{{ operation_group }}'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) {% endfor %} diff --git a/test/multiapi/AcceptanceTests/asynctests/multiapi_base.py b/test/multiapi/AcceptanceTests/asynctests/multiapi_base.py index 0efdfd54bf7..bc98879a78a 100644 --- a/test/multiapi/AcceptanceTests/asynctests/multiapi_base.py +++ b/test/multiapi/AcceptanceTests/asynctests/multiapi_base.py @@ -111,7 +111,7 @@ async def test_specificy_api_version_operation_mixin(self, client): @pytest.mark.parametrize('api_version', ["3.0.0"]) @pytest.mark.asyncio async def test_specify_api_version_with_no_mixin(self, client): - with pytest.raises(NotImplementedError): + with pytest.raises(ValueError): await client.test_one(id=1, message="This should throw") # OPERATION GROUP ONE diff --git a/test/multiapi/AcceptanceTests/multiapi_base.py b/test/multiapi/AcceptanceTests/multiapi_base.py index b880f4b782c..350a15fcc73 100644 --- a/test/multiapi/AcceptanceTests/multiapi_base.py +++ b/test/multiapi/AcceptanceTests/multiapi_base.py @@ -102,7 +102,7 @@ def test_specificy_api_version_operation_mixin(self, client): @pytest.mark.parametrize('api_version', ["3.0.0"]) def test_specify_api_version_with_no_mixin(self, client): - with pytest.raises(NotImplementedError): + with pytest.raises(ValueError): client.test_one(id=1, message="This should throw") # OPERATION GROUP ONE diff --git a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_multiapi_service_client.py index 42b73724da5..ac94cc136e5 100644 --- a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from .v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_operations_mixin.py index 99f2897bd01..010ec0cbc12 100644 --- a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/_operations_mixin.py @@ -50,7 +50,7 @@ def begin_test_lro( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -84,7 +84,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -115,7 +115,7 @@ def test_one( elif api_version == '2.0.0': from .v2.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -138,7 +138,7 @@ def test_paging( if api_version == '3.0.0': from .v3.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_multiapi_service_client.py index 16212c547a2..84188086716 100644 --- a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from ..v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) async def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_operations_mixin.py index 33c38147571..9c3cc589aca 100644 --- a/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/Multiapi/multiapi/aio/_operations_mixin.py @@ -46,7 +46,7 @@ async def begin_test_lro( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -80,7 +80,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -111,7 +111,7 @@ async def test_one( elif api_version == '2.0.0': from ..v2.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -134,7 +134,7 @@ def test_paging( if api_version == '3.0.0': from ..v3.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_multiapi_service_client.py index 9e936e40633..c139f409e4c 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from .v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_operations_mixin.py index 58ed83b81ec..28ee5a334a9 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/_operations_mixin.py @@ -50,7 +50,7 @@ def begin_test_lro( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -84,7 +84,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -115,7 +115,7 @@ def test_one( elif api_version == '2.0.0': from .v2.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -138,7 +138,7 @@ def test_paging( if api_version == '3.0.0': from .v3.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_multiapi_service_client.py index e82dbde5134..cf986cdd63d 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from ..v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) async def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_operations_mixin.py index 697d7b2c8a6..477941dbdc5 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCredentialDefaultPolicy/multiapicredentialdefaultpolicy/aio/_operations_mixin.py @@ -46,7 +46,7 @@ async def begin_test_lro( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -80,7 +80,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -111,7 +111,7 @@ async def test_one( elif api_version == '2.0.0': from ..v2.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -134,7 +134,7 @@ def test_paging( if api_version == '3.0.0': from ..v3.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_multiapi_custom_base_url_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_multiapi_custom_base_url_service_client.py index 2d8fafb1cd5..e02ccf3569e 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_multiapi_custom_base_url_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_multiapi_custom_base_url_service_client.py @@ -66,7 +66,7 @@ def __init__( elif api_version == '2.0.0': base_url = '{Endpoint}/multiapiCustomBaseUrl/v2' else: - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) self._config = MultiapiCustomBaseUrlServiceClientConfiguration(credential, endpoint, **kwargs) self._client = PipelineClient(base_url=base_url, config=self._config, **kwargs) super(MultiapiCustomBaseUrlServiceClient, self).__init__( @@ -91,7 +91,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '2.0.0': from .v2 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) def close(self): self._client.close() diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_operations_mixin.py index 2c14ab492d4..1d98f610fc4 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/_operations_mixin.py @@ -44,7 +44,7 @@ def test( elif api_version == '2.0.0': from .v2.operations import MultiapiCustomBaseUrlServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test'".format(api_version)) + raise ValueError("API version {} does not have operation 'test'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_multiapi_custom_base_url_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_multiapi_custom_base_url_service_client.py index df0cdfb343e..bc9d104d0eb 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_multiapi_custom_base_url_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_multiapi_custom_base_url_service_client.py @@ -66,7 +66,7 @@ def __init__( elif api_version == '2.0.0': base_url = '{Endpoint}/multiapiCustomBaseUrl/v2' else: - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) self._config = MultiapiCustomBaseUrlServiceClientConfiguration(credential, endpoint, **kwargs) self._client = AsyncPipelineClient(base_url=base_url, config=self._config, **kwargs) super(MultiapiCustomBaseUrlServiceClient, self).__init__( @@ -91,7 +91,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '2.0.0': from ..v2 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) async def close(self): await self._client.close() diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_operations_mixin.py index 6dbd05b7f2b..931971b01d7 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiCustomBaseUrl/multiapicustombaseurl/aio/_operations_mixin.py @@ -40,7 +40,7 @@ async def test( elif api_version == '2.0.0': from ..v2.aio.operations import MultiapiCustomBaseUrlServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test'".format(api_version)) + raise ValueError("API version {} does not have operation 'test'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_multiapi_service_client.py index a9c006cf798..1abc1f4fcac 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from .v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_operations_mixin.py index 1e9a2b411e9..3e1712f3b5f 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/_operations_mixin.py @@ -49,7 +49,7 @@ def begin_test_lro( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -83,7 +83,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -114,7 +114,7 @@ def test_one( elif api_version == '2.0.0': from .v2.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -137,7 +137,7 @@ def test_paging( if api_version == '3.0.0': from .v3.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_multiapi_service_client.py index 7980bbe4180..4b4960dd112 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from ..v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) async def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_operations_mixin.py index bad329f1d12..ef83d2ecc8b 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiDataPlane/multiapidataplane/aio/_operations_mixin.py @@ -45,7 +45,7 @@ async def begin_test_lro( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -79,7 +79,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -110,7 +110,7 @@ async def test_one( elif api_version == '2.0.0': from ..v2.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -133,7 +133,7 @@ def test_paging( if api_version == '3.0.0': from ..v3.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_multiapi_service_client.py index e06b1229205..27d78e704d4 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from .v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_operations_mixin.py index e7a856df72f..d57d45f8176 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiNoAsync/multiapinoasync/_operations_mixin.py @@ -50,7 +50,7 @@ def begin_test_lro( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -84,7 +84,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -115,7 +115,7 @@ def test_one( elif api_version == '2.0.0': from .v2.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -138,7 +138,7 @@ def test_paging( if api_version == '3.0.0': from .v3.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_multiapi_service_client.py index 7e63f59594f..7590039451a 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from .v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from .v3.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_operations_mixin.py index b8b50254439..ba53d49a045 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/_operations_mixin.py @@ -50,7 +50,7 @@ def begin_test_lro( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -84,7 +84,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from .v1.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -115,7 +115,7 @@ def test_one( elif api_version == '2.0.0': from .v2.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -138,7 +138,7 @@ def test_paging( if api_version == '3.0.0': from .v3.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_multiapi_service_client.py b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_multiapi_service_client.py index e3dcef9e1bf..fd2b9e3ab97 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_multiapi_service_client.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_multiapi_service_client.py @@ -94,7 +94,7 @@ def models(cls, api_version=DEFAULT_API_VERSION): elif api_version == '3.0.0': from ..v3 import models return models - raise NotImplementedError("API version {} is not available".format(api_version)) + raise ValueError("API version {} is not available".format(api_version)) @property def operation_group_one(self): @@ -112,7 +112,7 @@ def operation_group_one(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupOneOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_one'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_one'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) @property @@ -128,7 +128,7 @@ def operation_group_two(self): elif api_version == '3.0.0': from ..v3.aio.operations import OperationGroupTwoOperations as OperationClass else: - raise NotImplementedError("API version {} does not have operation group 'operation_group_two'".format(api_version)) + raise ValueError("API version {} does not have operation group 'operation_group_two'".format(api_version)) return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version))) async def close(self): diff --git a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_operations_mixin.py b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_operations_mixin.py index c274c843bc4..99137334437 100644 --- a/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_operations_mixin.py +++ b/test/multiapi/Expected/AcceptanceTests/MultiapiWithSubmodule/multiapiwithsubmodule/submodule/aio/_operations_mixin.py @@ -46,7 +46,7 @@ async def begin_test_lro( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -80,7 +80,7 @@ def begin_test_lro_and_paging( if api_version == '1.0.0': from ..v1.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'begin_test_lro_and_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -111,7 +111,7 @@ async def test_one( elif api_version == '2.0.0': from ..v2.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_one'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_one'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config @@ -134,7 +134,7 @@ def test_paging( if api_version == '3.0.0': from ..v3.aio.operations import MultiapiServiceClientOperationsMixin as OperationClass else: - raise NotImplementedError("API version {} does not have operation 'test_paging'".format(api_version)) + raise ValueError("API version {} does not have operation 'test_paging'".format(api_version)) mixin_instance = OperationClass() mixin_instance._client = self._client mixin_instance._config = self._config