Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Erroneous report of non-executing test statements #10

Closed
nedbat opened this issue Jun 26, 2009 · 2 comments
Closed

Erroneous report of non-executing test statements #10

nedbat opened this issue Jun 26, 2009 · 2 comments
Labels
bug Something isn't working

Comments

@nedbat
Copy link
Owner

nedbat commented Jun 26, 2009

Originally reported by James Bennett (Bitbucket: ubernostrum, GitHub: ubernostrum)


Running coverage on the current tip of my application django-flashpolicies (see http://bitbucket.org/ubernostrum/django-flashpolicies/src/359248e9e4d4/), I get an odd report; coverage doesn't think that the entire view tests file is executing:

Name                                                                             Stmts   Exec  Cover   Missing
--------------------------------------------------------------------------------------------------------------
/Users/jbennett/dev/hg/personal/django-flashpolicies/flashpolicies/tests/views      36     25    69%   28-29, 41-45, 58-59, 72-73

Those lines in the view tests file //are// executing, however, as I can verify by dropping print statements or changing the assertions so that the tests will fail.

The lines reported as missed all have in common that they come immediately after calls to xml.dom.minidom.parseString, but I've no idea why that would cause a problem or whether it's really related.


@nedbat
Copy link
Owner Author

nedbat commented Jun 28, 2009

OK, that was a big one. Read the comments in <<changeset 507072f95bdf (bb)>> to learn the whole sad story...

@nedbat
Copy link
Owner Author

nedbat commented Jun 28, 2009

Pyexpat bug filed: http://bugs.python.org/issue6359

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant