From b1e5b1daf88f4438238c194f5ecf4dfbfcee3f66 Mon Sep 17 00:00:00 2001 From: Viktor Klang Date: Mon, 22 May 2017 10:22:40 +0200 Subject: [PATCH] Clarifies the meaning of "stochastic" for skipStochasticTests() --- .../java/org/reactivestreams/tck/PublisherVerification.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tck/src/main/java/org/reactivestreams/tck/PublisherVerification.java b/tck/src/main/java/org/reactivestreams/tck/PublisherVerification.java index a9acf18f..43145ecf 100644 --- a/tck/src/main/java/org/reactivestreams/tck/PublisherVerification.java +++ b/tck/src/main/java/org/reactivestreams/tck/PublisherVerification.java @@ -128,7 +128,9 @@ public long maxElementsFromPublisher() { /** * Override and return {@code true} in order to skip executing tests marked as {@code Stochastic}. - * Such tests MAY sometimes fail even though the impl + * Stochastic in this case means that the Rule is impossible or infeasible to deterministically verify— + * usually this means that this test case can yield false positives ("be green") even if for some case, + * the given implementation may violate the tested behaviour. */ public boolean skipStochasticTests() { return false;