Skip to content

Commit b5b1401

Browse files
committed
fix lint & prettifier error
1 parent 667c94c commit b5b1401

File tree

5 files changed

+7
-10
lines changed

5 files changed

+7
-10
lines changed

src/generator/html-generator/genCode.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,7 @@ function generateSource(codes: CodeCoverage[], uncoveredlines: UncoveredLines):
2929
if (uncoveredlines.has(index + 1)) {
3030
// IMPORTANT! to add "nocode" here to preventing prettify from adding unwanted pln class
3131
str.push('<span class="missing-if-branch nocode" title="Branch not taken">!</span>' + escape(code.source));
32-
}
33-
else {
32+
} else {
3433
str.push(escape(code.source));
3534
}
3635
}

src/interface.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ export class FunctionCoverageResult {
115115
];
116116
result.branchCoverageRate = Rate.summarize(infos.map((info) => info.branchCoverageRate));
117117
for (const info of infos) {
118-
info.uncoveredlines.forEach(line => result.uncoveredlines.add(line));
118+
for (const line of info.uncoveredlines) result.uncoveredlines.add(line);
119119
for (const [lineIndex, count] of info.sourceUsedCount.entries()) {
120120
const srcLineUsedCount = result.sourceUsedCount.get(lineIndex);
121121
result.sourceUsedCount.set(lineIndex, srcLineUsedCount === undefined ? count : srcLineUsedCount + count);

src/parser/singleFileAnalysis.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export class SingleFileCoverageAnalysis {
3030
// SingleFileCoverageAnalysis contains FileCoverageResult
3131
if (results.length === 0) return;
3232
for (const functionCovResult of results) {
33-
functionCovResult.uncoveredlines.forEach(line => this.result.uncoveredlines.add(line));
33+
for (const line of functionCovResult.uncoveredlines) this.result.uncoveredlines.add(line);
3434
for (const [lineIndex, count] of functionCovResult.sourceUsedCount.entries()) {
3535
const srcLineUsedCount = this.result.sourceUsedCount[lineIndex - 1];
3636
if (srcLineUsedCount === undefined) {

src/parser/singleFunctionAnalysis.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -76,10 +76,10 @@ export class SingleFunctionCoverageAnalysis {
7676
for (const [currentBasicBlock, branchesForThatBasicBlock] of this.branchGraph) {
7777
let used = 0;
7878
for (const isCovered of branchesForThatBasicBlock.values()) {
79-
if (!isCovered) {
80-
this.notFullyCoveredBasicBlock.add(currentBasicBlock);
81-
} else {
79+
if (isCovered) {
8280
used++;
81+
} else {
82+
this.notFullyCoveredBasicBlock.add(currentBasicBlock);
8383
}
8484
}
8585
this.result.branchCoverageRate.used += used;

tests-ts/test/parser/singleFileAnalysis.test.ts

+1-3
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,7 @@ describe("singleFileAnalysis", () => {
7575

7676
test("setUnTestedFunction error", () => {
7777
const analyzer = new SingleFileCoverageAnalysis("main", source);
78-
expect(() => analyzer.setUnTestedFunction([[30, 31]])).toThrow(
79-
"unknown error: There is no 29 Line in file main"
80-
);
78+
expect(() => analyzer.setUnTestedFunction([[30, 31]])).toThrow("unknown error: There is no 29 Line in file main");
8179
});
8280

8381
test("merge error", () => {

0 commit comments

Comments
 (0)