Implement sync testflight config command

.gitignore

@@ -30,4 +30,4 @@ Temporary Items
 /Packages
 /*.xcodeproj
 xcuserdata/
-config/auth.yml
+/config

Sources/AppStoreConnectCLI/Commands/TestFlight/Sync/TestFlightPullCommand.swift

@@ -0,0 +1,30 @@
+// Copyright 2020 Itty Bitty Apps Pty Ltd
+
+import ArgumentParser
+import Foundation
+import FileSystem
+
+struct TestFlightPullCommand: CommonParsableCommand {
+
+    static var configuration = CommandConfiguration(
+        commandName: "pull",
+        abstract: "Pull down existing TestFlight configuration, refreshing local configuration files."
+    )
+
+    @OptionGroup()
+    var common: CommonOptions
+
+    @Option(
+        default: "./config/apps",
+        help: "Path to the folder containing the TestFlight configuration."
+    ) var outputPath: String
+
+    func run() throws {
+        let service = try makeService()
+
+        let configs = try service.pullTestFlightConfigurations()
+
+        try configs.save(in: outputPath)
+    }
+
+}

Sources/AppStoreConnectCLI/Commands/TestFlight/Sync/TestFlightPushCommand.swift

@@ -0,0 +1,238 @@
+// Copyright 2020 Itty Bitty Apps Pty Ltd
+
+import ArgumentParser
+import FileSystem
+import Foundation
+import Model
+
+struct TestFlightPushCommand: CommonParsableCommand {
+
+    static var configuration =  CommandConfiguration(
+        commandName: "push",
+        abstract: "Push the local configuration to TestFlight."
+    )
+
+    @OptionGroup()
+    var common: CommonOptions
+
+    @Option(
+        default: "./config/apps",
+        help: "Path to the folder containing the TestFlight configuration."
+    )
+    var inputPath: String
+
+    @Flag(help: "Perform a dry run.")
+    var dryRun: Bool
+
+    func run() throws {
+        let service = try makeService()
+
+        print("Loading local TestFlight configs... \n")
+
+        let localConfigurations = try [TestFlightConfiguration](from: inputPath)
+
+        print("Loading server TestFlight configs... \n")
+        let serverConfigurations = try service.pullTestFlightConfigurations()
+
+        let actions = compare(
+            serverConfigurations: serverConfigurations,
+            with: localConfigurations
+        )
+
+        if dryRun {
+            render(actions: actions)
+        } else {
+            try process(actions: actions, with: service)
+        }
+
+        print("Refreshing local configurations...")
+
+        try service.pullTestFlightConfigurations().save(in: inputPath)
+
+        print("Refreshing completed.")
+    }
+
+    func render(actions: [AppSyncActions]) {
+        print("'Dry Run' mode activated, changes will not be applied. \n")
+
+        actions.forEach {
+            print("\($0.app.name ?? ""):")
+            // 1. app testers
+            print("- Testers in App: ")
+            $0.appTestersSyncActions.forEach { $0.render(dryRun: dryRun) }
+
+            // 2. BetaGroups in App
+            print("- BetaGroups in App: ")
+            $0.betaGroupSyncActions.forEach { $0.render(dryRun: dryRun) }
+
+            // 3. Testers in BetaGroup
+            print("- Testers In Beta Group: ")
+            $0.testerInGroupsAction.forEach {
+                print("\($0.betaGroup.groupName):")
+                $0.testerActions.forEach { $0.render(dryRun: dryRun) }
+            }
+        }
+    }
+
+    private func process(actions: [AppSyncActions], with service: AppStoreConnectService) throws {
+        try actions.forEach { appAction in
+            print("\(appAction.app.name ?? ""): ")
+            // 1. app testers
+            print("- Testers in App: ")
+            try processAppTesterActions(
+                appAction.appTestersSyncActions,
+                appId: appAction.app.id,
+                service: service
+            )
+
+            // 2. beta groups in app
+            print("- BetaGroups in App: ")
+            try processBetagroupsActions(
+                appAction.betaGroupSyncActions,
+                appId: appAction.app.id,
+                service: service
+            )
+
+            // 3. testers in beta group
+            print("- Testers In Beta Group: ")
+            try appAction.testerInGroupsAction.forEach {
+                try processTestersInBetaGroupActions(
+                    $0.testerActions,
+                    betagroupId: $0.betaGroup.id!,
+                    appTesters: appAction.appTesters,
+                    service: service
+                )
+            }
+        }
+    }
+
+    private func compare(
+        serverConfigurations: [TestFlightConfiguration],
+        with localConfigurations: [TestFlightConfiguration]
+    ) -> [AppSyncActions] {
+        return serverConfigurations.compactMap { serverConfiguration in
+            guard
+                let localConfiguration = localConfigurations
+                    .first(where: { $0.app.id == serverConfiguration.app.id }) else {
+                return nil
+            }
+
+            let appTesterSyncActions = SyncResourceComparator(
+                localResources: localConfiguration.testers,
+                serverResources: serverConfiguration.testers
+            )
+            .compare()
+
+            let betaGroupSyncActions = SyncResourceComparator(
+                localResources: localConfiguration.betagroups,
+                serverResources: serverConfiguration.betagroups
+            )
+            .compare()
+
+            let testerInGroupsAction = localConfiguration.betagroups.compactMap { localBetagroup -> BetaTestersInGroupActions?  in
+                guard
+                    let serverBetaGroup = serverConfiguration
+                        .betagroups
+                        .first(where: { $0.id == localBetagroup.id }) else {
+                    return nil
+                }
+
+                return BetaTestersInGroupActions(
+                    betaGroup: localBetagroup,
+                    testerActions: SyncResourceComparator(
+                        localResources: localBetagroup.testers,
+                        serverResources: serverBetaGroup.testers
+                    )
+                    .compare()
+                )
+            }
+
+            return AppSyncActions(
+                app: localConfiguration.app,
+                appTesters: localConfiguration.testers,
+                appTestersSyncActions: appTesterSyncActions,
+                betaGroupSyncActions: betaGroupSyncActions,
+                testerInGroupsAction: testerInGroupsAction
+            )
+        }
+    }
+
+    func processAppTesterActions(_ actions: [SyncAction<FileSystem.BetaTester>], appId: String, service: AppStoreConnectService) throws {
+        try actions.forEach { action in
+            switch action {
+            case .delete(let betatester):
+                try service.removeTesterFromApp(testerEmail: betatester.email, appId: appId)
+                action.render(dryRun: dryRun)
+            default:
+                return
+            }
+        }
+    }
+
+    func processBetagroupsActions(_ actions: [SyncAction<FileSystem.BetaGroup>], appId: String, service: AppStoreConnectService) throws {
+        try actions.forEach { action in
+            switch action {
+            case .create(let betagroup):
+                _ = try service.createBetaGroup(
+                    appId: appId,
+                    groupName: betagroup.groupName,
+                    publicLinkEnabled: betagroup.publicLinkEnabled ?? false,
+                    publicLinkLimit: betagroup.publicLinkLimit
+                )
+                action.render(dryRun: dryRun)
+            case .delete(let betagroup):
+                try service.deleteBetaGroup(with: betagroup.id!)
+                action.render(dryRun: dryRun)
+            case .update(let betagroup):
+                try service.updateBetaGroup(betaGroup: betagroup)
+                action.render(dryRun: dryRun)
+            }
+        }
+    }
+
+    func processTestersInBetaGroupActions(
+        _ actions: [SyncAction<FileSystem.BetaGroup.EmailAddress>],
+        betagroupId: String,
+        appTesters: [FileSystem.BetaTester],
+        service: AppStoreConnectService
+    ) throws {
+        let deletingEmailsWithStrategy = actions
+            .compactMap { (action: SyncAction<FileSystem.BetaGroup.EmailAddress>) ->
+                (email: String, strategy: SyncAction<FileSystem.BetaGroup.EmailAddress>)? in
+                if case .delete(let email) = action {
+                    return (email, action)
+                }
+                return nil
+            }
+
+        try service.removeTestersFromGroup(
+            emails: deletingEmailsWithStrategy.map { $0.email },
+            groupId: betagroupId
+        )
+
+        deletingEmailsWithStrategy.forEach { $0.strategy.render(dryRun: dryRun) }
+
+        let creatingTestersWithStrategy = actions
+            .compactMap { (strategy: SyncAction<FileSystem.BetaGroup.EmailAddress>) ->
+                (tester: FileSystem.BetaTester, strategy: SyncAction<FileSystem.BetaGroup.EmailAddress>)? in
+                if case .create(let email) = strategy,
+                   let betatester = appTesters.first(where: { $0.email == email }) {
+                    return (betatester, strategy)
+                }
+                return nil
+            }
+
+            try creatingTestersWithStrategy.forEach {
+                try service.inviteBetaTesterToGroups(
+                    email: $0.tester.email,
+                    groupId: betagroupId,
+                    firstName: $0.tester.firstName,
+                    lastName: $0.tester.lastName
+                )
+
+            $0.strategy.render(dryRun: dryRun)
+        }
+
+    }
+
+}

Sources/AppStoreConnectCLI/Commands/TestFlight/Sync/TestFlightSyncCommand.swift

@@ -0,0 +1,14 @@
+// Copyright 2020 Itty Bitty Apps Pty Ltd
+
+import ArgumentParser
+
+struct TestFlightSyncCommand: ParsableCommand {
+    static var configuration = CommandConfiguration(
+        commandName: "sync",
+        abstract: "Sync information about testflight with provided configuration file.",
+        subcommands: [
+            TestFlightPullCommand.self,
+            TestFlightPushCommand.self,
+        ]
+    )
+}

Sources/AppStoreConnectCLI/Commands/TestFlight/TestFlightCommand.swift

@@ -13,6 +13,7 @@ public struct TestFlightCommand: ParsableCommand {
             TestFlightBetaTestersCommand.self,
             TestFlightBuildsCommand.self,
             TestFlightPreReleaseVersionCommand.self,
+            TestFlightSyncCommand.self,
         ])
 
     public init() {

Sources/AppStoreConnectCLI/Model/API/Sales.Filter+ExpressibleByArgument.swift

@@ -9,7 +9,7 @@ private typealias Filter = DownloadSalesAndTrendsReports.Filter
 extension Filter.Frequency: ExpressibleByArgument, CustomStringConvertible {
     private typealias AllCases = [Filter.Frequency]
 
-    public static var allCases: AllCases {
+    private static var allCases: AllCases {
         [.DAILY, .MONTHLY, .WEEKLY, .YEARLY]
     }
 
@@ -25,7 +25,7 @@ extension Filter.Frequency: ExpressibleByArgument, CustomStringConvertible {
 extension Filter.ReportType: ExpressibleByArgument, CustomStringConvertible {
     private typealias AllCases = [Filter.ReportType]
 
-    public static var allCases: AllCases {
+    private static var allCases: AllCases {
         [.SALES, .PRE_ORDER, .NEWSSTAND, .SUBSCRIPTION, .SUBSCRIPTION_EVENT, .SUBSCRIBER]
     }
 
@@ -41,7 +41,7 @@ extension Filter.ReportType: ExpressibleByArgument, CustomStringConvertible {
 extension Filter.ReportSubType: ExpressibleByArgument, CustomStringConvertible {
     private typealias AllCases = [Filter.ReportSubType]
 
-    public static var allCases: AllCases {
+    private static var allCases: AllCases {
         [.SUMMARY, .DETAILED, .OPT_IN]
     }
 

Sources/AppStoreConnectCLI/Model/BetaGroup.swift

@@ -3,6 +3,7 @@
 import AppStoreConnect_Swift_SDK
 import Combine
 import Foundation
+import FileSystem
 import struct Model.App
 import struct Model.BetaGroup
 import SwiftyTextTable
@@ -57,3 +58,15 @@ extension BetaGroup {
         )
     }
 }
+
+extension FileSystem.BetaGroup: SyncResourceProcessable {
+
+    var compareIdentity: String {
+        id ?? ""
+    }
+
+    var syncResultText: String {
+        groupName
+    }
+
+}

Sources/AppStoreConnectCLI/Model/BetaTester.swift

@@ -3,6 +3,7 @@
 import AppStoreConnect_Swift_SDK
 import Combine
 import Foundation
+import FileSystem
 import struct Model.BetaTester
 import SwiftyTextTable
 
@@ -58,3 +59,25 @@ extension BetaTester: ResultRenderable, TableInfoProvider {
         ]
     }
 }
+
+extension FileSystem.BetaTester: SyncResourceProcessable {
+
+    var syncResultText: String {
+        email
+    }
+
+    var compareIdentity: String {
+        email
+    }
+
+}
+
+extension String: SyncResourceProcessable {
+    var syncResultText: String {
+        self
+    }
+
+    var compareIdentity: String {
+        self
+    }
+}