PackageModelTests: add case for destination v2

Author: MaxDesiatov

Sources/PackageModel/Destination.swift

@@ -12,7 +12,6 @@
 
 import Basics
 import Foundation
-import SystemPackage
 import TSCBasic
 
 import struct TSCUtility.Triple
@@ -285,11 +284,11 @@ extension Destination {
             try self.init(
                 hostTriple: Triple(destination.hostTriple),
                 destinationTriple: Triple(destination.destinationTriple),
-                sdkRootDir: AbsolutePath(validating: destination.sdkDir, relativeTo: destinationDirectory),
+                sdkRootDir: AbsolutePath(validating: destination.sdkRootDir, relativeTo: destinationDirectory),
                 toolchainBinDir: AbsolutePath(validating: destination.toolchainBinDir, relativeTo: destinationDirectory),
                 extraFlags: .init(
                     cCompilerFlags: destination.extraCCFlags,
-                    cxxCompilerFlags: destination.extraCPPFlags,
+                    cxxCompilerFlags: destination.extraCXXFlags,
                     swiftCompilerFlags: destination.extraSwiftCFlags,
                     linkerFlags: destination.extraLinkerFlags
                 )
@@ -323,12 +322,12 @@ fileprivate struct DestinationInfoV1: Codable {
 }
 
 fileprivate struct DestinationInfoV2: Codable {
-    let sdkDir: String
+    let sdkRootDir: String
     let toolchainBinDir: String
     let hostTriple: String
     let destinationTriple: String
     let extraCCFlags: [String]
     let extraSwiftCFlags: [String]
-    let extraCPPFlags: [String]
+    let extraCXXFlags: [String]
     let extraLinkerFlags: [String]
 }

Tests/PackageModelTests/DestinationTests.swift

@@ -19,6 +19,7 @@ import XCTest
 private let bundleRootPath = try! AbsolutePath(validating: "/tmp/cross-toolchain")
 private let toolchainBinDir = bundleRootPath.appending(.init("swift.xctoolchain/usr/bin"))
 private let sdkRootDir = bundleRootPath.appending(.init("ubuntu-jammy.sdk"))
+private let hostTriple = "arm64-apple-darwin22.1.0"
 private let destinationTriple = "x86_64-unknown-linux-gnu"
 private let extraFlags = BuildFlags(
     cCompilerFlags: ["-fintegrated-as"],
@@ -40,11 +41,29 @@ private let destinationV1JSON =
     }
     """#
 
-class DestinationTests: XCTestCase {
+private let destinationV2JSON =
+    #"""
+    {
+        "version": 2,
+        "sdkRootDir": "\#(sdkRootDir)",
+        "toolchainBinDir": "\#(toolchainBinDir)",
+        "hostTriple": "\#(hostTriple)",
+        "destinationTriple": "\#(destinationTriple)",
+        "extraCCFlags": \#(extraFlags.cCompilerFlags),
+        "extraSwiftCFlags": \#(extraFlags.swiftCompilerFlags),
+        "extraCXXFlags": \#(extraFlags.cxxCompilerFlags),
+        "extraLinkerFlags": \#(extraFlags.linkerFlags)
+    }
+    """#
+
+final class DestinationTests: XCTestCase {
     func testDestinationCodable() throws {
-        let fs = InMemoryFileSystem(files: ["/sdk/destination.json": ByteString(encodingAsUTF8: destinationV1JSON)])
+        let fs = InMemoryFileSystem(files: [
+            "/sdk/destinationV1.json": ByteString(encodingAsUTF8: destinationV1JSON),
+            "/sdk/destinationV2.json": ByteString(encodingAsUTF8: destinationV2JSON),
+        ])
 
-        let destinationV1 = try Destination(fromFile: AbsolutePath(validating: "/sdk/destination.json"), fileSystem: fs)
+        let destinationV1 = try Destination(fromFile: AbsolutePath(validating: "/sdk/destinationV1.json"), fileSystem: fs)
 
         var flagsWithoutLinkerFlags = extraFlags
         flagsWithoutLinkerFlags.linkerFlags = []
@@ -58,5 +77,18 @@ class DestinationTests: XCTestCase {
                 extraFlags: flagsWithoutLinkerFlags
             )
         )
+
+        let destinationV2 = try Destination(fromFile: AbsolutePath(validating: "/sdk/destinationV2.json"), fileSystem: fs)
+
+        XCTAssertEqual(
+            destinationV2,
+            Destination(
+                hostTriple: try Triple(hostTriple),
+                destinationTriple: try Triple(destinationTriple),
+                sdkRootDir: sdkRootDir,
+                toolchainBinDir: toolchainBinDir,
+                extraFlags: extraFlags
+            )
+        )
     }
 }