-
Notifications
You must be signed in to change notification settings - Fork 536
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
Test rollback document suggested modification #699
base: 3.2.x
Are you sure you want to change the base?
Conversation
|
@@ -36,7 +36,8 @@ class ExampleSpec extends Specification { | |||
|
|||
The `Rollback` annotation ensures that each test method runs in a transaction that is rolled back. Generally this is desirable because you do not want your tests depending on order or application state. | |||
|
|||
In Grails 3.0 tests rely on `grails.transaction.Rollback` annotation to bind the session in integration tests. Though each test method transaction is rolled back, the `setup()` method uses a separate transaction that is not rolled back. | |||
In Grails 3.0 tests rely on `grails.transaction.Rollback` annotation to bind the session in integration tests. Though each test method transaction is rolled back, the `setup()` method uses a separate transaction that is not rolled back and the `setup()` method will be executed automatically when the tests is executed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this ... is not rolled back and the setup()...
should be broken into a sentence like this:
is not rolled back. The setup()
@@ -72,6 +73,11 @@ import spock.lang.* | |||
@Integration | |||
@Rollback | |||
class BookSpec extends Specification { | |||
|
|||
void setup() { | |||
// Do not use this method |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These two lines should not be commented out in the example, and the comment should be
// This line will not roll back
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me know what you think of these changes and we can merge it as i like the idea
No description provided.