Django

Code

Ticket #5442 (closed: fixed)

Opened 1 year ago

Last modified 1 year ago

Workaround for jython isclass() bug

Reported by: leo.soto@gmail.com Assigned to: nobody
Milestone: Component: Testing framework
Version: SVN Keywords: jython, sprintsept14
Cc: Triage Stage: Ready for checkin
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

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

jython_isclass_workaround.patch (0.7 kB) - added by leo.soto@gmail.com on 09/14/07 00:08:46.
Workaround for the bug until it's fixed on jython

Change History

09/14/07 00:08:46 changed by leo.soto@gmail.com

  • attachment jython_isclass_workaround.patch added.

Workaround for the bug until it's fixed on jython

09/14/07 01:06:21 changed by anonymous

  • needs_better_patch changed.
  • has_patch set to 1.
  • needs_tests changed.
  • needs_docs changed.

09/14/07 09:44:06 changed by Fredrik Lundh <fredrik@pythonware.com>

  • stage changed from Unreviewed to Ready 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.

09/14/07 10:14:09 changed by Fredrik Lundh <fredrik@pythonware.com>

  • keywords changed from jython to jython, sprintsept14.

09/14/07 12:17:40 changed by adrian

  • status changed from new to closed.
  • resolution set to fixed.

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

Add/Change #5442 (Workaround for jython isclass() bug)




Change Properties
Action