Skip to content

Commit 11ecc1b

Browse files
knutwalkerjjaderbergbreakanalysis
committed
Change expected test message for concurrency validation
Co-Authored-By: Jonatan Jäderberg <jonatan.jaderberg@neotechnology.com> Co-Authored-By: Jacob Sznajdman <breakanalysis@gmail.com>
1 parent 83b1247 commit 11ecc1b

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

algo/src/test/java/org/neo4j/graphalgo/beta/pregel/examples/WeaklyConnectedComponentsPregelTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ void shouldFailWithConcurrency10() {
8888
.build();
8989
});
9090

91-
assertThat(illegalArgumentException, containsMessage("The configured concurrency value is too high"));
91+
assertThat(illegalArgumentException, containsMessage("The configured `concurrency` value is too high"));
9292
}
9393

9494
@Test

proc/common/src/test/java/org/neo4j/graphalgo/config/ConcurrencyConfigTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,8 @@ void shouldThrowOnTooHighConcurrency() {
5353

5454
assertError(
5555
query,
56-
"The configured concurrency value is too high. " +
57-
"The maximum allowed concurrency value is 4 but 10 was configured."
56+
"The configured `concurrency` value is too high. " +
57+
"The maximum allowed `concurrency` value is 4 but 10 was configured."
5858
);
5959
}
6060

@@ -64,8 +64,8 @@ void shouldThrowOnTooHighReadConcurrency() {
6464

6565
assertError(
6666
query,
67-
"The configured concurrency value is too high. " +
68-
"The maximum allowed concurrency value is 4 but 9 was configured."
67+
"The configured `readConcurrency` value is too high. " +
68+
"The maximum allowed `readConcurrency` value is 4 but 9 was configured."
6969
);
7070
}
7171

@@ -75,8 +75,8 @@ void shouldThrowOnTooHighWriteConcurrency() {
7575

7676
assertError(
7777
query,
78-
"The configured concurrency value is too high. " +
79-
"The maximum allowed concurrency value is 4 but 12 was configured."
78+
"The configured `writeConcurrency` value is too high. " +
79+
"The maximum allowed `writeConcurrency` value is 4 but 12 was configured."
8080
);
8181
}
8282
}

proc/test/src/main/java/org/neo4j/graphalgo/AlgoBaseProcTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,7 @@ default void shouldThrowWhenTooManyCoresOnLimited() {
359359
() -> method.invoke(proc, configMap, Collections.emptyMap())
360360
);
361361
assertEquals(IllegalArgumentException.class, ex.getCause().getClass());
362-
assertThat(ex.getCause().getMessage(), containsString("The configured concurrency value is too high"));
362+
assertThat(ex.getCause().getMessage(), containsString("The configured `readConcurrency` value is too high"));
363363
})
364364
);
365365
}

0 commit comments

Comments
 (0)