diff --git a/Sources/DangerDependencies/Fake.swift b/Sources/DangerDependencies/Fake.swift index e69de29..8b13789 100644 --- a/Sources/DangerDependencies/Fake.swift +++ b/Sources/DangerDependencies/Fake.swift @@ -0,0 +1 @@ + diff --git a/Sources/DangerSwiftCoverage/XcodeBuild/XcodeBuildCoverageParser.swift b/Sources/DangerSwiftCoverage/XcodeBuild/XcodeBuildCoverageParser.swift index 74829d5..e27a278 100644 --- a/Sources/DangerSwiftCoverage/XcodeBuild/XcodeBuildCoverageParser.swift +++ b/Sources/DangerSwiftCoverage/XcodeBuild/XcodeBuildCoverageParser.swift @@ -16,7 +16,9 @@ enum XcodeBuildCoverageParser: XcodeBuildCoverageParsing { var coverage = try JSONDecoder().decode(XcodeBuildCoverage.self, from: data) coverage = coverage.filteringTargets(notOn: files, excludedTargets: excludedTargets) - return Report(messages: ["Project coverage: \(coverage.percentageCoverage.description)%"], - sections: coverage.targets.map { ReportSection(fromTarget: $0) }) + let targets = coverage.targets.map { ReportSection(fromTarget: $0) } + let messages = !targets.isEmpty ? ["Project coverage: \(coverage.percentageCoverage.description)%"] : [] + + return Report(messages: messages, sections: targets) } } diff --git a/Tests/DangerSwiftCoverageTests/XcodeBuildCoverageParserTests.swift b/Tests/DangerSwiftCoverageTests/XcodeBuildCoverageParserTests.swift index fcaf57a..e620bbb 100644 --- a/Tests/DangerSwiftCoverageTests/XcodeBuildCoverageParserTests.swift +++ b/Tests/DangerSwiftCoverageTests/XcodeBuildCoverageParserTests.swift @@ -83,7 +83,8 @@ final class XcodeBuildCoverageParserTests: XCTestCase { let result = try! XcodeBuildCoverageParser.coverage(xcresultBundlePath: "derived", files: files, excludedTargets: [], coverageFileFinder: FakeXcodeCoverageFileFinder.self, xcCovParser: MockedXcCovJSONParser.self) - XCTAssertEqual(result.messages, ["Project coverage: 50.09%"]) + XCTAssertEqual(result.messages, []) + XCTAssertEqual(result.sections.count, 0) } }