Opened 17 years ago

Closed 17 years ago

#5442 closed (fixed)

Workaround for jython isclass() bug

Reported by: leo.soto@… Owned by: nobody
Component: Testing framework Version: dev
Severity: Keywords: jython, sprintsept14
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

_doctest.py uses isclass(), which isn't 100% reliable on Jython, see http://sourceforge.net/tracker/index.php?func=detail&aid=1786009&group_id=12867&atid=112867.

Attachments (1)

jython_isclass_workaround.patch (727 bytes ) - added by leo.soto@… 17 years ago.
Workaround for the bug until it's fixed on jython

Download all attachments as: .zip

Change History (5)

by leo.soto@…, 17 years ago

Workaround for the bug until it's fixed on jython

comment:1 by anonymous, 17 years ago

Has patch: set

comment:2 by Fredrik Lundh <fredrik@…>, 17 years ago

Triage Stage: UnreviewedReady for checkin

I would have put the "patch_isclass" definition inside the if-statement (no need to add stuff to the module namespace on other platforms), but since this is test code (right?), that doesn't really matter that much.

comment:3 by Fredrik Lundh <fredrik@…>, 17 years ago

Keywords: sprintsept14 added

comment:4 by Adrian Holovaty, 17 years ago

Resolution: fixed
Status: newclosed

(In [6194]) Fixed #5442 -- Added Jython workaround in django.test._doctest. Thanks, leo.soto@…

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