refactor(compat): move tests to ":compat"

This makes use of test fixtures.
See fbf2dd0

Assisted-by: Claude Opus 4.6

Issue 20547

Author: david-allison

AnkiDroid/build.gradle

@@ -458,6 +458,8 @@ dependencies {
     implementation libs.leakcanary.android
 
     testImplementation project(":libanki:testutils")
+    testImplementation testFixtures(project(":common"))
+    testImplementation testFixtures(project(":compat"))
 
     // A path for a testing library which provide Parameterized Test
     testImplementation libs.junit.jupiter

AnkiDroid/jacoco.gradle

@@ -129,7 +129,7 @@ tasks.register('jacocoTestReport', JacocoReport) {
 }
 
 // modules do not yet support flavors (play/full)
-def modulesToUnitTest = [":common", ":libanki"]
+def modulesToUnitTest = [":common", ":libanki", ":compat"]
 
 // A unit-test only report task
 tasks.register('jacocoUnitTestReport', JacocoReport) {report ->

common/build.gradle.kts

@@ -10,6 +10,7 @@ android {
     // but we can define files in 'com.ichi2.anki' inside 'common'
     // even with this namespace
     namespace = "com.ichi2.anki.common"
+    testFixtures.enable = true
     compileSdk =
         libs.versions.compileSdk
             .get()
@@ -61,4 +62,11 @@ dependencies {
     androidTestImplementation(libs.androidx.test.junit)
     androidTestImplementation(libs.androidx.espresso.core)
     testImplementation(kotlin("test"))
+
+    testFixturesImplementation(libs.hamcrest)
+    testFixturesImplementation(libs.jakewharton.timber)
+    testFixturesImplementation(libs.androidx.annotation)
+    // Required so the ExperimentalCoroutinesApi opt-in (applied globally) doesn't cause
+    // an "unresolved" warning, which is treated as an error due to allWarningsAsErrors
+    testFixturesImplementation(libs.kotlinx.coroutines.core)
 }

…rc/test/java/com/ichi2/anki/TestUtils.kt …ixtures/java/com/ichi2/anki/TestUtils.ktAnkiDroid/src/test/java/com/ichi2/anki/TestUtils.kt renamed to common/src/testFixtures/java/com/ichi2/anki/TestUtils.kt AnkiDroid/src/test/java/com/ichi2/anki/TestUtils.kt renamed to common/src/testFixtures/java/com/ichi2/anki/TestUtils.kt

@@ -18,8 +18,6 @@ package com.ichi2.anki
 
 import com.ichi2.utils.FileOperation.Companion.getFileContentsBytes
 import java.io.File
-import java.lang.Exception
-import java.lang.StringBuilder
 import java.security.MessageDigest
 
 open class TestUtils {

…a/com/ichi2/testutils/FileSystemUtils.kt …a/com/ichi2/testutils/FileSystemUtils.ktAnkiDroid/src/test/java/com/ichi2/testutils/FileSystemUtils.kt renamed to common/src/testFixtures/java/com/ichi2/testutils/FileSystemUtils.kt AnkiDroid/src/test/java/com/ichi2/testutils/FileSystemUtils.kt renamed to common/src/testFixtures/java/com/ichi2/testutils/FileSystemUtils.kt

@@ -17,7 +17,6 @@
 package com.ichi2.testutils
 
 import androidx.annotation.CheckResult
-import org.acra.util.IOUtils
 import org.hamcrest.CoreMatchers
 import org.hamcrest.MatcherAssert
 import timber.log.Timber
@@ -129,7 +128,7 @@ fun createTransientFile(
             .pathString,
     ).also {
         it.deleteOnExit()
-        IOUtils.writeStringToFile(it, content)
+        it.writeText(content)
     }
 
 /** Creates a sub-directory with the given name which is deleted on exit */

…est/java/com/ichi2/testutils/FileUtil.kt …res/java/com/ichi2/testutils/FileUtil.ktAnkiDroid/src/test/java/com/ichi2/testutils/FileUtil.kt renamed to common/src/testFixtures/java/com/ichi2/testutils/FileUtil.kt AnkiDroid/src/test/java/com/ichi2/testutils/FileUtil.kt renamed to common/src/testFixtures/java/com/ichi2/testutils/FileUtil.kt

@@ -17,7 +17,6 @@
 package com.ichi2.testutils
 
 import com.ichi2.anki.model.Directory
-import org.acra.util.IOUtils
 import java.io.File
 import java.util.Scanner
 
@@ -74,7 +73,7 @@ fun File.addTempFile(
     content: String = "default content",
 ): File =
     File(this, fileName).also {
-        IOUtils.writeStringToFile(it, content)
+        it.writeText(content)
         it.deleteOnExit()
     }
 

…ava/com/ichi2/testutils/HamcrestUtils.kt …ava/com/ichi2/testutils/HamcrestUtils.ktAnkiDroid/src/test/java/com/ichi2/testutils/HamcrestUtils.kt renamed to common/src/testFixtures/java/com/ichi2/testutils/HamcrestUtils.kt AnkiDroid/src/test/java/com/ichi2/testutils/HamcrestUtils.kt renamed to common/src/testFixtures/java/com/ichi2/testutils/HamcrestUtils.kt

@@ -7,6 +7,7 @@ plugins {
 
 android {
     namespace = "com.ichi2.anki.compat"
+    testFixtures.enable = true
     compileSdk =
         libs.versions.compileSdk
             .get()
@@ -40,9 +41,23 @@ dependencies {
     implementation(libs.androidx.appcompat)
     implementation(libs.jakewharton.timber)
 
+    testImplementation(testFixtures(project(":common")))
+    testImplementation(testFixtures(project(":compat")))
     testImplementation(libs.junit.jupiter)
     testImplementation(libs.junit.vintage.engine)
     testImplementation(libs.hamcrest)
     testImplementation(libs.junit.platform.launcher)
+    testImplementation(libs.mockk)
+    testImplementation(libs.mockito.kotlin)
     testImplementation(kotlin("test"))
+
+    testFixturesImplementation(project(":common"))
+    testFixturesImplementation(testFixtures(project(":common")))
+    testFixturesImplementation(libs.mockk)
+    testFixturesImplementation(libs.mockito.kotlin)
+    testFixturesImplementation(libs.junit.vintage.engine)
+    testFixturesImplementation(kotlin("test"))
+    // Required so the ExperimentalCoroutinesApi opt-in (applied globally) doesn't cause
+    // an "unresolved" warning, which is treated as an error due to allWarningsAsErrors
+    testFixturesImplementation(libs.kotlinx.coroutines.core)
 }

…st/java/com/ichi2/utils/FileOperation.kt …es/java/com/ichi2/utils/FileOperation.ktAnkiDroid/src/test/java/com/ichi2/utils/FileOperation.kt renamed to common/src/testFixtures/java/com/ichi2/utils/FileOperation.kt AnkiDroid/src/test/java/com/ichi2/utils/FileOperation.kt renamed to common/src/testFixtures/java/com/ichi2/utils/FileOperation.kt

@@ -20,7 +20,6 @@ import com.ichi2.anki.compat.hasFiles
 import com.ichi2.compat.Test21And26
 import com.ichi2.testutils.HamcrestUtils.containsInAnyOrder
 import com.ichi2.testutils.withTempFile
-import org.acra.util.IOUtils
 import org.hamcrest.CoreMatchers.equalTo
 import org.hamcrest.CoreMatchers.not
 import org.hamcrest.CoreMatchers.nullValue
@@ -106,7 +105,7 @@ class DirectoryTest : Test21And26() {
     @Test
     fun has_files_is_true_if_file() {
         val dir = createValidTempDir()
-        IOUtils.writeStringToFile(File(dir.directory, "aa.txt"), "aa")
+        File(dir.directory, "aa.txt").writeText("aa")
         MatcherAssert.assertThat(
             "non-empty directory should have files",
             dir.hasFiles(),