#7981 closed (fixed)
serializers_regress test doesn't properly leave transaction management
| Reported by: | Leo Soto M. | Owned by: | nobody |
|---|---|---|---|
| Component: | Testing framework | Version: | dev |
| Severity: | Keywords: | jython | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
On source:/django/trunk/tests/regressiontests/serializers_regress/tests.py, if an exception occurs while transaction management is active, the transaction management block isn't properly "closed". This causes test suite crashes because the database can't be flushed later without committing or rollbacking the transaction (I'm experimenting them on my Jython testing).
The attaches patch ensures that leave_transaction_managed is always executed.
Attachments (1)
Change History (3)
by , 16 years ago
| Attachment: | 7981_serializers_regress_tests_fix_transaction_management.diff added |
|---|
comment:1 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
(In [8099]) Fixed #7981 -- Wrap the manual transaction management in the
serializers_regress tests in some "try...finally" blocks. Patch from Leo Soto.