chore: Run CLI unit tests in CI

.github/workflows/continuous-integration.yml

@@ -15,7 +15,7 @@ jobs:
       matrix:
         # This matrix runs tests on iOS sim & Mac, on oldest & newest supported Xcodes
         runner:
-          - macos-13
+          - macos-14
           - macos-15
         xcode:
           - Xcode_15.2
@@ -30,7 +30,7 @@ jobs:
           - 'platform=macOS'
         exclude:
           # Don't run old macOS with new Xcode
-          - runner: macos-13
+          - runner: macos-14
             xcode: Xcode_16
           # Don't run new macOS with old Xcode
           - runner: macos-15
@@ -101,6 +101,14 @@ jobs:
           java-version: 17
       - name: Tools Versions
         run: ./scripts/ci_steps/log_tool_versions.sh
+      - name: Run CLI Unit Tests
+        if: ${{ matrix.destination == 'platform=macOS' }}
+        run: |
+          cd AWSSDKSwiftCLI
+          swift test
+          cd ../SPRCLI
+          unset AWS_SWIFT_SDK_USE_LOCAL_DEPS
+          swift build
       - name: Prepare Protocol & Unit Tests
         run: |
           ./scripts/ci_steps/prepare_protocol_and_unit_tests.sh
@@ -182,6 +190,13 @@ jobs:
         run: ./scripts/ci_steps/install_native_linux_dependencies.sh
       - name: Tools Versions
         run: ./scripts/ci_steps/log_tool_versions.sh
+      - name: Run CLI Unit Tests
+        run: |
+          cd AWSSDKSwiftCLI
+          swift test
+          cd ../SPRCLI
+          unset AWS_SWIFT_SDK_USE_LOCAL_DEPS
+          swift build
       - name: Prepare Protocol & Unit Tests
         run: |
           ./scripts/ci_steps/prepare_protocol_and_unit_tests.sh

AWSSDKSwiftCLI/Sources/AWSSDKSwiftCLI/Commands/AWSSDKSwiftCLI/Subcommands/PrepareRelease.swift

