(tomee) branch main updated: Tweak ManagedScheduledExecutorServiceTest timing (which is often causing CI failures) The bad feeling remains that I only fixed 1 of 30 ugly timings :/

[rzo1]

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

rzo1 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomee.git


The following commit(s) were added to refs/heads/main by this push:
     new 1d0d439486 Tweak ManagedScheduledExecutorServiceTest timing (which is 
often causing CI failures) The bad feeling remains that I only fixed 1 of 30 
ugly timings :/
1d0d439486 is described below

commit 1d0d439486c2967183b0c610b7d664fdb6ff0602
Author: Richard Zowalla <r...@apache.org>
AuthorDate: Mon Apr 22 10:08:05 2024 +0200

    Tweak ManagedScheduledExecutorServiceTest timing (which is often causing CI 
failures)
    The bad feeling remains that I only fixed 1 of 30 ugly timings :/
---
 .../apache/openejb/threads/ManagedScheduledExecutorServiceTest.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/container/openejb-core/src/test/java/org/apache/openejb/threads/ManagedScheduledExecutorServiceTest.java
 
b/container/openejb-core/src/test/java/org/apache/openejb/threads/ManagedScheduledExecutorServiceTest.java
index 13d902f467..ac87560ea1 100644
--- 
a/container/openejb-core/src/test/java/org/apache/openejb/threads/ManagedScheduledExecutorServiceTest.java
+++ 
b/container/openejb-core/src/test/java/org/apache/openejb/threads/ManagedScheduledExecutorServiceTest.java
@@ -119,15 +119,15 @@ public class ManagedScheduledExecutorServiceTest {
         assertFalse(future.isCancelled());
 
         //Should easily get 5 invocations within 1 second
-        counter.await(3, TimeUnit.SECONDS);
+        counter.await(5, TimeUnit.SECONDS);
 
         future.cancel(true);
         assertEquals("Counter did not count down in time", 0L, 
counter.getCount());
 
         final boolean done = future.isDone();
-        assertTrue(done);
+        assertTrue("Future should be done", done);
         final boolean cancelled = future.isCancelled();
-        assertTrue(cancelled);
+        assertTrue("Future should be cancelled", cancelled);
     }
 
     @Test