cruseakshay commented on code in PR #64941:
URL: https://github.com/apache/airflow/pull/64941#discussion_r3254388203
##########
airflow-core/tests/unit/always/test_providers_manager.py:
##########
@@ -258,6 +259,90 @@ def test_dialects(self):
assert len(dialect_class_names) == 3
assert dialect_class_names == ["default", "mssql", "postgresql"]
+ def test_discover_object_storage_providers(self):
+ providers_manager = ProvidersManager()
+ providers_manager._provider_dict = LazyDictWithCache()
+ providers_manager._provider_dict["airflow.providers.amazon"] =
ProviderInfo(
Review Comment:
Added `test_object_storage_providers` that mirrors test_dialects: drives the
assertions through the public `ProvidersManager.object_storage_providers
property` without any manual injection into _provider_dict, so it covers the
real auto-registration chain.
The existing manual addition tests are kept for the unit-level coverage, in
line with the test_providers_manager_register_dialects + test_dialects.
--
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]
