-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
Package name is missing in XML report. #235
Comments
Can you provide a test case that demonstrates the regression? |
Original comment by schlauch (Bitbucket: schlauch, GitHub: schlauch) I attached a small project which illustrates the problem. It also contains the coverage XML reports which I created with version 3.5.3 and 3.6.: For more information see the README.txt. |
Original comment by Camilo Nova (Bitbucket: camilonova, GitHub: camilonova) Also happens to me as well for versions 3.6 and 3.7 |
Original comment by Lex Berezhny (Bitbucket: eukreign, GitHub: eukreign) @camilonova try my pull request #28 it resolves this issue and also adds support for defining package depth |
Original comment by Matt Wright (Bitbucket: mattupstate, GitHub: mattupstate) I just downgraded from 3.7.1 to 3.5.0. With 3.7.1, package name was missing. With 3.5.0, package name was _not_missing. |
Original comment by Camilo Nova (Bitbucket: camilonova, GitHub: camilonova) @nedbat Can you take a look again at this issue? seems https://bitbucket.org/ned/coveragepy/pull-request/28/package-depth-for-xml-report fixes it. |
Original comment by Camilo Nova (Bitbucket: camilonova, GitHub: camilonova) @nedbat the pull request was updated so it can be easily integrated. Will be great if you take and action soon on this mayor bug to fix it. |
Original comment by Camilo Nova (Bitbucket: camilonova, GitHub: camilonova) @nedbat do you have any updates on this? |
Everyone, sorry for the ridiculous regression, and for not attending to it sooner. This is now fixed in fa3b3f5777d0 (bb). |
Original comment by Camilo Nova (Bitbucket: camilonova, GitHub: camilonova) Thank you, can you put it on pypi ? |
Sure, here it is: https://pypi.python.org/pypi/coverage/4.0a3 |
Originally reported by schlauch (Bitbucket: schlauch, GitHub: schlauch)
Since 3.6 the package name is missing. I think the problem is the optimization of the package name. See xmlreport.py line 95 (xml_file):
I used coverage.py 3.6 on Windows 7 with Python 2.7.3
In 3.5 os.path.split has been used... However, this has been removed by commit:
https://bitbucket.org/ned/coveragepy/commits/6af3ba6d47e49420662c5e9f2d37f0f479ae6949
The text was updated successfully, but these errors were encountered: