[coverage] Simplify cobertura source paths for reporting.

Review Request #2918 — Created Oct. 1, 2015 and submitted — Latest diff uploaded

jtrobec
pants
jtrobec/coverage/cobertura-src-paths
2300
a405d3e...
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.

Loading...