test(bigtable): only async-iterate streaming results in materialized view metadata test

read_row, row_exists, and the other non-streaming methods return concrete
values; async-iterating them raised a TypeError that the surrounding
except block silently swallowed, which could mask real errors.

Author: axyjo

packages/google-cloud-bigtable/tests/unit/data/_async/test_client.py

@@ -1788,8 +1788,12 @@ async def test_call_metadata(self, include_app_profile, fn_name, fn_args, gapic_
         view = self._make_one(client, app_profile_id=profile)
         try:
             test_fn = view.__getattribute__(fn_name)
-            maybe_stream = await test_fn(*fn_args)
-            [i async for i in maybe_stream]
+            result = await test_fn(*fn_args)
+            if fn_name == "read_rows_stream":
+                # only streams are async-iterable; the other methods return
+                # concrete values, and iterating them would raise a TypeError
+                # that the except block below hides
+                [i async for i in result]
         except Exception:
             # we expect an exception from attempting to call the mock
             pass

packages/google-cloud-bigtable/tests/unit/data/_sync_autogen/test_client.py

@@ -1477,8 +1477,9 @@ def test_call_metadata(self, include_app_profile, fn_name, fn_args, gapic_fn):
         view = self._make_one(client, app_profile_id=profile)
         try:
             test_fn = view.__getattribute__(fn_name)
-            maybe_stream = test_fn(*fn_args)
-            [i for i in maybe_stream]
+            result = test_fn(*fn_args)
+            if fn_name == "read_rows_stream":
+                [i for i in result]
         except Exception:
             pass
         assert rpc_mock.call_count == 1