[PR #16773] Make the Settings Model tests into proper CI tests #30991

Open
opened 2026-01-31 09:44:24 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/16773

State: closed
Merged: Yes


This pull request removes the need for the SettingsModel tests to run in
a UAP harness and puts them into the standard CI rotation.

This required some changes to Run-Tests.ps1 to ensure that the right
te.exe is selected for each test harness. It's a bit annoying, but for
things that depend on a resources.pri, that file must be in the same
directory as the EXE that is hosting the test. Not the DLL, mind you,
the EXE. In our case, that's TE.ProcessHost.exe

The bulk of the change is honestly namespace tidying.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/16773 **State:** closed **Merged:** Yes --- This pull request removes the need for the SettingsModel tests to run in a UAP harness and puts them into the standard CI rotation. This required some changes to `Run-Tests.ps1` to ensure that the right `te.exe` is selected for each test harness. It's a bit annoying, but for things that depend on a `resources.pri`, that file must be in the same directory as the EXE that is hosting the test. Not the DLL, mind you, the EXE. In our case, that's `TE.ProcessHost.exe` The bulk of the change is honestly namespace tidying.
claunia added the pull-request label 2026-01-31 09:44:24 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30991