Feature/Introduce PersonNameComponents for Google, Facebook, LinkedIn…

… sign in (#33)

Co-authored-by: Yll Fejziu <[email protected]>

.github/workflows/Tests.yml

@@ -7,7 +7,7 @@ on:
 
 jobs:
   Tests:
-    runs-on: macos-14-xlarge
+    runs-on: macos-15-xlarge
     steps:
     - name: Cancel previous jobs
       uses: styfle/[email protected]
@@ -21,8 +21,8 @@ jobs:
         xcode-version: latest-stable
 
     - name: Build project
-      run: set -o pipefail && env NSUnbufferedIO=YES xcodebuild build-for-testing -destination 'name=iPhone 14 Pro' -scheme 'PovioKitAuth-Package' | xcpretty
+      run: set -o pipefail && env NSUnbufferedIO=YES xcodebuild build-for-testing -destination 'name=iPhone 16 Pro,OS=18.0' -scheme 'PovioKitAuth-Package' | xcpretty
 
     - name: Run tests
-      run: set -o pipefail && env NSUnbufferedIO=YES xcodebuild test-without-building -destination 'name=iPhone 14 Pro' -scheme 'PovioKitAuth-Package' | xcpretty
-      
+      run: set -o pipefail && env NSUnbufferedIO=YES xcodebuild test-without-building -destination 'name=iPhone 16 Pro,OS=18.0' -scheme 'PovioKitAuth-Package' | xcpretty
+

Sources/Apple/AppleAuthenticator+Models.swift

@@ -25,13 +25,7 @@ public extension AppleAuthenticator {
 
     /// User full name represented by `givenName` and `familyName`
     public var name: String? {
-      guard let givenName = nameComponents?.givenName else {
-        return nameComponents?.familyName
-      }
-      guard let familyName = nameComponents?.familyName else {
-        return givenName
-      }
-      return "\(givenName) \(familyName)"
+      nameComponents?.name
     }
   }
 

Sources/Core/PersonNameComponents+Extension.swift

@@ -0,0 +1,42 @@
+//
+//  PersonNameComponents+Extension.swift
+//  PovioKitAuth
+//
+//  Created by Egzon Arifi on 09/11/2024.
+//
+
+import AuthenticationServices
+import Foundation
+
+public extension PersonNameComponents {
+  var name: String? {
+    guard let givenName else {
+      return familyName
+    }
+    guard let familyName else {
+      return givenName
+    }
+    return "\(givenName) \(familyName)"
+  }
+}
+
+public extension PersonNameComponents {
+  init(
+    namePrefix: String? = .none,
+    middleName: String? = .none,
+    givenName: String? = .none,
+    familyName: String? = .none,
+    nameSuffix: String? = .none,
+    nickname: String? = .none,
+    phoneticRepresentation: PersonNameComponents? = .none
+  ) {
+    self.init()
+    self.namePrefix = namePrefix
+    self.familyName = familyName
+    self.middleName = middleName
+    self.givenName = givenName
+    self.nameSuffix = nameSuffix
+    self.nickname = nickname
+    self.phoneticRepresentation = phoneticRepresentation
+  }
+}

Sources/Facebook/FacebookAuthenticator+Models.swift

@@ -12,9 +12,14 @@ public extension FacebookAuthenticator {
   struct Response {
     public let userId: String
     public let token: String
-    public let name: String?
+    public let nameComponents: PersonNameComponents?
     public let email: String?
     public let expiresAt: Date
+
+    /// User full name represented by `givenName` and `familyName`
+    public var name: String? {
+      nameComponents?.name
+    }
   }
 
   struct GraphResponse: Decodable {
@@ -36,11 +41,3 @@ public extension FacebookAuthenticator {
     }
   }
 }
-
-public extension FacebookAuthenticator.GraphResponse {
-  var displayName: String {
-    [firstName, lastName]
-      .compactMap { $0 }
-      .joined(separator: " ")
-  }
-}

Sources/Facebook/FacebookAuthenticator.swift

@@ -110,7 +110,10 @@ private extension FacebookAuthenticator {
             let authResponse = Response(
               userId: object.id,
               token: token.tokenString,
-              name: object.displayName,
+              nameComponents: PersonNameComponents(
+                givenName: object.firstName,
+                familyName: object.lastName
+              ),
               email: object.email,
               expiresAt: token.expirationDate
             )

Sources/Google/GoogleAuthenticator+Models.swift

@@ -7,15 +7,21 @@
 //
 
 import Foundation
+import PovioKitAuthCore
 
 public extension GoogleAuthenticator {
   struct Response {
     public let userId: String?
     public let idToken: String?
     public let accessToken: String
     public let refreshToken: String
-    public let name: String?
+    public let nameComponents: PersonNameComponents?
     public let email: String?
     public let expiresAt: Date?
+
+    /// User full name represented by `givenName` and `familyName`
+    public var name: String? {
+      nameComponents?.name
+    }
   }
 }

Sources/Google/GoogleAuthenticator.swift

@@ -107,7 +107,10 @@ private extension GIDGoogleUser {
       idToken: idToken?.tokenString,
       accessToken: accessToken.tokenString,
       refreshToken: refreshToken.tokenString,
-      name: profile?.name,
+      nameComponents: PersonNameComponents(
+        givenName: profile?.givenName,
+        familyName: profile?.familyName
+      ),
       email: profile?.email,
       expiresAt: accessToken.expirationDate
     )

Sources/LinkedIn/LinkedInAuthenticator+Models.swift

@@ -86,8 +86,13 @@ public extension LinkedInAuthenticator {
   struct Response {
     public let userId: String
     public let token: String
-    public let name: String
+    public let nameComponents: PersonNameComponents
     public let email: String
     public let expiresAt: Date
+
+    /// User full name represented by `givenName` and `familyName`
+    public var name: String? {
+      nameComponents.name
+    }
   }
 }

Sources/LinkedIn/LinkedInAuthenticator.swift

@@ -39,11 +39,13 @@ extension LinkedInAuthenticator: Authenticator {
 
     storage.set(true, forKey: storageIsAuthenticatedKey)
 
-    let name = "\(profileResponse.localizedFirstName) \(profileResponse.localizedLastName)"
     return Response(
       userId: profileResponse.id,
       token: authResponse.accessToken,
-      name: name,
+      nameComponents: PersonNameComponents(
+        givenName: profileResponse.localizedFirstName,
+        familyName: profileResponse.localizedLastName
+      ),
       email: emailResponse.emailAddress,
       expiresAt: authResponse.expiresIn
     )