Added possibility to parse detailed report with implicit test ref id (#…

…11)
dodobrands · Dec 15, 2023 · f29382e · f29382e
1 parent 8fcb242
commit f29382e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/Sources/DBXCResultParser/Models/DTO/DTO+Helpers.swift b/Sources/DBXCResultParser/Models/DTO/DTO+Helpers.swift
@@ -18,7 +18,8 @@ extension OverviewReportDTO {
 }
 
 extension DetailedReportDTO {
-    init(from xcresultPath: URL, refId: String) throws {
+    init(from xcresultPath: URL, refId: String? = nil) throws {
+        let refId = try (refId ?? OverviewReportDTO(from: xcresultPath).testsRefId)
         let tempFilePath = try Constants.tempFilePath
         try Shell.execute("xcrun xcresulttool get --path \(xcresultPath.relativePath) --format json --id \(refId) > \(tempFilePath.relativePath)")
         let data = try Data(contentsOf: tempFilePath)

diff --git a/Tests/DBXCResultParserTests/DetailedReportDTOTests.swift b/Tests/DBXCResultParserTests/DetailedReportDTOTests.swift
@@ -19,8 +19,12 @@ class DetailedReportDTOTests: XCTestCase {
         try super.tearDownWithError()
     }
 
-    func test() throws {
+    func test_parseWithExplicitRefId() throws {
         let overviewReport = try OverviewReportDTO(from: Constants.unitTestsReportPath)
         XCTAssertNoThrow(try DetailedReportDTO(from: Constants.unitTestsReportPath, refId: overviewReport.testsRefId))
     }
+
+    func test_parseWithImplicitRefId() throws {
+        XCTAssertNoThrow(try DetailedReportDTO(from: Constants.unitTestsReportPath))
+    }
 }