Re enable config overrides test (#427)

opendistro-for-elasticsearch · Sep 17, 2020 · 54e61c5 · 54e61c5
1 parent 9a2624c
commit 54e61c5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/...a/com/amazon/opendistro/elasticsearch/performanceanalyzer/rca/framework/core/RcaConf.java b/...a/com/amazon/opendistro/elasticsearch/performanceanalyzer/rca/framework/core/RcaConf.java
@@ -243,7 +243,7 @@ public <T> T readRcaConfig(String rcaName, String key, T defaultValue, Predicate
     return setting;
   }
 
-  public static boolean updateAllRcaConfFiles(final Set<String> mutedRcas, final Set<String> mutedDeciders,
+  public boolean updateAllRcaConfFiles(final Set<String> mutedRcas, final Set<String> mutedDeciders,
       final Set<String> mutedActions) {
     boolean updateStatus = true;
     // update all rca.conf files
@@ -260,7 +260,7 @@ public static boolean updateAllRcaConfFiles(final Set<String> mutedRcas, final S
     return updateStatus;
   }
 
-  private static boolean updateRcaConf(String originalFilePath, final Set<String> mutedRcas,
+  private boolean updateRcaConf(String originalFilePath, final Set<String> mutedRcas,
       final Set<String> mutedDeciders, final Set<String> mutedActions) {
 
     String updatedPath = originalFilePath + ".updated";

diff --git a/...distro/elasticsearch/performanceanalyzer/config/overrides/ConfigOverridesApplierTest.java b/...distro/elasticsearch/performanceanalyzer/config/overrides/ConfigOverridesApplierTest.java
@@ -35,13 +35,11 @@
 import java.util.List;
 import java.util.Set;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 import org.mockito.ArgumentCaptor;
 import org.mockito.Captor;
 import org.mockito.Mock;
 
-@Ignore
 public class ConfigOverridesApplierTest {
 
   private static final String RCA1 = "rca1";