Skip to content

Commit 3843912

Browse files
committed
fixed: failing test
1 parent e789264 commit 3843912

3 files changed

Lines changed: 22 additions & 11 deletions

File tree

rqueue-core/src/test/java/com/github/sonus21/rqueue/core/RqueueMessageTemplateTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ class RqueueMessageTemplateTest extends TestBase {
5353
@Mock
5454
private RedisConnectionFactory redisConnectionFactory;
5555

56+
@Mock
5657
private ListOperations<String, RqueueMessage> listOperations;
5758

5859
@Mock

rqueue-core/src/test/java/com/github/sonus21/rqueue/listener/RqueueMiddlewareTest.java

Lines changed: 19 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@
7070
import java.util.UUID;
7171
import java.util.concurrent.Callable;
7272
import java.util.concurrent.Executor;
73+
import java.util.concurrent.ExecutorService;
7374
import java.util.concurrent.Executors;
7475
import lombok.extern.slf4j.Slf4j;
7576
import org.junit.jupiter.api.BeforeEach;
@@ -328,19 +329,26 @@ void logAndRateLimiterMiddleware() throws TimedOutException {
328329
})
329330
.when(messageHandler)
330331
.handleMessage(any());
331-
Executor executor = Executors.newSingleThreadExecutor();
332+
ExecutorService executor = Executors.newFixedThreadPool(4);
332333
long startTime = System.currentTimeMillis();
333-
for (RqueueMessage message : messages) {
334-
executor.execute(new RqueueExecutor(
335-
rqueueBeanProvider,
336-
queueStateMgr,
337-
newArrayList(logMiddleware, testRateLimiter),
338-
postProcessingHandler,
339-
message,
340-
queueDetail,
341-
queueThreadPool));
334+
try {
335+
for (RqueueMessage message : messages) {
336+
executor.execute(new RqueueExecutor(
337+
rqueueBeanProvider,
338+
queueStateMgr,
339+
newArrayList(logMiddleware, testRateLimiter),
340+
postProcessingHandler,
341+
message,
342+
queueDetail,
343+
queueThreadPool));
344+
}
345+
TimeoutUtils.waitFor(
346+
() -> testRateLimiter.jobs.size() == jobCount,
347+
20_000L,
348+
"all jobs to proceed");
349+
} finally {
350+
executor.shutdownNow();
342351
}
343-
TimeoutUtils.waitFor(() -> testRateLimiter.jobs.size() == jobCount, "all jobs to proceed");
344352
long endTime = System.currentTimeMillis();
345353
// we need to round since rate is at second resolution and execution in millis so we round this
346354
// to next second

rqueue-core/src/test/java/com/github/sonus21/rqueue/web/service/RqueueUtilityServiceTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,8 @@ class RqueueUtilityServiceTest extends TestBase {
6666

6767
@Mock
6868
private RqueueWebConfig rqueueWebConfig;
69+
70+
@Mock
6971
private RqueueMessageTemplate rqueueMessageTemplate;
7072

7173
@Mock

0 commit comments

Comments
 (0)