Skip to content

Commit 90d912e

Browse files
committed
Fix test_management_mode_access
1 parent e99dfcd commit 90d912e

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

src/query/service/tests/it/interpreters/access/management_mode_access.rs

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -158,14 +158,18 @@ async fn test_management_mode_access() -> Result<()> {
158158
for group in groups {
159159
for test in group.tests {
160160
let (plan, _, _) = planner.plan_sql(test.query).await?;
161-
let interpreter = InterpreterFactoryV2::get(ctx.clone(), &plan)?;
162-
let res = interpreter.execute(ctx.clone()).await;
163-
assert_eq!(
164-
test.is_err,
165-
res.is_err(),
166-
"in test case:{:?}",
167-
(group.name, test.name)
168-
);
161+
if test.is_err {
162+
let res = InterpreterFactoryV2::get(ctx.clone(), &plan);
163+
assert_eq!(
164+
test.is_err,
165+
res.is_err(),
166+
"in test case:{:?}",
167+
(group.name, test.name)
168+
);
169+
} else {
170+
let interpreter = InterpreterFactoryV2::get(ctx.clone(), &plan)?;
171+
interpreter.execute(ctx.clone()).await?;
172+
}
169173
}
170174
}
171175

0 commit comments

Comments
 (0)