Fixed 2.16 integ test failures

Signed-off-by: Vamsi Manohar <[email protected]>
vmmusings · Jul 30, 2024 · c208d76 · c208d76
1 parent 6b8ee3d
commit c208d76
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/integ-test/src/test/java/org/opensearch/sql/datasource/DataSourceEnabledIT.java b/integ-test/src/test/java/org/opensearch/sql/datasource/DataSourceEnabledIT.java
@@ -7,8 +7,10 @@
 
 import static org.opensearch.sql.legacy.TestUtils.getResponseBody;
 
+import java.io.IOException;
 import lombok.SneakyThrows;
 import org.json.JSONObject;
+import org.junit.AfterClass;
 import org.junit.Assert;
 import org.junit.Test;
 import org.opensearch.client.Request;
@@ -23,6 +25,13 @@ protected boolean preserveClusterUponCompletion() {
     return false;
   }
 
+  @AfterClass
+  protected static void deleteSelfDataSourceCreated() throws IOException {
+    Request deleteRequest = getDeleteDataSourceRequest("self");
+    Response deleteResponse = client().performRequest(deleteRequest);
+    Assert.assertEquals(204, deleteResponse.getStatusLine().getStatusCode());
+  }
+
   @Test
   public void testAsyncQueryAPIFailureIfSettingIsDisabled() {
     setDataSourcesEnabled("transient", false);

diff --git a/integ-test/src/test/java/org/opensearch/sql/legacy/OpenSearchSQLRestTestCase.java b/integ-test/src/test/java/org/opensearch/sql/legacy/OpenSearchSQLRestTestCase.java
@@ -195,7 +195,9 @@ protected static void wipeAllOpenSearchIndices(RestClient client) throws IOExcep
         try {
           // System index, mostly named .opensearch-xxx or .opendistro-xxx, are not allowed to
           // delete
-          if (!indexName.startsWith(".opensearch") && !indexName.startsWith(".opendistro")) {
+          if (!indexName.startsWith(".opensearch")
+              && !indexName.startsWith(".opendistro")
+              && !indexName.startsWith(".ql")) {
             client.performRequest(new Request("DELETE", "/" + indexName));
           }
         } catch (Exception e) {