From 4888c568652bcac50600fc67987e293206c34b1a Mon Sep 17 00:00:00 2001 From: Ali Satwat Khan Date: Mon, 27 Jul 2026 04:06:47 +0500 Subject: [PATCH 1/3] Fix azure-appconfiguration linting for pylint 4.0.6 --- .../_audience_error_handling_policy.py | 3 + .../_azure_appconfiguration_client.py | 2 +- .../_azure_appconfiguration_requests.py | 8 +- .../azure/appconfiguration/_models.py | 75 +++++++++++++++++-- .../azure/appconfiguration/_sync_token.py | 2 +- .../_azure_appconfiguration_client_async.py | 2 +- .../appconfiguration/aio/_sync_token_async.py | 2 +- .../tests/test_audience.py | 2 +- .../test_audience_error_handling_live.py | 2 +- ...test_audience_error_handling_live_async.py | 2 +- .../tests/test_audience_policy.py | 2 +- .../test_azure_appconfiguration_client.py | 4 +- ...est_azure_appconfiguration_client_async.py | 4 +- .../tests/test_consistency.py | 2 +- .../azure-appconfiguration/tests/test_http.py | 4 +- .../tests/test_query_param_policy.py | 4 +- .../tests/test_sync_token_policy.py | 5 +- 17 files changed, 101 insertions(+), 24 deletions(-) diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_audience_error_handling_policy.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_audience_error_handling_policy.py index 57ffd18ed019..297e2a98e4ad 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_audience_error_handling_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_audience_error_handling_policy.py @@ -26,6 +26,9 @@ class AudienceErrorHandlingPolicy(SansIOHTTPPolicy): """ A policy to handle audience-related authentication errors for Azure App Configuration. Raises a ClientAuthenticationError with a helpful message if the audience is missing or incorrect. + + :param has_audience: Indicates if the expected audience is set for the authentication token. + :type has_audience: bool """ def __init__(self, has_audience: bool = False): diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_client.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_client.py index 89d0ebcf9151..e2461d11cd7d 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_client.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_client.py @@ -45,7 +45,7 @@ from ._audience_error_handling_policy import AudienceErrorHandlingPolicy -class AzureAppConfigurationClient: +class AzureAppConfigurationClient: # pylint: disable=docstring-keyword-should-match-keyword-only """Represents a client that calls restful API of Azure App Configuration service. :param str base_url: Base url of the service. diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_requests.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_requests.py index 4f248d0f47e3..c7546c01140e 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_requests.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_requests.py @@ -15,7 +15,13 @@ class AppConfigRequestsCredentialsPolicy(HTTPPolicy): - """Implementation of request-oauthlib except and retry logic.""" + """Implementation of request-oauthlib except and retry logic. + + :param credential: The credential used to authenticate requests. + :type credential: ~azure.core.credentials.AzureKeyCredential + :param str endpoint: The App Configuration endpoint. + :param str id_credential: The credential identifier used to sign requests. + """ def __init__(self, credential: AzureKeyCredential, endpoint: str, id_credential: str): super(AppConfigRequestsCredentialsPolicy, self).__init__() diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py index 19d50919c47c..6f5b882879c2 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py @@ -122,7 +122,18 @@ def _to_generated(self) -> KeyValue: class FeatureFlagConfigurationSetting(ConfigurationSetting): # pylint: disable=too-many-instance-attributes - """A configuration setting that stores a feature flag value.""" + """A configuration setting that stores a feature flag value. + + :param feature_id: The identity of the configuration setting. + :type feature_id: str + :keyword enabled: The value indicating whether the feature flag is enabled. + A feature is OFF if enabled is false. If enabled is true, then the feature is ON + if there are no conditions or if all conditions are satisfied. Default value is False. + :paramtype enabled: bool + :keyword filters: Filters that must run on the client and be evaluated as true for the feature + to be considered enabled. + :paramtype filters: list[dict[str, Any]] or None + """ etag: str """A value representing the current state of the resource.""" @@ -289,7 +300,13 @@ def _to_generated(self) -> KeyValue: class SecretReferenceConfigurationSetting(ConfigurationSetting): - """A configuration value that references a configuration setting secret.""" + """A configuration value that references a configuration setting secret. + + :param key: The key of the configuration setting. + :type key: str + :param secret_id: The identity of the configuration setting. + :type secret_id: str + """ etag: str """A value representing the current state of the resource.""" @@ -405,7 +422,15 @@ def _to_generated(self) -> KeyValue: class ConfigurationSettingsFilter: - """Enables filtering of configuration settings.""" + """Enables filtering of configuration settings. + + :keyword key: Filters configuration settings by their key field. Required. + :paramtype key: str + :keyword label: Filters configuration settings by their label field. + :paramtype label: str or None + :keyword tags: Filters key-values by their tags field. + :paramtype tags: list[str] or None + """ key: str """Filters configuration settings by their key field. Required.""" @@ -429,7 +454,25 @@ def __init__(self, *, key: str, label: Optional[str] = None, tags: Optional[List class ConfigurationSnapshot: # pylint: disable=too-many-instance-attributes - """A point-in-time snapshot of configuration settings.""" + """A point-in-time snapshot of configuration settings. + + :param filters: A list of filters used to filter the key-values included in the configuration snapshot. + Required. + :type filters: list[~azure.appconfiguration.ConfigurationSettingsFilter] + :keyword composition_type: The composition type describes how the key-values within the configuration + snapshot are composed. The 'key' composition type ensures there are no two key-values + containing the same key. The 'key_label' composition type ensures there are no two key-values + containing the same key and label. Known values are: "key" and "key_label". + :paramtype composition_type: str or None + :keyword retention_period: The amount of time, in seconds, that a configuration snapshot will remain in the + archived state before expiring. This property is only writable during the creation of a configuration + snapshot. If not specified, the default lifetime of key-value revisions will be used. + :paramtype retention_period: int or None + :keyword tags: The tags of the configuration snapshot. + :paramtype tags: dict[str, str] or None + :keyword description: The description of the configuration snapshot. + :paramtype description: str or None + """ name: Optional[str] """The name of the configuration snapshot.""" @@ -584,7 +627,11 @@ def _to_generated(self) -> GeneratedConfigurationSnapshot: class ConfigurationSettingLabel: - """The label info of a configuration setting.""" + """The label info of a configuration setting. + + :keyword name: The configuration setting label name. + :paramtype name: str or None + """ name: Optional[str] """The name of the ConfigurationSetting label.""" @@ -602,7 +649,11 @@ def _return_deserialized_and_headers(_, deserialized, response_headers): class ConfigurationSettingPropertiesPagedBase: # pylint:disable=too-many-instance-attributes - """Base class for iterable of ConfigurationSetting properties.""" + """Base class for iterable of ConfigurationSetting properties. + + :param command: The command to execute for pagination. + :type command: Callable + """ etag: str """The current etag""" @@ -672,7 +723,11 @@ def _extract_data_cb_base(self, get_next_return) -> tuple: class ConfigurationSettingPropertiesPaged( ConfigurationSettingPropertiesPagedBase, PageIterator ): # pylint:disable=too-many-instance-attributes - """An iterable of ConfigurationSetting properties.""" + """An iterable of ConfigurationSetting properties. + + :param command: The command to execute for pagination. + :type command: Callable + """ def __init__(self, command: Callable, **kwargs: Any): super().__init__(command, **kwargs) @@ -733,7 +788,11 @@ def __next__(self) -> Iterator[ReturnType]: class ConfigurationSettingPropertiesPagedAsync( ConfigurationSettingPropertiesPagedBase, AsyncPageIterator ): # pylint:disable=too-many-instance-attributes - """An iterable of ConfigurationSetting properties.""" + """An iterable of ConfigurationSetting properties. + + :param command: The command to execute for pagination. + :type command: Callable + """ def __init__(self, command: Callable, **kwargs: Any): ConfigurationSettingPropertiesPagedBase.__init__(self, command, **kwargs) diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_sync_token.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_sync_token.py index c1ab0d8e8541..303f0ad87b02 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_sync_token.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_sync_token.py @@ -59,7 +59,7 @@ def from_sync_token_string(cls, sync_token): return None -class SyncTokenPolicy(SansIOHTTPPolicy): +class SyncTokenPolicy(SansIOHTTPPolicy): # pylint: disable=docstring-keyword-should-match-keyword-only """A simple policy that enable the given callback with the response. :keyword callback raw_response_hook: Callback function. Will be invoked on response. diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_azure_appconfiguration_client_async.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_azure_appconfiguration_client_async.py index 66040aa9f80a..40fd12a29f53 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_azure_appconfiguration_client_async.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_azure_appconfiguration_client_async.py @@ -46,7 +46,7 @@ from .._audience_error_handling_policy import AudienceErrorHandlingPolicy -class AzureAppConfigurationClient: +class AzureAppConfigurationClient: # pylint: disable=docstring-keyword-should-match-keyword-only """Represents a client that calls restful API of Azure App Configuration service. :param str base_url: Base url of the service. diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_sync_token_async.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_sync_token_async.py index efff90353314..2f416b5b8d5c 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_sync_token_async.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/aio/_sync_token_async.py @@ -31,7 +31,7 @@ from .._sync_token import SyncToken -class AsyncSyncTokenPolicy(SansIOHTTPPolicy): +class AsyncSyncTokenPolicy(SansIOHTTPPolicy): # pylint: disable=docstring-keyword-should-match-keyword-only """A simple policy that enable the given callback with the response. :keyword callback raw_response_hook: Callback function. Will be invoked on response. diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py index 874cefd7d0bf..593b32ad5aed 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py @@ -4,7 +4,7 @@ # license information. # -------------------------------------------------------------------------- -from azure.appconfiguration._audience import get_audience +from azure.appconfiguration._audience import get_audience # pylint: disable=no-cross-package-private-import # Expected scope constants _EXPECTED_PUBLIC_CLOUD_AUDIENCE = "https://appconfig.azure.com/" diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py index 6e511153af10..463bd4a89907 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py @@ -7,7 +7,7 @@ from testcase import AppConfigTestCase from consts import APPCONFIGURATION_ENDPOINT_STRING from devtools_testutils import EnvironmentVariableLoader, recorded_by_proxy -from azure.appconfiguration._audience_error_handling_policy import ( +from azure.appconfiguration._audience_error_handling_policy import ( # pylint: disable=no-cross-package-private-import AudienceErrorHandlingPolicy, ) diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py index 16a81b668234..cc5284578dd2 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py @@ -8,7 +8,7 @@ from consts import APPCONFIGURATION_ENDPOINT_STRING from devtools_testutils import EnvironmentVariableLoader from devtools_testutils.aio import recorded_by_proxy_async -from azure.appconfiguration._audience_error_handling_policy import ( +from azure.appconfiguration._audience_error_handling_policy import ( # pylint: disable=no-cross-package-private-import AudienceErrorHandlingPolicy, ) diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py index 4f025b399e02..5c8aee772db1 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py @@ -5,7 +5,7 @@ # ------------------------------------------------------------------------- import pytest from azure.core.exceptions import ClientAuthenticationError -from azure.appconfiguration._audience_error_handling_policy import ( +from azure.appconfiguration._audience_error_handling_policy import ( # pylint: disable=no-cross-package-private-import AudienceErrorHandlingPolicy, AAD_AUDIENCE_ERROR_CODE, NO_AUDIENCE_ERROR_MESSAGE, diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py index b3e48edb4285..2c934a7ab5e0 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py @@ -1324,7 +1324,9 @@ def send(self, request: PipelineRequest, **kwargs) -> PipelineResponse: def new_method(request): request.http_request.headers["Authorization"] = str(uuid4()) - from azure.appconfiguration._azure_appconfiguration_requests import AppConfigRequestsCredentialsPolicy + from azure.appconfiguration._azure_appconfiguration_requests import ( # pylint: disable=no-cross-package-private-import + AppConfigRequestsCredentialsPolicy, + ) # Store the method to restore later temp = AppConfigRequestsCredentialsPolicy._signed_request diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py index 92dda975b3a9..3df74c3fd820 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py @@ -1372,7 +1372,9 @@ async def send(self, request: PipelineRequest, **kwargs) -> PipelineResponse: def new_method(request): request.http_request.headers["Authorization"] = str(uuid4()) - from azure.appconfiguration._azure_appconfiguration_requests import AppConfigRequestsCredentialsPolicy + from azure.appconfiguration._azure_appconfiguration_requests import ( # pylint: disable=no-cross-package-private-import + AppConfigRequestsCredentialsPolicy, + ) # Store the method to restore later temp = AppConfigRequestsCredentialsPolicy._signed_request diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py index a97671d93f80..57ea260c09e1 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py @@ -17,7 +17,7 @@ SecretReferenceConfigurationSetting, FILTER_PERCENTAGE, ) -from azure.appconfiguration._generated.models import ( +from azure.appconfiguration._generated.models import ( # pylint: disable=no-cross-package-private-import KeyValue, KeyValueFilter, Snapshot as GeneratedSnapshot, diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py index 8e2800f081bf..67297d6c5a3b 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py @@ -9,10 +9,10 @@ from azure.core.credentials import AzureKeyCredential from azure.core.pipeline.transport import HttpRequest from azure.core.pipeline import PipelineRequest -from azure.appconfiguration._azure_appconfiguration_requests import ( +from azure.appconfiguration._azure_appconfiguration_requests import ( # pylint: disable=no-cross-package-private-import AppConfigRequestsCredentialsPolicy, ) -from azure.appconfiguration._utils import parse_connection_string +from azure.appconfiguration._utils import parse_connection_string # pylint: disable=no-cross-package-private-import def test_parse_connection_string_returns_http_endpoint(): diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py index 91a3238ff77e..f8bd1245e693 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py @@ -7,7 +7,9 @@ import pytest from azure.core.pipeline.transport import HttpRequest from azure.core.pipeline import PipelineRequest -from azure.appconfiguration._query_param_policy import QueryParamPolicy +from azure.appconfiguration._query_param_policy import ( # pylint: disable=no-cross-package-private-import + QueryParamPolicy, +) TEST_URL = "https://example.com" diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py index c4e25e7bd6e6..4f99ef6444a1 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py @@ -6,7 +6,10 @@ from azure.core.pipeline.transport import HttpRequest, HttpResponse from azure.core.pipeline import PipelineRequest, PipelineResponse -from azure.appconfiguration._sync_token import SyncToken, SyncTokenPolicy +from azure.appconfiguration._sync_token import ( # pylint: disable=no-cross-package-private-import + SyncToken, + SyncTokenPolicy, +) def test_parse_sync_token(): From 37e3cc72dcb147675e757b430bf31e2b397c89e8 Mon Sep 17 00:00:00 2001 From: Ali Satwat Khan Date: Mon, 27 Jul 2026 04:22:48 +0500 Subject: [PATCH 2/3] Potential fix for pull request updating docstring Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- .../azure-appconfiguration/azure/appconfiguration/_models.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py index 6f5b882879c2..bc09cb033cbd 100644 --- a/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py +++ b/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_models.py @@ -304,7 +304,7 @@ class SecretReferenceConfigurationSetting(ConfigurationSetting): :param key: The key of the configuration setting. :type key: str - :param secret_id: The identity of the configuration setting. + :param secret_id: The identifier of the secret referenced by this configuration setting. :type secret_id: str """ From e8ae7c204ae8501fc1e83b72f50061bfb690e746 Mon Sep 17 00:00:00 2001 From: Ali Satwat Khan Date: Fri, 31 Jul 2026 03:35:45 +0500 Subject: [PATCH 3/3] Remove per-test Pylint suppressions --- .../azure-appconfiguration/tests/test_audience.py | 2 +- .../tests/test_audience_error_handling_live.py | 2 +- .../tests/test_audience_error_handling_live_async.py | 2 +- .../azure-appconfiguration/tests/test_audience_policy.py | 2 +- .../tests/test_azure_appconfiguration_client.py | 4 +--- .../tests/test_azure_appconfiguration_client_async.py | 4 +--- .../azure-appconfiguration/tests/test_consistency.py | 2 +- .../azure-appconfiguration/tests/test_http.py | 4 ++-- .../azure-appconfiguration/tests/test_query_param_policy.py | 4 +--- .../azure-appconfiguration/tests/test_sync_token_policy.py | 5 +---- 10 files changed, 11 insertions(+), 20 deletions(-) diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py index 593b32ad5aed..874cefd7d0bf 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience.py @@ -4,7 +4,7 @@ # license information. # -------------------------------------------------------------------------- -from azure.appconfiguration._audience import get_audience # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._audience import get_audience # Expected scope constants _EXPECTED_PUBLIC_CLOUD_AUDIENCE = "https://appconfig.azure.com/" diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py index 463bd4a89907..6e511153af10 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py @@ -7,7 +7,7 @@ from testcase import AppConfigTestCase from consts import APPCONFIGURATION_ENDPOINT_STRING from devtools_testutils import EnvironmentVariableLoader, recorded_by_proxy -from azure.appconfiguration._audience_error_handling_policy import ( # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._audience_error_handling_policy import ( AudienceErrorHandlingPolicy, ) diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py index cc5284578dd2..16a81b668234 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py @@ -8,7 +8,7 @@ from consts import APPCONFIGURATION_ENDPOINT_STRING from devtools_testutils import EnvironmentVariableLoader from devtools_testutils.aio import recorded_by_proxy_async -from azure.appconfiguration._audience_error_handling_policy import ( # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._audience_error_handling_policy import ( AudienceErrorHandlingPolicy, ) diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py index 5c8aee772db1..4f025b399e02 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_audience_policy.py @@ -5,7 +5,7 @@ # ------------------------------------------------------------------------- import pytest from azure.core.exceptions import ClientAuthenticationError -from azure.appconfiguration._audience_error_handling_policy import ( # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._audience_error_handling_policy import ( AudienceErrorHandlingPolicy, AAD_AUDIENCE_ERROR_CODE, NO_AUDIENCE_ERROR_MESSAGE, diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py index 2c934a7ab5e0..b3e48edb4285 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py @@ -1324,9 +1324,7 @@ def send(self, request: PipelineRequest, **kwargs) -> PipelineResponse: def new_method(request): request.http_request.headers["Authorization"] = str(uuid4()) - from azure.appconfiguration._azure_appconfiguration_requests import ( # pylint: disable=no-cross-package-private-import - AppConfigRequestsCredentialsPolicy, - ) + from azure.appconfiguration._azure_appconfiguration_requests import AppConfigRequestsCredentialsPolicy # Store the method to restore later temp = AppConfigRequestsCredentialsPolicy._signed_request diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py index 3df74c3fd820..92dda975b3a9 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py @@ -1372,9 +1372,7 @@ async def send(self, request: PipelineRequest, **kwargs) -> PipelineResponse: def new_method(request): request.http_request.headers["Authorization"] = str(uuid4()) - from azure.appconfiguration._azure_appconfiguration_requests import ( # pylint: disable=no-cross-package-private-import - AppConfigRequestsCredentialsPolicy, - ) + from azure.appconfiguration._azure_appconfiguration_requests import AppConfigRequestsCredentialsPolicy # Store the method to restore later temp = AppConfigRequestsCredentialsPolicy._signed_request diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py index 57ea260c09e1..a97671d93f80 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py @@ -17,7 +17,7 @@ SecretReferenceConfigurationSetting, FILTER_PERCENTAGE, ) -from azure.appconfiguration._generated.models import ( # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._generated.models import ( KeyValue, KeyValueFilter, Snapshot as GeneratedSnapshot, diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py index 67297d6c5a3b..8e2800f081bf 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_http.py @@ -9,10 +9,10 @@ from azure.core.credentials import AzureKeyCredential from azure.core.pipeline.transport import HttpRequest from azure.core.pipeline import PipelineRequest -from azure.appconfiguration._azure_appconfiguration_requests import ( # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._azure_appconfiguration_requests import ( AppConfigRequestsCredentialsPolicy, ) -from azure.appconfiguration._utils import parse_connection_string # pylint: disable=no-cross-package-private-import +from azure.appconfiguration._utils import parse_connection_string def test_parse_connection_string_returns_http_endpoint(): diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py index f8bd1245e693..91a3238ff77e 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_query_param_policy.py @@ -7,9 +7,7 @@ import pytest from azure.core.pipeline.transport import HttpRequest from azure.core.pipeline import PipelineRequest -from azure.appconfiguration._query_param_policy import ( # pylint: disable=no-cross-package-private-import - QueryParamPolicy, -) +from azure.appconfiguration._query_param_policy import QueryParamPolicy TEST_URL = "https://example.com" diff --git a/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py b/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py index 4f99ef6444a1..c4e25e7bd6e6 100644 --- a/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py +++ b/sdk/appconfiguration/azure-appconfiguration/tests/test_sync_token_policy.py @@ -6,10 +6,7 @@ from azure.core.pipeline.transport import HttpRequest, HttpResponse from azure.core.pipeline import PipelineRequest, PipelineResponse -from azure.appconfiguration._sync_token import ( # pylint: disable=no-cross-package-private-import - SyncToken, - SyncTokenPolicy, -) +from azure.appconfiguration._sync_token import SyncToken, SyncTokenPolicy def test_parse_sync_token():