Skip to content

Commit 0e74165

Browse files
committed
tests adapted to full names
1 parent 015db36 commit 0e74165

File tree

1 file changed

+26
-22
lines changed

1 file changed

+26
-22
lines changed

tests/test_sdk.py

Lines changed: 26 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -101,15 +101,15 @@ def check_model(self, model, database, mock_post):
101101
pred_df = model.predict(query, params={ 'x': '1' })
102102

103103
check_sql_call(mock_post,
104-
f'SELECT m.* FROM (SELECT a FROM t1) as t JOIN {model.project.name}.{model_name} AS m USING x="1"')
104+
f'SELECT m.* FROM (SELECT * FROM {query.database} (select a from t1)) AS t JOIN {model.project.name}.{model_name} AS m USING x="1"')
105105
assert (pred_df == pd.DataFrame(data_out)).all().bool()
106106

107107
# time series prediction
108108
query = database.query('select * from t1 where type="house" and saledate>latest')
109109
model.predict(query)
110110

111111
check_sql_call(mock_post,
112-
f"SELECT m.* FROM (SELECT * FROM t1 WHERE type = 'house' AND saledate > LATEST) as t JOIN {model.project.name}.{model_name} AS m")
112+
f'SELECT m.* FROM (SELECT * FROM {query.database} (select * from t1 where type="house" and saledate>latest)) as t JOIN {model.project.name}.{model_name} AS m')
113113
assert (pred_df == pd.DataFrame(data_out)).all().bool()
114114

115115
# ----------- model managing --------------
@@ -176,12 +176,12 @@ def check_table(self, table, mock_post):
176176
table2 = table.filter(a=3, b='2').limit(3)
177177
table2.fetch()
178178
str(table2)
179-
check_sql_call(mock_post, f'SELECT * FROM {table2.name} WHERE a = 3 AND b = \'2\' LIMIT 3')
179+
check_sql_call(mock_post, f'SELECT * FROM {table2.db.name}.{table2.name} WHERE a = 3 AND b = \'2\' LIMIT 3')
180180

181181
# last
182182
table2 = table.filter(a=3).track('type')
183183
table2.fetch()
184-
check_sql_call(mock_post, f'SELECT * FROM {table2.name} WHERE a = 3 AND type > last')
184+
check_sql_call(mock_post, f'SELECT * FROM {table2.db.name}.{table2.name} WHERE a = 3 AND type > last')
185185

186186

187187
class Test(BaseFlow):
@@ -476,6 +476,7 @@ def check_database(self, database, mock_post):
476476
sql = 'select * from tbl1'
477477
query = database.query(sql)
478478
assert query.sql == sql
479+
table0 = database.tables.tbl0
479480

480481
result = pd.DataFrame([{'s': '1'}, {'s': 'a'}])
481482
response_mock(mock_post, result)
@@ -498,8 +499,8 @@ def check_database(self, database, mock_post):
498499
self.check_table(table)
499500

500501
# create from query
501-
table2 = database.create_table('t2', query)
502-
check_sql_call(mock_post, f'create table {database.name}.t2 (select * from tbl1)')
502+
table2 = database.create_table('t2', table0)
503+
check_sql_call(mock_post, f'create table {database.name}.t2 (select * from {database.name}.tbl0)')
503504

504505
assert table2.name == 't2'
505506
self.check_table(table2)
@@ -508,14 +509,14 @@ def check_database(self, database, mock_post):
508509
table1 = database.get_table('t1')
509510
table1 = table1.filter(b=2)
510511
table3 = database.create_table('t3', table1)
511-
check_sql_call(mock_post, f'create table {database.name}.t3 (SELECT * FROM t1 WHERE b = 2)')
512+
check_sql_call(mock_post, f'create table {database.name}.t3 (SELECT * FROM {table1.db.name}.t1 WHERE b = 2)')
512513

513514
assert table3.name == 't3'
514515
self.check_table(table3)
515516

516517
# drop table
517518
database.drop_table('t3')
518-
check_sql_call(mock_post, f'drop table t3')
519+
check_sql_call(mock_post, f'drop table {database.name}.t3')
519520

520521

521522
@patch('requests.Session.post')
@@ -942,6 +943,8 @@ def check_database(self, database, mock_post):
942943
query = database.query(sql)
943944
assert query.sql == sql
944945

946+
table0 = database.tables.tbl0
947+
945948
result = pd.DataFrame([{'s': '1'}, {'s': 'a'}])
946949
response_mock(mock_post, result)
947950

@@ -964,12 +967,12 @@ def check_database(self, database, mock_post):
964967
self.check_table(table)
965968

966969
# create from query
967-
table2 = database.tables.create('t2', query)
968-
check_sql_call(mock_post, f'create table {database.name}.t2 (select * from tbl1)')
970+
table2 = database.tables.create('t2', table0)
971+
check_sql_call(mock_post, f'create table {database.name}.t2 (select * from {database.name}.tbl0)')
969972

