Skip to content

Commit 2f505e4

Browse files
committed
fix tests
Signed-off-by: Lantao Jin <ltjin@amazon.com>
1 parent 3add0ef commit 2f505e4

4 files changed

Lines changed: 14 additions & 5 deletions

File tree

core/src/test/java/org/opensearch/sql/calcite/CalciteRelNodeVisitorSearchSimpleTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,11 @@
1111

1212
import org.junit.jupiter.api.BeforeEach;
1313
import org.junit.jupiter.api.Test;
14+
import org.mockito.Mock;
1415
import org.opensearch.sql.ast.dsl.AstDSL;
1516
import org.opensearch.sql.ast.tree.Relation;
1617
import org.opensearch.sql.ast.tree.Search;
18+
import org.opensearch.sql.datasource.DataSourceService;
1719

1820
/**
1921
* Simple tests for CalciteRelNodeVisitor.visitSearch method. Tests basic functionality without
@@ -22,10 +24,11 @@
2224
public class CalciteRelNodeVisitorSearchSimpleTest {
2325

2426
private CalciteRelNodeVisitor visitor;
27+
@Mock DataSourceService dataSourceService;
2528

2629
@BeforeEach
2730
public void setUp() {
28-
visitor = new CalciteRelNodeVisitor();
31+
visitor = new CalciteRelNodeVisitor(dataSourceService);
2932
}
3033

3134
@Test

core/src/test/java/org/opensearch/sql/calcite/CalciteRexNodeVisitorTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
import org.opensearch.sql.ast.expression.LambdaFunction;
3030
import org.opensearch.sql.ast.expression.QualifiedName;
3131
import org.opensearch.sql.calcite.utils.CalciteToolsHelper;
32+
import org.opensearch.sql.datasource.DataSourceService;
3233
import org.opensearch.sql.executor.QueryType;
3334

3435
@ExtendWith(MockitoExtension.class)
@@ -44,6 +45,7 @@ public class CalciteRexNodeVisitorTest {
4445
@Mock RelDataType accType;
4546
@Mock QualifiedName functionArg1;
4647
@Mock QualifiedName functionArg2;
48+
@Mock DataSourceService dataSourceService;
4749

4850
static CalciteRexNodeVisitor visitor;
4951
static CalciteRelNodeVisitor relNodeVisitor;
@@ -57,7 +59,7 @@ public class CalciteRexNodeVisitorTest {
5759

5860
@BeforeEach
5961
public void setUpContext() {
60-
relNodeVisitor = new CalciteRelNodeVisitor();
62+
relNodeVisitor = new CalciteRelNodeVisitor(dataSourceService);
6163
visitor = new CalciteRexNodeVisitor(relNodeVisitor);
6264
when(relBuilder.getRexBuilder()).thenReturn(rexBuilder);
6365
when(rexBuilder.getTypeFactory()).thenReturn(TYPE_FACTORY);

0 commit comments

Comments
 (0)