Allow system index warning in OpenSearchRestTestCase.refreshAllIndices (

#14635) * Allow system index warning Signed-off-by: Craig Perkins <[email protected]> * Add to CHANGELOG Signed-off-by: Craig Perkins <[email protected]> * Address code review comments Signed-off-by: Craig Perkins <[email protected]> --------- Signed-off-by: Craig Perkins <[email protected]>
opensearch-project · Jul 5, 2024 · f14b5c8 · f14b5c8
1 parent 74230b7
commit f14b5c8
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -36,6 +36,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 - Make the class CommunityIdProcessor final ([#14448](https:/opensearch-project/OpenSearch/pull/14448))
 - Allow @InternalApi annotation on classes not meant to be constructed outside of the OpenSearch core ([#14575](https:/opensearch-project/OpenSearch/pull/14575))
 - Add @InternalApi annotation to japicmp exclusions ([#14597](https:/opensearch-project/OpenSearch/pull/14597))
+- Allow system index warning in OpenSearchRestTestCase.refreshAllIndices ([#14635](https:/opensearch-project/OpenSearch/pull/14635))
 
 ### Deprecated
 

diff --git a/test/framework/src/main/java/org/opensearch/test/rest/OpenSearchRestTestCase.java b/test/framework/src/main/java/org/opensearch/test/rest/OpenSearchRestTestCase.java
@@ -708,11 +708,15 @@ protected void refreshAllIndices() throws IOException {
  requestOptions.setWarningsHandler(warnings -> {
  if (warnings.isEmpty()) {
  return false;
- } else if (warnings.size() > 1) {
- return true;
- } else {
- return warnings.get(0).startsWith("this request accesses system indices:") == false;
  }
+ boolean allSystemIndexWarnings = true;
+ for (String warning : warnings) {
+ if (!warning.startsWith("this request accesses system indices:")) {
+ allSystemIndexWarnings = false;
+ break;
+ }
+ }
+ return !allSystemIndexWarnings;
  });
  refreshRequest.setOptions(requestOptions);
  client().performRequest(refreshRequest);