[MS-1470] Scanning noew checks that there are not enough captures before updating the scanning progress state

feature/external-credential/src/main/java/com/simprints/feature/externalcredential/screens/scanocr/ExternalCredentialScanOcrViewModel.kt

@@ -147,7 +147,8 @@ internal class ExternalCredentialScanOcrViewModel @AssistedInject constructor(
                 val scannedMfidDocument =
                     scanMfidDocumentUseCase(bitmap = cropped, documentType = ocrDocumentType, config = mfidConfig) ?: return@launch
                 Simber.d("Detected OCR")
-                if (isScanningInProgress) {
+                val notEnoughCaptures = scannedMfidDocuments.size < ocrConfig.capturesRequired
+                if (isScanningInProgress && notEnoughCaptures) {
                     scannedMfidDocuments += scannedMfidDocument
                     updateState {
                         ScanOcrState.ScanningInProgress(

feature/external-credential/src/test/java/com/simprints/feature/externalcredential/screens/scanocr/ExternalCredentialScanOcrViewModelTest.kt

@@ -325,6 +325,53 @@ internal class ExternalCredentialScanOcrViewModelTest {
         assertThat(result).isEqualTo(R.string.mfid_type_ghana_id_card)
     }
 
+    @Test
+    fun `processImage does not add documents or update state once required captures are reached`() = runTest {
+        val mockScannedDocument = mockk<ScannedMfidDocument>()
+        val mockNormalizedBitmap = mockk<Bitmap>()
+        val mockCroppedBitmap = mockk<Bitmap>()
+
+        coEvery { normalizeBitmapToPreviewUseCase(bitmap, cropConfig) } returns mockNormalizedBitmap
+        coEvery { cropDocumentFromPreviewUseCase(mockNormalizedBitmap, any()) } returns mockCroppedBitmap
+        coEvery { scanMfidDocumentUseCase(mockCroppedBitmap, documentType, any()) } returns mockScannedDocument
+
+        val observer = viewModel.scanOcrStateLiveData.test()
+        viewModel.startScanning()
+
+        val capturesRequired = (observer.value() as ScanOcrState.ScanningInProgress).scansRequired
+        repeat(capturesRequired) { viewModel.processImage(bitmap, cropConfig) }
+
+        val stateAtCapacity = observer.value() as ScanOcrState.ScanningInProgress
+        assertThat(stateAtCapacity.successfulCaptures).isEqualTo(capturesRequired)
+
+        viewModel.processImage(bitmap, cropConfig)
+
+        assertThat(observer.value()).isEqualTo(stateAtCapacity)
+    }
+
+    @Test
+    fun `processImage emits ScanningInProgress with enough captures exactly once`() = runTest {
+        val mockScannedDocument = mockk<ScannedMfidDocument>()
+        val mockNormalizedBitmap = mockk<Bitmap>()
+        val mockCroppedBitmap = mockk<Bitmap>()
+
+        coEvery { normalizeBitmapToPreviewUseCase(bitmap, cropConfig) } returns mockNormalizedBitmap
+        coEvery { cropDocumentFromPreviewUseCase(mockNormalizedBitmap, any()) } returns mockCroppedBitmap
+        coEvery { scanMfidDocumentUseCase(mockCroppedBitmap, documentType, any()) } returns mockScannedDocument
+
+        val observer = viewModel.scanOcrStateLiveData.test()
+        viewModel.startScanning()
+
+        val capturesRequired = (observer.value() as ScanOcrState.ScanningInProgress).scansRequired
+        repeat(capturesRequired + 2) { viewModel.processImage(bitmap, cropConfig) }
+
+        val completionTriggers = observer
+            .valueHistory()
+            .filterIsInstance<ScanOcrState.ScanningInProgress>()
+            .count { it.successfulCaptures >= it.scansRequired }
+        assertThat(completionTriggers).isEqualTo(1)
+    }
+
     @Test
     fun `processImage does not append documents or update state after scanning is complete`() = runTest {
         val mockScannedDocument = mockk<ScannedMfidDocument>()