🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Author: gcf-owl-bot[bot]

google/cloud/spanner_v1/__init__.py

@@ -64,10 +64,7 @@
 from .types.type import TypeAnnotationCode
 from .types.type import TypeCode
 from .data_types import JsonObject
-from .transaction import (
-    BatchTransactionId,
-    DefaultTransactionOptions
-)
+from .transaction import BatchTransactionId, DefaultTransactionOptions
 
 from google.cloud.spanner_v1 import param_types
 from google.cloud.spanner_v1.client import Client
@@ -152,5 +149,5 @@
     "SpannerClient",
     "SpannerAsyncClient",
     "BatchTransactionId",
-    "DefaultTransactionOptions"
+    "DefaultTransactionOptions",
 )

google/cloud/spanner_v1/client.py

@@ -166,7 +166,7 @@ class Client(ClientWithProject):
     :type default_transaction_options: :class:`~google.cloud.spanner_v1.DefaultTransactionOptions`
         or :class:`dict`
     :param default_transaction_options: (Optional) Default options to use for all transactions.
-    
+
     :raises: :class:`ValueError <exceptions.ValueError>` if both ``read_only``
              and ``admin`` are :data:`True`
     """
@@ -188,7 +188,7 @@ def __init__(
         route_to_leader_enabled=True,
         directed_read_options=None,
         observability_options=None,
-        default_transaction_options: Optional[DefaultTransactionOptions] =None,
+        default_transaction_options: Optional[DefaultTransactionOptions] = None,
     ):
         self._emulator_host = _get_spanner_emulator_host()
 
@@ -253,7 +253,9 @@ def __init__(
         if default_transaction_options is None:
             default_transaction_options = DefaultTransactionOptions()
         elif not isinstance(default_transaction_options, DefaultTransactionOptions):
-            raise TypeError("default_transaction_options must be an instance of DefaultTransactionOptions")
+            raise TypeError(
+                "default_transaction_options must be an instance of DefaultTransactionOptions"
+            )
 
         self._default_transaction_options = default_transaction_options
 
@@ -504,7 +506,9 @@ def directed_read_options(self, directed_read_options):
         self._directed_read_options = directed_read_options
 
     @default_transaction_options.setter
-    def default_transaction_options(self, default_transaction_options:DefaultTransactionOptions):
+    def default_transaction_options(
+        self, default_transaction_options: DefaultTransactionOptions
+    ):
         """Sets default_transaction_options for the client
         :type default_transaction_options: :class:`~google.cloud.spanner_v1.DefaultTransactionOptions`
             or :class:`dict`
@@ -513,6 +517,8 @@ def default_transaction_options(self, default_transaction_options:DefaultTransac
         if default_transaction_options is None:
             default_transaction_options = DefaultTransactionOptions()
         elif not isinstance(default_transaction_options, DefaultTransactionOptions):
-            raise TypeError("default_transaction_options must be an instance of DefaultTransactionOptions")
+            raise TypeError(
+                "default_transaction_options must be an instance of DefaultTransactionOptions"
+            )
 
         self._default_transaction_options = default_transaction_options

google/cloud/spanner_v1/transaction.py

@@ -656,6 +656,9 @@ class BatchTransactionId:
     session_id: str
     read_timestamp: Any
 
+
 @dataclass
 class DefaultTransactionOptions:
-    isolation_level: TransactionOptions.IsolationLevel = TransactionOptions.IsolationLevel.ISOLATION_LEVEL_UNSPECIFIED
+    isolation_level: TransactionOptions.IsolationLevel = (
+        TransactionOptions.IsolationLevel.ISOLATION_LEVEL_UNSPECIFIED
+    )

tests/unit/test_client.py

@@ -16,11 +16,7 @@
 
 import os
 import mock
-from google.cloud.spanner_v1 import (
-    DirectedReadOptions,
-    DefaultTransactionOptions
-)
-
+from google.cloud.spanner_v1 import DirectedReadOptions, DefaultTransactionOptions
 
 
 def _make_credentials():
@@ -57,7 +53,9 @@ class TestClient(unittest.TestCase):
             "auto_failover_disabled": True,
         },
     }
-    DEFAULT_TRANSACTION_OPTIONS = DefaultTransactionOptions(isolation_level="SERIALIZABLE")
+    DEFAULT_TRANSACTION_OPTIONS = DefaultTransactionOptions(
+        isolation_level="SERIALIZABLE"
+    )
 
     def _get_target_class(self):
         from google.cloud import spanner

tests/unit/test_database.py

@@ -25,7 +25,11 @@
 from google.api_core.retry import Retry
 from google.protobuf.field_mask_pb2 import FieldMask
 
-from google.cloud.spanner_v1 import RequestOptions, DirectedReadOptions, DefaultTransactionOptions
+from google.cloud.spanner_v1 import (
+    RequestOptions,
+    DirectedReadOptions,
+    DefaultTransactionOptions,
+)
 
 DML_WO_PARAM = """
 DELETE FROM citizens

tests/unit/test_session.py

@@ -85,7 +85,9 @@ def _make_one(self, *args, **kwargs):
 
     @staticmethod
     def _make_database(
-        name=DATABASE_NAME, database_role=None, default_transaction_options=DefaultTransactionOptions()
+        name=DATABASE_NAME,
+        database_role=None,
+        default_transaction_options=DefaultTransactionOptions(),
     ):
         database = mock.create_autospec(Database, instance=True)
         database.name = name
@@ -1776,7 +1778,9 @@ def test_run_in_transaction_w_isolation_level_at_client(self):
         gax_api = self._make_spanner_api()
         gax_api.begin_transaction.return_value = TransactionPB(id=b"FACEDACE")
         database = self._make_database(
-            default_transaction_options=DefaultTransactionOptions(isolation_level="SERIALIZABLE")
+            default_transaction_options=DefaultTransactionOptions(
+                isolation_level="SERIALIZABLE"
+            )
         )
         database.spanner_api = gax_api
         session = self._make_one(database)