App Config - Connection String testing

[mrm9084]

Description

Currently the only difference between Entra ID and Connection string in python is this method:

azure-sdk-for-python/sdk/appconfiguration/azure-appconfiguration/azure/appconfiguration/_azure_appconfiguration_client.py

Lines 113 to 138 in d5ce43f

	def from_connection_string(cls, connection_string: str, **kwargs: Any) -> "AzureAppConfigurationClient":
	"""Create AzureAppConfigurationClient from a Connection String.
	:param str connection_string: Connection String
	(one of the access keys of the Azure App Configuration resource)
	used to access the Azure App Configuration.
	:return: An AzureAppConfigurationClient authenticated with the connection string
	:rtype: ~azure.appconfiguration.AzureAppConfigurationClient
	Example
	.. code-block:: python
	from azure.appconfiguration import AzureAppConfigurationClient
	connection_str = "<my connection string>"
	client = AzureAppConfigurationClient.from_connection_string(connection_str)
	"""
	endpoint, id_credential, secret = parse_connection_string(connection_string)
	# AzureKeyCredential type is for internal use, it's not exposed in public API.
	return cls(
	credential=AzureKeyCredential(secret), # type: ignore[arg-type]
	base_url=endpoint,
	id_credential=id_credential,
	**kwargs,
	)

which makes a token type that is then used the same as Entra ID tokens. We don't need duplicate tests everywhere for connection string.

• I've moved all tests to be Entra ID
• Removed any duplicate tests in connection string files
• Now connection string testing is actual unit testing.

All SDK Contribution checklist:

• The pull request does not introduce [breaking changes]
• CHANGELOG is updated for new features, bug fixes or other significant changes.
• I have read the contribution guidelines.

General Guidelines and Best Practices

• Title of the pull request is clear and informative.
• There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

• Pull request includes test coverage for the included changes.

[Copilot]

Pull request overview

This PR refactors the Azure App Configuration Python test suite to reduce duplicated coverage between connection-string auth and Microsoft Entra ID auth, shifting most live tests to Entra ID and keeping connection-string validation primarily as unit tests.

Changes:

• Updated test case helpers to set up data using Entra ID clients and removed connection-string-specific helper paths.
• Removed large connection-string live test suites and added focused unit tests for from_connection_string behavior (sync + async).
• Updated perf stress tests and documentation to use an endpoint + DefaultAzureCredential instead of a connection string.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
sdk/appconfiguration/azure-appconfiguration/tests/testcase.py	Simplifies setup to Entra ID, removes connection-string helper methods.
sdk/appconfiguration/azure-appconfiguration/tests/asynctestcase.py	Aligns async test helper creation/setup with Entra ID path.
sdk/appconfiguration/azure-appconfiguration/tests/test_consistency.py	Switches live consistency tests from connection string env var to endpoint env var.
sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client.py	Removes connection-string live tests; adds unit tests for from_connection_string.
sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_async.py	Removes connection-string live tests; adds async unit tests for from_connection_string.
sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_aad.py	Updates live tests to use the unified client creation path and adds additional test coverage.
sdk/appconfiguration/azure-appconfiguration/tests/test_azure_appconfiguration_client_aad_async.py	Updates live async tests similarly and adds coverage for additional behaviors.
sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live.py	Updates audience live tests to use the unified client creation path.
sdk/appconfiguration/azure-appconfiguration/tests/test_audience_error_handling_live_async.py	Updates async audience live tests to use the unified client creation path.
sdk/appconfiguration/azure-appconfiguration/tests/perfstress_tests/get.py	Switches perf test auth to Entra ID via endpoint + DefaultAzureCredential.
sdk/appconfiguration/azure-appconfiguration/tests/perfstress_tests/set.py	Switches perf test auth to Entra ID via endpoint + DefaultAzureCredential.
sdk/appconfiguration/azure-appconfiguration/tests/perfstress_tests/README.md	Updates perf test required env var from connection string to endpoint.
sdk/appconfiguration/azure-appconfiguration/assets.json	Updates recorded assets tag reference.

[avanigupta]

is this a duplicate of TestAppConfigurationClientUnitTest underneath this?

[avanigupta]

same here, why different aad class?