Change authTokens to be a function instead

Author: yim-lee

Sources/PackageCollections/PackageCollections+Configuration.swift

@@ -1,7 +1,7 @@
 /*
  This source file is part of the Swift.org open source project
 
- Copyright (c) 2020 Apple Inc. and the Swift project authors
+ Copyright (c) 2020-2021 Apple Inc. and the Swift project authors
  Licensed under Apache License v2.0 with Runtime Library Exception
 
  See http://swift.org/LICENSE.txt for license information
@@ -16,9 +16,9 @@ extension PackageCollections {
         // JSONPackageCollectionValidator: maximumPackageCount, maximumMajorVersionCount, maximumMinorVersionCount
 
         /// Auth tokens for the collections or metadata provider
-        public var authTokens: [AuthTokenType: String]?
+        public var authTokens: () -> [AuthTokenType: String]?
 
-        public init(authTokens: [AuthTokenType: String]? = nil) {
+        public init(authTokens: @escaping () -> [AuthTokenType: String]? = { nil }) {
             self.authTokens = authTokens
         }
     }

Sources/PackageCollections/PackageCollections.swift

@@ -21,11 +21,11 @@ public struct PackageCollections: PackageCollectionsProtocol {
     static let isSupportedPlatform = false
     #endif
 
-    var configuration: Configuration
+    let configuration: Configuration
     private let diagnosticsEngine: DiagnosticsEngine?
     private let storageContainer: (storage: Storage, owned: Bool)
     private let collectionProviders: [Model.CollectionSourceType: PackageCollectionProvider]
-    var metadataProvider: PackageMetadataProvider
+    let metadataProvider: PackageMetadataProvider
 
     private var storage: Storage {
         self.storageContainer.storage
@@ -67,14 +67,6 @@ public struct PackageCollections: PackageCollectionsProtocol {
         }
     }
 
-    public mutating func updateAuthTokens(_ authTokens: [AuthTokenType: String]?) {
-        self.configuration.authTokens = authTokens
-        if var metadataProvider = self.metadataProvider as? GitHubPackageMetadataProvider {
-            metadataProvider.configuration.authTokens = authTokens
-            self.metadataProvider = metadataProvider
-        }
-    }
-
     // MARK: - Collections
 
     public func listCollections(identifiers: Set<PackageCollectionsModel.CollectionIdentifier>? = nil,

Sources/PackageCollections/Providers/GitHubPackageMetadataProvider.swift

@@ -194,7 +194,7 @@ struct GitHubPackageMetadataProvider: PackageMetadataProvider {
         options.validResponseCodes = validResponseCodes
         options.authorizationProvider = { url in
             url.host.flatMap { host in
-                self.configuration.authTokens?[.github(host)].flatMap { token in
+                self.configuration.authTokens()?[.github(host)].flatMap { token in
                     "token \(token)"
                 }
             }
@@ -213,11 +213,11 @@ struct GitHubPackageMetadataProvider: PackageMetadataProvider {
 
     public struct Configuration {
         public var apiLimitWarningThreshold: Int
-        public var authTokens: [AuthTokenType: String]?
+        public var authTokens: () -> [AuthTokenType: String]?
         public var cacheTTLInSeconds: Int
         public var cacheSize: Int
 
-        public init(authTokens: [AuthTokenType: String]? = nil,
+        public init(authTokens: @escaping () -> [AuthTokenType: String]? = { nil },
                     apiLimitWarningThreshold: Int? = nil,
                     cacheTTLInSeconds: Int? = nil,
                     cacheSize: Int? = nil) {

Tests/PackageCollectionsTests/GitHubPackageMetadataProviderTests.swift

@@ -200,7 +200,7 @@ class GitHubPackageMetadataProviderTests: XCTestCase {
         httpClient.configuration.circuitBreakerStrategy = .none
         httpClient.configuration.retryStrategy = .none
         var provider = GitHubPackageMetadataProvider(httpClient: httpClient)
-        provider.configuration.authTokens = authTokens
+        provider.configuration.authTokens = { authTokens }
         let reference = PackageReference(repository: RepositorySpecifier(url: repoURL))
         XCTAssertThrowsError(try tsc_await { callback in provider.get(reference, callback: callback) }, "should throw error") { error in
             XCTAssertEqual(error as? GitHubPackageMetadataProvider.Errors, .invalidAuthToken(apiURL))
@@ -290,7 +290,7 @@ class GitHubPackageMetadataProviderTests: XCTestCase {
         httpClient.configuration.requestHeaders!.add(name: "Cache-Control", value: "no-cache")
         var configuration = GitHubPackageMetadataProvider.Configuration()
         if let token = ProcessEnv.vars["GITHUB_API_TOKEN"] {
-            configuration.authTokens = [.github("api.github.com"): token]
+            configuration.authTokens = { [.github("api.github.com"): token] }
         }
         configuration.apiLimitWarningThreshold = 50
         let provider = GitHubPackageMetadataProvider(configuration: configuration, httpClient: httpClient)

Tests/PackageCollectionsTests/PackageCollectionsTests.swift

@@ -19,31 +19,35 @@ import TSCUtility
 
 final class PackageCollectionsTests: XCTestCase {
     func testUpdateAuthTokens() throws {
-        let configuration = PackageCollections.Configuration()
+        var authTokens: [AuthTokenType: String]? = [:]
+
+        let configuration = PackageCollections.Configuration(authTokens: { authTokens })
         let storage = makeMockStorage()
         defer { XCTAssertNoThrow(try storage.close()) }
 
-        let metadataProviderConfig = GitHubPackageMetadataProvider.Configuration(cacheTTLInSeconds: -1)
+        // disable cache for this test to avoid setup/cleanup
+        let metadataProviderConfig = GitHubPackageMetadataProvider.Configuration(authTokens: configuration.authTokens, cacheTTLInSeconds: -1)
         let metadataProvider = GitHubPackageMetadataProvider(configuration: metadataProviderConfig)
-        var packageCollections = PackageCollections(configuration: configuration, storage: storage, collectionProviders: [:], metadataProvider: metadataProvider)
+        let packageCollections = PackageCollections(configuration: configuration, storage: storage, collectionProviders: [:], metadataProvider: metadataProvider)
 
-        XCTAssertNil(packageCollections.configuration.authTokens)
+        XCTAssertEqual(0, packageCollections.configuration.authTokens()?.count)
         do {
             guard let githubMetadataProvider = packageCollections.metadataProvider as? GitHubPackageMetadataProvider else {
                 return XCTFail("Expected GitHubPackageMetadataProvider")
             }
-            XCTAssertNil(githubMetadataProvider.configuration.authTokens)
+            XCTAssertEqual(0, githubMetadataProvider.configuration.authTokens()?.count)
         }
 
-        let authTokens: [AuthTokenType: String] = [.github("github.test"): "topsekret"]
-        packageCollections.updateAuthTokens(authTokens)
+        authTokens![.github("github.test")] = "topsekret"
 
-        XCTAssertEqual(authTokens, packageCollections.configuration.authTokens)
+        // Check that authTokens change is propagated to PackageMetadataProvider
+        XCTAssertEqual(1, packageCollections.configuration.authTokens()?.count)
         do {
             guard let githubMetadataProvider = packageCollections.metadataProvider as? GitHubPackageMetadataProvider else {
                 return XCTFail("Expected GitHubPackageMetadataProvider")
             }
-            XCTAssertEqual(authTokens, githubMetadataProvider.configuration.authTokens)
+            XCTAssertEqual(1, githubMetadataProvider.configuration.authTokens()?.count)
+            XCTAssertEqual(authTokens, githubMetadataProvider.configuration.authTokens())
         }
     }