@@ -235,13 +235,16 @@ struct PrepareRelease {
         previousVersion: Version
     ) throws {
         let commits = try Process.git.listOfCommitsBetween("HEAD", "\(previousVersion)")
-
+        let featuresReader = FeaturesReader(repoPath: repoPath)
+
         let releaseNotes = try ReleaseNotesBuilder(
             previousVersion: previousVersion,
             newVersion: newVersion,
             repoOrg: repoOrg,
             repoType: repoType,
-            commits: commits
+            commits: commits,
+            features: featuresReader.getFeaturesFromFile(),
+            featuresIDToServiceName: featuresReader.getFeaturesIDToServiceNameDictFromFile()
         ).build()
 
         let manifest = ReleaseManifest(

AWSSDKSwiftCLI/Sources/AWSSDKSwiftCLI/Models/Features.swift

@@ -8,25 +8,26 @@
 import Foundation
 import AWSCLIUtils
 
-struct FeaturesReader: Decodable {
-    private let requestFilePath: String
-    private let mappingFilePath: String
+struct FeaturesReader {
+    private let repoPath: String
+    private let requestFile = "build-request.json"
+    private let mappingFile = "feature-service-id.json"
 
     public init(
-        requestFilePath: String = "../build-request.json",
-        mappingFilePath: String = "../feature-service-id.json"
+        repoPath: String = "."
     ) {
-        self.requestFilePath = requestFilePath
-        self.mappingFilePath = mappingFilePath
+        self.repoPath = repoPath
     }
 
     public func getFeaturesFromFile() throws -> Features {
-        let fileContents = try FileManager.default.loadContents(atPath: requestFilePath)
+        let path = URL(fileURLWithPath: repoPath).appendingPathComponent(requestFile, isDirectory: false).path
+        let fileContents = try FileManager.default.loadContents(atPath: path)
         return try JSONDecoder().decode(Features.self, from: fileContents)
     }
 
     public func getFeaturesIDToServiceNameDictFromFile() throws -> [String: String] {
-        let fileContents = try FileManager.default.loadContents(atPath: mappingFilePath)
+        let path = URL(fileURLWithPath: repoPath).appendingPathComponent(mappingFile, isDirectory: false).path
+        let fileContents = try FileManager.default.loadContents(atPath: path)
         return try JSONDecoder().decode([String: String].self, from: fileContents)
     }
 }

AWSSDKSwiftCLI/Sources/AWSSDKSwiftCLI/Models/ReleaseNotesBuilder.swift

@@ -15,7 +15,8 @@ struct ReleaseNotesBuilder {
     let repoOrg: PrepareRelease.Org
     let repoType: PrepareRelease.Repo
     let commits: [String]
-    var featuresReader: FeaturesReader = FeaturesReader()
+    let features: Features
+    let featuresIDToServiceName: [String: String]
 
     // MARK: - Build
 
@@ -41,9 +42,7 @@ struct ReleaseNotesBuilder {
     }
 
     func buildServiceChangeSection() throws -> [String] {
-        let features = try featuresReader.getFeaturesFromFile()
-        let mapping = try featuresReader.getFeaturesIDToServiceNameDictFromFile()
-        return buildServiceFeatureSection(features, mapping) + buildServiceDocSection(features, mapping)
+        return buildServiceFeatureSection(features, featuresIDToServiceName) + buildServiceDocSection(features, featuresIDToServiceName)
     }
 
     private func buildServiceFeatureSection(

AWSSDKSwiftCLI/Tests/AWSSDKSwiftCLITests/Models/ReleaseNotesBuilderTests.swift

@@ -209,10 +209,8 @@ class ReleaseNotesBuilderTests: XCTestCase {
             repoType: .awsSdkSwift,
             commits: testCommits,
             // Parametrize behavior of FeaturesReader with paths used to create JSON test files
-            featuresReader: FeaturesReader(
-                requestFilePath: "build-request.json",
-                mappingFilePath: "feature-service-id.json"
-            )
+            features: FeaturesReader().getFeaturesFromFile(),
+            featuresIDToServiceName: FeaturesReader().getFeaturesIDToServiceNameDictFromFile()
         )
     }
 }

SPRCLI/Package.swift

@@ -9,7 +9,7 @@ let package = Package(
     ],
     dependencies: [
         .package(url: "https://github.com/apple/swift-argument-parser", from: "1.2.0"),
-        .package(url: "https://github.com/awslabs/aws-sdk-swift", from: "0.46.0"),
+        .package(url: "https://github.com/awslabs/aws-sdk-swift", from: "1.0.0"),
         .package(path: "../AWSSDKSwiftCLI"),
     ],
     targets: [

SPRCLI/Sources/SPR/Process+SPR.swift

@@ -6,7 +6,6 @@
 //
 
 import Foundation
-import PackageDescription
 import AWSCLIUtils
 
 extension Process {

SPRCLI/Sources/SPR/SPRPublisher.swift

@@ -65,7 +65,7 @@ public struct SPRPublisher {
     }
 
     private mutating func setOptions() async {
-        await SDKLoggingSystem.initialize(logLevel: .error)
+        await SDKLoggingSystem().initialize(logLevel: .error)
         let env = ProcessInfo.processInfo.environment
         bucket = bucket ?? env["AWS_SDK_SPR_BUCKET"]
         if region.isEmpty {

SPRCLI/Sources/SPR/UpdateList.swift

@@ -63,9 +63,9 @@ extension SPRPublisher {
                 throw Error("URL is invalid")
             }
             return baseURL
-                .appending(component: scope)
-                .appending(component: name)
-                .appending(component: version)
+                .appendingPathComponent(scope)
+                .appendingPathComponent(name)
+                .appendingPathComponent(version)
         }
     }
 }

SPRCLI/Sources/SPR/UploadArchive.swift

@@ -15,7 +15,7 @@ extension SPRPublisher {
 
     mutating func uploadArchive() async throws {
         let tmpDirFileURL = FileManager.default.temporaryDirectory
-        let archiveFileURL = tmpDirFileURL.appending(component: "\(UUID().uuidString).zip")
+        let archiveFileURL = tmpDirFileURL.appendingPathComponent("\(UUID().uuidString).zip")
         let archiveProcess = Process.SPR.archive(name: name, packagePath: path, archiveFileURL: archiveFileURL)
         _ = try _runReturningStdOut(archiveProcess)
         guard FileManager.default.fileExists(atPath: urlPath(archiveFileURL)) else {

SPRCLI/Sources/SPR/UploadManifest.swift

@@ -15,7 +15,7 @@ extension SPRPublisher {
 
     func uploadManifest() async throws {
         let packageFileURL = URL(fileURLWithPath: path).standardizedFileURL
-        let manifestFileURL = packageFileURL.appending(component: "Package.swift")
+        let manifestFileURL = packageFileURL.appendingPathComponent("Package.swift")
         let s3Client = try S3Client(region: region)
         try await verify(s3Client: s3Client)
         try await upload(s3Client: s3Client, manifestFileURL: manifestFileURL)

SPRCLI/Sources/SPR/UploadMetadata.swift

@@ -47,7 +47,8 @@ extension SPRPublisher {
     }
 
     private func createMetadata() -> PackageInfo {
-        let now = Date().ISO8601Format()
+        let formatter = ISO8601DateFormatter()
+        let now = formatter.string(from: Date())
         let organization = PackageInfo.Metadata.Author.Organization(name: "Amazon Web Services", email: nil, description: nil, url: URL(string: "https://aws.amazon.com/")!)
         let author = PackageInfo.Metadata.Author(name: "AWS SDK for Swift Team", email: nil, description: nil, organization: organization, url: nil)
         let resource = Resource(name: "source-archive", type: "application/zip", checksum: checksum, signing: nil)