This is an automated email from the ASF dual-hosted git repository.

fjtiradosarti pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-kie-kogito-apps.git


The following commit(s) were added to refs/heads/main by this push:
     new c6575e650 Avoid potential concurrent exception (#2199)
c6575e650 is described below

commit c6575e6504f228bbf8f05cccc047577cb89fa0b4
Author: Francisco Javier Tirado Sarti 
<[email protected]>
AuthorDate: Thu Feb 27 16:15:02 2025 +0100

    Avoid potential concurrent exception (#2199)
    
    If the list of events in the published (which global) gets modified by
    another test, then a concurrent exception will be thrown.
    In other words, the test is potentially flaky, so it is safer to copy
    the list to check it holds the expected data.
---
 .../java/org/kie/kogito/jobs/embedded/EmbeddedJobsServiceTest.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/jobs-service/kogito-addons-jobs-service/kogito-addons-quarkus-jobs/src/test/java/org/kie/kogito/jobs/embedded/EmbeddedJobsServiceTest.java
 
b/jobs-service/kogito-addons-jobs-service/kogito-addons-quarkus-jobs/src/test/java/org/kie/kogito/jobs/embedded/EmbeddedJobsServiceTest.java
index 33e0c8be4..719c2b5dc 100644
--- 
a/jobs-service/kogito-addons-jobs-service/kogito-addons-quarkus-jobs/src/test/java/org/kie/kogito/jobs/embedded/EmbeddedJobsServiceTest.java
+++ 
b/jobs-service/kogito-addons-jobs-service/kogito-addons-quarkus-jobs/src/test/java/org/kie/kogito/jobs/embedded/EmbeddedJobsServiceTest.java
@@ -18,6 +18,7 @@
  */
 package org.kie.kogito.jobs.embedded;
 
+import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.CountDownLatch;
 import java.util.function.Consumer;
@@ -84,7 +85,7 @@ public class EmbeddedJobsServiceTest {
 
         latch.await();
 
-        List<DataEvent<?>> events = publisher.getEvents();
+        List<DataEvent<?>> events = new ArrayList<>(publisher.getEvents());
 
         Assertions.assertEquals(8, events.size());
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to