Kotest: Add infix modifier to collections matchers

[jensim]

val found = repo.findAll()
found shouldContainExactly listOf("one", "two", "three")

//Before
fun <T, C : Collection<T>> C?.shouldContainExactly(expected: C) = this should containExactly(expected)

//After
infix fun <T, C : Collection<T>> C?.shouldContainExactly(expected: C) = this should containExactly(expected)

This is not applicable to vararg param functions due to limitation in language, but it would look better in for all collections matchers where it is possible.

[sksamuel]

Yes good idea.

[jensim]

I forked master and created a branch with the infix added

[sksamuel]

Excellent. Please make a PR and I'll merge.

[jensim]

https://github.com/kotlintest/kotlintest/pull/405

[sksamuel]

I'll include this in 3.1.10

[jensim]

@sksamuel When is 3.1.10 due?

[sksamuel]

I'll try and do it tonight. I need to branch from master and cherry pick a few commits.

[sksamuel]

3.1.10 is released.

[jensim]

Hitting refresh button at
https://search.maven.org/search?q=g:io.kotlintest%20AND%20a:kotlintest-assertions%20AND%20v:3.1.10
😁

[sksamuel]

You'll be waiting hours for that UI to update.
Once it's in the repo you're good to go
http://repo1.maven.org/maven2/io/kotlintest/kotlintest-assertions/

On Thu, 13 Sep 2018 at 09:23, Jens [e-mail] wrote:

Hitting refresh button at

https://search.maven.org/search?q=g:io.kotlintest%20AND%20a:kotlintest-assertions%20AND%20v:3.1.10
😁

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/issues/404#issuecomment-420925202,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtZGv20HWBFBdtH3GVu7TEObaj7RPK_ks5uahYAgaJpZM4WlMj7
.

[jensim]

Thanks @sksamuel !