cleaned up code from review, removed unnecessary code and files

Author: kiraksi

sqlalchemy_bigquery/test.py

@@ -85,8 +85,6 @@ def literal(value, type_=None):
                 if type_ is not None:
                     assert type_ is self.datatype
 
-                import sqlalchemy.sql.sqltypes
-
                 return sqlalchemy.sql.elements.literal(value, self.datatype)
 
             with mock.patch("sqlalchemy.testing.suite.test_types.literal", literal):
@@ -127,7 +125,6 @@ def test_update(self):
             # Had to pass in data as a dict object in 2.0
             r = connection.execute(t.update().where(t.c.id == 2), dict(data="d2_new"))
             assert not r.is_insert
-            # assert not r.returns_rows
 
             eq_(
                 connection.execute(t.select().order_by(t.c.id)).fetchall(),
@@ -139,7 +136,6 @@ def test_delete(self):
             connection = config.db.connect()
             r = connection.execute(t.delete().where(t.c.id == 2))
             assert not r.is_insert
-            # assert not r.returns_rows
             eq_(
                 connection.execute(t.select().order_by(t.c.id)).fetchall(),
                 [(1, "d1"), (3, "d3")],
@@ -526,7 +522,6 @@ def test_update(self):
             t = self.tables.plain_pk
             r = config.db.execute(t.update().where(t.c.id == 2), data="d2_new")
             assert not r.is_insert
-            # assert not r.returns_rows
 
             eq_(
                 config.db.execute(t.select().order_by(t.c.id)).fetchall(),
@@ -537,7 +532,6 @@ def test_delete(self):
             t = self.tables.plain_pk
             r = config.db.execute(t.delete().where(t.c.id == 2))
             assert not r.is_insert
-            # assert not r.returns_rows
             eq_(
                 config.db.execute(t.select().order_by(t.c.id)).fetchall(),
                 [(1, "d1"), (3, "d3")],
@@ -671,7 +665,6 @@ def test_update(self):
             t = self.tables.plain_pk
             r = config.db.execute(t.update().where(t.c.id == 2), data="d2_new")
             assert not r.is_insert
-            # assert not r.returns_rows
 
             eq_(
                 config.db.execute(t.select().order_by(t.c.id)).fetchall(),
@@ -682,7 +675,6 @@ def test_delete(self):
             t = self.tables.plain_pk
             r = config.db.execute(t.delete().where(t.c.id == 2))
             assert not r.is_insert
-            # assert not r.returns_rows
             eq_(
                 config.db.execute(t.select().order_by(t.c.id)).fetchall(),
                 [(1, "d1"), (3, "d3")],

tests/sqlalchemy_dialect_compliance/test_dialect_compliance.py

@@ -286,11 +286,6 @@ def test_reflect_select(table, engine_using_test_dataset, table_using_test_datas
         assert isinstance(table.c["nested_record.record.name"].type, types.String)
         assert isinstance(table.c.array.type, types.ARRAY)
 
-        # Force unique column labels using `use_labels` below to deal
-        # with BQ sometimes complaining about duplicate column names
-        # when a destination table is specified, even though no
-        # destination table is specified. When this test was written,
-        # `use_labels` was forced by the dialect.
         with engine_using_test_dataset.connect() as conn:
             rows = conn.execute(
                 table.select().set_label_style(