TCK: Provide test classes working directly on JDK9 interfaces

[ktoso]

There's two ways to go about this I guess:

Proposed style of rolling it out:

• change TCK code to work on JDK9 interfaces directly
• make current Verification classes go through adapters provided via Create RS <-> JDK9 bridge jar #294
• add new Verification classes (named like "FlowPublisher...Verification" etc, which new users can use then

This would move "us" to test the JDK things directly, which is the end goal and allow slowly moving away from having to use the adapters for most impls (as impls move to implementing the JDK interfaces)

[viktorklang]

@ktoso I believe you wanted to take a stab at this? :)

[ktoso]

Yes, absolutely - work in progress already.
Terribly jet-lagged though, hope a good nights sleep today will get me hacking on to finish this up on monday US time.

[viktorklang]

Awesome. Let me know how I can help.

…

-- Cheers, √

On Oct 2, 2017 05:43, "ktoso" ***@***.***> wrote: Yes, absolutely - work in progress already. Terribly jet-lagged though, hope a good nights sleep today will get me hacking on to finish this up on monday US time. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#388 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAqd3cPx4aHm3RRuOIuJ97dmnLp0D4Nks5soFvNgaJpZM4OWiq_> .

[akarnokd]

This should be simply a copy-paste of the current TCK in a new module and replacing org.reactivestreams with java.util.concurrent.Flow, right?

[ktoso]

That would not be nice for maintenance though ;)

…

On 2 October 2017 at 02:09:03, David Karnok ***@***.***) wrote: This should be simply a copy-paste of the current TCK in a new module and replacing org.reactivestreams with java.util.concurrent.Flow, right? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#388 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAHYk_0xDS36nYUva9KLHLPa7lV8KZomks5soKgvgaJpZM4OWiq_> .

[akarnokd]

I'm not a Gradle guru but I'm sure it can be automatically synchronized via scripting.

[akarnokd]

I'm not a Gradle guru but I'm sure it can be automatically synchronized via scripting.

Nope, it's still tedious as the references have to be carefully replaced since the containing package is org.reactivestreams too.