@W-21933885: [MSDK Android] App Attestation Implementation (Fix TokenMigrationActivityTest By Explicitly Nulling appAttestationClient In Spy)

Author: JohnsonEricAtSalesforce

libs/test/SalesforceSDKTest/src/com/salesforce/androidsdk/ui/TokenMigrationActivityTest.kt

@@ -40,6 +40,7 @@ import com.salesforce.androidsdk.accounts.UserAccountManager
 import com.salesforce.androidsdk.analytics.AnalyticsPublishingWorker
 import com.salesforce.androidsdk.analytics.logger.SalesforceLogger
 import com.salesforce.androidsdk.app.SalesforceSDKManager
+import com.salesforce.androidsdk.auth.AppAttestationClient
 import com.salesforce.androidsdk.auth.OAuth2.FRONTDOOR_URL_KEY
 import com.salesforce.androidsdk.config.OAuthConfig
 import com.salesforce.androidsdk.rest.RestClient
@@ -80,6 +81,7 @@ class TokenMigrationActivityTest {
     private lateinit var mockUser: UserAccount
     private lateinit var mockClientManager: com.salesforce.androidsdk.rest.ClientManager
     private lateinit var savedFactory: ViewModelProvider.Factory
+    private var savedAppAttestationClient: AppAttestationClient? = null
 
     @Before
     fun setUp() {
@@ -113,12 +115,17 @@ class TokenMigrationActivityTest {
         // Get the REAL SalesforceSDKManager instance and save properties
         val realSdkManager = SalesforceSDKManager.getInstance()
         savedFactory = realSdkManager.loginViewModelFactory
+        savedAppAttestationClient = realSdkManager.appAttestationClient
 
         // Mock only the clientManager on the real instance using spyk
         val sdkManagerSpy = spyk(realSdkManager)
         every { sdkManagerSpy.clientManager } returns mockClientManager
         every { mockClientManager.peekRestClient(any<UserAccount>()) } returns mockRestClient
 
+        // CRITICAL: Set appAttestationClient to null to prevent blocking network calls
+        // in LoginViewModel.getAuthorizationUrl() and OAuth2.makeTokenEndpointRequest()
+        every { sdkManagerSpy.appAttestationClient } returns null
+
         // Replace getInstance() to return our spy instead of completely mocking the object
         mockkStatic(SalesforceSDKManager::class)
         every { SalesforceSDKManager.getInstance() } returns sdkManagerSpy
@@ -136,10 +143,11 @@ class TokenMigrationActivityTest {
 
     @After
     fun tearDown() {
-        // Restore the loginViewModelFactory on the real instance before unmocking
+        // Restore the loginViewModelFactory and appAttestationClient on the real instance before unmocking
         try {
             val realSdkManager = SalesforceSDKManager.getInstance()
             realSdkManager.loginViewModelFactory = savedFactory
+            realSdkManager.appAttestationClient = savedAppAttestationClient
         } catch (e: Exception) {
             // If getInstance() is still mocked, this will fail - unmockkAll will clean it up
         }