Added extensions for virtual time testing

reactor / reactor-kotlin-extensions

Reactor Kotlin Support

Apache License 2.0

143 stars 21 forks source link

Added extensions for virtual time testing #19

Closed mickeelm closed 3 years ago

mickeelm commented 3 years ago

Came across some virtual time-testing scenarios in my own project and I felt that these were missing, wrote them and shared if you are interested in taking them in.

I'm of course open to discussion and perhaps I'm over-testing them, but I felt it was necessary to show that they actually use the methods that they are intended to bridge.

mickeelm commented 3 years ago

So, this PR is turning four months today. Just a friendly ping...I realize the kotlin-extensions isn't super prio though.

simonbasle commented 3 years ago

Damn @mickeelm I'm so sorry... Yeah I haven't looked at the PR list in a long time, looks like I was only subscribed to issues 😢

This PR makes total sense, but I have a comment on the tests.

mickeelm commented 3 years ago

No worries, I forgot the PR existed even :)

I'll have a look and make some changes.