refactor(test): rename 'TestClass' to 'AnkiTest'

`com.ichi2.anki.libanki.testutils.TestClass`
`com.ichi2.anki.libanki.testutils.AnkiTest`

This is now for 'LibAnki' only, enforced by the module
the name makes the intention of the class much more clear

Author: david-allison

AnkiDroid/src/test/java/com/ichi2/anki/CardBrowserTest.kt

@@ -84,7 +84,7 @@ import com.ichi2.anki.libanki.CardType
 import com.ichi2.anki.libanki.Note
 import com.ichi2.anki.libanki.NotetypeJson
 import com.ichi2.anki.libanki.QueueType
-import com.ichi2.anki.libanki.testutils.TestClass
+import com.ichi2.anki.libanki.testutils.AnkiTest
 import com.ichi2.anki.model.CardsOrNotes.CARDS
 import com.ichi2.anki.model.CardsOrNotes.NOTES
 import com.ichi2.anki.model.SortType
@@ -1633,7 +1633,7 @@ suspend fun CardBrowserViewModel.setFlagFilterSync(flag: Flag) {
     searchJob?.join()
 }
 
-fun TestClass.flagCardForNote(
+fun AnkiTest.flagCardForNote(
     n: Note,
     flag: Flag,
 ) {

AnkiDroid/src/test/java/com/ichi2/anki/RobolectricTest.kt

@@ -46,7 +46,7 @@ import com.ichi2.anki.libanki.Collection
 import com.ichi2.anki.libanki.Note
 import com.ichi2.anki.libanki.NotetypeJson
 import com.ichi2.anki.libanki.Storage
-import com.ichi2.anki.libanki.testutils.TestClass
+import com.ichi2.anki.libanki.testutils.AnkiTest
 import com.ichi2.anki.observability.ChangeManager
 import com.ichi2.anki.observability.undoableOp
 import com.ichi2.anki.preferences.sharedPrefs
@@ -88,7 +88,7 @@ import timber.log.Timber
 import kotlin.test.assertNotNull
 
 open class RobolectricTest :
-    TestClass,
+    AnkiTest,
     AndroidTest {
     @Suppress("PLATFORM_CLASS_MAPPED_TO_KOTLIN")
     private fun Any.wait(timeMs: Long) = (this as Object).wait(timeMs)

AnkiDroid/src/test/java/com/ichi2/anki/browser/CardBrowserViewModelTest.kt

@@ -64,7 +64,7 @@ import com.ichi2.anki.libanki.Note
 import com.ichi2.anki.libanki.QueueType
 import com.ichi2.anki.libanki.QueueType.ManuallyBuried
 import com.ichi2.anki.libanki.QueueType.New
-import com.ichi2.anki.libanki.testutils.TestClass
+import com.ichi2.anki.libanki.testutils.AnkiTest
 import com.ichi2.anki.model.CardsOrNotes
 import com.ichi2.anki.model.SortType
 import com.ichi2.anki.model.SortType.NO_SORTING
@@ -1287,7 +1287,7 @@ internal suspend fun CardBrowserViewModel.invokeInitialSearch() {
     Timber.v("initial search completed")
 }
 
-private fun TestClass.assertAllSuspended(context: String) {
+private fun AnkiTest.assertAllSuspended(context: String) {
     val cards = col.findCards("").map { col.getCard(it) }
     assertThat("performance", cards.size, lessThan(10))
 
@@ -1300,7 +1300,7 @@ private fun TestClass.assertAllSuspended(context: String) {
     }
 }
 
-private fun TestClass.assertAllUnsuspended(context: String) {
+private fun AnkiTest.assertAllUnsuspended(context: String) {
     val cards = col.findCards("").map { col.getCard(it) }
     assertThat("performance", cards.size, lessThan(10))
 
@@ -1313,18 +1313,18 @@ private fun TestClass.assertAllUnsuspended(context: String) {
     }
 }
 
-private fun TestClass.suspendAll() {
+private fun AnkiTest.suspendAll() {
     col.findCards("").also { cards ->
         col.sched.suspendCards(col.findCards(""))
         Timber.d("suspended %d cards", cards.size)
     }
 }
 
-private fun TestClass.suspendCards(vararg cardIds: CardId) {
+private fun AnkiTest.suspendCards(vararg cardIds: CardId) {
     col.sched.suspendCards(ids = cardIds.toList())
 }
 
-private fun TestClass.suspendNote(note: Note) {
+private fun AnkiTest.suspendNote(note: Note) {
     col.sched.suspendCards(note.cardIds(col))
 }
 

AnkiDroid/src/test/java/com/ichi2/anki/instanteditor/InstantEditorViewModelTest.kt

@@ -23,7 +23,7 @@ import com.ichi2.anki.RobolectricTest
 import com.ichi2.anki.instantnoteeditor.InstantEditorViewModel
 import com.ichi2.anki.instantnoteeditor.InstantNoteEditorActivity
 import com.ichi2.anki.instantnoteeditor.SaveNoteResult
-import com.ichi2.anki.libanki.testutils.TestClass
+import com.ichi2.anki.libanki.testutils.AnkiTest
 import org.junit.Assert.assertEquals
 import org.junit.Assert.assertTrue
 import org.junit.Test
@@ -231,7 +231,7 @@ class InstantEditorViewModelTest : RobolectricTest() {
         }
 
     companion object {
-        fun TestClass.runInstantEditorViewModelTest(
+        fun AnkiTest.runInstantEditorViewModelTest(
             initViewModel: () -> InstantEditorViewModel = { InstantEditorViewModel() },
             testBody: suspend InstantEditorViewModel.() -> Unit,
         ) = runTest {

AnkiDroid/src/test/java/com/ichi2/testutils/JvmTest.kt

@@ -24,7 +24,7 @@ import com.ichi2.anki.common.time.TimeManager
 import com.ichi2.anki.ioDispatcher
 import com.ichi2.anki.libanki.Collection
 import com.ichi2.anki.libanki.Storage
-import com.ichi2.anki.libanki.testutils.TestClass
+import com.ichi2.anki.libanki.testutils.AnkiTest
 import com.ichi2.anki.observability.ChangeManager
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.runBlocking
@@ -44,7 +44,7 @@ import org.robolectric.junit.rules.TimeoutRule
 import timber.log.Timber
 import timber.log.Timber.Forest.plant
 
-open class JvmTest : TestClass {
+open class JvmTest : AnkiTest {
     @get:Rule
     val timeoutRule: TimeoutRule = TimeoutRule.seconds(60)
 

…ava/com/ichi2/testutils/ext/TestClass.kt …java/com/ichi2/testutils/ext/AnkiTest.ktAnkiDroid/src/test/java/com/ichi2/testutils/ext/TestClass.kt renamed to AnkiDroid/src/test/java/com/ichi2/testutils/ext/AnkiTest.kt AnkiDroid/src/test/java/com/ichi2/testutils/ext/TestClass.kt renamed to AnkiDroid/src/test/java/com/ichi2/testutils/ext/AnkiTest.kt

@@ -20,10 +20,10 @@ import anki.collection.OpChanges
 import com.ichi2.anki.libanki.Consts
 import com.ichi2.anki.libanki.Note
 import com.ichi2.anki.libanki.NotetypeJson
-import com.ichi2.anki.libanki.testutils.TestClass
+import com.ichi2.anki.libanki.testutils.AnkiTest
 import com.ichi2.anki.observability.undoableOp
 
-suspend fun TestClass.addBasicNoteWithOp(
+suspend fun AnkiTest.addBasicNoteWithOp(
     fields: List<String> = listOf("foo", "bar"),
     noteType: NotetypeJson = col.notetypes.byName("Basic")!!,
 ): Note =

…chi2/anki/libanki/testutils/TestClass.kt …ichi2/anki/libanki/testutils/AnkiTest.ktlibanki/testutils/src/main/java/com/ichi2/anki/libanki/testutils/TestClass.kt renamed to libanki/testutils/src/main/java/com/ichi2/anki/libanki/testutils/AnkiTest.kt libanki/testutils/src/main/java/com/ichi2/anki/libanki/testutils/TestClass.kt renamed to libanki/testutils/src/main/java/com/ichi2/anki/libanki/testutils/AnkiTest.kt

@@ -28,7 +28,6 @@ import com.ichi2.anki.libanki.NotetypeJson
 import com.ichi2.anki.libanki.Notetypes
 import com.ichi2.anki.libanki.QueueType
 import com.ichi2.anki.libanki.exception.ConfirmModSchemaException
-import com.ichi2.anki.libanki.testutils.TestCollectionManager
 import com.ichi2.anki.libanki.testutils.ext.addNote
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.test.StandardTestDispatcher
@@ -47,7 +46,7 @@ import kotlin.time.Duration.Companion.milliseconds
  *
  * Android (AnkiDroid/Robolectric) is not required for these tests to run
  */
-interface TestClass {
+interface AnkiTest {
     val col: Collection
 
     val collectionManager: TestCollectionManager
@@ -243,7 +242,7 @@ interface TestClass {
     /** Helper method to update a card */
     fun Card.update(update: Card.() -> Unit): Card {
         update(this)
-        this@TestClass.col.updateCard(this, skipUndoEntry = true)
+        this@AnkiTest.col.updateCard(this, skipUndoEntry = true)
         return this
     }