chore: [autoapprove] Update black and isort to latest versions

.github/.OwlBot.lock.yaml

@@ -13,5 +13,5 @@
 # limitations under the License.
 docker:
   image: gcr.io/cloud-devrel-public-resources/owlbot-python:latest
-  digest: sha256:fac304457974bb530cc5396abd4ab25d26a469cd3bc97cbfb18c8d4324c584eb
-# created: 2023-10-02T21:31:03.517640371Z
+  digest: sha256:08e34975760f002746b1d8c86fdc90660be45945ee6d9db914d1508acdf9a547
+# created: 2023-10-09T14:06:13.397766266Z

.kokoro/requirements.txt

@@ -467,9 +467,9 @@ typing-extensions==4.4.0 \
     --hash=sha256:1511434bb92bf8dd198c12b1cc812e800d4181cfcb867674e0f8279cc93087aa \
     --hash=sha256:16fa4864408f655d35ec496218b85f79b3437c829e93320c7c9215ccfd92489e
     # via -r requirements.in
-urllib3==1.26.12 \
-    --hash=sha256:3fa96cf423e6987997fc326ae8df396db2a8b7c667747d47ddd8ecba91f4a74e \
-    --hash=sha256:b930dd878d5a8afb066a637fbb35144fe7901e3b209d1cd4f524bd0e9deee997
+urllib3==1.26.17 \
+    --hash=sha256:24d6a242c28d29af46c3fae832c36db3bbebcc533dd1bb549172cd739c82df21 \
+    --hash=sha256:94a757d178c9be92ef5539b8840d48dc9cf1b2709c9d6b588232a055c524458b
     # via
     #   requests
     #   twine

.pre-commit-config.yaml

@@ -22,7 +22,7 @@ repos:
     -   id: end-of-file-fixer
     -   id: check-yaml
 -   repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 23.7.0
     hooks:
     - id: black
 -   repo: https://github.com/pycqa/flake8

docs/snippets.py

@@ -448,7 +448,6 @@ def test_bigtable_create_table():
 
 
 def test_bigtable_list_tables():
-
     # [START bigtable_api_list_tables]
     from google.cloud.bigtable import Client
 

docs/snippets_table.py

@@ -964,7 +964,6 @@ def test_bigtable_create_family_gc_nested():
 
 
 def test_bigtable_row_data_cells_cell_value_cell_values():
