Fixes #5972: Profile name accepts invalid inputs like single dot (.) or single space ( )

[ShankhanilSaha]

Explanation

Fixes #5972

• Updates utility/src/main/java/org/oppia/android/util/profile/ProfileNameValidator.kt to only allow names which have only alphabets and spaces.
• Now names with single dots(.) and single spaces( ) are not allowed anymore which previously were.
• Creates function createDefaultProfile() in domain/src/main/java/org/oppia/android/domain/profile/ProfileManagementController.kt to create a default profile during onboarding with no checks other than checking no accounts are already present to prevent create a invalid account.
• Create exception type PROFILES_ALREADY_EXIST for function to return if condition not met.
• Replace addProfile() with createDefaultProfile() in createDefaultAdminProfile() and createDefaultProfile().

Essential Checklist

• [x] The PR title and explanation each start with "Fix #bugnum: " (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
• [x] Any changes to scripts/assets files have their rationale included in the PR explanation.
• [x] The PR follows the style guide.
• [x] The PR does not contain any unnecessary code changes from Android Studio (reference).
• [x] The PR is made from a branch that's not called "develop" and is up-to-date with "develop".
• [x] The PR is assigned to the appropriate reviewers (reference).

[ShankhanilSaha]

Hi @adhiamboperes, the tests are primarily failing because of the tests in utility/src/test/java/org/oppia/android/util/profile/ProfileNameValidatorTest.kt should i completely remove the tests which are verifying names which were formerly allowed? or should i update them to something like the code in the attatchment which will expect false to formerly allowed names.

[ShankhanilSaha]

@adhiamboperes could you please check the error logs once, some tests were failing previously so i fixed them by changing the functions to addAdminProfileAndWait() which fixed the issue of the test looking for a profile before it was even created. But now it seems like there is some maven related dependency downloading issue which i believe has not arrived from my code, could be wrong.

[theMr17]

@ShankhanilSaha I've re-run the tests, they should work fine now.

[ShankhanilSaha]

@adhiamboperes upon making the file changes in this PR a few tests fail in app/src/sharedTest/java/org/oppia/android/app/onboarding/CreateProfileFragmentTest.kt and testing/src/test/java/org/oppia/android/testing/profile/ProfileTestHelperTest.kt even tho tests are unrelated to the changes since they don't use any name which passed previously but gets rejected with the changes. I could use some help with this one.

[adhiamboperes]

Hi @ShankhanilSaha, those tests are failing because they rely on the ProfileManagementController, which relies on the ProfileNameValidator

[adhiamboperes]

Looking at the traces, it appears that the failing functions rely on an exception that has changed, so there are changes needed downstream as well. I need to review the code first.

[adhiamboperes]

CreateProfileFragmentTest

@ShankhanilSaha, the failing tests in CreateProfileFragmentTest and ProfileTestHelperTest are most likely due to the fact that we are no longer allowing empty names, but in the OnboardingFragmentPresenter, we have a createDefaultProfile function which is meant to create a dummy profile, to be filled later. This lead to the creation of fun createDefaultAdminProfile() in ProfileTestHelper as a helper functions for the scenarios where we need a default empty profile.

The failing tests are probably usages of ProfileTestHelper.createDefaultAdminProfile(). Please review each of the failing tests and see if any usage of createDefaultAdminProfile can be replaced by addOnlyAdminProfile, and in cases where the function cannot be replaced(because it is the functionality under test), or you are unsure, please raise these in the group chat for resolution.

[ShankhanilSaha]

Hi @adhiamboperes , in CreateProfileFragmentTest and ProfileTestHelperTest, the appropriate files are not loaded for the addAdminProfileAndWait() function.

However, profileTestHelper has another function addOnlyAdminProfile(). Unlike createDefaultAdminProfile(), this starts the setup fresh and ensures there is initially only a single admin profile. I believe this is more suitable for tests, and it fixes the failures (verified locally).

That said, since this doesn't relate directly to the name validation logic, I am not sure why createDefaultAdminProfile() causes the failures. I noticed in previous tests I fixed that createDefaultAdminProfile() was often the root cause. It may not be suitable for tests due to slow execution (I tried using addAdminProfileAndWait() because the logs showed the test checking for a profile before the function had finished creating it).

[adhiamboperes]

Hi @ShankhanilSaha,

That said, since this doesn't relate directly to the name validation logic, I am not sure why createDefaultAdminProfile() causes the failures. I noticed in previous tests I fixed that createDefaultAdminProfile() was often the root cause. It may not be suitable for tests due to slow execution (I tried using addAdminProfileAndWait() because the logs showed the test checking for a profile before the function had finished creating it).

createDefaultAdminProfile() fails because if you look at the implementation of the function, it creates a profile with an empty name. The new validation logic prohibits empty names. So the function should be updated to have a profile name, e.g. John.

[adhiamboperes]

Please also remember to update the default profile in OnboardingFragmentPresenter to have a default name, "Admin" while keeping the rest of the fields blank as they are. Refer to my previous explanation quoted below.

@ShankhanilSaha, the failing tests in CreateProfileFragmentTest and ProfileTestHelperTest are most likely due to the fact that we are no longer allowing empty names, but in the OnboardingFragmentPresenter, we have a createDefaultProfile function which is meant to create a dummy profile, to be filled later. This lead to the creation of fun createDefaultAdminProfile() in ProfileTestHelper as a helper functions for the scenarios where we need a default empty profile.

[ShankhanilSaha]

Hi @adhiamboperes could you please let me know about the new function replacing the createDefaultAdminProfile() in brief.

[adhiamboperes]

Hi Shankhanil, Please create a new function in the ProfileManagementController for creating a default profile. It would have no validations(such as the name validation) and all fields would have default values. It should however check that there are no existing profiles(to avoid misuse).

This function should only be used on the onboarding fargment presenter(where the default profile is currently being created) and the test helper(also as a replacement of the create default profile helper).

Please let me know if you need any further support.

[ShankhanilSaha]

Hi @adhiamboperes , i am primarily referring to the addProfile() fucntion in the ProfileManagementController file as it is somewhat similar to the function I need to make. Had some questions about the default profile being created, the parameters being passed name: String, pin: String, avatarImagePath: Uri?, allowDownloadAccess: Boolean, colorRgb: Int, isAdmin: Boolean you said should all be default values. Should I just hard code the default values in the function definition or can just make the newProfileBuilder without any parameters and it somehow automatically handles them. If not the later what should the default values be such as name, etc. Also should the profile be admin?

[adhiamboperes]

Hi @adhiamboperes , i am primarily referring to the addProfile() fucntion in the ProfileManagementController file as it is somewhat similar to the function I need to make. Had some questions about the default profile being created, the parameters being passed name: String, pin: String, avatarImagePath: Uri?, allowDownloadAccess: Boolean, colorRgb: Int, isAdmin: Boolean you said should all be default values. Should I just hard code the default values in the function definition or can just make the newProfileBuilder without any parameters and it somehow automatically handles them. If not the later what should the default values be such as name, etc. Also should the profile be admin?

You should hard code them in the arguments, similar to what is passed in the createDefaultAdminProfile()

[ShankhanilSaha]

Hi @adhiamboperes , for checking if there are no existing profile

if (it.profilesCount > 0) {
    //Returns in case profiles 
    return@storeDataWithCustomChannelAsync Pair(it, ProfileActionStatus.PROFILES_EXIST) do exist in the db
}

what should I pair in the return statement? Shall I add a attribute in ProfileActionStatus called PROFILES_EXIST? Also should the default profile have admin access?

[ShankhanilSaha]

Also this is my proposed function, i just took addProfile() as a reference. Could you please take a look and let me know the changes needed?

    fun createDefaultProfile(): DataProvider<Any?> {
    val deferred = profileDataStore.storeDataWithCustomChannelAsync(
      updateInMemoryCache = true
    ) {
      if (it.profilesCount > 0) {
        return@storeDataWithCustomChannelAsync Pair(it, ProfileActionStatus.PROFILES_EXIST)
      }

      val nextProfileId = it.nextProfileId
      val profileDir = directoryManagementUtil.getOrCreateDir(nextProfileId.toString())

      val newProfile = Profile.newBuilder().apply {
        this.name = ""
        this.pin = ""
        this.allowDownloadAccess = false
        this.allowInLessonQuickLanguageSwitching = false
        this.id = ProfileId.newBuilder().setInternalId(nextProfileId).build()
        dateCreatedTimestampMs = oppiaClock.getCurrentTimeMs()
        this.isAdmin = false
        readingTextSize = ReadingTextSize.MEDIUM_TEXT_SIZE
        numberOfLogins = 0

        avatar = ProfileAvatar.newBuilder().apply {
          avatarColorRgb = -10710042
        }.build()
      }.build()

      val wasProfileEverAdded = it.profilesCount > 0

      val profileDatabaseBuilder =
        it.toBuilder()
          .putProfiles(nextProfileId, newProfile)
          .setWasProfileEverAdded(wasProfileEverAdded)
          .setNextProfileId(nextProfileId + 1)
      Pair(profileDatabaseBuilder.build(), ProfileActionStatus.SUCCESS)
    }
    return dataProviders.createInMemoryDataProviderAsync(ADD_PROFILE_PROVIDER_ID) {
      return@createInMemoryDataProviderAsync getDeferredResult(null, name, deferred)
    }
  }

[oppiabot[bot]]

Unassigning @ShankhanilSaha since a re-review was requested. @ShankhanilSaha, please make sure you have addressed all review comments. Thanks!

[ShankhanilSaha]

@adhiamboperes The function is created and applied in the necessary places, Please take a look. Also the tests seem to be failing again for those dependencies download issue.

[oppiabot[bot]]

Unassigning @adhiamboperes since the review is done.

[oppiabot[bot]]

Hi @ShankhanilSaha, it looks like some changes were requested on this pull request by @adhiamboperes. PTAL. Thanks!

[ShankhanilSaha]

@adhiamboperes I have added the mentioned tests and addressed the requested changes. Please take a look.

[oppiabot[bot]]

Unassigning @ShankhanilSaha since a re-review was requested. @ShankhanilSaha, please make sure you have addressed all review comments. Thanks!

[oppiabot[bot]]

Unassigning @adhiamboperes since the review is done.

[oppiabot[bot]]

Hi @ShankhanilSaha, it looks like some changes were requested on this pull request by @adhiamboperes. PTAL. Thanks!

[github-actions[bot]]

Coverage Report

Results

Number of files assessed: 129 Overall Coverage: 86.31% Coverage Analysis: PASS :white_check_mark:

Passing coverage

Files with passing code coverage

File	Coverage	Lines Hit	Status	Min Required
LintCommon.kt scripts/src/java/org/oppia/android/scripts/lint/LintCommon.kt	96.15%	75 / 78	:white_check_mark:	70%
AndroidLintRunner.kt scripts/src/java/org/oppia/android/scripts/lint/AndroidLintRunner.kt	73.36%	190 / 259	:white_check_mark:	70%
LintProjectDescription.kt scripts/src/java/org/oppia/android/scripts/lint/LintProjectDescription.kt	80.95%	323 / 399	:white_check_mark:	70%
LintModelCreator.kt scripts/src/java/org/oppia/android/scripts/lint/LintModelCreator.kt	81.82%	180 / 220	:white_check_mark:	70%
RepositoryFile.kt scripts/src/java/org/oppia/android/scripts/common/RepositoryFile.kt	96.15%	25 / 26	:white_check_mark:	70%
TransformAndroidManifest.kt scripts/src/java/org/oppia/android/scripts/build/TransformAndroidManifest.kt	100.00%	72 / 72	:white_check_mark:	70%
ComputeChangedFiles.kt scripts/src/java/org/oppia/android/scripts/ci/ComputeChangedFiles.kt	96.53%	167 / 173	:white_check_mark:	70%
ComputeAffectedTests.kt scripts/src/java/org/oppia/android/scripts/ci/ComputeAffectedTests.kt	95.88%	163 / 170	:white_check_mark:	70%
RegexPatternValidationCheck.kt scripts/src/java/org/oppia/android/scripts/regex/RegexPatternValidationCheck.kt	100.00%	104 / 104	:white_check_mark:	70%
AccessibilityLabelCheck.kt scripts/src/java/org/oppia/android/scripts/label/AccessibilityLabelCheck.kt	100.00%	68 / 68	:white_check_mark:	70%
ProfileNameValidator.kt utility/src/main/java/org/oppia/android/util/profile/ProfileNameValidator.kt	100.00%	13 / 13	:white_check_mark:	70%
EventBundleCreator.kt utility/src/main/java/org/oppia/android/util/logging/EventBundleCreator.kt	75.28%	338 / 449	:white_check_mark:	70%
ExplorationStorageTestModule.kt domain/src/main/java/org/oppia/android/domain/exploration/testing/ExplorationStorageTestModule.kt	100.00%	4 / 4	:white_check_mark:	70%
FakeExplorationRetriever.kt domain/src/main/java/org/oppia/android/domain/exploration/testing/FakeExplorationRetriever.kt	100.00%	10 / 10	:white_check_mark:	70%
TranslationController.kt domain/src/main/java/org/oppia/android/domain/translation/TranslationController.kt	94.12%	160 / 170	:white_check_mark:	70%
LanguageConfigRetriever.kt domain/src/main/java/org/oppia/android/domain/locale/LanguageConfigRetriever.kt	100.00%	5 / 5	:white_check_mark:	70%
FeatureFlagsLogger.kt domain/src/main/java/org/oppia/android/domain/oppialogger/analytics/FeatureFlagsLogger.kt	100.00%	22 / 22	:white_check_mark:	70%
AppStartupStateController.kt domain/src/main/java/org/oppia/android/domain/onboarding/AppStartupStateController.kt	91.57%	76 / 83	:white_check_mark:	70%
InitializeDefaultLocaleRule.kt testing/src/main/java/org/oppia/android/testing/junit/InitializeDefaultLocaleRule.kt	100.00%	85 / 85	:white_check_mark:	100% *

* represents tests with custom overridden pass/fail coverage thresholds

Exempted coverage

Files exempted from coverage

File	Exemption Reason
TestPlatformParameterModule.kt testing/src/main/java/org/oppia/android/testing/platformparameter/TestPlatformParameterModule.kt	This file is exempted from having a test file; skipping coverage check.
ExplorationCheckpointTestHelper.kt testing/src/main/java/org/oppia/android/testing/lightweightcheckpointing/ExplorationCheckpointTestHelper.kt	This file is incompatible with code coverage tooling; skipping coverage check.
GenericViewMatchers.kt testing/src/main/java/org/oppia/android/testing/espresso/GenericViewMatchers.kt	This file is exempted from having a test file; skipping coverage check.
OppiaTestRule.kt testing/src/main/java/org/oppia/android/testing/OppiaTestRule.kt	This file is exempted from having a test file; skipping coverage check.
OppiaTestAnnotations.kt testing/src/main/java/org/oppia/android/testing/OppiaTestAnnotations.kt	This file is exempted from having a test file; skipping coverage check.
ProfileTestHelper.kt testing/src/main/java/org/oppia/android/testing/profile/ProfileTestHelper.kt	This file is incompatible with code coverage tooling; skipping coverage check.
StoryProgressTestHelper.kt testing/src/main/java/org/oppia/android/testing/story/StoryProgressTestHelper.kt	This file is incompatible with code coverage tooling; skipping coverage check.
FeatureFlagConstants.kt utility/src/main/java/org/oppia/android/util/platformparameter/FeatureFlagConstants.kt	This file is exempted from having a test file; skipping coverage check.
NetworkConnectionDebugUtilModule.kt utility/src/main/java/org/oppia/android/util/networking/NetworkConnectionDebugUtilModule.kt	This file is exempted from having a test file; skipping coverage check.
ListItemLeadingMarginSpan.kt utility/src/main/java/org/oppia/android/util/parser/html/ListItemLeadingMarginSpan.kt	This file is incompatible with code coverage tooling; skipping coverage check.
HtmlParser.kt utility/src/main/java/org/oppia/android/util/parser/html/HtmlParser.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ViewBindingShimImpl.kt app/src/main/java/org/oppia/android/app/shim/ViewBindingShimImpl.kt	This file is exempted from having a test file; skipping coverage check.
IntentFactoryShimImpl.kt app/src/main/java/org/oppia/android/app/shim/IntentFactoryShimImpl.kt	This file is exempted from having a test file; skipping coverage check.
ViewBindingShim.kt app/src/main/java/org/oppia/android/app/shim/ViewBindingShim.kt	This file is exempted from having a test file; skipping coverage check.
IntentFactoryShim.kt app/src/main/java/org/oppia/android/app/shim/IntentFactoryShim.kt	This file is exempted from having a test file; skipping coverage check.
ProfileProgressActivity.kt app/src/main/java/org/oppia/android/app/profileprogress/ProfileProgressActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
CompletedStoryListActivity.kt app/src/main/java/org/oppia/android/app/completedstorylist/CompletedStoryListActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
CompletedStoryListFragment.kt app/src/main/java/org/oppia/android/app/completedstorylist/CompletedStoryListFragment.kt	This file is exempted from having a test file; skipping coverage check.
OptionsActivity.kt app/src/main/java/org/oppia/android/app/options/OptionsActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ReadingTextSizeFragment.kt app/src/main/java/org/oppia/android/app/options/ReadingTextSizeFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
AudioLanguageSelectionViewModel.kt app/src/main/java/org/oppia/android/app/options/AudioLanguageSelectionViewModel.kt	This file is exempted from having a test file; skipping coverage check.
AudioLanguageFragmentPresenterV1.kt app/src/main/java/org/oppia/android/app/options/AudioLanguageFragmentPresenterV1.kt	This file is exempted from having a test file; skipping coverage check.
AppLanguageFragment.kt app/src/main/java/org/oppia/android/app/options/AppLanguageFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
OptionsFragment.kt app/src/main/java/org/oppia/android/app/options/OptionsFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
AudioLanguageFragment.kt app/src/main/java/org/oppia/android/app/options/AudioLanguageFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
WalkthroughTopicListFragment.kt app/src/main/java/org/oppia/android/app/walkthrough/topiclist/WalkthroughTopicListFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ApplicationInjector.kt app/src/main/java/org/oppia/android/app/application/ApplicationInjector.kt	This file is exempted from having a test file; skipping coverage check.
AbstractOppiaApplication.kt app/src/main/java/org/oppia/android/app/application/AbstractOppiaApplication.kt	This file is exempted from having a test file; skipping coverage check.
ApplicationComponent.kt app/src/main/java/org/oppia/android/app/application/ApplicationComponent.kt	This file is exempted from having a test file; skipping coverage check.
ApplicationInjectorProvider.kt app/src/main/java/org/oppia/android/app/application/ApplicationInjectorProvider.kt	This file is exempted from having a test file; skipping coverage check.
ProfileChooserFragmentTestActivity.kt app/src/main/java/org/oppia/android/app/testing/ProfileChooserFragmentTestActivity.kt	This file is exempted from having a test file; skipping coverage check.
TopicTestActivityForStory.kt app/src/main/java/org/oppia/android/app/testing/TopicTestActivityForStory.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ExplorationActivity.kt app/src/main/java/org/oppia/android/app/player/exploration/ExplorationActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
StateFragment.kt app/src/main/java/org/oppia/android/app/player/state/StateFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ClassroomListFragment.kt app/src/main/java/org/oppia/android/app/classroom/ClassroomListFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ClassroomListFragmentPresenter.kt app/src/main/java/org/oppia/android/app/classroom/ClassroomListFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
SplashActivityPresenter.kt app/src/main/java/org/oppia/android/app/splash/SplashActivityPresenter.kt	This file is exempted from having a test file; skipping coverage check.
SplashActivity.kt app/src/main/java/org/oppia/android/app/splash/SplashActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
LifecycleSafeTimerFactory.kt app/src/main/java/org/oppia/android/app/utility/lifecycle/LifecycleSafeTimerFactory.kt	This file is exempted from having a test file; skipping coverage check.
OngoingTopicListActivity.kt app/src/main/java/org/oppia/android/app/ongoingtopiclist/OngoingTopicListActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
OngoingTopicListFragment.kt app/src/main/java/org/oppia/android/app/ongoingtopiclist/OngoingTopicListFragment.kt	This file is exempted from having a test file; skipping coverage check.
TopicFragmentPresenter.kt app/src/main/java/org/oppia/android/app/topic/TopicFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
ViewPagerAdapter.kt app/src/main/java/org/oppia/android/app/topic/ViewPagerAdapter.kt	This file is exempted from having a test file; skipping coverage check.
QuestionPlayerActivity.kt app/src/main/java/org/oppia/android/app/topic/questionplayer/QuestionPlayerActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicActivity.kt app/src/main/java/org/oppia/android/app/topic/TopicActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicFragment.kt app/src/main/java/org/oppia/android/app/topic/TopicFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
RevisionCardFragment.kt app/src/main/java/org/oppia/android/app/topic/revisioncard/RevisionCardFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
RevisionCardActivity.kt app/src/main/java/org/oppia/android/app/topic/revisioncard/RevisionCardActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicLessonsFragment.kt app/src/main/java/org/oppia/android/app/topic/lessons/TopicLessonsFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicRevisionFragment.kt app/src/main/java/org/oppia/android/app/topic/revision/TopicRevisionFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicInfoFragment.kt app/src/main/java/org/oppia/android/app/topic/info/TopicInfoFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicTab.kt app/src/main/java/org/oppia/android/app/topic/TopicTab.kt	This file is exempted from having a test file; skipping coverage check.
ConceptCardFragment.kt app/src/main/java/org/oppia/android/app/topic/conceptcard/ConceptCardFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicPracticeFragment.kt app/src/main/java/org/oppia/android/app/topic/practice/TopicPracticeFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicViewModel.kt app/src/main/java/org/oppia/android/app/topic/TopicViewModel.kt	This file is exempted from having a test file; skipping coverage check.
AdminAuthActivity.kt app/src/main/java/org/oppia/android/app/profile/AdminAuthActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ProfileLoginFragment.kt app/src/main/java/org/oppia/android/app/profile/ProfileLoginFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ProfileChooserFragmentPresenterV1.kt app/src/main/java/org/oppia/android/app/profile/ProfileChooserFragmentPresenterV1.kt	This file is exempted from having a test file; skipping coverage check.
AdminAuthActivityPresenter.kt app/src/main/java/org/oppia/android/app/profile/AdminAuthActivityPresenter.kt	This file is exempted from having a test file; skipping coverage check.
ProfileChooserFragment.kt app/src/main/java/org/oppia/android/app/profile/ProfileChooserFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ProfileLoginFragmentPresenter.kt app/src/main/java/org/oppia/android/app/profile/ProfileLoginFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
ProfileChooserFragmentPresenter.kt app/src/main/java/org/oppia/android/app/profile/ProfileChooserFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
ProfileChooserActivity.kt app/src/main/java/org/oppia/android/app/profile/ProfileChooserActivity.kt	This file is exempted from having a test file; skipping coverage check.
ProfileChooserActivityPresenter.kt app/src/main/java/org/oppia/android/app/profile/ProfileChooserActivityPresenter.kt	This file is exempted from having a test file; skipping coverage check.
ExitProfileDialogFragment.kt app/src/main/java/org/oppia/android/app/drawer/ExitProfileDialogFragment.kt	This file is exempted from having a test file; skipping coverage check.
ResumeLessonActivity.kt app/src/main/java/org/oppia/android/app/resumelesson/ResumeLessonActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ProfileEditDeletionDialogFragment.kt app/src/main/java/org/oppia/android/app/settings/profile/ProfileEditDeletionDialogFragment.kt	This file is exempted from having a test file; skipping coverage check.
AdminIntroActivity.kt app/src/main/java/org/oppia/android/app/onboarding/AdminIntroActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
OnboardingProfileTypeFragment.kt app/src/main/java/org/oppia/android/app/onboarding/OnboardingProfileTypeFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
OnboardingFragmentPresenter.kt app/src/main/java/org/oppia/android/app/onboarding/OnboardingFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
AdminIntroFragmentPresenter.kt app/src/main/java/org/oppia/android/app/onboarding/AdminIntroFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
AdminIntroFragment.kt app/src/main/java/org/oppia/android/app/onboarding/AdminIntroFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
OnboardingFragment.kt app/src/main/java/org/oppia/android/app/onboarding/OnboardingFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
AudioLanguageFragmentPresenter.kt app/src/main/java/org/oppia/android/app/onboarding/AudioLanguageFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
CreateProfileFragment.kt app/src/main/java/org/oppia/android/app/onboarding/CreateProfileFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
OnboardingProfileTypeFragmentPresenter.kt app/src/main/java/org/oppia/android/app/onboarding/OnboardingProfileTypeFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
CreateProfileActivity.kt app/src/main/java/org/oppia/android/app/onboarding/CreateProfileActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
AdminIntroActivityPresenter.kt app/src/main/java/org/oppia/android/app/onboarding/AdminIntroActivityPresenter.kt	This file is exempted from having a test file; skipping coverage check.
WavyBackgroundView.kt app/src/main/java/org/oppia/android/app/onboarding/WavyBackgroundView.kt	This file is exempted from having a test file; skipping coverage check.
CreateProfileFragmentPresenter.kt app/src/main/java/org/oppia/android/app/onboarding/CreateProfileFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
HomeFragmentPresenter.kt app/src/main/java/org/oppia/android/app/home/HomeFragmentPresenter.kt	This file is exempted from having a test file; skipping coverage check.
RecentlyPlayedActivity.kt app/src/main/java/org/oppia/android/app/home/recentlyplayed/RecentlyPlayedActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
HomeActivity.kt app/src/main/java/org/oppia/android/app/home/HomeActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
InjectableAppCompatActivity.kt app/src/main/java/org/oppia/android/app/activity/InjectableAppCompatActivity.kt	This file is exempted from having a test file; skipping coverage check.
ActivityComponentImpl.kt app/src/main/java/org/oppia/android/app/activity/ActivityComponentImpl.kt	This file is exempted from having a test file; skipping coverage check.
FeatureFlagsFragment.kt app/src/main/java/org/oppia/android/app/devoptions/featureflags/FeatureFlagsFragment.kt	This file is incompatible with code coverage tooling; skipping coverage check.
FAQSingleActivity.kt app/src/main/java/org/oppia/android/app/help/faq/faqsingle/FAQSingleActivity.kt	This file is incompatible with code coverage tooling; skipping coverage check.
FragmentComponentImpl.kt app/src/main/java/org/oppia/android/app/fragment/FragmentComponentImpl.kt	This file is exempted from having a test file; skipping coverage check.
Constants.kt data/src/main/java/org/oppia/android/data/backends/gae/Constants.kt	This file is exempted from having a test file; skipping coverage check.
PlatformParameterControllerInjectorProvider.kt domain/src/main/java/org/oppia/android/domain/platformparameter/PlatformParameterControllerInjectorProvider.kt	This file is exempted from having a test file; skipping coverage check.
PlatformParameterControllerInjector.kt domain/src/main/java/org/oppia/android/domain/platformparameter/PlatformParameterControllerInjector.kt	This file is exempted from having a test file; skipping coverage check.
FeatureFlagsMapBindingModule.kt domain/src/main/java/org/oppia/android/domain/platformparameter/FeatureFlagsMapBindingModule.kt	This file is exempted from having a test file; skipping coverage check.
FeatureFlagBindingModule.kt domain/src/main/java/org/oppia/android/domain/platformparameter/FeatureFlagBindingModule.kt	This file is exempted from having a test file; skipping coverage check.
ExplorationCheckpointController.kt domain/src/main/java/org/oppia/android/domain/exploration/lightweightcheckpointing/ExplorationCheckpointController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ExplorationProgressController.kt domain/src/main/java/org/oppia/android/domain/exploration/ExplorationProgressController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ExplorationStorageModule.kt domain/src/main/java/org/oppia/android/domain/exploration/ExplorationStorageModule.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ExplorationActiveTimeController.kt domain/src/main/java/org/oppia/android/domain/exploration/ExplorationActiveTimeController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ExplorationDataController.kt domain/src/main/java/org/oppia/android/domain/exploration/ExplorationDataController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
SpotlightStateController.kt domain/src/main/java/org/oppia/android/domain/spotlight/SpotlightStateController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ClassroomController.kt domain/src/main/java/org/oppia/android/domain/classroom/ClassroomController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
HintHandlerDebugImpl.kt domain/src/main/java/org/oppia/android/domain/hintsandsolution/HintHandlerDebugImpl.kt	This file is incompatible with code coverage tooling; skipping coverage check.
HintHandlerProdImpl.kt domain/src/main/java/org/oppia/android/domain/hintsandsolution/HintHandlerProdImpl.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicListController.kt domain/src/main/java/org/oppia/android/domain/topic/TopicListController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
TopicController.kt domain/src/main/java/org/oppia/android/domain/topic/TopicController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ProfileManagementController.kt domain/src/main/java/org/oppia/android/domain/profile/ProfileManagementController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
QuestionAssessmentProgressController.kt domain/src/main/java/org/oppia/android/domain/question/QuestionAssessmentProgressController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ModifyLessonProgressController.kt domain/src/main/java/org/oppia/android/domain/devoptions/ModifyLessonProgressController.kt	This file is incompatible with code coverage tooling; skipping coverage check.
StateRetriever.kt domain/src/main/java/org/oppia/android/domain/util/StateRetriever.kt	This file is incompatible with code coverage tooling; skipping coverage check.
ItemSelectionInputDoesNotContainAtLeastOneOfRuleClassifierProvider.kt domain/src/main/java/org/oppia/android/domain/classify/rules/itemselectioninput/ItemSelectionInputDoesNotContainAtLeastOneOfRuleClassifierProvider.kt	This file is incompatible with code coverage tooling; skipping coverage check.
StateGraph.kt domain/src/main/java/org/oppia/android/domain/state/StateGraph.kt	This file is exempted from having a test file; skipping coverage check.

Refer test_file_exemptions.textproto for the comprehensive list of file exemptions and their required coverage percentages.

To learn more, visit the Oppia Android Code Coverage wiki page

[oppiabot[bot]]

Assigning @manas-yu for code owner reviews. Thanks!

[ShankhanilSaha]

@adhiamboperes Missed the typo during code review, thanks! I Have pushed the change requested.