[RunAllTests] Consolidate testing to ensure all tests are covered in Bazel CI

app/app_test.bzl

@@ -1,25 +1,28 @@
-load("//:oppia_android_test.bzl", "oppia_android_test")
+load("//:oppia_android_test.bzl", "oppia_android_module_level_test")
 
 # TODO(#1620): Remove module-specific test macros once Gradle is removed
-def app_test(name, srcs, test_class, deps, **kwargs):
-    '''
+def app_test(name, processed_src, test_path_prefix, filtered_tests, deps, **kwargs):
+    """
     Creates individual tests for test files in the app module.
 
     Args:
-        name: str. The name of the Kotlin test file without the '.kt' suffix.
-        src: list of str. The list of test files to be run.
-        test_class: str. The package of the src file. Example: If the src is 'FakeEventLoggerTest.kt',
-            then the test_class would be "org.oppia.testing.FakeEventLoggerTest".
+        name: str. The relative path to the Kotlin test file.
+        processed_src: str. The source to a processed version of the test that should be used
+            instead of the original.
+        test_path_prefix: str. The prefix of the test path (which is used to extract the qualified
+            class name of the test suite).
+        filtered_tests: list of str. The test files that should not have tests defined for them.
         deps: list of str. The list of dependencies needed to build and run this test.
-    '''
-
-    oppia_android_test(
+    """
+    oppia_android_module_level_test(
         name = name,
-        srcs = srcs + ["src/test/java/DataBinderMapperImpl.java"],
+        processed_src = processed_src,
+        filtered_tests = filtered_tests,
+        test_path_prefix = test_path_prefix,
+        deps = deps,
         custom_package = "org.oppia.android.app.test",
-        test_class = test_class,
         test_manifest = "src/test/AndroidManifest.xml",
-        deps = deps,
+        additional_srcs = ["src/test/java/DataBinderMapperImpl.java"],
         enable_data_binding = True,
         **kwargs
     )

app/src/test/java/org/oppia/android/app/testing/options/OptionsFragmentTest.kt

@@ -74,6 +74,11 @@ import org.robolectric.annotation.LooperMode
 import javax.inject.Inject
 import javax.inject.Singleton
 
+private const val KEY_READING_TEXT_SIZE_PREFERENCE_TITLE = "READING_TEXT_SIZE_PREFERENCE"
+private const val APP_LANGUAGE_PREFERENCE_TITLE_EXTRA_KEY =
+  "AppLanguageActivity.app_language_preference_title"
+private const val KEY_AUDIO_LANGUAGE_PREFERENCE_TITLE = "AUDIO_LANGUAGE_PREFERENCE"
+
 @RunWith(AndroidJUnit4::class)
 @Config(application = OptionsFragmentTest.TestApplication::class)
 class OptionsFragmentTest {
@@ -112,12 +117,7 @@ class OptionsFragmentTest {
         click()
       )
       intended(hasComponent(ReadingTextSizeActivity::class.java.name))
-      intended(
-        hasExtra(
-          ReadingTextSizeActivity.KEY_READING_TEXT_SIZE_PREFERENCE_TITLE,
-          READING_TEXT_SIZE
-        )
-      )
+      intended(hasExtra(KEY_READING_TEXT_SIZE_PREFERENCE_TITLE, READING_TEXT_SIZE))
     }
   }
 
@@ -135,12 +135,7 @@ class OptionsFragmentTest {
         click()
       )
       intended(hasComponent(AppLanguageActivity::class.java.name))
-      intended(
-        hasExtra(
-          AppLanguageActivity.APP_LANGUAGE_PREFERENCE_TITLE_EXTRA_KEY,
-          APP_LANGUAGE
-        )
-      )
+      intended(hasExtra(APP_LANGUAGE_PREFERENCE_TITLE_EXTRA_KEY, APP_LANGUAGE))
     }
   }
 
@@ -158,12 +153,7 @@ class OptionsFragmentTest {
         click()
       )
       intended(hasComponent(AudioLanguageActivity::class.java.name))
-      intended(
-        hasExtra(
-          AudioLanguageActivity.KEY_AUDIO_LANGUAGE_PREFERENCE_TITLE,
-          AUDIO_LANGUAGE
-        )
-      )
+      intended(hasExtra(KEY_AUDIO_LANGUAGE_PREFERENCE_TITLE, AUDIO_LANGUAGE))
     }
   }