Skip to content

Commit b9eeda0

Browse files
Improve test time and flaky schedule test (#2183)
Try running CI on 16 core runners
1 parent abc5323 commit b9eeda0

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ on:
88
jobs:
99
unit_test_edge:
1010
name: Unit test with in-memory test service [Edge]
11-
runs-on: ubuntu-latest
11+
runs-on: ubuntu-latest-16-cores
1212
timeout-minutes: 30
1313
steps:
1414
- name: Checkout repo
@@ -47,7 +47,7 @@ jobs:
4747

4848
unit_test_jdk8:
4949
name: Unit test with docker service [JDK8]
50-
runs-on: ubuntu-latest
50+
runs-on: ubuntu-latest-16-cores
5151
timeout-minutes: 30
5252
steps:
5353
- name: Checkout repo

.github/workflows/coverage.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ on:
66

77
jobs:
88
code-coverage:
9-
runs-on: ubuntu-latest
9+
runs-on: ubuntu-latest-16-cores
1010
steps:
1111
- name: Checkout
1212
uses: actions/checkout@v4

temporal-sdk/src/test/java/io/temporal/client/schedules/ScheduleTest.java

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -260,14 +260,15 @@ public void triggerScheduleNoPolicy() {
260260

261261
@Test(timeout = 30000)
262262
public void backfillSchedules() {
263-
// assumeTrue("skipping for test server", SDKTestWorkflowRule.useExternalService);
264-
Instant now = Instant.now();
263+
Instant backfillTime = Instant.ofEpochSecond(100000);
265264
ScheduleClient client = createScheduleClient();
266265
// Create schedule
267266
ScheduleOptions options =
268267
ScheduleOptions.newBuilder()
269268
.setBackfills(
270-
Arrays.asList(new ScheduleBackfill(now.minusMillis(20500), now.minusMillis(10000))))
269+
Arrays.asList(
270+
new ScheduleBackfill(
271+
backfillTime.minusMillis(20500), backfillTime.minusMillis(10000))))
271272
.build();
272273
String scheduleId = UUID.randomUUID().toString();
273274
Schedule schedule =
@@ -283,17 +284,17 @@ public void backfillSchedules() {
283284

284285
handle.backfill(
285286
Arrays.asList(
286-
new ScheduleBackfill(now.minusMillis(5500), now.minusMillis(2500)),
287-
new ScheduleBackfill(now.minusMillis(2500), now)));
287+
new ScheduleBackfill(backfillTime.minusMillis(5500), backfillTime.minusMillis(2500)),
288+
new ScheduleBackfill(backfillTime.minusMillis(2500), backfillTime)));
288289
waitForActions(handle, 15);
289290
// Cleanup schedule
290291
handle.delete();
291292
// Try to backfill a deleted schedule
292293
try {
293294
handle.backfill(
294295
Arrays.asList(
295-
new ScheduleBackfill(now.minusMillis(5500), now.minusMillis(2500)),
296-
new ScheduleBackfill(now.minusMillis(2500), now)));
296+
new ScheduleBackfill(backfillTime.minusMillis(5500), backfillTime.minusMillis(2500)),
297+
new ScheduleBackfill(backfillTime.minusMillis(2500), backfillTime)));
297298
Assert.fail();
298299
} catch (ScheduleException e) {
299300
}

0 commit comments

Comments
 (0)