PIR: Add profile saving validation (#6614)

Task/Issue URL:
https://app.asana.com/1/137249556945/project/488551667048375/task/1211075568938200?focus=true

### Description
Validates that all profiles are saved on ´saveProfile´ message.

### Steps to test this PR
https://app.asana.com/1/137249556945/task/1211099146322592?focus=true

### UI changes
No UI changes

Author: landomen

pir/pir-impl/src/main/java/com/duckduckgo/pir/impl/dashboard/messaging/handlers/PirWebSaveProfileMessageHandler.kt

@@ -73,16 +73,29 @@ class PirWebSaveProfileMessageHandler @Inject constructor(
 
         appCoroutineScope.launch(dispatcherProvider.io()) {
             val profiles = pirWebOnboardingStateHolder.toUserProfiles()
-            repository.saveUserProfiles(profiles)
+            if (!repository.saveUserProfiles(profiles)) {
+                logcat { "PIR-WEB: PirWebSaveProfileMessageHandler: failed to save all user profiles" }
+                jsMessaging.sendResponse(
+                    jsMessage = jsMessage,
+                    response = PirWebMessageResponse.DefaultResponse.ERROR,
+                )
+                return@launch
+            }
 
-            // TODO check if all profiles were saved successfully
             jsMessaging.sendResponse(
                 jsMessage,
                 response = PirWebMessageResponse.DefaultResponse.SUCCESS,
             )
 
-            context.startForegroundService(Intent(context, PirForegroundScanService::class.java))
-            scanScheduler.scheduleScans()
+            // start the initial scan at this point as startScanAndOptOut message is not reliable
+            startAndScheduleInitialScan()
+
+            pirWebOnboardingStateHolder.clear()
         }
     }
+
+    private fun startAndScheduleInitialScan() {
+        context.startForegroundService(Intent(context, PirForegroundScanService::class.java))
+        scanScheduler.scheduleScans()
+    }
 }

pir/pir-impl/src/main/java/com/duckduckgo/pir/impl/dashboard/state/PirWebOnboardingStateHolder.kt

@@ -139,4 +139,10 @@ class PirWebOnboardingStateHolder @Inject constructor() {
         }
         return profiles
     }
+
+    fun clear() {
+        names.clear()
+        addresses.clear()
+        birthYear = null
+    }
 }

pir/pir-impl/src/main/java/com/duckduckgo/pir/impl/store/PirRepository.kt

@@ -124,7 +124,7 @@ interface PirRepository {
 
     suspend fun replaceUserProfile(userProfile: UserProfile)
 
-    suspend fun saveUserProfiles(userProfiles: List<UserProfile>)
+    suspend fun saveUserProfiles(userProfiles: List<UserProfile>): Boolean
 
     suspend fun getEmailForBroker(dataBroker: String): String
 
@@ -436,9 +436,10 @@ internal class RealPirRepository(
         }
     }
 
-    override suspend fun saveUserProfiles(userProfiles: List<UserProfile>) =
+    override suspend fun saveUserProfiles(userProfiles: List<UserProfile>): Boolean =
         withContext(dispatcherProvider.io()) {
-            userProfileDao.insertUserProfiles(userProfiles)
+            val insertResult = userProfileDao.insertUserProfiles(userProfiles)
+            insertResult.size == userProfiles.size
         }
 
     override suspend fun getEmailForBroker(dataBroker: String): String =

pir/pir-impl/src/main/java/com/duckduckgo/pir/impl/store/db/UserProfileDao.kt

@@ -33,7 +33,7 @@ interface UserProfileDao {
     fun insertUserProfile(userProfile: UserProfile)
 
     @Insert(onConflict = OnConflictStrategy.REPLACE)
-    fun insertUserProfiles(userProfiles: List<UserProfile>)
+    fun insertUserProfiles(userProfiles: List<UserProfile>): List<Long>
 
     @Query("DELETE from pir_user_profile")
     fun deleteAllProfiles()