Review fixes

getindata · Oct 17, 2024 · c277429 · c277429
1 parent 2d1ffe9
commit c277429
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/...onnectors/http/internal/table/lookup/querycreators/ElasticSearchLiteQueryCreatorTest.java b/...onnectors/http/internal/table/lookup/querycreators/ElasticSearchLiteQueryCreatorTest.java
@@ -20,7 +20,7 @@ public class ElasticSearchLiteQueryCreatorTest {
 
 
     @Test
-    public void testWithEmptyLookup() {
+    public void testWithEmptyLookupResult() {
 
         // GIVEN
         LookupRow lookupRow = new LookupRow();

diff --git a/...data/connectors/http/internal/table/lookup/querycreators/GenericJsonQueryCreatorTest.java b/...data/connectors/http/internal/table/lookup/querycreators/GenericJsonQueryCreatorTest.java
@@ -52,7 +52,6 @@ public void shouldSerializeToJson() {
         row.setField(0, 11);
         row.setField(1, StringData.fromString("myUuid"));
 
-        this.jsonQueryCreator.createLookupQuery(row);
         LookupQueryInfo lookupQuery = this.jsonQueryCreator.createLookupQuery(row);
         assertThat(lookupQuery.getBodyBasedUrlQueryParameters().isEmpty());
         assertThat(lookupQuery.getLookupQuery()).isEqualTo("{\"id\":11,\"uuid\":\"myUuid\"}");
@@ -64,8 +63,6 @@ public void shouldSerializeToJsonTwice() {
         row.setField(0, 11);
         row.setField(1, StringData.fromString("myUuid"));
 
-        this.jsonQueryCreator.createLookupQuery(row);
-
         // Call createLookupQuery two times
         // to check that serialization schema is not opened Two times.
         this.jsonQueryCreator.createLookupQuery(row);