Fixed 2.16 integ test failures (#2871)

Signed-off-by: Vamsi Manohar <[email protected]>
opensearch-project · Jul 30, 2024 · 103c416 · 103c416
1 parent ba82e12
commit 103c416
Show file tree

Hide file tree

Showing 2 changed files with 13 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
@@ -38,6 +38,7 @@ public void testDataSourceCreationWithDefaultSettings() {
     assertDataSourceCount(1);
     assertSelectFromDataSourceReturnsSuccess();
     assertSelectFromDummyIndexInValidDataSourceDataSourceReturnsDoesNotExist();
+    deleteSelfDataSourceCreated();
   }
 
   @Test
@@ -52,6 +53,8 @@ public void testAfterPreviousEnable() {
     assertDataSourceCount(0);
     assertSelectFromDataSourceReturnsDoesNotExist();
     assertAsyncQueryApiDisabled();
+    setDataSourcesEnabled("transient", true);
+    deleteSelfDataSourceCreated();
   }
 
   @SneakyThrows
@@ -142,4 +145,11 @@ private Response performRequest(Request request) {
       return e.getResponse();
     }
   }
+
+  @SneakyThrows
+  private void deleteSelfDataSourceCreated() {
+    Request deleteRequest = getDeleteDataSourceRequest("self");
+    Response deleteResponse = client().performRequest(deleteRequest);
+    Assert.assertEquals(204, deleteResponse.getStatusLine().getStatusCode());
+  }
 }
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) {