coleheflin commented on PR #65879: URL: https://github.com/apache/airflow/pull/65879#issuecomment-4322570749
## Manual Testing Completed The following manual tests were run locally against the Breeze development environment: **Import sanity check** ✅ All four new classes import cleanly: ``` from airflow.providers.microsoft.azure.hooks.analysis_services import AzureAnalysisServicesHook from airflow.providers.microsoft.azure.operators.analysis_services import AzureAnalysisServicesRefreshOperator from airflow.providers.microsoft.azure.sensors.analysis_services import AzureAnalysisServicesSensor from airflow.providers.microsoft.azure.triggers.analysis_services import AzureAnalysisServicesRefreshTrigger # All imports OK ``` **DAG authoring** ✅ A DAG using `AzureAnalysisServicesRefreshOperator` parses without errors inside Breeze. **Connection type UI** ✅ The `azure_analysis_services` connection type appears in Admin → Connections with the correct custom field labels (Region Endpoint, Client ID, Client Secret, Tenant ID) and hidden fields (Schema, Port, Extra). **Unit tests** ✅ (40/40 passing) All hook, operator, sensor, and trigger tests pass with no live Azure credentials required. The only remaining test is the end-to-end system test against a live Azure Analysis Services instance, which requires real credentials. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
