Arrays of XCResultValueTypes do not parse correctly (#52)

Change Description: This change addresses an issue where arrays (such as [Double] in the performance metrics object) were not properly being decoded. This was due to the need to decode into the intermediate type of XCResultValueType in order to get to the true values underneath.

Test Plan/Testing Performed: Added the xcresult that @OdNairy gave in #51. Added some explicit tests to double-check that the values returned match what we would expect. Also did a quick test if we removed one of the enum cases like "CPU Time" that the metric type enum would properly handle new metric identifier values from new xcresult versions.

Author: abotkin-cpi

Sources/XCParseCore/ActionTestPerformanceMetricSummary.swift

@@ -8,6 +8,40 @@
 
 import Foundation
 
+
+public enum ActionTestPerformanceMetric : ExpressibleByStringLiteral {
+    case ClockMonotonicTime
+    case CPUCycles
+    case CPUInstructionsRetired
+    case CPUTime
+    case DiskLogicalWrites
+    case MemoryPhysical
+    case MemoryPhysicalPeak
+    case Unknown(name: String)
+
+    public init(stringLiteral name: String) {
+        switch name {
+        case "com.apple.dt.XCTMetric_Clock.time.monotonic":
+            self = .ClockMonotonicTime
+        case "com.apple.dt.XCTMetric_CPU.cycles":
+            self = .CPUCycles
+        case "com.apple.dt.XCTMetric_CPU.instructions_retired":
+            self = .CPUInstructionsRetired
+        case "com.apple.dt.XCTMetric_CPU.time":
+            self = .CPUTime
+        case "com.apple.dt.XCTMetric_Disk.logical.writes":
+            self = .DiskLogicalWrites
+        case "com.apple.dt.XCTMetric_Memory.physical":
+            self = .MemoryPhysical
+        case "com.apple.dt.XCTMetric_Memory.physical-peak":
+            self = .MemoryPhysicalPeak
+        default:
+            self = .Unknown(name: name)
+        }
+    }
+}
+
+
 open class ActionTestPerformanceMetricSummary : Codable {
     public let displayName: String
     public let unitOfMeasurement: String
@@ -20,6 +54,12 @@ open class ActionTestPerformanceMetricSummary : Codable {
     public let maxRegression: Double?
     public let maxStandardDeviation: Double?
 
+    // Derived
+    public var metricType : ActionTestPerformanceMetric {
+        let identifierString = identifier ?? "Identifier Missing"
+        return ActionTestPerformanceMetric(stringLiteral: identifierString)
+    }
+
     enum ActionTestPerformanceMetricSummaryCodingKeys: String, CodingKey {
         case displayName
         case unitOfMeasurement

Sources/XCParseCore/XCPResultDecoding.swift

@@ -350,6 +350,11 @@ extension KeyedDecodingContainer {
             let resultObj = try container.decode(XCResultObject.self)
             if let type = resultObj.type.getType() as? T.Type {
                 list.append(try tmpContainer.decode(type))
+            } else if let type = resultObj.type.getType() as? XCResultValueType.Type {
+                let decodedValue = try tmpContainer.decode(type)
+                if let value = decodedValue.getValue() as? T {
+                    list.append(value)
+                }
             }
         }
         return list