Support for Spock

[phejl]

It would be great to have a support for Spock. So one could use the library like this:

import dev.morling.jfrunit.*
import spock.lang.Specification

import java.time.Duration

import static dev.morling.jfrunit.JfrEventsAssert.*
import static dev.morling.jfrunit.ExpectedEvent.*

@JfrEventTest
class JfrSpec extends Specification {

    JfrEvents jfrEvents = new JfrEvents()

    @EnableEvent('jdk.GarbageCollection')
    @EnableEvent('jdk.ThreadSleep')
    def 'should Have GC And Sleep Events'() throws Exception {
        when:
        System.gc()
        sleep(1000)

        then:
        assertThat(jfrEvents).contains(event('jdk.GarbageCollection'))
        assertThat(jfrEvents).contains(
                event('jdk.ThreadSleep').with('time', Duration.ofSeconds(1)))
    }
}

[aalmiray]

AFAICT Spock 2.0 supports Junit Platform's Vintage and Jupiter engines which would allow Junit5 extension to be seamlessly consumed. Adding @leonard84

[gunnarmorling]

Interesting idea, I certainly could see us doing that. What's the language used above, Groovy?

[aalmiray]

That is indeed Groovy.

[leonard84]

Spock 2.0 doesn't support junit-jupiter extensions (the jupiter API is far to complex for an easy support), although there is still best-effort support for junit4 rules.

The best thing would be to write a native Spock extension.
FYI, you can write Spock extensions in Java.

[gunnarmorling]

That is indeed Groovy.

I see. So I like that apparently this would provide a way to avoid the need for the explicit awaitEvents() call between the given/when and then. We might wrap those blocks in Lambda expressions for a Java API of course, alternatively.

[leonard84]

I like that apparently this would provide a way to avoid the need for the explicit awaitEvents() call between the given/when and then.

This might not be possible to achieve via extensions at the moment, as spock doesn't expose those blocks at runtime to extensions. There is an open issue spockframework/spock#538 that touches on this.

[phejl]

I think I might have a PR soon :)

[phejl]

#38

[phejl]

@gunnarmorling I guess this can be closed now.

[gunnarmorling]

Indeed, thx!