Skip to content

Commit a659ba7

Browse files
committed
Typos and formatting
1 parent ec3af86 commit a659ba7

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

platform-tests/src/test/java/org/junit/platform/engine/support/hierarchical/ConcurrentHierarchicalTestExecutorServiceTests.java

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -412,7 +412,7 @@ void workIsStolenInReverseOrder() throws Exception {
412412
CyclicBarrier cyclicBarrier = new CyclicBarrier(2);
413413
Executable behavior = cyclicBarrier::await;
414414

415-
// With half of the leaves to executed normally
415+
// With half of the leaves to be executed normally
416416
var leaf1a = new TestTaskStub(ExecutionMode.CONCURRENT, behavior) //
417417
.withName("leaf1a").withLevel(2);
418418
var leaf1b = new TestTaskStub(ExecutionMode.CONCURRENT, behavior) //
@@ -437,18 +437,16 @@ void workIsStolenInReverseOrder() throws Exception {
437437
assertThat(List.of(root, leaf1a, leaf1b, leaf1c, leaf2a, leaf2b, leaf2c)) //
438438
.allSatisfy(TestTaskStub::assertExecutedSuccessfully);
439439

440-
// If the last node must be stolen.
440+
// If the last node was stolen.
441441
assertThat(leaf1a.executionThread).isNotEqualTo(leaf2c.executionThread);
442-
// Then must follow that the last half of the nodes were stolen
442+
// Then it must follow that the last half of the nodes were stolen
443443
assertThat(Stream.of(leaf1a, leaf1b, leaf1c, leaf2a, leaf2b, leaf2c)).extracting(
444-
TestTaskStub::executionThread).containsExactly(leaf1a.executionThread, //
445-
leaf1a.executionThread, //
446-
leaf1a.executionThread, //
447-
leaf2c.executionThread, //
448-
leaf2c.executionThread, //
449-
leaf2c.executionThread //
444+
TestTaskStub::executionThread).containsExactly( //
445+
leaf1a.executionThread, leaf1a.executionThread, leaf1a.executionThread, //
446+
leaf2c.executionThread, leaf2c.executionThread, leaf2c.executionThread //
450447
);
451-
assertAll(() -> assertThat(leaf1a.startTime).isBeforeOrEqualTo(leaf1b.startTime),
448+
assertAll( //
449+
() -> assertThat(leaf1a.startTime).isBeforeOrEqualTo(leaf1b.startTime),
452450
() -> assertThat(leaf1b.startTime).isBeforeOrEqualTo(leaf1c.startTime),
453451
() -> assertThat(leaf2a.startTime).isAfterOrEqualTo(leaf2b.startTime),
454452
() -> assertThat(leaf2b.startTime).isAfterOrEqualTo(leaf2c.startTime));

0 commit comments

Comments
 (0)