970973
# create with replace
971-
database.tables.create('t2', query, replace=True)
972-
check_sql_call(mock_post, f'create or replace table {database.name}.t2 (select * from tbl1)')
974+
database.tables.create('t2', table0, replace=True)
975+
check_sql_call(mock_post, f'create or replace table {database.name}.t2 (select * from {database.name}.tbl0)')
973976

974977

975978
assert table2.name == 't2'
@@ -978,11 +981,11 @@ def check_database(self, database, mock_post):
978981
# -- insert into table --
979982
# from dataframe
980983
table2.insert(pd.DataFrame([{'s': '1', 'x': 1}, {'s': 'a', 'x': 2}]))
981-
check_sql_call(mock_post, "INSERT INTO t2(s, x) VALUES ('1', 1), ('a', 2)")
984+
check_sql_call(mock_post, f"INSERT INTO {table2.db.name}.t2(s, x) VALUES ('1', 1), ('a', 2)")
982985

983986
# from query
984987
table2.insert(query)
985-
check_sql_call(mock_post, f"INSERT INTO {database.name}.t2 (select * from tbl1)")
988+
check_sql_call(mock_post, f"INSERT INTO {database.name}.t2 (select * from {query.database}(select * from tbl1))")
986989

987990
# -- delete in table --
988991
table2.delete(a=1, b='2')
@@ -991,24 +994,24 @@ def check_database(self, database, mock_post):
991994
# -- update table --
992995
# from query
993996
table2.update(query, on=['a', 'b'])
994-
check_sql_call(mock_post, f"UPDATE {database.name}.t2 ON a, b FROM (select * from tbl1)")
997+
check_sql_call(mock_post, f"UPDATE {database.name}.t2 ON a, b FROM (select * from {query.database}(select * from tbl1))")
995998

996999
# from dict
9971000
table2.update({'a': '1', 'b': 1}, filters={'x': 3})
998-
check_sql_call(mock_post, f"UPDATE t2 SET a='1', b=1 WHERE x=3")
1001+
check_sql_call(mock_post, f"UPDATE {table2.db.name}.t2 SET a='1', b=1 WHERE x=3")
9991002

10001003
# create from table
10011004
table1 = database.tables.t1
10021005
table1 = table1.filter(b=2)
10031006
table3 = database.tables.create('t3', table1)
1004-
check_sql_call(mock_post, f'create table {database.name}.t3 (SELECT * FROM t1 WHERE b = 2)')
1007+
check_sql_call(mock_post, f'create table {database.name}.t3 (SELECT * FROM {table1.db.name}.t1 WHERE b = 2)')
10051008

10061009
assert table3.name == 't3'
10071010
self.check_table(table3)
10081011

10091012
# drop table
10101013
database.tables.drop('t3')
1011-
check_sql_call(mock_post, f'drop table t3')
1014+
check_sql_call(mock_post, f'drop table {database.name}.t3')
10121015

10131016
@patch('requests.Session.post')
10141017
def check_project_jobs(self, project, mock_post):
@@ -1124,7 +1127,7 @@ def check_project_kb(self, project, model, database, mock_post):
11241127
check_sql_call(
11251128
mock_post,
11261129
f''' insert into {project.name}.{kb.name} (
1127-
select * from tbl2 where a=1
1130+
select * from {database.name}.tbl2 where a=1
11281131
)'''
11291132
)
11301133

@@ -1148,7 +1151,7 @@ def check_project_kb(self, project, model, database, mock_post):
11481151
check_sql_call(
11491152
mock_post,
11501153
f'''
1151-
CREATE KNOWLEDGE_BASE kb2
1154+
CREATE KNOWLEDGE_BASE {project.name}.kb2
11521155
USING model={model_name},
11531156
metadata_columns=['date', 'author'],
11541157
k='v'
@@ -1168,7 +1171,7 @@ def check_project_kb(self, project, model, database, mock_post):
11681171
check_sql_call(
11691172
mock_post,
11701173
f'''
1171-
CREATE KNOWLEDGE_BASE kb2
1174+
CREATE KNOWLEDGE_BASE {project.name}.kb2
11721175
USING storage={table_name},
11731176
content_columns=['review'],
11741177
id_column='num'
@@ -1181,9 +1184,10 @@ def check_project_kb(self, project, model, database, mock_post):
11811184

11821185
check_sql_call(
11831186
mock_post,
1184-
"DROP KNOWLEDGE_BASE kb2"
1187+
f"DROP KNOWLEDGE_BASE {project.name}.kb2"
11851188
)
11861189

1190+
11871191
class TestAgents():
11881192
@patch('requests.Session.get')
11891193
def test_list(self, mock_get):

0 commit comments

Comments
 (0)