fix IT

Signed-off-by: xinyual <xinyual@amazon.com>

Author: xinyual

core/src/main/java/org/opensearch/sql/calcite/udf/textUDF/LocateFunction.java

@@ -14,8 +14,8 @@ public Object eval(Object... args) {
       return new IllegalArgumentException(
           "Invalid number of arguments, locate function expects 2 or 3 arguments");
     }
-    String stringText = (String) args[0];
-    String targetText = (String) args[1];
+    String stringText = (String) args[1];
+    String targetText = (String) args[0];
     if (stringText == null || targetText == null) {
       return null;
     }

core/src/main/java/org/opensearch/sql/calcite/utils/BuiltinFunctionUtils.java

@@ -243,6 +243,9 @@ static List<RexNode> translateArgument(
                     context.rexBuilder.makeLiteral(" ")));
         RTrimArgs.addAll(argList);
         return RTrimArgs;
+      case "STRCMP":
+        List<RexNode> StrcmpArgs = List.of(argList.get(1), argList.get(0));
+        return StrcmpArgs;
       case "ATAN":
         List<RexNode> AtanArgs = new ArrayList<>(argList);
         if (AtanArgs.size() == 1) {

integ-test/src/test/java/org/opensearch/sql/calcite/remote/nonfallback/NonFallbackCalciteTextFunctionIT.java

@@ -5,7 +5,6 @@
 
 package org.opensearch.sql.calcite.remote.nonfallback;
 
-import org.junit.Ignore;
 import org.opensearch.sql.calcite.remote.fallback.CalciteTextFunctionIT;
 
 public class NonFallbackCalciteTextFunctionIT extends CalciteTextFunctionIT {
@@ -14,28 +13,4 @@ public void init() throws Exception {
     super.init();
     disallowCalciteFallback();
   }
-
-  @Ignore("https://github.com/opensearch-project/sql/issues/3467")
-  @Override
-  public void testAscii() {}
-
-  @Ignore("https://github.com/opensearch-project/sql/issues/3467")
-  @Override
-  public void testLeft() {}
-
-  @Ignore("https://github.com/opensearch-project/sql/issues/3467")
-  @Override
-  public void testLocate() {}
-
-  @Ignore("https://github.com/opensearch-project/sql/issues/3467")
-  @Override
-  public void testReplace() {}
-
-  @Ignore("https://github.com/opensearch-project/sql/issues/3467")
-  @Override
-  public void testStrcmp() {}
-
-  @Ignore("https://github.com/opensearch-project/sql/issues/3467")
-  @Override
-  public void testSubstr() {}
 }

integ-test/src/test/java/org/opensearch/sql/calcite/standalone/CalcitePPLStringBuiltinFunctionIT.java

@@ -157,7 +157,7 @@ public void testLocate() {
     JSONObject actual =
         executeQuery(
             String.format(
-                "source=%s | where locate(name, 'Ja')=1 | fields name, age",
+                "source=%s | where locate('Ja', name)=1 | fields name, age",
                 TEST_INDEX_STATE_COUNTRY_WITH_NULL));
 
     verifySchema(actual, schema("name", "string"), schema("age", "integer"));