Fix IndexLifecycleRunnerTests.testRunAsyncActionReturningFalseEntersE…

…rror (#75125) (#75140)

Avoid ConcurrentModificationException by using a thread safe collection
in the test, as we write to the ILM history store in an async way.

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Andrei Dan <andrei.dan@elastic.co>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

x-pack/plugin/ilm/src/test/java/org/elasticsearch/xpack/ilm/IndexLifecycleRunnerTests.java

@@ -76,6 +76,7 @@
 import java.util.Objects;
 import java.util.SortedMap;
 import java.util.TreeMap;
+import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
@@ -1252,7 +1253,7 @@ public static MockPolicyStepsRegistry createMultiStepPolicyStepRegistry(String p
     private static class NoOpHistoryStore extends ILMHistoryStore {
         private static final Logger logger = LogManager.getLogger(NoOpHistoryStore.class);
 
-        private final List<ILMHistoryItem> items = new ArrayList<>();
+        private final List<ILMHistoryItem> items = new CopyOnWriteArrayList<>();
 
         NoOpHistoryStore(Client noopClient) {
             super(Settings.EMPTY, noopClient, null, null);