-
Notifications
You must be signed in to change notification settings - Fork 347
Add support for serialized rollback #353
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
Conversation
As per my comment in PR #308 (#308 (comment)) I think the fixture in this PR should be named to |
"""Enable serialized rollback after transaction test cases | ||
|
||
This fixture only has an effect when the ``transactional_db`` | ||
fixture is active, which happen as a side-effect of requesting |
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.
s/happen/happens/
ping. what's the status of this PR? Is there a newer version? What do you need help with? I kind of need this fixed ASAP and I can help clean up the rest of the code review if you can merge quickly. |
@atodorov |
this is a workaround for pytest-dev/pytest-django#353, pytest-dev/pytest-django#329 and #121.
this is a workaround for pytest-dev/pytest-django#353, pytest-dev/pytest-django#329 and #121.
this is a workaround for pytest-dev/pytest-django#353, pytest-dev/pytest-django#329 and #121.
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
A PR which will replace this PR is in #970. I intend to merge and release it in a few days. Comments and tests are welcome. |
Thanks to Aymeric Augustin, Daniel Hahler and Fábio C. Barrionuevo da Luz for previous work on this feature. Fix pytest-dev#329. Closes pytest-dev#353. Closes pytest-dev#721. Closes pytest-dev#919. Closes pytest-dev#956.
Fixes #329.
TODO:
This conflicts (in the sense of touching the same code) with #308.
Continuation/rebase of #331.