fix: add grpc check back to test modules to prevent ImportError on non-grpc environments

Author: hebaalazzeh

packages/google-api-core/tests/unit/gapic/test_client_cert.py

@@ -18,6 +18,14 @@
 
 import pytest
 
+# We need to skip this test module if grpc is not installed because importing
+# gapic_v1._client_cert will load gapic_v1/__init__, which unconditionally
+# imports gapic_v1.config, which imports grpc.
+try:
+    import grpc  # noqa: F401
+except ImportError:
+    pytest.skip("No GRPC", allow_module_level=True)
+
 from google.api_core.gapic_v1._client_cert import (
     _get_client_cert_source,
     _use_client_cert_effective,
@@ -60,12 +68,8 @@ def test_use_client_cert_effective_fallback():
                 _use_client_cert_effective()
 
 
-@mock.patch(
-    "google.auth.transport.mtls.has_default_client_cert_source", create=True
-)  # noqa: E501
-@mock.patch(
-    "google.auth.transport.mtls.default_client_cert_source", create=True
-)  # noqa: E501
+@mock.patch("google.auth.transport.mtls.has_default_client_cert_source", create=True)  # noqa: E501
+@mock.patch("google.auth.transport.mtls.default_client_cert_source", create=True)  # noqa: E501
 def test_get_client_cert_source(mock_default, mock_has_default):
     mock_default.return_value = b"default_cert"
     mock_has_default.return_value = True

packages/google-api-core/tests/unit/gapic/test_config_helpers.py

@@ -18,14 +18,20 @@
 
 import pytest
 
+# We need to skip this test module if grpc is not installed because importing
+# gapic_v1._config_helpers will load gapic_v1/__init__, which unconditionally
+# imports gapic_v1.config, which imports grpc.
+try:
+    import grpc  # noqa: F401
+except ImportError:
+    pytest.skip("No GRPC", allow_module_level=True)
+
 from google.auth.exceptions import MutualTLSChannelError
 
 from google.api_core.gapic_v1._config_helpers import _read_environment_variables
 
 
-@mock.patch(
-    "google.api_core.gapic_v1._config_helpers._use_client_cert_effective"
-)  # noqa: E501
+@mock.patch("google.api_core.gapic_v1._config_helpers._use_client_cert_effective")  # noqa: E501
 @mock.patch.dict(os.environ, clear=True)
 def test_read_environment_variables(mock_effective):
     mock_effective.return_value = True

packages/google-api-core/tests/unit/gapic/test_method_helpers.py

@@ -14,6 +14,17 @@
 # limitations under the License.
 
 
+import pytest
+
+# We need to skip this test module if grpc is not installed because importing
+# gapic_v1._method_helpers will load gapic_v1/__init__, which unconditionally
+# imports gapic_v1.config, which imports grpc.
+try:
+    import grpc  # noqa: F401
+except ImportError:
+    pytest.skip("No GRPC", allow_module_level=True)
+
+
 from google.api_core.gapic_v1._method_helpers import _setup_request_id
 
 

packages/google-api-core/tests/unit/gapic/test_routing.py

@@ -17,6 +17,14 @@
 
 import pytest
 
+# We need to skip this test module if grpc is not installed because importing
+# gapic_v1._routing will load gapic_v1/__init__, which unconditionally
+# imports gapic_v1.config, which imports grpc.
+try:
+    import grpc  # noqa: F401
+except ImportError:
+    pytest.skip("No GRPC", allow_module_level=True)
+
 from google.auth.exceptions import MutualTLSChannelError
 
 from google.api_core.gapic_v1._routing import (
@@ -144,9 +152,7 @@ def test_get_universe_domain():
     )
 
     # fallback to default
-    assert (
-        _get_universe_domain(None, None, "default.com") == "default.com"
-    )  # noqa: E501
+    assert _get_universe_domain(None, None, "default.com") == "default.com"  # noqa: E501
 
 
 def test_get_universe_domain_strip():