fix: flaxy EventProcessorTest.triggerOnAllEventDeleteEventInstantlyAfterEvent (#3003)

csviri · metacosm · web-flow · commit 48513b9c0c99 · 2025-10-17T09:12:43.000+02:00
Signed-off-by: Attila Mészáros &lt;a_meszaros@apple.com&gt;
Co-authored-by: Chris Laprun &lt;metacosm@users.noreply.github.com&gt;
diff --git a/operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/event/EventProcessorTest.java b/operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/event/EventProcessorTest.java
@@ -562,7 +562,9 @@ void triggerOnAllEventDeleteEventInstantlyAfterEvent() {
     eventProcessor.handleEvent(prepareCRDeleteEvent1());
 
     waitUntilProcessingFinished(eventProcessor, TestUtils.testCustomResource1Id());
-    verify(reconciliationDispatcherMock, times(1)).handleExecution(any());
+    // it is expected to be usually called only once, but it can happen based on timing that called
+    // twice
+    verify(reconciliationDispatcherMock, atMost(2)).handleExecution(any());
   }
 
   @Test

Original file line number	Diff line number	Diff line change
`@@ -562,7 +562,9 @@ void triggerOnAllEventDeleteEventInstantlyAfterEvent() {`
`562`	`562`	`eventProcessor.handleEvent(prepareCRDeleteEvent1());`
`563`	`563`
`564`	`564`	`waitUntilProcessingFinished(eventProcessor, TestUtils.testCustomResource1Id());`
`565`		`- verify(reconciliationDispatcherMock, times(1)).handleExecution(any());`
	`565`	`+ // it is expected to be usually called only once, but it can happen based on timing that called`
	`566`	`+ // twice`
	`567`	`+ verify(reconciliationDispatcherMock, atMost(2)).handleExecution(any());`
`566`	`568`	`}`
`567`	`569`
`568`	`570`	`@Test`