Add an explanatory comment for making ranges unique

This commit is contained in:
Leonid Logvinov 2018-03-12 12:56:22 +01:00
parent e474096119
commit 20826e0f08
No known key found for this signature in database
GPG Key ID: 0DD294BFDE8C95D4

View File

@ -42,6 +42,7 @@ export class CoverageManager {
const coverageEntriesDescription = collectCoverageEntries(contractData.sourceCodes[fileIndex], fileName);
let sourceRanges = _.map(coveredPcs, coveredPc => pcToSourceRange[coveredPc]);
sourceRanges = _.compact(sourceRanges); // Some PC's don't map to a source range and we just ignore them.
// By default lodash does a shallow object comparasion. We JSON.stringify them and compare as strings.
sourceRanges = _.uniqBy(sourceRanges, s => JSON.stringify(s)); // We don't care if one PC was covered multiple times within a single transaction
sourceRanges = _.filter(sourceRanges, sourceRange => sourceRange.fileName === fileName);
const branchCoverage: BranchCoverage = {};