Skip to content

Commit fd62ac0

Browse files
committed
refactor: fix unit tests
Signed-off-by: ClSlaid <cailue@bupt.edu.cn>
1 parent 3ed7396 commit fd62ac0

File tree

2 files changed

+18
-1
lines changed

2 files changed

+18
-1
lines changed

src/query/ast/tests/it/testdata/statement.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ SHOW DATABASES
66
ShowDatabases(
77
ShowDatabasesStmt {
88
catalog: None,
9+
full: false,
910
limit: None,
1011
},
1112
)
@@ -19,6 +20,7 @@ SHOW DATABASES
1920
ShowDatabases(
2021
ShowDatabasesStmt {
2122
catalog: None,
23+
full: false,
2224
limit: None,
2325
},
2426
)

src/query/service/tests/it/servers/http/http_query_handlers.rs

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,9 +184,10 @@ async fn test_simple_sql() -> Result<()> {
184184
assert_eq!(status, StatusCode::OK, "{:?}", result);
185185
assert!(result.error.is_none(), "{:?}", result);
186186
assert!(result.schema.is_some(), "{:?}", result);
187+
// has only one column
187188
assert_eq!(
188189
result.schema.as_ref().unwrap().fields().len(),
189-
2,
190+
1,
190191
"{:?}",
191192
result
192193
);
@@ -202,6 +203,20 @@ async fn test_show_databases() -> Result<()> {
202203
assert_eq!(status, StatusCode::OK, "{:?}", result);
203204
assert!(result.error.is_none(), "{:?}", result);
204205
assert!(result.schema.is_some(), "{:?}", result);
206+
// has only one field: name
207+
assert_eq!(
208+
result.schema.as_ref().unwrap().fields().len(),
209+
1,
210+
"{:?}",
211+
result
212+
);
213+
214+
let sql = "show full databases";
215+
let (status, result) = post_sql(sql, 1).await?;
216+
assert_eq!(status, StatusCode::OK, "{:?}", result);
217+
assert!(result.error.is_none(), "{:?}", result);
218+
assert!(result.schema.is_some(), "{:?}", result);
219+
// has two fields: catalog, name
205220
assert_eq!(
206221
result.schema.as_ref().unwrap().fields().len(),
207222
2,

0 commit comments

Comments
 (0)