Skip to content

Commit 95aca1d

Browse files
committed
Revert "SA20: Address deprecation warnings in SqlAlchemyDictTypeTest"
This reverts commit b20faba.
1 parent b20faba commit 95aca1d

File tree

1 file changed

+14
-13
lines changed

1 file changed

+14
-13
lines changed

src/crate/client/sqlalchemy/tests/dict_test.py

+14-13
Original file line numberDiff line numberDiff line change
@@ -37,22 +37,22 @@
3737

3838

3939
class SqlAlchemyDictTypeTest(TestCase):
40+
4041
def setUp(self):
4142
self.engine = sa.create_engine('crate://')
42-
metadata = sa.MetaData()
43-
self.mytable = sa.Table('mytable',
44-
metadata,
43+
# FIXME: Deprecated with SA20.
44+
metadata = sa.MetaData(bind=self.engine)
45+
self.mytable = sa.Table('mytable', metadata,
4546
sa.Column('name', sa.String),
46-
sa.Column('data', Craty),
47-
autoload_with=self.engine)
47+
sa.Column('data', Craty))
4848

4949
def assertSQL(self, expected_str, actual_expr):
5050
self.assertEqual(expected_str, str(actual_expr).replace('\n', ''))
5151

5252
def test_select_with_dict_column(self):
5353
mytable = self.mytable
5454
self.assertSQL(
55-
"SELECT mytable.data[:data_1] AS anon_1 FROM mytable",
55+
"SELECT mytable.data['x'] AS anon_1 FROM mytable",
5656
select(mytable.c.data['x'])
5757
)
5858

@@ -61,7 +61,7 @@ def test_select_with_dict_column_where_clause(self):
6161
s = select(mytable.c.data).\
6262
where(mytable.c.data['x'] == 1)
6363
self.assertSQL(
64-
"SELECT mytable.data FROM mytable WHERE mytable.data[:data_1] = :param_1",
64+
"SELECT mytable.data FROM mytable WHERE mytable.data['x'] = ?",
6565
s
6666
)
6767

@@ -71,7 +71,7 @@ def test_select_with_dict_column_nested_where(self):
7171
s = s.where(mytable.c.data['x']['y'] == 1)
7272
self.assertSQL(
7373
"SELECT mytable.name FROM mytable " +
74-
"WHERE mytable.data[:data_1][:param_1] = :param_2",
74+
"WHERE mytable.data['x']['y'] = ?",
7575
s
7676
)
7777

@@ -80,7 +80,7 @@ def test_select_with_dict_column_where_clause_gt(self):
8080
s = select(mytable.c.data).\
8181
where(mytable.c.data['x'] > 1)
8282
self.assertSQL(
83-
"SELECT mytable.data FROM mytable WHERE mytable.data[:data_1] > :param_1",
83+
"SELECT mytable.data FROM mytable WHERE mytable.data['x'] > ?",
8484
s
8585
)
8686

@@ -90,18 +90,19 @@ def test_select_with_dict_column_where_clause_other_col(self):
9090
s = s.where(mytable.c.data['x'] == mytable.c.name)
9191
self.assertSQL(
9292
"SELECT mytable.name FROM mytable " +
93-
"WHERE mytable.data[:data_1] = mytable.name",
93+
"WHERE mytable.data['x'] = mytable.name",
9494
s
9595
)
9696

9797
def test_update_with_dict_column(self):
9898
mytable = self.mytable
9999
stmt = mytable.update().\
100100
where(mytable.c.name == 'Arthur Dent').\
101-
values({mytable.c.data['x']: "Trillian"})
102-
101+
values({
102+
"data['x']": "Trillian"
103+
})
103104
self.assertSQL(
104-
"UPDATE mytable SET data[:data_1]=:param_1 WHERE mytable.name = :name_1",
105+
"UPDATE mytable SET data['x'] = ? WHERE mytable.name = ?",
105106
stmt
106107
)
107108

0 commit comments

Comments
 (0)