SettingsModelLocalTests failures are on the rise #18517

Closed
opened 2026-01-31 06:16:29 +00:00 by claunia · 3 comments
Owner

Originally created by @j4james on GitHub (Sep 21, 2022).

Windows Terminal version

Commit 1ce22a87e6

Windows build number

10.0.19044.1889

Other Software

No response

Steps to reproduce

!. Check out a recent version of the project.
2. Build it.
3. Run the unit tests.

Expected Behavior

All tests should pass.

Actual Behavior

A number of SettingsModelLocalTests are failing:

SettingsModelLocalTests::DeserializationTests::TestCloseOnExitParsing [Failed]
SettingsModelLocalTests::SerializationTests::Profile [Failed]
SettingsModelLocalTests::SerializationTests::Actions [Failed]
SettingsModelLocalTests::SerializationTests::LegacyFontSettings [Failed]

If I remember correctly, the first one has been around for a while now. I'm afraid I've just been ignoring it and hoping somebody else would notice. The others I think are fairly recent. Although I may be misremembering which are old and which are new.

Originally created by @j4james on GitHub (Sep 21, 2022). ### Windows Terminal version Commit 1ce22a87e6fc204b5fd98642723aa0a51a503749 ### Windows build number 10.0.19044.1889 ### Other Software _No response_ ### Steps to reproduce !. Check out a recent version of the project. 2. Build it. 3. Run the unit tests. ### Expected Behavior All tests should pass. ### Actual Behavior A number of `SettingsModelLocalTests` are failing: SettingsModelLocalTests::DeserializationTests::TestCloseOnExitParsing [Failed] SettingsModelLocalTests::SerializationTests::Profile [Failed] SettingsModelLocalTests::SerializationTests::Actions [Failed] SettingsModelLocalTests::SerializationTests::LegacyFontSettings [Failed] If I remember correctly, the first one has been around for a while now. I'm afraid I've just been ignoring it and hoping somebody else would notice. The others I think are fairly recent. Although I may be misremembering which are old and which are new.
Author
Owner

@j4james commented on GitHub (Sep 21, 2022):

Looking at the test logs for those last three, it seems the failures are a result of a change in serialization of the font size from int to float (I suspect PR #14013).

The older one I suspect is the result of the new Automatic mode for CloseOnExit (PR #13560).

@j4james commented on GitHub (Sep 21, 2022): Looking at the test logs for those last three, it seems the failures are a result of a change in serialization of the font size from int to float (I suspect PR #14013). The older one I suspect is the result of the new `Automatic` mode for `CloseOnExit` (PR #13560).
Author
Owner

@lhecker commented on GitHub (Sep 21, 2022):

I keep forgetting that we don't run them in the CI... 😞

@lhecker commented on GitHub (Sep 21, 2022): I keep forgetting that we don't run them in the CI... 😞
Author
Owner

@ghost commented on GitHub (Jan 24, 2023):

:tada:This issue was addressed in #14064, which has now been successfully released as Windows Terminal Preview v1.17.1023.🎉

Handy links:

@ghost commented on GitHub (Jan 24, 2023): :tada:This issue was addressed in #14064, which has now been successfully released as `Windows Terminal Preview v1.17.1023`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.17.1023) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#18517