test: simplify template previewer tests

the collection media dir is necessary for running `prepareCardTextForDisplay` when a card is shown, hence why a different CollectionStorageMode was necessary

since `prepareCardTextForDisplay` isn't necessary for any of the current tests and a different CollectionStorageMode requires a heavier test configuration with Robolectric, its usage is being modified.

Author: BrayanDSO

AnkiDroid/src/main/java/com/ichi2/anki/previewer/CardViewerViewModel.kt

@@ -18,6 +18,7 @@ package com.ichi2.anki.previewer
 import android.media.MediaPlayer
 import android.net.Uri
 import androidx.annotation.CallSuper
+import androidx.annotation.VisibleForTesting
 import androidx.core.net.toFile
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
@@ -116,7 +117,8 @@ abstract class CardViewerViewModel :
     /** From the [desktop code](https://github.com/ankitects/anki/blob/1ff55475b93ac43748d513794bcaabd5d7df6d9d/qt/aqt/reviewer.py#L358) */
     private suspend fun mungeQA(text: String) = typeAnsFilter(prepareCardTextForDisplay(text))
 
-    private suspend fun prepareCardTextForDisplay(text: String): String =
+    @VisibleForTesting
+    suspend fun prepareCardTextForDisplay(text: String): String =
         replaceAvRefsWithPlayButtons(
             text = withCol { media.escapeMediaFilenames(text) },
             renderOutput = currentCard.await().let { card -> withCol { card.renderOutput(this) } },

AnkiDroid/src/test/java/com/ichi2/anki/previewer/TemplatePreviewerViewModelFileSystemTest.kt

@@ -17,20 +17,29 @@ package com.ichi2.anki.previewer
 
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import com.ichi2.anki.NotetypeFile
-import com.ichi2.anki.libanki.testutils.AnkiTest
 import com.ichi2.testutils.JvmTest
+import io.mockk.coEvery
+import io.mockk.spyk
 import org.hamcrest.CoreMatchers.equalTo
 import org.hamcrest.MatcherAssert.assertThat
 import org.junit.Rule
 import org.junit.Test
 import org.junit.rules.TemporaryFolder
 import org.junit.runner.RunWith
+import kotlin.test.assertNotEquals
 
 @RunWith(AndroidJUnit4::class)
 class TemplatePreviewerViewModelTest : JvmTest() {
     @get:Rule
     val tempDirectory = TemporaryFolder()
 
+    private fun createViewModel(arguments: TemplatePreviewerArguments): TemplatePreviewerViewModel =
+        spyk(TemplatePreviewerViewModel(arguments)).apply {
+            // the default implementation requires the Collection media directory,
+            // which needs a Robolectric setup with CollectionStorageMode.IN_MEMORY_WITH_MEDIA or ON_DISK
+            coEvery { prepareCardTextForDisplay(any()) } answers { firstArg() }
+        }
+
     @Test
     fun `getCurrentTabIndex returns the correct tab if the first cloze isn't 1 and ord isn't 0`() =
         runClozeTest(ord = 6, fields = mutableListOf("{{c7::foo}} {{c4::bar}} {{c9::ha}}")) {
@@ -57,9 +66,20 @@ class TemplatePreviewerViewModelTest : JvmTest() {
         ) {
             onPageFinished(false)
             assertThat(this.cardsWithEmptyFronts!!.await()[0], equalTo(false))
-            assertThat(this.cardsWithEmptyFronts!!.await()[1], equalTo(true))
+            assertThat(this.cardsWithEmptyFronts.await()[1], equalTo(true))
         }
 
+    @Test
+    fun `card ords are changed`() {
+        runClozeTest(tempDirectory = tempDirectory, fields = mutableListOf("{{c1::one}} {{c2::bar}}")) {
+            onPageFinished(false)
+            val ord1 = currentCard.await().ord
+            onTabSelected(1)
+            val ord2 = currentCard.await().ord
+            assertNotEquals(ord1, ord2)
+        }
+    }
+
     private fun runClozeTest(
         ord: Int = 0,
         fields: MutableList<String>? = null,
@@ -79,25 +99,25 @@ class TemplatePreviewerViewModelTest : JvmTest() {
                 tags = mutableListOf(),
                 ord = ord,
             )
-        val viewModel = TemplatePreviewerViewModel(arguments)
+        val viewModel = createViewModel(arguments)
         block(viewModel)
     }
-}
 
-fun AnkiTest.runClozeTest(
-    ord: Int = 0,
-    tempDirectory: TemporaryFolder,
-    fields: MutableList<String>? = null,
-    block: suspend TemplatePreviewerViewModel.() -> Unit,
-) = runTest {
-    val notetype = col.notetypes.byName("Cloze")!!
-    val arguments =
-        TemplatePreviewerArguments(
-            notetypeFile = NotetypeFile(tempDirectory.root, notetype),
-            fields = fields ?: mutableListOf("{{c1::foo}} {{c2::bar}}", "anki"),
-            tags = mutableListOf(),
-            ord = ord,
-        )
-    val viewModel = TemplatePreviewerViewModel(arguments)
-    block(viewModel)
+    private fun runClozeTest(
+        ord: Int = 0,
+        tempDirectory: TemporaryFolder,
+        fields: MutableList<String>? = null,
+        block: suspend TemplatePreviewerViewModel.() -> Unit,
+    ) = runTest {
+        val notetype = col.notetypes.byName("Cloze")!!
+        val arguments =
+            TemplatePreviewerArguments(
+                notetypeFile = NotetypeFile(tempDirectory.root, notetype),
+                fields = fields ?: mutableListOf("{{c1::foo}} {{c2::bar}}", "anki"),
+                tags = mutableListOf(),
+                ord = ord,
+            )
+        val viewModel = createViewModel(arguments)
+        block(viewModel)
+    }
 }