Skip to content

Fix part of #5035: Add SplashActivity locale reset test - #6540

Open
Abhishekgit01 wants to merge 1 commit into
oppia:developfrom
Abhishekgit01:splash-locale-reset-demo-5035
Open

Abhishekgit01 wants to merge 1 commit into
oppia:developfrom
Abhishekgit01:splash-locale-reset-demo-5035

Conversation

@Abhishekgit01

Copy link
Copy Markdown
Contributor

Explanation

Fixes part of #5035

Adds a SplashActivity test for reopening the activity after onboarding state is reset while the app-scoped locale handler remains initialized. It checks that the second launch routes to onboarding and retains the display locale from the first launch.

Essential Checklist

  • The PR title starts with Fix part of #5035: ....
  • The explanation section above starts with Fixes part of #5035.
  • Any changes to scripts/assets files have their rationale included (no such files are changed).
  • The PR follows the style guide.
  • The PR does not contain unnecessary code changes from Android Studio.
  • The PR is made from a branch that's not called develop and is up-to-date with develop (checked against cd3f5f5415ffbda8f7e7730304376bdb571ac2f7 immediately before publication).
  • The PR is assigned to the appropriate reviewers. Oppia’s guide says to request a reviewer in a PR comment (e.g. @Reviewer, PTAL) after the PR is created.

Testing

  • bazel --batch test --nocache_test_results --test_output=errors //app/src/sharedTest/java/org/oppia/android/app/splash:SplashActivityTest — passed; 135 test cases, zero failures or errors.
  • KtLint for SplashActivityTest.kt — passed.
  • Repository pre-push checks (KtLint, Checkstyle, Buf, Buildifier) — passed.
  • git diff --check — passed.

Disclosure of LLM Usage

  • Did you use AI/LLMs when working on this PR? Yes.
  • If yes, describe the extent AI was used: AI tools assisted with codebase investigation, the test approach, test implementation, and local verification.

@Abhishekgit01
Abhishekgit01 requested a review from a team as a code owner November 5, 2026 19:30
@Neer-rn Neer-rn self-assigned this Nov 9, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants