FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Feature/multiline json by realInt2k · Pull Request #2183 · coveragepy/coveragepy · GitHub

Feature/multiline json - #2183

Open
realInt2k wants to merge 4 commits into
coveragepy:mainfrom
realInt2k:feature/multiline_json
Open

Feature/multiline json#2183
realInt2k wants to merge 4 commits into
coveragepy:mainfrom
realInt2k:feature/multiline_json

Conversation

Copy link
Copy Markdown

I added the multiline map thing from html report for json report

I also added --multiline option with comment so it's optional.

Purpose: I'm doing a thing where I get coverage for base execution, then coverage for another execution, and I want to diff them. With the original coverage.py, I wouldn't get multiline statement and have to do extra static analysis to get the full statement

Copy link
Copy Markdown
Author
for example, original command would just give 1218

nedbat commented May 30, 2026

Copy link
Copy Markdown
Member

I need to review this more closely, but some things stand out:

  • There is no test for the new code path in jsonreport.py
  • I don't understand why jsonreport.py needs to examine the source text itself. Isn't that information already analyzed someplace?

Copy link
Copy Markdown
Author

Yes, I did not write any test for the multiline purpose yet (I tried to look for similar test case such as --pretty-print to get a bootstrap but couldn't, so far, just a simple cmdline call test)

don't understand why jsonreport.py needs to examine the source text itself. Isn't that information already analyzed someplace?

I don't think I fully understand this. By "examining the source text" you mean the mapping between the line and the parent statement that I did in my pull request?


I wanted this function because I saw a discrepancy between json's report in the "executed lines" and the green lines in html report. Turned out json report only report covered statements, not actual lines (so a covered multiline statement would only have the first of its line reported)

The mappping (multiline map) was directly taken from FileReporter class, which was used in html report to show all the covered lines.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL