feat: Add ability to pass routes into configuration (#74)

cbaker6 · web-flow · commit 7c11bd334017 · 2024-08-04T09:21:27.000-07:00
* feat: Add ability to pass routes into configuration

* lint
diff --git a/Package.resolved b/Package.resolved
diff --git a/Package.swift b/Package.swift
@@ -1,4 +1,4 @@
-// swift-tools-version:5.6
+// swift-tools-version:5.7
 import PackageDescription
 
 // swiftlint:disable line_length
@@ -21,11 +21,11 @@ let package = Package(
     dependencies: [
         .package(
             url: "https://github.com/vapor/vapor.git",
-                .upToNextMajor(from: "4.102.1")
+            .upToNextMajor(from: "4.102.1")
         ),
         .package(
             url: "https://github.com/netreconlab/Parse-Swift.git",
-                 .upToNextMajor(from: "5.11.1")
+            .upToNextMajor(from: "5.11.2")
         )
     ],
     targets: [
@@ -43,10 +43,14 @@ let package = Package(
                 // the `.unsafeFlags` construct required by SwiftPM, this flag is recommended for Release
                 // builds. See <https://github.com/swift-server/guides/blob/main/docs/building.md#building-for-production> for details.
                 .unsafeFlags(["-cross-module-optimization"], .when(configuration: .release))
-            ]),
-        .testTarget(name: "ParseServerSwiftTests", dependencies: [
-            .target(name: "ParseServerSwift"),
-            .product(name: "XCTVapor", package: "vapor")
-        ])
+            ]
+        ),
+        .testTarget(
+            name: "ParseServerSwiftTests",
+            dependencies: [
+                .target(name: "ParseServerSwift"),
+                .product(name: "XCTVapor", package: "vapor")
+            ]
+        )
     ]
 )
diff --git a/README.md b/README.md
@@ -122,7 +122,10 @@ enum Entrypoint {
         let executorTakeoverSuccess = NIOSingletons.unsafeTryInstallSingletonPosixEventLoopGroupAsConcurrencyGlobalExecutor()
         app.logger.debug("Running with \(executorTakeoverSuccess ? "SwiftNIO" : "standard") Swift Concurrency default executor")
 
-        try await parseServerSwiftConfigure(app)
+        try await parseServerSwiftConfigure(
+            app,
+            using: exampleRoutes
+        )
         try await app.execute()
         try await app.asyncShutdown()
     }
diff --git a/Sources/App/entrypoint.swift b/Sources/App/entrypoint.swift
@@ -28,7 +28,10 @@ enum Entrypoint {
             "Running with \(executorTakeoverSuccess ? "SwiftNIO" : "standard") Swift Concurrency default executor"
         )
 
-        try await parseServerSwiftConfigure(app)
+        try await parseServerSwiftConfigure(
+            app,
+            using: exampleRoutes
+        )
         try await app.execute()
         try await app.asyncShutdown()
     }
diff --git a/Sources/ParseServerSwift/Parse.swift b/Sources/ParseServerSwift/Parse.swift
@@ -30,42 +30,53 @@ public var configuration: ParseServerConfiguration {
  server URL.
  - warning: Be sure to call this method before calling `try routes(app)`.
  */
-public func initialize(_ configuration: ParseServerConfiguration,
-                       app: Application) async throws {
+public func initialize(
+    _ configuration: ParseServerConfiguration,
+    app: Application
+) async throws {
     try await initializeServer(configuration, app: app)
 }
 
-func initialize(_ configuration: ParseServerConfiguration,
-                app: Application,
-                testing: Bool) async throws {
+func initialize(
+    _ configuration: ParseServerConfiguration,
+    app: Application,
+    testing: Bool
+) async throws {
     var configuration = configuration
     configuration.isTesting = testing
     try await initialize(configuration, app: app)
 }
 
-func initializeServer(_ configuration: ParseServerConfiguration,
-                      app: Application) async throws {
+func initializeServer(
+    _ configuration: ParseServerConfiguration,
+    app: Application
+) async throws {
 
     // Parse uses tailored encoders/decoders. These can be retrieved from any ParseObject
     ContentConfiguration.global.use(encoder: User.getEncoder(), for: .json)
     ContentConfiguration.global.use(decoder: User.getDecoder(), for: .json)
 
     guard let parseServerURL = URL(string: configuration.primaryParseServerURLString) else {
-        throw ParseError(code: .otherCause,
-                         message: "Could not make a URL from the Parse Server string")
+        let error = ParseError(
+            code: .otherCause,
+            message: "Could not make a URL from the Parse Server string"
+        )
+        throw error
     }
 
     if !configuration.isTesting {
         try setConfiguration(configuration)
         do {
             // Initialize the Parse-Swift SDK. Add any additional parameters you need
-            try await ParseSwift.initialize(applicationId: configuration.applicationId,
-                                            primaryKey: configuration.primaryKey,
-                                            serverURL: parseServerURL,
-                                            // POST all queries instead of using GET.
-                                            usingPostForQuery: true,
-                                            // Do not use cache for anything.
-                                            requestCachePolicy: .reloadIgnoringLocalCacheData) { _, completionHandler in
+            try await ParseSwift.initialize(
+                applicationId: configuration.applicationId,
+                primaryKey: configuration.primaryKey,
+                serverURL: parseServerURL,
+                // POST all queries instead of using GET.
+                usingPostForQuery: true,
+                // Do not use cache for anything.
+                requestCachePolicy: .reloadIgnoringLocalCacheData
+            ) { _, completionHandler in
                 // Setup to use default certificate pinning. See Parse-Swift docs for more info
                 completionHandler(.performDefaultHandling, nil)
             }
diff --git a/Sources/ParseServerSwift/configure.swift b/Sources/ParseServerSwift/configure.swift
@@ -9,7 +9,8 @@ import Vapor
  */
 public func parseServerSwiftConfigure(
     _ app: Application,
-    with configuration: ParseServerConfiguration? = nil
+    with configuration: ParseServerConfiguration? = nil,
+    using routes: ((Application) throws -> Void)
 ) async throws {
     // Initialize ParseServerSwift
     let configuration = try configuration ?? ParseServerConfiguration(app: app)
diff --git a/Sources/ParseServerSwift/routes.swift b/Sources/ParseServerSwift/routes.swift
@@ -2,7 +2,7 @@ import Vapor
 import ParseSwift
 
 // swiftlint:disable:next cyclomatic_complexity function_body_length
-func routes(_ app: Application) throws {
+public func exampleRoutes(_ app: Application) throws {
 
     // A typical route in Vapor.
     app.get { req in
@@ -15,8 +15,10 @@ func routes(_ app: Application) throws {
     }
 
     // A simple Parse Hook Function route that returns "Hello World".
-    app.post("hello",
-             name: "hello") { req async throws -> ParseHookResponse<String> in
+    app.post(
+        "hello",
+        name: "hello"
+    ) { req async throws -> ParseHookResponse<String> in
         // Note that `ParseHookResponse<String>` means a "successful"
         // response will return a "String" type.
         if let error: ParseHookResponse<String> = checkHeaders(req) {
@@ -39,8 +41,10 @@ func routes(_ app: Application) throws {
     }
 
     // Another simple Parse Hook Function route that returns the version of the server.
-    app.post("version",
-             name: "version") { req async throws -> ParseHookResponse<String> in
+    app.post(
+        "version",
+        name: "version"
+    ) { req async throws -> ParseHookResponse<String> in
         // Note that `ParseHookResponse<String>` means a "successful"
         // response will return a "String" type.
         if let error: ParseHookResponse<String> = checkHeaders(req) {
@@ -89,9 +93,13 @@ func routes(_ app: Application) throws {
     }
 
     // A Parse Hook Trigger route.
-    app.post("score", "save", "before",
-             object: GameScore.self,
-             trigger: .beforeSave) { req async throws -> ParseHookResponse<GameScore> in
+    app.post(
+        "score",
+        "save",
+        "before",
+        object: GameScore.self,
+        trigger: .beforeSave
+    ) { req async throws -> ParseHookResponse<GameScore> in
         // Note that `ParseHookResponse<GameScore>` means a "successful"
         // response will return a "GameScore" type.
         if let error: ParseHookResponse<GameScore> = checkHeaders(req) {
@@ -117,9 +125,13 @@ func routes(_ app: Application) throws {
     }
 
     // Another Parse Hook Trigger route.
-    app.post("score", "find", "before",
-             object: GameScore.self,
-             trigger: .beforeFind) { req async throws -> ParseHookResponse<[GameScore]> in
+    app.post(
+        "score",
+        "find",
+        "before",
+        object: GameScore.self,
+        trigger: .beforeFind
+    ) { req async throws -> ParseHookResponse<[GameScore]> in
         // Note that `ParseHookResponse<[GameScore]>` means a "successful"
         // response will return a "[GameScore]" type.
         if let error: ParseHookResponse<[GameScore]> = checkHeaders(req) {
@@ -144,9 +156,13 @@ func routes(_ app: Application) throws {
     }
 
     // Another Parse Hook Trigger route.
-    app.post("user", "login", "after",
-             object: User.self,
-             trigger: .afterLogin) { req async throws -> ParseHookResponse<Bool> in
+    app.post(
+        "user",
+        "login",
+        "after",
+        object: User.self,
+        trigger: .afterLogin
+    ) { req async throws -> ParseHookResponse<Bool> in
         // Note that `ParseHookResponse<Bool>` means a "successful"
         // response will return a "Bool" type. Bool is the standard response with
         // a "true" response meaning everything is okay or continue.
@@ -161,8 +177,12 @@ func routes(_ app: Application) throws {
     }
 
     // A Parse Hook Trigger route for `ParseFile`.
-    app.on("file", "save", "before",
-           trigger: .beforeSave) { req async throws -> ParseHookResponse<Bool> in
+    app.on(
+        "file",
+        "save",
+        "before",
+        trigger: .beforeSave
+    ) { req async throws -> ParseHookResponse<Bool> in
         // Note that `ParseHookResponse<Bool>` means a "successful"
         // response will return a "Bool" type. Bool is the standard response with
         // a "true" response meaning everything is okay or continue. Sending "false"
@@ -178,8 +198,12 @@ func routes(_ app: Application) throws {
     }
 
     // Another Parse Hook Trigger route for `ParseFile`.
-    app.post("file", "delete", "before",
-             trigger: .beforeDelete) { req async throws -> ParseHookResponse<Bool> in
+    app.post(
+        "file",
+        "delete",
+        "before",
+        trigger: .beforeDelete
+    ) { req async throws -> ParseHookResponse<Bool> in
         // Note that `ParseHookResponse<Bool>` means a "successful"
         // response will return a "Bool" type. Bool is the standard response with
         // a "true" response meaning everything is okay or continue.
@@ -194,8 +218,11 @@ func routes(_ app: Application) throws {
     }
 
     // A Parse Hook Trigger route for `ParseLiveQuery`.
-    app.post("connect", "before",
-             trigger: .beforeConnect) { req async throws -> ParseHookResponse<Bool> in
+    app.post(
+        "connect",
+        "before",
+        trigger: .beforeConnect
+    ) { req async throws -> ParseHookResponse<Bool> in
         // Note that `ParseHookResponse<Bool>` means a "successful"
         // response will return a "Bool" type. Bool is the standard response with
         // a "true" response meaning everything is okay or continue.
@@ -210,9 +237,13 @@ func routes(_ app: Application) throws {
     }
 
     // Another Parse Hook Trigger route for `ParseLiveQuery`.
-    app.post("score", "subscribe", "before",
-             object: GameScore.self,
-             trigger: .beforeSubscribe) { req async throws -> ParseHookResponse<Bool> in
+    app.post(
+        "score",
+        "subscribe",
+        "before",
+        object: GameScore.self,
+        trigger: .beforeSubscribe
+    ) { req async throws -> ParseHookResponse<Bool> in
         // Note that `ParseHookResponse<Bool>` means a "successful"
         // response will return a "Bool" type. Bool is the standard response with
         // a "true" response meaning everything is okay or continue.
@@ -227,9 +258,13 @@ func routes(_ app: Application) throws {
     }
 
     // Another Parse Hook Trigger route for `ParseLiveQuery`.
-    app.post("score", "event", "after",
-             object: GameScore.self,
-             trigger: .afterEvent) { req async throws -> ParseHookResponse<Bool> in
+    app.post(
+        "score",
+        "event",
+        "after",
+        object: GameScore.self,
+        trigger: .afterEvent
+    ) { req async throws -> ParseHookResponse<Bool> in
         // Note that `ParseHookResponse<Bool>` means a "successful"
         // response will return a "Bool" type. Bool is the standard response with
         // a "true" response meaning everything is okay or continue.
diff --git a/Tests/ParseServerSwiftTests/AppTests.swift b/Tests/ParseServerSwiftTests/AppTests.swift

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,10 @@ enum Entrypoint {`
`28`	`28`	`"Running with \(executorTakeoverSuccess ? "SwiftNIO" : "standard") Swift Concurrency default executor"`
`29`	`29`	`)`
`30`	`30`
`31`		`- try await parseServerSwiftConfigure(app)`
	`31`	`+ try await parseServerSwiftConfigure(`
	`32`	`+ app,`
	`33`	`+ using: exampleRoutes`
	`34`	`+ )`
`32`	`35`	`try await app.execute()`
`33`	`36`	`try await app.asyncShutdown()`
`34`	`37`	`}`