Skip to content

Restore MSTest.TestAdapter runsettings translations regressed by OneLocBuild #9912#9928

Merged
Evangelink merged 1 commit into
mainfrom
dev/amauryleve/ubiquitous-eureka
Jul 14, 2026
Merged

Restore MSTest.TestAdapter runsettings translations regressed by OneLocBuild #9912#9928
Evangelink merged 1 commit into
mainfrom
dev/amauryleve/ubiquitous-eureka

Conversation

@Evangelink

Copy link
Copy Markdown
Member

Problem

Main is red. MSTest.Acceptance.IntegrationTests.RunSettingsTests.UnsupportedRunSettingsEntriesAreFlagged_Localization fails for the fr-FR and it-IT cases: the localized runsettings warnings are emitted in English instead of the expected French/Italian text, so the AssertOutputMatchesRegex / AssertOutputContains assertions fail.

Root cause

The OneLocBuild auto check-in #9912 (fd05c119c) regressed src/Adapter/MSTest.TestAdapter/Resources/xlf/PlatformAdapterResources.*.xlf: it reset every trans-unit from state="translated" back to state="new" with the English <source> text as the <target>. Since MSTestRunSettings.cs reads these strings via PlatformAdapterResources.UnsupportedRunsettingsLoggers / UnsupportedRunsettingsDatacollectors, non-English runs emitted English text.

This is a recurring regression on these duplicated strings — earlier fixes were #9868 and #9897, and the loc bot clobbered them again.

Fix

Restored the pre-#9912 translated .xlf files (all 13 languages):

git checkout fd05c119c^ -- src/Adapter/MSTest.TestAdapter/Resources/xlf/PlatformAdapterResources.*.xlf

Only <target> values changed (English state="new" → the previously approved translations state="translated"). No .resx change, and every <source> / unit ID is untouched.

Verification

Note

The underlying loc-pipeline issue (OneLocBuild repeatedly resetting these strings to state="new") is infrastructure-level and worth a separate follow-up so it stops recurring.

…ocBuild #9912

OneLocBuild check-in #9912 reset every PlatformAdapterResources trans-unit from state=translated back to state=new (English source), breaking the localized runsettings warnings. This made UnsupportedRunSettingsEntriesAreFlagged_Localization fail because fr-FR/it-IT runs emitted English text.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 62a5b826-22ea-4fbb-a585-8fdb7f9ce345
Copilot AI review requested due to automatic review settings July 14, 2026 08:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Restores approved MSTest.TestAdapter translations overwritten by OneLocBuild, fixing localized runsettings warnings.

Changes:

  • Restores 11 translated resources across 13 locales.
  • Preserves source text, identifiers, placeholders, and locked terms.
  • Restores French and Italian acceptance-test expectations.
Show a summary per file
File Description
PlatformAdapterResources.cs.xlf Restores Czech translations.
PlatformAdapterResources.de.xlf Restores German translations.
PlatformAdapterResources.es.xlf Restores Spanish translations.
PlatformAdapterResources.fr.xlf Restores French translations.
PlatformAdapterResources.it.xlf Restores Italian translations.
PlatformAdapterResources.ja.xlf Restores Japanese translations.
PlatformAdapterResources.ko.xlf Restores Korean translations.
PlatformAdapterResources.pl.xlf Restores Polish translations.
PlatformAdapterResources.pt-BR.xlf Restores Brazilian Portuguese translations.
PlatformAdapterResources.ru.xlf Restores Russian translations.
PlatformAdapterResources.tr.xlf Restores Turkish translations.
PlatformAdapterResources.zh-Hans.xlf Restores Simplified Chinese translations.
PlatformAdapterResources.zh-Hant.xlf Restores Traditional Chinese translations.

Review details

  • Files reviewed: 13/13 changed files
  • Comments generated: 0
  • Review effort level: Medium

@Evangelink Evangelink added the state/needs-review Awaiting review from the team. label Jul 14, 2026
@Evangelink
Evangelink merged commit 398e9d1 into main Jul 14, 2026
18 checks passed
@Evangelink
Evangelink deleted the dev/amauryleve/ubiquitous-eureka branch July 14, 2026 08:58
Evangelink added a commit that referenced this pull request Jul 15, 2026
…ocBuild #9912

OneLocBuild check-in #9912 (fd05c11) reset the PlatformAdapterResources
runsettings trans-units from state="translated" to state="new" with English
target text, causing RunSettingsTests.UnsupportedRunSettingsEntriesAreFlagged_Localization
(fr-FR/it-IT) to fail on Debug legs. Restores the approved translations from the
main-branch fix #9928 (398e9d1). Only <target> values change; sources/IDs/.resx
untouched, so XliffTasks stays green.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: c5a5f648-c002-4866-adb6-ddf4a284850b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state/needs-review Awaiting review from the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants