Open Bug 1640467 Opened 4 years ago Updated 2 years ago

[Automated review] Code coverage message is missing the paths of the uncovered files

Categories

(Developer Infrastructure :: Source Code Analysis, defect, P3)

Tracking

(Not tracked)

People

(Reporter: mccr8, Unassigned)

References

Details

Phabricator URL: https://phabricator.services.mozilla.com/D76598

The message posted in Phabricator tells me that there's 1 defect, but if I click through on the link to the Mozilla Code Review bot there's almost 200 issues. I was only able to find what it was talking about because I already knew. Maybe the message could say to search for no-coverage or something. Assuming that's what the 1 defect is. I'm not sure. (It is also almost impossible to actually view the page because it is 2 or 3 times wider than my screen, and there's no horizontal scrolling bar.)

The severity field is not set for this bug.
:andi, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(bpostelnicu)

marco any thoughts here?

Flags: needinfo?(bpostelnicu) → needinfo?(mcastelluccio)

There seems to be a bug in this specific case.
Usually, the reviewbot comment contains the actual paths (listed after we found some files which had no coverage and are being modified in this patch:), and there should be a reviewbot comment for each found issue.

Flags: needinfo?(mcastelluccio)
Summary: [Automated review] Code coverage message not specific enough → [Automated review] Code coverage message is missing the paths of the uncovered files
Priority: -- → P3
Product: Firefox Build System → Developer Infrastructure
Severity: -- → S3
You need to log in before you can comment on or make changes to this bug.