Skip to content

Commit 8dbe718

Browse files
committed
Remove vague statements, be more specific in others
1 parent 5c2746a commit 8dbe718

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

tck/src/main/java/org/reactivestreams/tck/support/PublisherVerificationRules.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ public interface PublisherVerificationRules {
3030
void required_validate_boundedDepthOfOnNextAndRequestRecursion() throws Exception;
3131
/**
3232
* Asks for a {@code Publisher} that should emit exactly one item and complete (both within a
33-
* timeout specified by {@link org.reactivestreams.tck.TestEnvironment#defaultTimeoutMillis()}).
33+
* timeout specified by {@link org.reactivestreams.tck.TestEnvironment#defaultTimeoutMillis()})
34+
* in response to a request(1).
3435
* <p>The test is not executed if {@link org.reactivestreams.tck.PublisherVerification#maxElementsFromPublisher()} returns zero.
35-
* <p>This test mostly ensures that the {@code Publisher} implementation is actually operational.
3636
* If this test fails (likely with a timeout error), the following could be checked within the {@code Publisher} implementation:
3737
* <ul>
3838
* <li>the {@code Publisher.subscribe(Subscriber)} method has actual implementation,</li>
@@ -52,8 +52,8 @@ public interface PublisherVerificationRules {
5252
* Asks for a {@code Publisher} that should emit exactly three items and complete (all within a
5353
* timeout specified by {@link org.reactivestreams.tck.TestEnvironment#defaultTimeoutMillis()}).
5454
* <p>The test is not executed if {@link org.reactivestreams.tck.PublisherVerification#maxElementsFromPublisher()} is less than 3.
55-
* <p>This test mostly ensures that the {@code Publisher} implementation is actually operational.
56-
* If this test fails (likely with a timeout error), the following could be checked within the {@code Publisher} implementation:
55+
* <p>The tests requests one-by-one and verifies each single response item arrives in time.
56+
* <p>If this test fails (likely with a timeout error), the following could be checked within the {@code Publisher} implementation:
5757
* <ul>
5858
* <li>the {@code Publisher.subscribe(Subscriber)} method has actual implementation,</li>
5959
* <li>in the {@code Publisher.subscribe(Subscriber)} method, if there is an upstream {@code Publisher},
@@ -189,7 +189,7 @@ public interface PublisherVerificationRules {
189189
/**
190190
* Asks for an empty {@code Publisher} and calls and verifies if {@code onSubscribe} was called before any calls
191191
* to the other {@code onXXX} methods.
192-
* <p>Note that this test doesn't request anything yet an {@code onNext} is not considered as a failure.
192+
* <p>Note that this test doesn't request anything, however, an {@code onNext} is not considered as a failure.
193193
* <p>If this test fails, the following could be checked within the {@code Publisher} implementation:
194194
* <ul>
195195
* <li>the {@code TestEnvironment} has large enough timeout specified in case the {@code Publisher} has some time-delay behavior,</li>

0 commit comments

Comments
 (0)