Opened 14 years ago

Closed 14 years ago

Last modified 11 years ago

#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


On source:/django/trunk/tests/regressiontests/serializers_regress/, 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)

7981_serializers_regress_tests_fix_transaction_management.diff (3.2 KB) - added by Leo Soto M. 14 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 14 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

(In [8099]) Fixed #7981 -- Wrap the manual transaction management in the
serializers_regress tests in some "try...finally" blocks. Patch from Leo Soto.

comment:2 Changed 11 years ago by Jacob

milestone: 1.0 beta

Milestone 1.0 beta deleted

Note: See TracTickets for help on using tickets.
Back to Top