Raise error on provider not found

CloudVE · Apr 13, 2024 · 9cf27ee · 9cf27ee
1 parent b38e6f0
commit 9cf27ee
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/cloudbridge/factory.py b/cloudbridge/factory.py
@@ -69,10 +69,7 @@ def discover_providers(self):
         """
         for _, modname, _ in pkgutil.iter_modules(providers.__path__):
             log.debug("Importing provider: %s", modname)
-            try:
-                self._import_provider(modname)
-            except Exception as e:
-                log.debug("Could not import provider: %s", e)
+            self._import_provider(modname)
 
     def _import_provider(self, module_name):
         """

diff --git a/tests/test_cloud_factory.py b/tests/test_cloud_factory.py
@@ -37,9 +37,10 @@ def test_get_provider_class_invalid(self):
         self.assertIsNone(CloudProviderFactory().get_provider_class("aws1"))
 
     def test_find_provider_include_mocks(self):
+        providers = CloudProviderFactory().get_all_provider_classes()
         self.assertTrue(
             any(cls for cls
-                in CloudProviderFactory().get_all_provider_classes()
+                in providers
                 if issubclass(cls, TestMockHelperMixin)),
             "expected to find at least one mock provider")