Prob fixes the flaky test in ChangePolicy

opensearch-project · Aug 2, 2021 · feca54b · feca54b
1 parent f31a5c6
commit feca54b
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/...g/opensearch/indexmanagement/indexstatemanagement/resthandler/RestChangePolicyActionIT.kt b/...g/opensearch/indexmanagement/indexstatemanagement/resthandler/RestChangePolicyActionIT.kt
@@ -470,11 +470,15 @@ class RestChangePolicyActionIT : IndexStateManagementRestTestCase() {
         // speed up to third execution where we transition to second state
         updateManagedIndexConfigStartTime(firstManagedIndexConfig)
 
+        logger.info("time before check")
         waitFor {
-            getExplainManagedIndexMetaData(firstIndex).let {
-                assertEquals(it.copy(stateMetaData = it.stateMetaData?.copy(name = secondState.name)), it)
-            }
+//            getExplainManagedIndexMetaData(firstIndex).let {
+//                assertEquals(it.copy(stateMetaData = it.stateMetaData?.copy(name = secondState.name)), it)
+//            }
+            assertEquals(secondState.name, getExplainManagedIndexMetaData(firstIndex).stateMetaData?.name)
+            logger.info("Explain firstIndex before change policy: ${getExplainManagedIndexMetaData(firstIndex)}")
         }
+        logger.info("time after check")
 
         // create second index
         val (secondIndex) = createIndex("second_index", policy.id)
@@ -501,6 +505,7 @@ class RestChangePolicyActionIT : IndexStateManagementRestTestCase() {
         val responseMap = response.asMap()
         logger.info("Change policy response: $responseMap")
         assertAffectedIndicesResponseIsEqual(expectedResponse, responseMap)
+        fail()
 
         waitFor {
             // The first managed index should not have a change policy added to it as it should of been filtered out from the states filter