Add logging and tests

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>

Author: pvaneck

sdk/identity/azure-identity/azure/identity/_credentials/default.py

@@ -212,6 +212,11 @@ def __init__(self, **kwargs: Any) -> None:  # pylint: disable=too-many-statement
                 broker_credential_args["client_id"] = interactive_browser_client_id
 
             credentials.append(broker_credential_class(**broker_credential_args))
+        else:
+            _LOGGER.info(
+                "InteractiveBrowserBrokerCredential unavailable. Brokered authentication in DefaultAzureCredential is "
+                "only available on Windows and requires the 'azure-identity-broker' package to be installed."
+            )
 
         within_dac.set(False)
         super(DefaultAzureCredential, self).__init__(*credentials)

sdk/identity/azure-identity/dev_requirements.txt

@@ -1,4 +1,5 @@
 ../../core/azure-core
+../azure-identity-broker
 aiohttp>=3.0
 typing_extensions>=3.7.2
 -e ../../../tools/azure-sdk-tools

sdk/identity/azure-identity/tests/test_default.py

@@ -3,6 +3,7 @@
 # Licensed under the MIT License.
 # ------------------------------------
 import os
+import sys
 
 from azure.core.credentials import AccessToken, AccessTokenInfo
 from azure.identity import (
@@ -432,3 +433,22 @@ def test_validate_cloud_shell_credential_in_dac():
         DefaultAzureCredential(identity_config={"client_id": "foo"})
         DefaultAzureCredential(identity_config={"object_id": "foo"})
         DefaultAzureCredential(identity_config={"resource_id": "foo"})
+
+
+@pytest.mark.skipif(not sys.platform.startswith("win"), reason="tests Windows-specific behavior")
+def test_broker_credential():
+    """Test that DefaultAzureCredential uses the broker credential when available"""
+    with patch("azure.identity.broker.InteractiveBrowserBrokerCredential") as mock_credential:
+        DefaultAzureCredential()
+    assert mock_credential.call_count == 1, "InteractiveBrowserBrokerCredential should be instantiated once"
+
+
+@pytest.mark.skipif(not sys.platform.startswith("win"), reason="tests Windows-specific behavior")
+def test_broker_credential_requirements_not_installed():
+    """Test that DefaultAzureCredential does not use the broker credential when requirements are not installed"""
+
+    with patch.dict("sys.modules", {"azure.identity.broker": None}):
+        with patch("azure.identity.broker.InteractiveBrowserBrokerCredential") as mock_credential:
+            DefaultAzureCredential()
+
+    assert mock_credential.call_count == 0, "InteractiveBrowserBrokerCredential should not be instantiated"