Opened 18 years ago

Closed 12 years ago

#3343 closed Bug (wontfix)

Doctest find() doesn't work for model subpackage (models/__init__.py)

Reported by: anonymous Owned by: nobody
Component: Testing framework Version: dev
Severity: Normal Keywords:
Cc: noah@…, jesse.lovelace@… Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

#2982 addresses a small issue with splitting up models into a subpackage. This works fine, except the test runner can no longer find doctests in the other package files.

This is the default doctest behavior, to avoid recursing too far on packages. I have found that removing the module filter works just fine on all my model subpackages, but I am not sure if there are other consequences of removing this...

Maybe there is a better way to address this issue?

Attachments (1)

doctest_module_package.patch (555 bytes ) - added by noah@… 18 years ago.

Download all attachments as: .zip

Change History (9)

by noah@…, 18 years ago

comment:1 by Simon G. <dev@…>, 18 years ago

Component: UncategorizedUnit test system
Owner: changed from Jacob to Adrian Holovaty
Triage Stage: UnreviewedAccepted

comment:2 by anonymous, 17 years ago

Cc: jesse.lovelace@… added

comment:3 by James Bennett, 17 years ago

This needs to wait for a resolution to #3591.

comment:4 by Antti Kaihola, 17 years ago

There's a more complicated patch for this in #5751.

I didn't try this, but your approach might drop file/linenum information from error messages.

comment:5 by Łukasz Rekucki, 14 years ago

Severity: Normal
Type: Bug

comment:6 by Aymeric Augustin, 13 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:7 by Aymeric Augustin, 13 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:8 by Aymeric Augustin, 12 years ago

Resolution: wontfix
Status: newclosed

Time has passed since this ticket was accepted, and it no longer looks like a good idea. The trend is to remove custom support for doctests, not to extend it. See #18727 for more details.

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