Skip to content

Commit 349c772

Browse files
committed
fix(plugin-coverage): calculate coverage for no found entities as full coverage
1 parent 238ed3b commit 349c772

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/plugin-coverage/src/lib/runner/lcov/utils.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { NumberRange } from './types';
77
* @returns coverage between 0 and 1
88
*/
99
export function calculateCoverage(hit: number, found: number): number {
10-
return found > 0 ? hit / found : 0;
10+
return found > 0 ? hit / found : 1;
1111
}
1212

1313
export function mergeConsecutiveNumbers(numberArr: number[]): NumberRange[] {

packages/plugin-coverage/src/lib/runner/lcov/utils.unit.test.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ describe('calculateCoverage', () => {
1010
expect(calculateCoverage(0, 25)).toBe(0);
1111
});
1212

13-
it('should calculate zero coverage when no entity was covered or found', () => {
14-
expect(calculateCoverage(0, 0)).toBe(0);
13+
it('should assign full coverage when no entity was found', () => {
14+
expect(calculateCoverage(0, 0)).toBe(1);
1515
});
1616
});
1717

0 commit comments

Comments
 (0)