style(bigtable): use f-string and 'cannot' in MutationsBatcher.mutate

Author: mutianf

packages/google-cloud-bigtable/google/cloud/bigtable/batcher.py

@@ -189,14 +189,13 @@ def mutate(self, row):
         """
         if not isinstance(row, DirectRow):
             raise TypeError(
-                "Bulk processing can not be applied for conditional or append mutations."
+                "Bulk processing cannot be applied for conditional or append mutations."
             )
         if row.table is not None and row.table.name != self.table.name:
             from google.cloud.bigtable.table import TableMismatchError
 
             raise TableMismatchError(
-                "Row %s is a part of %s table. Current table: %s"
-                % (row.row_key, row.table.name, self.table.name)
+                f"Row {row.row_key} is a part of {row.table.name} table. Current table: {self.table.name}"
             )
         self._batcher.append(RowMutationEntry(row.row_key, row._get_mutations()))
 

packages/google-cloud-bigtable/tests/unit/v2_client/test_batcher.py

@@ -397,19 +397,19 @@ def test_mutations_batcher_mutate_wrong_row_type(_setup_batcher):
     with MutationsBatcher(table=table, flush_count=1) as mutation_batcher:
         with pytest.raises(
             TypeError,
-            match="Bulk processing can not be applied for conditional or append mutations.",
+            match="Bulk processing cannot be applied for conditional or append mutations.",
         ):
             mutation_batcher.mutate(cond_row)
 
         with pytest.raises(
             TypeError,
-            match="Bulk processing can not be applied for conditional or append mutations.",
+            match="Bulk processing cannot be applied for conditional or append mutations.",
         ):
             mutation_batcher.mutate(append_row)
 
         with pytest.raises(
             TypeError,
-            match="Bulk processing can not be applied for conditional or append mutations.",
+            match="Bulk processing cannot be applied for conditional or append mutations.",
         ):
             mutation_batcher.mutate_rows([cond_row])
 
@@ -424,9 +424,9 @@ def test_mutations_batcher_mutate_table_mismatch(_setup_batcher):
     row = DirectRow(row_key=b"row_key", table=other_table)
     row.set_cell("cf1", b"c1", b"val")
 
-    expected_msg = r"Row b'row_key' is a part of %s table\. Current table: %s" % (
-        other_table.name,
-        table.name,
+    expected_msg = (
+        rf"Row b'row_key' is a part of {other_table.name} table\. "
+        rf"Current table: {table.name}"
     )
 
     with MutationsBatcher(table=table, flush_count=1) as mutation_batcher: