Fix: Update type hints for various BigQuery files (#2206)

* Fix: Update type hints for various BigQuery files This commit addresses Issue #2132 by updating type hints in the following files: - google/cloud/bigquery/external_config.py - google/cloud/bigquery/job/base.py - google/cloud/bigquery/routine/routine.py - google/cloud/bigquery/schema.py - google/cloud/bigquery/table.py These changes improve code clarity and maintainability by providing more accurate type information. * updates type hints across the board --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Author: chalmerlowe

google/cloud/bigquery/external_config.py

@@ -22,6 +22,7 @@
 
 import base64
 import copy
+import typing
 from typing import Any, Dict, FrozenSet, Iterable, Optional, Union
 
 from google.cloud.bigquery._helpers import _to_bytes
@@ -835,10 +836,10 @@ def schema(self):
  See
  https://cloud.google.com/bigquery/docs/reference/rest/v2/tables#ExternalDataConfiguration.FIELDS.schema
  """
- # TODO: The typehinting for this needs work. Setting this pragma to temporarily
- # manage a pytype issue that came up in another PR. See Issue: #2132
- prop = self._properties.get("schema", {}) # type: ignore
- return [SchemaField.from_api_repr(field) for field in prop.get("fields", [])] # type: ignore
+ prop: Dict[str, Any] = typing.cast(
+  Dict[str, Any], self._properties.get("schema", {})
+ )
+ return [SchemaField.from_api_repr(field) for field in prop.get("fields", [])]
 
  @schema.setter
  def schema(self, value):

google/cloud/bigquery/job/base.py

@@ -435,9 +435,7 @@ def __init__(self, job_id, client):
  @property
  def configuration(self) -> _JobConfig:
  """Job-type specific configurtion."""
- # TODO: The typehinting for this needs work. Setting this pragma to temporarily
- # manage a pytype issue that came up in another PR. See Issue: #2132
- configuration = self._CONFIG_CLASS() # pytype: disable=not-callable
+ configuration: _JobConfig = self._CONFIG_CLASS() # pytype: disable=not-callable
  configuration._properties = self._properties.setdefault("configuration", {})
  return configuration
 

google/cloud/bigquery/routine/routine.py

@@ -518,23 +518,17 @@ def __init__(self):
  @property
  def project(self):
  """str: ID of the project containing the routine."""
- # TODO: The typehinting for this needs work. Setting this pragma to temporarily
- # manage a pytype issue that came up in another PR. See Issue: #2132
- return self._properties["projectId"] # pytype: disable=typed-dict-error
+ return self._properties.get("projectId", "")
 
  @property
  def dataset_id(self):
  """str: ID of dataset containing the routine."""
- # TODO: The typehinting for this needs work. Setting this pragma to temporarily
- # manage a pytype issue that came up in another PR. See Issue: #2132
- return self._properties["datasetId"] # pytype: disable=typed-dict-error
+ return self._properties.get("datasetId", "")
 
  @property
  def routine_id(self):
  """str: The routine ID."""
- # TODO: The typehinting for this needs work. Setting this pragma to temporarily
- # manage a pytype issue that came up in another PR. See Issue: #2132
- return self._properties["routineId"] # pytype: disable=typed-dict-error
+ return self._properties.get("routineId", "")
 
  @property
  def path(self):

google/cloud/bigquery/schema.py

@@ -232,11 +232,9 @@ def __init__(
  if max_length is not _DEFAULT_VALUE:
  self._properties["maxLength"] = max_length
  if policy_tags is not _DEFAULT_VALUE:
- # TODO: The typehinting for this needs work. Setting this pragma to temporarily
- # manage a pytype issue that came up in another PR. See Issue: #2132
  self._properties["policyTags"] = (
- policy_tags.to_api_repr() # pytype: disable=attribute-error
- if policy_tags is not None
+ policy_tags.to_api_repr()
+ if isinstance(policy_tags, PolicyTagList)
  else None
  )
  if isinstance(range_element_type, str):

google/cloud/bigquery/table.py

@@ -137,9 +137,9 @@ def _reference_getter(table):
  return TableReference(dataset_ref, table.table_id)
 
 
-# TODO: The typehinting for this needs work. Setting this pragma to temporarily
-# manage a pytype issue that came up in another PR. See Issue: #2132
-def _view_use_legacy_sql_getter(table):
+def _view_use_legacy_sql_getter(
+ table: Union["Table", "TableListItem"]
+) -> Optional[bool]:
  """bool: Specifies whether to execute the view with Legacy or Standard SQL.
 
  This boolean specifies whether to execute the view with Legacy SQL
@@ -151,15 +151,16 @@ def _view_use_legacy_sql_getter(table):
  ValueError: For invalid value types.
  """
 
- view = table._properties.get("view") # type: ignore
+ view: Optional[Dict[str, Any]] = table._properties.get("view")
  if view is not None:
  # The server-side default for useLegacySql is True.
- return view.get("useLegacySql", True)  # type: ignore
+ return view.get("useLegacySql", True) if view is not None else True
  # In some cases, such as in a table list no view object is present, but the
  # resource still represents a view. Use the type as a fallback.
  if table.table_type == "VIEW":
  # The server-side default for useLegacySql is True.
  return True
+ return None # explicit return statement to appease mypy
 
 
 class _TableBase: