tkalkirill commented on code in PR #1833:
URL: https://github.com/apache/ignite-3/pull/1833#discussion_r1145792819
##########
modules/core/src/testFixtures/java/org/apache/ignite/internal/testframework/matchers/CompletableFutureMatcher.java:
##########
@@ -70,22 +73,23 @@ private CompletableFutureMatcher(Matcher<T> matcher) {
* @param matcher Matcher to forward the result of the completable future.
* @param timeout Timeout.
* @param timeoutTimeUnit {@link TimeUnit} for timeout.
- * @param causeOfFail If {@code null}, the future should be completed
successfully, otherwise it specifies the class of cause
- * throwable.
+ * @param causeOfFail If {@code null}, the future should be completed
successfully, otherwise it specifies class of cause throwable.
Review Comment:
Fix it
##########
modules/core/src/testFixtures/java/org/apache/ignite/internal/testframework/matchers/CompletableFutureMatcher.java:
##########
@@ -70,22 +73,23 @@ private CompletableFutureMatcher(Matcher<T> matcher) {
* @param matcher Matcher to forward the result of the completable future.
* @param timeout Timeout.
* @param timeoutTimeUnit {@link TimeUnit} for timeout.
- * @param causeOfFail If {@code null}, the future should be completed
successfully, otherwise it specifies the class of cause
- * throwable.
+ * @param causeOfFail If {@code null}, the future should be completed
successfully, otherwise it specifies class of cause throwable.
+ * @param errorMessageFragment Fragment that must be a substring of a
error message (if {@code null}, message won't be checked).
Review Comment:
fix it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
