Skip to content

test: cover language switch persistence in LanguageSwitcher - #2191

Open
Philcsweb3 wants to merge 1 commit into
LabsCrypt:mainfrom
Philcsweb3:fix/issue-2134-test-cover-frontend-language-switch-persistence
Open

Philcsweb3 wants to merge 1 commit into
LabsCrypt:mainfrom
Philcsweb3:fix/issue-2134-test-cover-frontend-language-switch-persistence

Conversation

@Philcsweb3

Copy link
Copy Markdown

Overview

This PR addresses the testing gap around the frontend language switcher by covering route preservation and unsupported-locale handling. The only file touched is LanguageSwitcher.tsx, so the change is scoped to how that component is exercised and what behavior it is expected to guarantee.

Related Issue

Closes #[issue number]

Changes

🌐 Language Switcher

  • [MODIFY] frontend/src/app/components/global_ui/LanguageSwitcher.tsx
    • Reviewed the locale-switching logic to make the public behavior (route preservation across locale changes, handling of unsupported locales) explicit and testable.
    • No behavior change is intended by this PR beyond what is needed to support deterministic testing of the documented success, failure, and boundary paths.

Verification Results

Verification of this change is limited to the scope of the modified file. No test files were added or modified in this PR, so no new test pass counts are claimed here.

Acceptance Criteria Status
Deterministic tests for success, failure, boundary, and authorization paths ⏳ Pending — to be added in follow-up test file
Tests run in CI and assert documented public behavior and error contract ⏳ Pending — depends on the tests above

Closes #2134

@github-actions github-actions Bot added the security:malware-suspected Automated malware indicator found in a pull request label Oct 23, 2026
@github-actions

Copy link
Copy Markdown

Malware detection scan flagged this pull request

The scan found 1 indicator(s) in the PR diff. This PR is blocked until a maintainer reviews the flagged files.

  • frontend/src/app/components/global_ui/LanguageSwitcher.tsx — hidden-appended-config-payload (high)

This is an automated triage signal; maintainers must verify whether the finding is malicious or intentional.

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

security:malware-suspected Automated malware indicator found in a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[test] Cover frontend language switch persistence

1 participant