-
     value = b"value_in_col1"
     row = Config.TABLE.row(b"row_key_1")
     row.set_cell(

noxfile.py

@@ -17,46 +17,48 @@
 # Generated by synthtool. DO NOT EDIT!
 
 from __future__ import absolute_import
+
 import os
 import pathlib
 import re
 import shutil
+from typing import Dict, List
 import warnings
 
 import nox
 
 FLAKE8_VERSION = "flake8==6.1.0"
-BLACK_VERSION = "black==22.3.0"
-ISORT_VERSION = "isort==5.10.1"
+BLACK_VERSION = "black[jupyter]==23.7.0"
+ISORT_VERSION = "isort==5.11.0"
 LINT_PATHS = ["docs", "google", "tests", "noxfile.py", "setup.py"]
 
 DEFAULT_PYTHON_VERSION = "3.8"
 
-UNIT_TEST_PYTHON_VERSIONS = ["3.7", "3.8", "3.9", "3.10", "3.11"]
+UNIT_TEST_PYTHON_VERSIONS: List[str] = ["3.7", "3.8", "3.9", "3.10", "3.11"]
 UNIT_TEST_STANDARD_DEPENDENCIES = [
     "mock",
     "asyncmock",
     "pytest",
     "pytest-cov",
     "pytest-asyncio",
 ]
-UNIT_TEST_EXTERNAL_DEPENDENCIES = []
-UNIT_TEST_LOCAL_DEPENDENCIES = []
-UNIT_TEST_DEPENDENCIES = []
-UNIT_TEST_EXTRAS = []
-UNIT_TEST_EXTRAS_BY_PYTHON = {}
-
-SYSTEM_TEST_PYTHON_VERSIONS = ["3.8"]
-SYSTEM_TEST_STANDARD_DEPENDENCIES = [
+UNIT_TEST_EXTERNAL_DEPENDENCIES: List[str] = []
+UNIT_TEST_LOCAL_DEPENDENCIES: List[str] = []
+UNIT_TEST_DEPENDENCIES: List[str] = []
+UNIT_TEST_EXTRAS: List[str] = []
+UNIT_TEST_EXTRAS_BY_PYTHON: Dict[str, List[str]] = {}
+
+SYSTEM_TEST_PYTHON_VERSIONS: List[str] = ["3.8"]
+SYSTEM_TEST_STANDARD_DEPENDENCIES: List[str] = [
     "mock",
     "pytest",
     "google-cloud-testutils",
 ]
-SYSTEM_TEST_EXTERNAL_DEPENDENCIES = []
-SYSTEM_TEST_LOCAL_DEPENDENCIES = []
-SYSTEM_TEST_DEPENDENCIES = []
-SYSTEM_TEST_EXTRAS = []
-SYSTEM_TEST_EXTRAS_BY_PYTHON = {}
+SYSTEM_TEST_EXTERNAL_DEPENDENCIES: List[str] = []
+SYSTEM_TEST_LOCAL_DEPENDENCIES: List[str] = []
+SYSTEM_TEST_DEPENDENCIES: List[str] = []
+SYSTEM_TEST_EXTRAS: List[str] = []
+SYSTEM_TEST_EXTRAS_BY_PYTHON: Dict[str, List[str]] = {}
 
 CURRENT_DIRECTORY = pathlib.Path(__file__).parent.absolute()
 
@@ -71,6 +73,7 @@
     "lint_setup_py",
     "blacken",
     "docs",
+    "format",
 ]
 
 # Error if a python version is missing
@@ -201,7 +204,6 @@ def unit(session):
 
 
 def install_systemtest_dependencies(session, *constraints):
-
     # Use pre-release gRPC for system tests.
     # Exclude version 1.52.0rc1 which has a known issue.
     # See https://github.com/grpc/grpc/issues/32163

tests/system/test_instance_admin.py

@@ -28,7 +28,6 @@ def _create_app_profile_helper(
     allow_transactional_writes=None,
     ignore_warnings=None,
 ):
-
     app_profile = instance.app_profile(
         app_profile_id=app_profile_id,
         routing_policy_type=routing_policy_type,

tests/unit/test_batcher.py

@@ -59,7 +59,6 @@ def callback_fn(response):
 def test_mutation_batcher_mutate_row():
     table = _Table(TABLE_NAME)
     with MutationsBatcher(table=table) as mutation_batcher:
-
         rows = [
             DirectRow(row_key=b"row_key"),
             DirectRow(row_key=b"row_key_2"),
@@ -75,7 +74,6 @@ def test_mutation_batcher_mutate_row():
 def test_mutation_batcher_mutate():
     table = _Table(TABLE_NAME)
     with MutationsBatcher(table=table) as mutation_batcher:
-
         row = DirectRow(row_key=b"row_key")
         row.set_cell("cf1", b"c1", 1)
         row.set_cell("cf1", b"c2", 2)
@@ -98,7 +96,6 @@ def test_mutation_batcher_flush_w_no_rows():
 def test_mutation_batcher_mutate_w_max_flush_count():
     table = _Table(TABLE_NAME)
     with MutationsBatcher(table=table, flush_count=3) as mutation_batcher:
-
         row_1 = DirectRow(row_key=b"row_key_1")
         row_2 = DirectRow(row_key=b"row_key_2")
         row_3 = DirectRow(row_key=b"row_key_3")
@@ -114,7 +111,6 @@ def test_mutation_batcher_mutate_w_max_flush_count():
 def test_mutation_batcher_mutate_w_max_mutations():
     table = _Table(TABLE_NAME)
     with MutationsBatcher(table=table) as mutation_batcher:
-
         row = DirectRow(row_key=b"row_key")
         row.set_cell("cf1", b"c1", 1)
         row.set_cell("cf1", b"c2", 2)
@@ -130,7 +126,6 @@ def test_mutation_batcher_mutate_w_max_row_bytes():
     with MutationsBatcher(
         table=table, max_row_bytes=3 * 1024 * 1024
     ) as mutation_batcher:
-
         number_of_bytes = 1 * 1024 * 1024
         max_value = b"1" * number_of_bytes
 
@@ -168,7 +163,6 @@ def test_mutations_batcher_context_manager_flushed_when_closed():
     with MutationsBatcher(
         table=table, max_row_bytes=3 * 1024 * 1024
     ) as mutation_batcher:
-
         number_of_bytes = 1 * 1024 * 1024
         max_value = b"1" * number_of_bytes
 

tests/unit/test_cluster.py

@@ -752,7 +752,6 @@ def test_cluster_update_w_partial_autoscaling_config():
         },
     ]
     for config in cluster_config:
-
         cluster = _make_cluster(
             CLUSTER_ID,
             instance,
@@ -927,7 +926,6 @@ def test_cluster_disable_autoscaling():
 
 
 def test_create_cluster_with_both_manual_and_autoscaling():
-
     from google.cloud.bigtable.instance import Instance
     from google.cloud.bigtable.enums import StorageType
 
@@ -955,7 +953,6 @@ def test_create_cluster_with_both_manual_and_autoscaling():
 
 
 def test_create_cluster_with_partial_autoscaling_config():
-
     from google.cloud.bigtable.instance import Instance
     from google.cloud.bigtable.enums import StorageType
 
@@ -996,7 +993,6 @@ def test_create_cluster_with_partial_autoscaling_config():
 
 
 def test_create_cluster_with_no_scaling_config():
-
     from google.cloud.bigtable.instance import Instance
     from google.cloud.bigtable.enums import StorageType
 

tests/unit/test_column_family.py

@@ -595,7 +595,6 @@ def test__gc_rule_from_pb_unknown_field_name():
     from google.cloud.bigtable.column_family import _gc_rule_from_pb
 
     class MockProto(object):
-
         names = []
 
         _pb = {}

tests/unit/test_instance.py

@@ -67,7 +67,6 @@ def _make_instance(*args, **kwargs):
 
 
 def test_instance_constructor_defaults():
-
     client = object()
     instance = _make_instance(INSTANCE_ID, client)
     assert instance.instance_id == INSTANCE_ID

tests/unit/test_row_data.py

@@ -1118,7 +1118,6 @@ def test_RRRM_build_updated_request_row_ranges_valid():
 
 
 class _MockCancellableIterator(object):
-
     cancel_calls = 0
 
     def __init__(self, *values):
@@ -1199,5 +1198,4 @@ def _read_rows_retry_exception(exc):
 
 
 class _Client(object):
-
     data_stub = None

tests/unit/test_table.py

@@ -1689,7 +1689,6 @@ def _do_mutate_retryable_rows_helper(
 
     expected_entries = []
     for row, prior_status in zip(rows, worker.responses_statuses):
-
         if prior_status is None or prior_status.code in RETRYABLES:
             mutations = row._get_mutations().copy()  # row clears on success
             entry = data_messages_v2_pb2.MutateRowsRequest.Entry(