test: add comprehensive Auth test coverage

- Add extensive test coverage for Auth client functionality
- Test password reset, email resend, phone resend operations
- Test admin user management (get, update, create, delete users)
- Test MFA operations (enroll, challenge, verify, unenroll, list factors)
- Test SSO sign-in with domain and provider ID
- Test user identity unlinking and reauthentication
- Test authenticator assurance level functionality
- Fix status codes in existing tests (200 -> 204 for appropriate endpoints)
- Add proper test mocks and assertions for all new test cases

Author: grdsdev

Sources/Auth/AuthClient.swift

@@ -97,16 +97,18 @@ public actor AuthClient {
     AuthClient.globalClientID += 1
     clientID = AuthClient.globalClientID
 
-    var adapters: [any RequestAdapter] = []
-
-    if let apiKey = configuration.headers["apikey"] {
-      adapters.append(SupabaseApiKeyAdapter(apiKey: apiKey))
+    var headers = HTTPHeaders(configuration.headers)
+    if headers["X-Client-Info"] == nil {
+      headers["X-Client-Info"] = "auth-swift/\(version)"
     }
-    adapters.append(SupabaseApiVersionAdapter())
+
+    headers["X-Supabase-Api-Version"] = apiVersions[._20240101]!.name.rawValue
 
     Dependencies[clientID] = Dependencies(
       configuration: configuration,
-      session: configuration.session.newSession(adapters: adapters),
+      session: configuration.session.newSession(adapters: [
+        DefaultHeadersRequestAdapter(headers: headers)
+      ]),
       api: APIClient(clientID: clientID),
       codeVerifierStorage: .live(clientID: clientID),
       sessionStorage: .live(clientID: clientID),

Sources/Auth/Internal/APIClient.swift

@@ -163,14 +163,16 @@ extension Alamofire.Session {
   }
 }
 
-struct SupabaseApiVersionAdapter: RequestAdapter {
+struct DefaultHeadersRequestAdapter: RequestAdapter {
+  let headers: HTTPHeaders
+
   func adapt(
     _ urlRequest: URLRequest,
     for session: Alamofire.Session,
-    completion: @escaping @Sendable (_ result: Result<URLRequest, any Error>) -> Void
+    completion: @escaping (Result<URLRequest, any Error>) -> Void
   ) {
-    var request = urlRequest
-    request.headers["X-Supabase-Api-Version"] = apiVersions[._20240101]!.name.rawValue
-    completion(.success(request))
+    var urlRequest = urlRequest
+    urlRequest.headers = urlRequest.headers.merging(with: headers)
+    completion(.success(urlRequest))
   }
 }

Tests/AuthTests/AuthClientTests.swift

@@ -9,6 +9,7 @@ import ConcurrencyExtras
 import CustomDump
 import InlineSnapshotTesting
 import Mocker
+import SnapshotTestingCustomDump
 import TestHelpers
 import XCTest
 
@@ -23,7 +24,6 @@ final class AuthClientTests: XCTestCase {
 
   var storage: InMemoryLocalStorage!
 
-  var http: HTTPClientMock!
   var sut: AuthClient!
 
   #if !os(Windows) && !os(Linux) && !os(Android)
@@ -38,7 +38,7 @@ final class AuthClientTests: XCTestCase {
     super.setUp()
     storage = InMemoryLocalStorage()
 
-    //    isRecording = true
+//        isRecording = true
   }
 
   override func tearDown() {
@@ -89,7 +89,7 @@ final class AuthClientTests: XCTestCase {
     Mock(
       url: clientURL.appendingPathComponent("logout"),
       ignoreQuery: true,
-      statusCode: 200,
+      statusCode: 204,
       data: [
         .post: Data()
       ]
@@ -134,7 +134,7 @@ final class AuthClientTests: XCTestCase {
       url: clientURL.appendingPathComponent("logout").appendingQueryItems([
         URLQueryItem(name: "scope", value: "others")
       ]),
-      statusCode: 200,
+      statusCode: 204,
       data: [
         .post: Data()
       ]
@@ -779,7 +779,7 @@ final class AuthClientTests: XCTestCase {
     Mock(
       url: clientURL.appendingPathComponent("otp"),
       ignoreQuery: true,
-      statusCode: 200,
+      statusCode: 204,
       data: [.post: Data()]
     )
     .snapshotRequest {
@@ -812,7 +812,7 @@ final class AuthClientTests: XCTestCase {
     Mock(
       url: clientURL.appendingPathComponent("otp"),
       ignoreQuery: true,
-      statusCode: 200,
+      statusCode: 204,
       data: [.post: Data()]
     )
     .snapshotRequest {
@@ -1277,7 +1277,7 @@ final class AuthClientTests: XCTestCase {
     Mock(
       url: clientURL.appendingPathComponent("recover"),
       ignoreQuery: true,
-      statusCode: 200,
+      statusCode: 204,
       data: [.post: Data()]
     )
     .snapshotRequest {
@@ -1307,7 +1307,7 @@ final class AuthClientTests: XCTestCase {
     Mock(
       url: clientURL.appendingPathComponent("resend"),
       ignoreQuery: true,
-      statusCode: 200,
+      statusCode: 204,
       data: [.post: Data()]
     )
     .snapshotRequest {
@@ -1398,7 +1398,7 @@ final class AuthClientTests: XCTestCase {
   func testReauthenticate() async throws {
     Mock(
       url: clientURL.appendingPathComponent("reauthenticate"),
-      statusCode: 200,
+      statusCode: 204,
       data: [.get: Data()]
     )
     .snapshotRequest {
@@ -2245,9 +2245,7 @@ final class AuthClientTests: XCTestCase {
       localStorage: storage,
       logger: nil,
       encoder: encoder,
-      fetch: { request in
-        try await session.data(for: request)
-      }
+      session: .init(configuration: sessionConfiguration)
     )
 
     let sut = AuthClient(configuration: configuration)