chore: update credentials file unit tests

Author: ohmayr

tests/asyncio/test_grpc_helpers_async.py

@@ -522,11 +522,12 @@ def test_create_channel_explicit_with_duplicate_credentials():
     target = "example:443"
 
     with pytest.raises(exceptions.DuplicateCredentialArgs) as excinfo:
-        grpc_helpers_async.create_channel(
-            target,
-            credentials_file="credentials.json",
-            credentials=mock.sentinel.credentials,
-        )
+        with pytest.warns(DeprecationWarning):
+            grpc_helpers_async.create_channel(
+                target,
+                credentials_file="credentials.json",
+                credentials=mock.sentinel.credentials,
+            )
 
     assert "mutually exclusive" in str(excinfo.value)
 
@@ -641,9 +642,10 @@ def test_create_channel_with_credentials_file(
     credentials_file = "/path/to/credentials/file.json"
     composite_creds = composite_creds_call.return_value
 
-    channel = grpc_helpers_async.create_channel(
-        target, credentials_file=credentials_file
-    )
+    with pytest.warns(DeprecationWarning):
+        channel = grpc_helpers_async.create_channel(
+            target, credentials_file=credentials_file
+        )
 
     google.auth.load_credentials_from_file.assert_called_once_with(
         credentials_file, scopes=None, default_scopes=None
@@ -670,9 +672,10 @@ def test_create_channel_with_credentials_file_and_scopes(
     credentials_file = "/path/to/credentials/file.json"
     composite_creds = composite_creds_call.return_value
 
-    channel = grpc_helpers_async.create_channel(
-        target, credentials_file=credentials_file, scopes=scopes
-    )
+    with pytest.warns(DeprecationWarning):
+        channel = grpc_helpers_async.create_channel(
+            target, credentials_file=credentials_file, scopes=scopes
+        )
 
     google.auth.load_credentials_from_file.assert_called_once_with(
         credentials_file, scopes=scopes, default_scopes=None
@@ -699,9 +702,10 @@ def test_create_channel_with_credentials_file_and_default_scopes(
     credentials_file = "/path/to/credentials/file.json"
     composite_creds = composite_creds_call.return_value
 
-    channel = grpc_helpers_async.create_channel(
-        target, credentials_file=credentials_file, default_scopes=default_scopes
-    )
+    with pytest.warns(DeprecationWarning):
+        channel = grpc_helpers_async.create_channel(
+            target, credentials_file=credentials_file, default_scopes=default_scopes
+        )
 
     google.auth.load_credentials_from_file.assert_called_once_with(
         credentials_file, scopes=None, default_scopes=default_scopes

tests/unit/operations_v1/test_operations_rest_client.py

@@ -369,7 +369,8 @@ def test_operations_client_client_options(
         )
 
     # Check the case credentials_file is provided
-    options = client_options.ClientOptions(credentials_file="credentials.json")
+    with pytest.warns(DeprecationWarning):
+        options = client_options.ClientOptions(credentials_file="credentials.json")
     with mock.patch.object(transport_class, "__init__") as patched:
         patched.return_value = None
         client = client_class(client_options=options, transport=transport_name)
@@ -539,7 +540,8 @@ def test_operations_client_client_options_credentials_file(
     client_class, transport_class, transport_name
 ):
     # Check the case credentials file is provided.
-    options = client_options.ClientOptions(credentials_file="credentials.json")
+    with pytest.warns(DeprecationWarning):
+        options = client_options.ClientOptions(credentials_file="credentials.json")
     if "async" in str(client_class):
         # TODO(): Add support for credentials file to async REST transport.
         with pytest.raises(core_exceptions.AsyncRestUnsupportedParameterError):
@@ -570,10 +572,17 @@ def test_operations_client_client_options_credentials_file(
     return_value=(mock.sentinel.credentials, mock.sentinel.project),
 )
 def test_list_operations_rest(google_auth_default, credentials_file):
-    sync_transport = transports.rest.OperationsRestTransport(
-        credentials_file=credentials_file,
-        http_options=HTTP_OPTIONS,
-    )
+    if credentials_file is not None:
+        with pytest.warns(DeprecationWarning):
+            sync_transport = transports.rest.OperationsRestTransport(
+                credentials_file=credentials_file,
+                http_options=HTTP_OPTIONS,
+            )
+    else:
+        sync_transport = transports.rest.OperationsRestTransport(
+            credentials_file=credentials_file,
+            http_options=HTTP_OPTIONS,
+        )
 
     client = AbstractOperationsClient(transport=sync_transport)
 
@@ -1130,10 +1139,11 @@ def test_transport_adc(client_class, transport_class, credentials):
 def test_operations_base_transport_error():
     # Passing both a credentials object and credentials_file should raise an error
     with pytest.raises(core_exceptions.DuplicateCredentialArgs):
-        transports.OperationsTransport(
-            credentials=ga_credentials.AnonymousCredentials(),
-            credentials_file="credentials.json",
-        )
+        with pytest.warns(DeprecationWarning):
+            transports.OperationsTransport(
+                credentials=ga_credentials.AnonymousCredentials(),
+                credentials_file="credentials.json",
+            )
 
 
 def test_operations_base_transport():
@@ -1171,10 +1181,11 @@ def test_operations_base_transport_with_credentials_file():
     ) as Transport:
         Transport.return_value = None
         load_creds.return_value = (ga_credentials.AnonymousCredentials(), None)
-        transports.OperationsTransport(
-            credentials_file="credentials.json",
-            quota_project_id="octopus",
-        )
+        with pytest.warns(DeprecationWarning):
+            transports.OperationsTransport(
+                credentials_file="credentials.json",
+                quota_project_id="octopus",
+            )
         load_creds.assert_called_once_with(
             "credentials.json",
             scopes=None,

tests/unit/test_client_options.py

@@ -27,19 +27,19 @@ def get_client_encrypted_cert():
 
 
 def test_constructor():
-
-    options = client_options.ClientOptions(
-        api_endpoint="foo.googleapis.com",
-        client_cert_source=get_client_cert,
-        quota_project_id="quote-proj",
-        credentials_file="path/to/credentials.json",
-        scopes=[
-            "https://www.googleapis.com/auth/cloud-platform",
-            "https://www.googleapis.com/auth/cloud-platform.read-only",
-        ],
-        api_audience="foo2.googleapis.com",
-        universe_domain="googleapis.com",
-    )
+    with pytest.warns(DeprecationWarning):
+        options = client_options.ClientOptions(
+            api_endpoint="foo.googleapis.com",
+            client_cert_source=get_client_cert,
+            quota_project_id="quote-proj",
+            credentials_file="path/to/credentials.json",
+            scopes=[
+                "https://www.googleapis.com/auth/cloud-platform",
+                "https://www.googleapis.com/auth/cloud-platform.read-only",
+            ],
+            api_audience="foo2.googleapis.com",
+            universe_domain="googleapis.com",
+        )
 
     assert options.api_endpoint == "foo.googleapis.com"
     assert options.client_cert_source() == (b"cert", b"key")
@@ -102,10 +102,11 @@ def test_constructor_with_api_key():
 
 def test_constructor_with_both_api_key_and_credentials_file():
     with pytest.raises(ValueError):
-        client_options.ClientOptions(
-            api_key="api-key",
-            credentials_file="path/to/credentials.json",
-        )
+        with pytest.warns(DeprecationWarning):
+            client_options.ClientOptions(
+                api_key="api-key",
+                credentials_file="path/to/credentials.json",
+            )
 
 
 def test_from_dict():

tests/unit/test_grpc_helpers.py

@@ -581,11 +581,12 @@ def test_create_channel_explicit_with_duplicate_credentials():
     target = "example.com:443"
 
     with pytest.raises(exceptions.DuplicateCredentialArgs):
-        grpc_helpers.create_channel(
-            target,
-            credentials_file="credentials.json",
-            credentials=mock.sentinel.credentials,
-        )
+        with pytest.warns(DeprecationWarning):
+            grpc_helpers.create_channel(
+                target,
+                credentials_file="credentials.json",
+                credentials=mock.sentinel.credentials,
+            )
 
 
 @mock.patch("grpc.compute_engine_channel_credentials")
@@ -710,7 +711,8 @@ def test_create_channel_with_credentials_file(
     credentials_file = "/path/to/credentials/file.json"
     composite_creds = composite_creds_call.return_value
 
-    channel = grpc_helpers.create_channel(target, credentials_file=credentials_file)
+    with pytest.warns(DeprecationWarning):
+        channel = grpc_helpers.create_channel(target, credentials_file=credentials_file)
 
     google.auth.load_credentials_from_file.assert_called_once_with(
         credentials_file, scopes=None, default_scopes=None
@@ -742,9 +744,10 @@ def test_create_channel_with_credentials_file_and_scopes(
     credentials_file = "/path/to/credentials/file.json"
     composite_creds = composite_creds_call.return_value
 
-    channel = grpc_helpers.create_channel(
-        target, credentials_file=credentials_file, scopes=scopes
-    )
+    with pytest.warns(DeprecationWarning):
+        channel = grpc_helpers.create_channel(
+            target, credentials_file=credentials_file, scopes=scopes
+        )
 
     google.auth.load_credentials_from_file.assert_called_once_with(
         credentials_file, scopes=scopes, default_scopes=None
@@ -776,9 +779,10 @@ def test_create_channel_with_credentials_file_and_default_scopes(
     credentials_file = "/path/to/credentials/file.json"
     composite_creds = composite_creds_call.return_value
 
-    channel = grpc_helpers.create_channel(
-        target, credentials_file=credentials_file, default_scopes=default_scopes
-    )
+    with pytest.warns(DeprecationWarning):
+        channel = grpc_helpers.create_channel(
+            target, credentials_file=credentials_file, default_scopes=default_scopes
+        )
 
     load_credentials_from_file.assert_called_once_with(
         credentials_file, scopes=None, default_scopes=default_scopes

tests/unit/test_python_version_support.py

@@ -178,17 +178,20 @@ def test_override_gapic_end_only():
             "google.api_core._python_version_support.PYTHON_VERSION_INFO",
             {version_tuple: overridden_info},
         ):
-            result_before_boundary = check_python_version(
-                today=custom_gapic_end + datetime.timedelta(days=-1)
-            )
+            with pytest.warns(FutureWarning):
+                result_before_boundary = check_python_version(
+                    today=custom_gapic_end + datetime.timedelta(days=-1)
+                )
             assert result_before_boundary == PythonVersionStatus.PYTHON_VERSION_EOL
 
-            result_at_boundary = check_python_version(today=custom_gapic_end)
+            with pytest.warns(FutureWarning):
+                result_at_boundary = check_python_version(today=custom_gapic_end)
             assert result_at_boundary == PythonVersionStatus.PYTHON_VERSION_EOL
 
-            result_after_boundary = check_python_version(
-                today=custom_gapic_end + datetime.timedelta(days=1)
-            )
+            with pytest.warns(FutureWarning):
+                result_after_boundary = check_python_version(
+                    today=custom_gapic_end + datetime.timedelta(days=1)
+                )
             assert (
                 result_after_boundary == PythonVersionStatus.PYTHON_VERSION_UNSUPPORTED
             )
@@ -217,7 +220,8 @@ def test_override_gapic_deprecation_only():
                 result_before_boundary == PythonVersionStatus.PYTHON_VERSION_SUPPORTED
             )
 
-            result_at_boundary = check_python_version(today=custom_gapic_dep)
+            with pytest.warns(FutureWarning):
+                result_at_boundary = check_python_version(today=custom_gapic_dep)
             assert result_at_boundary == PythonVersionStatus.PYTHON_VERSION_DEPRECATED