TestEnvironment constructor mixes up timeout, no signal timeout and poll timeout

[Azquelt]

If either the TestEnvironment() or TestEnvironment(long defaultTimeoutMillis, long defaultNoSignalsTimeoutMillis) constructor is used:

• defaultNoSignalsTimeoutMillis is set to the value intended for defaultTimeoutMillis
• defaultPollTimeoutMillis is set to the value intended for defaultNoSignalsTimeoutMillis

This is due to the way that arguments are passed here: https://github.com/reactive-streams/reactive-streams-jvm/blob/master/tck/src/main/java/org/reactivestreams/tck/TestEnvironment.java#L106-L108

  public TestEnvironment(long defaultTimeoutMillis, long defaultNoSignalsTimeoutMillis) {
    this(defaultTimeoutMillis, defaultTimeoutMillis, defaultNoSignalsTimeoutMillis);
  }

I'm not sure what the correct value for the third argument is, but the second argument should definitely be defaultNoSignalsTimeoutMillis rather than defaultTimeoutMillis.

[viktorklang]

@Azquelt Fixed in master!

[Azquelt]

Thanks 😄