Repository: microsoft/terminal (language: cpp)

Issue:
TerminalApp.Unit.Tests.manifest gets updated when building
### Windows Terminal version

Commit 09e99156727a92f84b309f18aee59ea7293b1264

### Windows build number

10.0.19041.1415

### Other Software

_No response_

### Steps to reproduce

1. Checkout the latest source (commit 09e99156727a92f84b309f18aee59ea7293b1264 or later).
2. Build the solution.
3. Look for any modified files in git status.

### Expected Behavior

There shouldn't be any modified files.

### Actual Behavior

The [TerminalApp.Unit.Tests.manifest](https://github.com/microsoft/terminal/blob/main/src/cascadia/ut_app/TerminalApp.Unit.Tests.manifest) file is being updated with the same changes that were made to the [WindowsTerminal.manifest](https://github.com/microsoft/terminal/blob/main/src/cascadia/WindowsTerminal/WindowsTerminal.manifest) file in PR #12462.

Shouldn't those changes have been committed at the same time as the WindowsTerminal manifest? Or if the unit test manifest is generated automatically, then maybe it shouldn't be committed at all.

Or am I the only one that's getting this?


Discussion / hints:
What, no, I always build the unit tests.... 😅 

I think there's at least one open PR that's got a similar update in it snuck in, but if you'd like to submit a PR just for this tiny bit then that's fine. 

Frankly those tests should probably be reworked into the broader localtests or fixed so they don't need to do that anymore, but 🤷 

Fix patch:
diff --git a/src/cascadia/ut_app/TerminalApp.Unit.Tests.manifest b/src/cascadia/ut_app/TerminalApp.Unit.Tests.manifest
index 47612326b28..8e1afe241f0 100644
--- a/src/cascadia/ut_app/TerminalApp.Unit.Tests.manifest
+++ b/src/cascadia/ut_app/TerminalApp.Unit.Tests.manifest
@@ -14,6 +14,11 @@
         <!-- This will cause unpackaged activation failures in XAML Islands. -->
         <!-- (We use unpackaged activation in test scenarios.) See GH#10265. -->
         <maxversiontested Id="10.0.18362.0"/>
+
+        <!-- You apparently are expected to include all the maxversiontested
+        entries separately. They are treated like an array. This one turns on Segoe
+        UI Variable, GH #12452 -->
+        <maxversiontested Id="10.0.22000.0"/>
         <supportedOS Id="{8e0f7a12-bfb3-4fe8-b9a5-48fd50a15a9a}" />
     </application>
   </compatibility>


Test patch: