Updates for pycodestyle. (#2973)

Author: daspecster

bigquery/unit_tests/test_dataset.py

@@ -838,7 +838,7 @@ def api_request(self, **kw):
 
         try:
             response, self._responses = self._responses[0], self._responses[1:]
-        except:
+        except IndexError:
             raise NotFound('miss')
         else:
             return response

bigquery/unit_tests/test_job.py

@@ -1904,7 +1904,7 @@ def api_request(self, **kw):
 
         try:
             response, self._responses = self._responses[0], self._responses[1:]
-        except:
+        except IndexError:
             raise NotFound('miss')
         else:
             return response

bigquery/unit_tests/test_table.py

@@ -1989,7 +1989,7 @@ def api_request(self, **kw):
 
         try:
             response, self._responses = self._responses[0], self._responses[1:]
-        except:
+        except IndexError:
             raise NotFound('miss')
         else:
             return response

bigtable/unit_tests/test_instance.py

@@ -254,8 +254,8 @@ def test_create(self):
             metadata=Any(
                 type_url=type_url,
                 value=metadata.SerializeToString(),
-                )
             )
+        )
 
         # Patch the stub used by the API method.
         client._instance_stub = stub = _FakeStub(response_pb)

core/unit_tests/test_exceptions.py

@@ -40,7 +40,7 @@ def test_ctor_explicit(self):
             'locationType': 'testing',
             'message': 'Testing',
             'reason': 'test',
-            }
+        }
         e = self._make_one('Testing', [ERROR])
         e.code = 600
         self.assertEqual(str(e), '600 Testing')
@@ -115,7 +115,7 @@ def test_miss_w_content_as_dict(self):
             'locationType': 'testing',
             'message': 'Testing',
             'reason': 'test',
-            }
+        }
         response = _Response(600)
         content = {"error": {"message": "Unknown Error", "errors": [ERROR]}}
         exception = self._call_fut(response, content)

datastore/google/cloud/datastore/transaction.py

@@ -186,7 +186,7 @@ def begin(self):
         try:
             self._id = self._client._connection.begin_transaction(
                 self.project)
-        except:
+        except:  # noqa: E722 do not use bare except, specify exception instead
             self._status = self._ABORTED
             raise
 

datastore/unit_tests/test__http.py

@@ -1051,11 +1051,11 @@ def test_allocate_ids_non_empty(self):
         before_key_pbs = [
             self._make_key_pb(PROJECT, id_=None),
             self._make_key_pb(PROJECT, id_=None),
-            ]
+        ]
         after_key_pbs = [
             self._make_key_pb(PROJECT),
             self._make_key_pb(PROJECT, id_=2345),
-            ]
+        ]
         rsp_pb = datastore_pb2.AllocateIdsResponse()
         rsp_pb.keys.add().CopyFrom(after_key_pbs[0])
         rsp_pb.keys.add().CopyFrom(after_key_pbs[1])

datastore/unit_tests/test_batch.py

@@ -121,7 +121,7 @@ def test_put_entity_w_completed_key(self):
             'baz': 'qux',
             'spam': [1, 2, 3],
             'frotz': [],  # will be ignored
-            }
+        }
         connection = _Connection()
         client = _Client(_PROJECT, connection)
         batch = self._make_one(client)

datastore/unit_tests/test_client.py

@@ -853,7 +853,7 @@ def test_query_explicit(self):
                 projection=PROJECTION,
                 order=ORDER,
                 distinct_on=DISTINCT_ON,
-                )
+            )
 
         self.assertIsInstance(query, _Dummy)
         self.assertEqual(query.args, (client,))

datastore/unit_tests/test_query.py

@@ -68,7 +68,7 @@ def test_ctor_explicit(self):
             projection=PROJECTION,
             order=ORDER,
             distinct_on=DISTINCT_ON,
-            )
+        )
         self.assertIs(query._client, client)
         self.assertEqual(query.project, _PROJECT)
         self.assertEqual(query.kind, _KIND)