Skip to content

Commit ba28cd1

Browse files
committed
Slight refactor
1 parent dc838a6 commit ba28cd1

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

Source/SwiftLintFramework/Coverage.swift

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,16 +51,21 @@ struct Coverage {
5151
observedCoverage += coverage.observedCoverage
5252
maximumCoverage += coverage.maximumCoverage
5353
}
54+
55+
func coverage(denominator: Int? = nil) -> Double {
56+
let denominator = denominator ?? maximumCoverage
57+
return denominator == 0 ? 0.0 : (Double(observedCoverage) / Double(denominator))
58+
}
5459
}
5560

5661
private let totalNumberOfRules: Int
5762
var coverage = Self.Coverage()
5863

5964
var enabledRulesCoverage: Double {
60-
coverage(denominator: coverage.maximumCoverage)
65+
coverage.coverage()
6166
}
6267
var allRulesCoverage: Double {
63-
coverage(denominator: coverage.numberOfLinesOfCode * totalNumberOfRules)
68+
coverage.coverage(denominator: coverage.numberOfLinesOfCode * totalNumberOfRules)
6469
}
6570
var report: String {
6671
"""
@@ -76,10 +81,6 @@ struct Coverage {
7681
mutating func addCoverage(for linter: CollectedLinter) {
7782
coverage.add(linter.file.coverage(for: linter.rules))
7883
}
79-
80-
private func coverage(denominator: Int) -> Double {
81-
denominator == 0 ? 0.0 : (Double(coverage.observedCoverage) / Double(denominator))
82-
}
8384
}
8485

8586
private extension SwiftLintFile {

0 commit comments

Comments
 (0)