[Spotless] Applying Google Code Format for integ-tests #8 (opensearch…

…-project#1962)

* spotless apply for 81 integ-test files (#327)

add ignore failures for build.gradle.

Reverting ignore for checkstyle in integ-test

Addressed PR comments.

Addressed PR comments to expand jav doc.

fixed string formatting

Fixed string formatting.

Fixed string formatting in MatchPhrasePrefixIT

Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com>

* Apply suggestions from code review

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com>

* address PR comments

Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com>

---------

Signed-off-by: Mitchell Gale <Mitchell.Gale@improving.com>
Signed-off-by: Mitchell Gale <Mitchell.gale@improving.com>
Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

integ-test/src/test/java/org/opensearch/sql/jdbc/CursorIT.java

@@ -111,7 +111,8 @@ public void select_all_no_cursor() {
       var query = String.format("SELECT * FROM %s", table);
       ResultSet rs = stmt.executeQuery(query);
       int rows = 0;
-      while (rs.next()) rows++;
+      while(rs.next())
+        rows++;
 
       var restResponse = executeRestQuery(query, null);
       assertEquals(rows, restResponse.getInt("total"));

integ-test/src/test/java/org/opensearch/sql/legacy/DateFormatIT.java

@@ -40,9 +40,9 @@ protected void init() throws Exception {
   }
 
   /**
-   * All the following tests use UTC as their date_format timezone as this is the same timezone of
-   * the data being queried. This is to prevent discrepancies in the OpenSearch query and the actual
-   * field data that is being checked for the integration tests.
+   * All the following tests use UTC as their date_format timezone as this is the same timezone
+   * of the data being queried. This is to prevent discrepancies in the OpenSearch query and the
+   * actual field data that is being checked for the integration tests.
    *
    * <p>Large LIMIT values were given for some of these queries since the default result size of the
    * query is 200 and this ends up excluding some of the expected values causing the assertion to

integ-test/src/test/java/org/opensearch/sql/legacy/MetaDataQueriesIT.java

@@ -26,8 +26,6 @@
 import org.opensearch.sql.legacy.utils.StringUtils;
 
 /**
- *
- *
  * <pre>
  * The following are tests for SHOW/DESCRIBE query support under Pretty Format Response protocol using JDBC format.
  * <p>

integ-test/src/test/java/org/opensearch/sql/legacy/MethodQueryIT.java

@@ -27,8 +27,6 @@ protected void init() throws Exception {
   }
 
   /**
-   *
-   *
    * <pre>
    * query
    * "query" : {
@@ -37,7 +35,6 @@ protected void init() throws Exception {
    *   }
    * }
    * </pre>
-   *
    * @throws IOException
    */
   @Test
@@ -53,8 +50,6 @@ public void queryTest() throws IOException {
   }
 
   /**
-   *
-   *
    * <pre>
    * matchQuery
    * "query" : {
@@ -66,7 +61,6 @@ public void queryTest() throws IOException {
    *   }
    * }
    * </pre>
-   *
    * @throws IOException
    */
   @Test
@@ -83,8 +77,6 @@ public void matchQueryTest() throws IOException {
   }
 
   /**
-   *
-   *
    * <pre>
    * matchQuery
    * {
@@ -126,7 +118,6 @@ public void matchQueryTest() throws IOException {
    *   }
    * }
    * </pre>
-   *
    * @throws IOException
    */
   @Test
@@ -183,8 +174,6 @@ public void negativeRegexpQueryTest() throws IOException {
   }
 
   /**
-   *
-   *
    * <pre>
    * wildcardQuery
    * l*e means leae ltae ...
@@ -194,7 +183,6 @@ public void negativeRegexpQueryTest() throws IOException {
    *   }
    * }
    * </pre>
-   *
    * @throws IOException
    */
   @Test
@@ -210,16 +198,13 @@ public void wildcardQueryTest() throws IOException {
   }
 
   /**
-   *
-   *
    * <pre>
    * matchPhraseQuery
    * "address" : {
    *   "query" : "671 Bristol Street",
    *   "type" : "phrase"
    * }
    * </pre>
-   *
    * @throws IOException
    */
   @Test

integ-test/src/test/java/org/opensearch/sql/ppl/ConvertTZFunctionIT.java

@@ -1,26 +1,28 @@
-/*
- * Copyright OpenSearch Contributors
- * SPDX-License-Identifier: Apache-2.0
- */
+  /*
+   * Copyright OpenSearch Contributors
+   * SPDX-License-Identifier: Apache-2.0
+   */
 
 package org.opensearch.sql.ppl;
 
-import static org.opensearch.sql.legacy.TestsConstants.TEST_INDEX_DATE;
-import static org.opensearch.sql.util.MatcherUtils.rows;
-import static org.opensearch.sql.util.MatcherUtils.schema;
-import static org.opensearch.sql.util.MatcherUtils.verifySchema;
-import static org.opensearch.sql.util.MatcherUtils.verifySome;
+  import org.json.JSONObject;
+  import org.junit.Test;
 
-import java.io.IOException;
-import org.json.JSONObject;
-import org.junit.Test;
+  import java.io.IOException;
 
-public class ConvertTZFunctionIT extends PPLIntegTestCase {
+  import static org.opensearch.sql.legacy.TestsConstants.TEST_INDEX_DATE;
+  import static org.opensearch.sql.util.MatcherUtils.rows;
+  import static org.opensearch.sql.util.MatcherUtils.schema;
+  import static org.opensearch.sql.util.MatcherUtils.verifySchema;
+  import static org.opensearch.sql.util.MatcherUtils.verifySome;
 
-  @Override
-  public void init() throws IOException {
-    loadIndex(Index.DATE);
-  }
+  public class ConvertTZFunctionIT extends PPLIntegTestCase {
+
+
+    @Override
+    public void init() throws IOException {
+      loadIndex(Index.DATE);
+    }
 
   @Test
   public void inRangeZeroToPositive() throws IOException {