Skip to content

Commit 754452c

Browse files
authored
raise timeout on flaky test (#10868)
1 parent fb7e259 commit 754452c

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

airbyte-workers/src/test/java/io/airbyte/workers/temporal/scheduling/ConnectionManagerWorkflowTest.java

+7-7
Original file line numberDiff line numberDiff line change
@@ -503,7 +503,7 @@ public void cancelRunning() throws InterruptedException {
503503
final Queue<ChangedStateEvent> eventQueue = testStateListener.events(testId);
504504
final List<ChangedStateEvent> events = new ArrayList<>(eventQueue);
505505

506-
for (ChangedStateEvent event : events) {
506+
for (final ChangedStateEvent event : events) {
507507
if (event.isValue()) {
508508
log.info("event = " + event);
509509
}
@@ -549,7 +549,7 @@ public void resetStart() throws InterruptedException {
549549
}
550550

551551
@RepeatedTest(10)
552-
@Timeout(value = 10,
552+
@Timeout(value = 30,
553553
unit = TimeUnit.SECONDS)
554554
@DisplayName("Test that resetting a running workflow cancels the running workflow")
555555
public void resetCancelRunningWorkflow() throws InterruptedException {
@@ -578,7 +578,7 @@ public void resetCancelRunningWorkflow() throws InterruptedException {
578578
final Queue<ChangedStateEvent> eventQueue = testStateListener.events(testId);
579579
final List<ChangedStateEvent> events = new ArrayList<>(eventQueue);
580580

581-
for (ChangedStateEvent event : events) {
581+
for (final ChangedStateEvent event : events) {
582582
if (event.isValue()) {
583583
log.info("event = " + event);
584584
}
@@ -667,7 +667,7 @@ public void updatedSignalReceivedWhileRunning() throws InterruptedException {
667667
final Queue<ChangedStateEvent> eventQueue = testStateListener.events(testId);
668668
final List<ChangedStateEvent> events = new ArrayList<>(eventQueue);
669669

670-
for (ChangedStateEvent event : events) {
670+
for (final ChangedStateEvent event : events) {
671671
if (event.isValue()) {
672672
log.info("event = " + event);
673673
}
@@ -883,7 +883,7 @@ public static Stream<Arguments> getSetupFailingFailingActivityBeforeRun() {
883883

884884
@ParameterizedTest
885885
@MethodSource("getSetupFailingFailingActivityBeforeRun")
886-
void testGetStuckBeforeRun(Thread mockSetup) throws InterruptedException {
886+
void testGetStuckBeforeRun(final Thread mockSetup) throws InterruptedException {
887887
mockSetup.run();
888888
Mockito.when(mConfigFetchActivity.getTimeToWait(Mockito.any())).thenReturn(new ScheduleRetrieverOutput(
889889
Duration.ZERO));
@@ -973,7 +973,7 @@ public static Stream<Arguments> getSetupFailingFailingActivityAfterRun() {
973973

974974
@ParameterizedTest
975975
@MethodSource("getSetupFailingFailingActivityAfterRun")
976-
void testGetStuckAfterRun(Consumer<ConnectionManagerWorkflow> signalSender, Thread mockSetup) throws InterruptedException {
976+
void testGetStuckAfterRun(final Consumer<ConnectionManagerWorkflow> signalSender, final Thread mockSetup) throws InterruptedException {
977977
mockSetup.run();
978978

979979
final UUID testId = UUID.randomUUID();
@@ -1055,7 +1055,7 @@ private static void startWorkflowAndWaitUntilReady(final ConnectionManagerWorkfl
10551055
while (!isReady) {
10561056
try {
10571057
isReady = workflow.getState() != null;
1058-
} catch (Exception e) {
1058+
} catch (final Exception e) {
10591059
log.info("retrying...");
10601060
Thread.sleep(100);
10611061
}

0 commit comments

Comments
 (0)