Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add interceptor around parameterized features #953

Merged
merged 1 commit into from
Feb 16, 2024
Merged

Conversation

timyates
Copy link
Contributor

When a feature is using @Unroll (which is implicit in recent versions of Spock), we needed to add the interceptor to the iteration interceptor handler instead of the standard handler.

This bug was visible in #911 where specs that were parameterized did not properly revert the transaction between iterations.

Closes #911

When a feature is using `@Unroll` (which is implicit in recent versions of Spock), we needed to
add the interceptor to the iteration interceptor handler instead of the standard handler.

This bug was visible in #911 where specs that were parameterized did not properly revert the
transaction between iterations.

Closes #911
@timyates timyates added lang: groovy Issues or features specific to Groovy type: bug Something isn't working labels Feb 16, 2024
@timyates timyates self-assigned this Feb 16, 2024
@timyates
Copy link
Contributor Author

Thanks @dstepanov for all the help 👍

Copy link

@sdelamo sdelamo merged commit d610996 into master Feb 16, 2024
@sdelamo sdelamo deleted the fix-spock-unroll branch February 16, 2024 13:56
timyates added a commit that referenced this pull request Feb 16, 2024
When a feature is using `@Unroll` (which is implicit in recent versions of Spock), we needed to
add the interceptor to the iteration interceptor handler instead of the standard handler.

This bug was visible in #911 where specs that were parameterized did not properly revert the
transaction between iterations.

Closes #911
sdelamo pushed a commit that referenced this pull request Feb 16, 2024
When a feature is using `@Unroll` (which is implicit in recent versions of Spock), we needed to
add the interceptor to the iteration interceptor handler instead of the standard handler.

This bug was visible in #911 where specs that were parameterized did not properly revert the
transaction between iterations.

Closes #911
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang: groovy Issues or features specific to Groovy type: bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Spock Unroll tests rollback regression
2 participants