test: Avoid circular import in Azure tests (#5280)

cloudinit/sources/helpers/azure.py

@@ -1,4 +1,7 @@
 # This file is part of cloud-init. See LICENSE file for license information.
+
+# TODO: Importing this file without first importing
+# cloudinit.sources.azure.errors will result in a circular import.
 import base64
 import json
 import logging

tests/unittests/reporting/test_reporting_hyperv.py

@@ -11,6 +11,11 @@
 from cloudinit import util
 from cloudinit.reporting import events, instantiated_handler_registry
 from cloudinit.reporting.handlers import HyperVKvpReportingHandler
+
+# TODO: Importing `errors` here is a hack to avoid a circular import.
+# Without it, we have a azure->errors->identity->azure import loop, but
+# long term we should restructure these modules to avoid the issue.
+from cloudinit.sources.azure import errors  # noqa: F401
 from cloudinit.sources.helpers import azure
 from tests.unittests.helpers import CiTestCase
 

tests/unittests/sources/azure/test_imds.py

@@ -10,6 +10,10 @@
 import requests
 import responses
 
+# TODO: Importing `errors` here is a hack to avoid a circular import.
+# Without it, we have a azure->errors->identity->azure import loop, but
+# long term we should restructure these modules to avoid the issue.
+from cloudinit.sources.azure import errors as _errors  # noqa: F401
 from cloudinit.sources.azure import imds
 from cloudinit.url_helper import UrlError, readurl