[coverage] Simplify cobertura source paths for reporting.

Review Request #2918 - Created Oct. 2, 2015 and submitted

Information
Justin Trobec
pants
jtrobec/coverage/cobertura-src-paths
2300
a405d3e...
Reviewers
pants-reviews
benjyw, stuhood

The existing approach to supplying cobertura reporting with access to coverage target source is somewhat complicated, largely with the intention of fooling cobertura into finding scala classes. Cobertura is not really designed to cover scala in the first place, so this seems fairly pointless. Removing that in favor of a much simpler approach that just passes the traget_base of relevant targets.

Manually tested on a couple of targets, and added validation to an existing integration test that looks for the annotated source file html and verifies it contains some src.

Stu Hood
Stu Hood
Justin Trobec
Review request changed

Status: Closed (submitted)

Change Summary:

Merged as 8671207c2df7e4812c1681c5df68c27f6d935082

Loading...