PM-26594: Move the QrCodeAnalyzer to the UI module

app/build.gradle.kts

@@ -271,7 +271,6 @@ dependencies {
     implementation(platform(libs.square.retrofit.bom))
     implementation(libs.square.retrofit)
     implementation(libs.timber)
-    implementation(libs.zxing.zxing.core)
 
     // For now we are restricted to running Compose tests for debug builds only
     debugImplementation(libs.androidx.compose.ui.test.manifest)

app/src/main/kotlin/com/x8bit/bitwarden/ui/vault/feature/qrcodescan/QrCodeScanScreen.kt

@@ -55,15 +55,15 @@ import com.bitwarden.ui.platform.base.util.annotatedStringResource
 import com.bitwarden.ui.platform.components.appbar.BitwardenTopAppBar
 import com.bitwarden.ui.platform.components.scaffold.BitwardenScaffold
 import com.bitwarden.ui.platform.components.util.rememberVectorPainter
+import com.bitwarden.ui.platform.feature.qrcodescan.util.QrCodeAnalyzer
+import com.bitwarden.ui.platform.feature.qrcodescan.util.QrCodeAnalyzerImpl
 import com.bitwarden.ui.platform.model.WindowSize
 import com.bitwarden.ui.platform.resource.BitwardenDrawable
 import com.bitwarden.ui.platform.resource.BitwardenString
 import com.bitwarden.ui.platform.theme.BitwardenTheme
 import com.bitwarden.ui.platform.theme.LocalBitwardenColorScheme
 import com.bitwarden.ui.platform.theme.color.darkBitwardenColorScheme
 import com.bitwarden.ui.platform.util.rememberWindowSize
-import com.x8bit.bitwarden.ui.vault.feature.qrcodescan.util.QrCodeAnalyzer
-import com.x8bit.bitwarden.ui.vault.feature.qrcodescan.util.QrCodeAnalyzerImpl
 import java.util.concurrent.Executors
 import kotlin.coroutines.resume
 import kotlin.coroutines.suspendCoroutine

app/src/test/kotlin/com/x8bit/bitwarden/ui/vault/feature/qrcodescan/QrCodeScanScreenTest.kt

@@ -3,9 +3,9 @@ package com.x8bit.bitwarden.ui.vault.feature.qrcodescan
 import androidx.camera.core.ImageProxy
 import androidx.compose.ui.test.onNodeWithText
 import com.bitwarden.core.data.repository.util.bufferedMutableSharedFlow
+import com.bitwarden.ui.platform.feature.qrcodescan.util.FakeQrCodeAnalyzer
 import com.bitwarden.ui.util.performCustomAccessibilityAction
 import com.x8bit.bitwarden.ui.platform.base.BitwardenComposeTest
-import com.x8bit.bitwarden.ui.vault.feature.qrcodescan.util.FakeQrCodeAnalyzer
 import io.mockk.every
 import io.mockk.mockk
 import io.mockk.verify

authenticator/build.gradle.kts

@@ -224,7 +224,6 @@ dependencies {
     implementation(libs.kotlinx.collections.immutable)
     implementation(libs.kotlinx.coroutines.android)
     implementation(libs.kotlinx.serialization)
-    implementation(libs.zxing.zxing.core)
 
     // For now we are restricted to running Compose tests for debug builds only
     debugImplementation(libs.androidx.compose.ui.test.manifest)

authenticator/src/main/kotlin/com/bitwarden/authenticator/ui/authenticator/feature/qrcodescan/QrCodeScanScreen.kt

@@ -50,8 +50,6 @@ import androidx.compose.ui.viewinterop.AndroidView
 import androidx.hilt.lifecycle.viewmodel.compose.hiltViewModel
 import androidx.lifecycle.compose.LocalLifecycleOwner
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
-import com.bitwarden.authenticator.ui.authenticator.feature.qrcodescan.util.QrCodeAnalyzer
-import com.bitwarden.authenticator.ui.authenticator.feature.qrcodescan.util.QrCodeAnalyzerImpl
 import com.bitwarden.authenticator.ui.platform.util.isPortrait
 import com.bitwarden.ui.platform.base.util.EventsEffect
 import com.bitwarden.ui.platform.base.util.annotatedStringResource
@@ -60,6 +58,8 @@ import com.bitwarden.ui.platform.base.util.standardHorizontalMargin
 import com.bitwarden.ui.platform.components.appbar.BitwardenTopAppBar
 import com.bitwarden.ui.platform.components.dialog.BitwardenBasicDialog
 import com.bitwarden.ui.platform.components.scaffold.BitwardenScaffold
+import com.bitwarden.ui.platform.feature.qrcodescan.util.QrCodeAnalyzer
+import com.bitwarden.ui.platform.feature.qrcodescan.util.QrCodeAnalyzerImpl
 import com.bitwarden.ui.platform.resource.BitwardenDrawable
 import com.bitwarden.ui.platform.resource.BitwardenString
 import com.bitwarden.ui.platform.theme.BitwardenTheme

authenticator/src/test/kotlin/com/bitwarden/authenticator/ui/authenticator/feature/qrcodescan/QrCodeScanScreenTest.kt

@@ -8,6 +8,7 @@ import androidx.compose.ui.test.onNodeWithText
 import androidx.compose.ui.test.performClick
 import com.bitwarden.authenticator.ui.platform.base.AuthenticatorComposeTest
 import com.bitwarden.core.data.repository.util.bufferedMutableSharedFlow
+import com.bitwarden.ui.platform.feature.qrcodescan.util.FakeQrCodeAnalyzer
 import io.mockk.every
 import io.mockk.just
 import io.mockk.mockk

ui/build.gradle.kts

@@ -57,6 +57,7 @@ dependencies {
     implementation(platform(libs.androidx.compose.bom))
     implementation(libs.androidx.activity.compose)
     implementation(libs.androidx.browser)
+    implementation(libs.androidx.camera.camera2)
     implementation(libs.androidx.compose.animation)
     implementation(libs.androidx.compose.material3)
     implementation(libs.androidx.compose.material3.adaptive)
@@ -72,6 +73,7 @@ dependencies {
     implementation(libs.kotlinx.serialization)
     implementation(libs.kotlinx.coroutines.core)
     implementation(libs.kotlinx.collections.immutable)
+    implementation(libs.zxing.zxing.core)
 
     // For now we are restricted to running Compose tests for debug builds only
     debugImplementation(libs.androidx.compose.ui.test.manifest)
@@ -86,6 +88,7 @@ dependencies {
     testImplementation(libs.androidx.compose.ui.test)
 
     testFixturesImplementation(libs.androidx.activity.compose)
+    testFixturesImplementation(libs.androidx.camera.camera2)
     testFixturesImplementation(libs.androidx.compose.ui.test)
     testFixturesImplementation(libs.androidx.navigation.compose)
     testFixturesImplementation(libs.google.hilt.android.testing)

app/src/main/kotlin/com/x8bit/bitwarden/ui/vault/feature/qrcodescan/util/QrCodeAnalyzer.kt → ui/src/main/kotlin/com/bitwarden/ui/platform/feature/qrcodescan/util/QrCodeAnalyzer.kt

@@ -1,4 +1,4 @@
-package com.x8bit.bitwarden.ui.vault.feature.qrcodescan.util
+package com.bitwarden.ui.platform.feature.qrcodescan.util
 
 import androidx.camera.core.ImageAnalysis
 import androidx.compose.runtime.Stable

authenticator/src/main/kotlin/com/bitwarden/authenticator/ui/authenticator/feature/qrcodescan/util/QrCodeAnalyzerImpl.kt → ui/src/main/kotlin/com/bitwarden/ui/platform/feature/qrcodescan/util/QrCodeAnalyzerImpl.kt

@@ -1,6 +1,7 @@
-package com.bitwarden.authenticator.ui.authenticator.feature.qrcodescan.util
+package com.bitwarden.ui.platform.feature.qrcodescan.util
 
 import androidx.camera.core.ImageProxy
+import com.bitwarden.annotation.OmitFromCoverage
 import com.google.zxing.BarcodeFormat
 import com.google.zxing.BinaryBitmap
 import com.google.zxing.DecodeHintType
@@ -14,6 +15,7 @@ import java.nio.ByteBuffer
  * A class setup to handle image analysis so that we can use the Zxing library
  * to scan QR codes and convert them to a string.
  */
+@OmitFromCoverage
 class QrCodeAnalyzerImpl : QrCodeAnalyzer {
 
     /**
@@ -31,20 +33,18 @@ class QrCodeAnalyzerImpl : QrCodeAnalyzer {
 
         val source = PlanarYUVLuminanceSource(
             image.planes[0].buffer.toByteArray(),
-            image.planes[0].rowStride,
+            image.width,
             image.height,
             0,
             0,
             image.width,
             image.height,
             false,
         )
-
         val binaryBitmap = BinaryBitmap(HybridBinarizer(source))
         try {
             val result = MultiFormatReader().decode(
-                /* image = */ binaryBitmap,
-                /* hints = */
+                binaryBitmap,
                 mapOf(
                     DecodeHintType.POSSIBLE_FORMATS to arrayListOf(BarcodeFormat.QR_CODE),
                     DecodeHintType.ALSO_INVERTED to true,
@@ -53,7 +53,8 @@ class QrCodeAnalyzerImpl : QrCodeAnalyzer {
 
             qrCodeRead = true
             onQrCodeScanned(result.text)
-        } catch (ignored: NotFoundException) {
+        } catch (_: NotFoundException) {
+            return
         } finally {
             image.close()
         }
@@ -63,5 +64,6 @@ class QrCodeAnalyzerImpl : QrCodeAnalyzer {
 /**
  * This function helps us prepare the byte buffer to be read.
  */
+@OmitFromCoverage
 private fun ByteBuffer.toByteArray(): ByteArray =
     ByteArray(rewind().remaining()).also { get(it) }

app/src/test/kotlin/com/x8bit/bitwarden/ui/vault/feature/qrcodescan/util/FakeQrCodeAnalyzer.kt → ui/src/testFixtures/kotlin/com/bitwarden/ui/platform/feature/qrcodescan/util/FakeQrCodeAnalyzer.kt

@@ -1,4 +1,4 @@
-package com.x8bit.bitwarden.ui.vault.feature.qrcodescan.util
+package com.bitwarden.ui.platform.feature.qrcodescan.util
 
 import androidx.camera.core.ImageProxy