[PinsStore] Add support for loading PinsStore schema 1

- <rdar://problem/34186002> [SR-5843]: SwiftPM needs to be resilient to schema changes

Author: aciidgh

Sources/Workspace/PinsStore.swift

@@ -76,6 +76,7 @@ public final class PinsStore {
  self.persistence = SimplePersistence(
  fileSystem: fileSystem,
  schemaVersion: PinsStore.schemaVersion,
+ supportedSchemaVersions: [1],
  statePath: pinsFile,
  prettyPrint: true)
  pinsMap = [:]
@@ -153,6 +154,28 @@ extension PinsStore: SimplePersistanceProtocol {
  self.pinsMap = try Dictionary(items: json.get("pins").map({ ($0.packageRef.identity, $0) }))
  }
 
+ public func restore(from json: JSON, supportedSchemaVersion: Int) throws {
+ switch supportedSchemaVersion {
+ case 1:
+ // We did not have concept of package identity in previous schema but we can
+ // compute it using the URL.
+ //
+ // FIXME: This logic will need to be updated when we require identity and package
+ // name to be same. <rdar://problem/33693433>
+ let pinsArray: [JSON] = try json.get("pins")
+ let pins: [Pin] = try pinsArray.map({ pinData in
+ let url: String = try pinData.get("repositoryURL")
+ let ref = PackageReference(
+ identity: PackageReference.computeIdentity(packageURL: url), path: url)
+ return try Pin(packageRef: ref, state: pinData.get("state"))
+ })
+ self.pinsMap = Dictionary(items: pins.map({ ($0.packageRef.identity, $0) }))
+
+ default:
+ fatalError()
+ }
+ }
+
  /// Saves the current state of pins.
  public func toJSON() -> JSON {
  return JSON([

Tests/WorkspaceTests/PinsStoreTests.swift

@@ -96,7 +96,46 @@ final class PinsStoreTests: XCTestCase {
  }
  }
 
+ func testLoadingSchema1() throws {
+ var fs = InMemoryFileSystem()
+ let pinsFile = AbsolutePath("/pinsfile.txt")
+
+ try fs.writeFileContents(pinsFile) {
+ $0 <<< """
+ {
+ "object": {
+ "pins": [
+ {
+ "package": "Clang_C",
+ "repositoryURL": "https://github.com/something/Clang_C.git",
+ "state": {
+ "branch": null,
+ "revision": "90a9574276f0fd17f02f58979423c3fd4d73b59e",
+ "version": "1.0.2"
+ }
+ },
+ {
+ "package": "Commandant",
+ "repositoryURL": "https://github.com/something/Commandant.git",
+ "state": {
+ "branch": null,
+ "revision": "c281992c31c3f41c48b5036c5a38185eaec32626",
+ "version": "0.12.0"
+ }
+ }
+ ]
+ },
+ "version": 1
+ }
+ """
+ }
+
+ let store = try PinsStore(pinsFile: pinsFile, fileSystem: fs)
+ XCTAssertEqual(store.pinsMap.keys.map{$0}.sorted(), ["clang_c", "commandant"])
+ }
+
  static var allTests = [
  ("testBasics", testBasics),
+ ("testLoadingSchema1", testLoadingSchema1),
  ]
 }