Page MenuHomePhabricator

Distinguish between "no coverage information" and "no test coverage" better
ClosedPublic

Authored by epriestley on Jan 18 2016, 3:11 PM.
Tags
None
Referenced Files
F15375499: D15044.id.diff
Wed, Mar 12, 11:33 PM
F15375498: D15044.diff
Wed, Mar 12, 11:32 PM
F15367156: D15044.id36336.diff
Tue, Mar 11, 4:22 PM
F15335172: D15044.id36336.diff
Sat, Mar 8, 2:38 PM
F15335171: D15044.id.diff
Sat, Mar 8, 2:38 PM
F15335170: D15044.diff
Sat, Mar 8, 2:38 PM
Unknown Object (File)
Feb 9 2025, 12:30 AM
Unknown Object (File)
Feb 9 2025, 12:30 AM
Subscribers
None

Details

Summary

Fixes T10169. Diffs with no build targets were incorrectly showing as though they had no test coverage, when we actually want to show them having no coverage information available.

Test Plan

Viewed an older revision, saw a column of "Not Executable" before change, now see no column.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Distinguish between "no coverage information" and "no test coverage" better.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 18 2016, 3:50 PM
This revision was automatically updated to reflect the committed changes.