Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#13332 closed (fixed)

UnitTests assume reference counting semantics

Reported by: Alex Gaynor Owned by: nobody
Component: Testing framework Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

This results in a *very* hard to debug failure on PyPy's hybrid GC.

Attachments (1)

django-unittest-refcount.diff (4.5 KB) - added by Alex Gaynor 6 years ago.

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by Alex Gaynor

comment:1 Changed 6 years ago by Alex Gaynor

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

comment:2 Changed 6 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

(In [12964]) Fixed #13332 -- Corrected the cleanup code in the test client to avoid a refcounting problem with signal handlers. This is a fix for the benefit of PyPy's hybrid GC. Thanks to Alex Gaynor for the report and patch.

comment:3 Changed 6 years ago by Russell Keith-Magee

(In [12965]) [1.1.X] Fixed #13332 -- Corrected the cleanup code in the test client to avoid a refcounting problem with signal handlers. This is a fix for the benefit of PyPy's hybrid GC. Thanks to Alex Gaynor for the report and patch.

Backport of r12964 from trunk.

comment:4 Changed 5 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

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