Make all test binaries consistent in Target and Project name #1239

Open
opened 2026-01-30 22:19:57 +00:00 by claunia · 0 comments
Owner

Originally created by @DHowett-MSFT on GitHub (May 21, 2019).

Probably another area where cleanup/reconciliation is needed.

The Host Unit Tests (the O.G. of our testing libraries) says

<ProjectName>Host.Tests.Unit</ProjectName>
<TargetName>Conhost.Unit.Tests</TargetName>

So at least the target name might need to be pluralized to match the pattern in the CI (i.e. Types.Unit.Tests).
I think ProjectName only affects the sort order inside Visual Studio so similar projects are sorted together visually.

I'd recommend aligning at least the TargetName (or confirming the CI pattern isn't looking for the plural) here and then filing a follow on task to make the test libraries named more consistently.

_Originally posted by @miniksa in https://github.com/render_node/MDIzOlB1bGxSZXF1ZXN0UmV2aWV3VGhyZWFkMTc1MTU5OTM4OnYy/pull_request_review_threads/discussion

Originally created by @DHowett-MSFT on GitHub (May 21, 2019). Probably another area where cleanup/reconciliation is needed. The Host Unit Tests (the O.G. of our testing libraries) says ``` <ProjectName>Host.Tests.Unit</ProjectName> <TargetName>Conhost.Unit.Tests</TargetName> ``` So at least the target name might need to be pluralized to match the pattern in the CI (i.e. `Types.Unit.Tests`). I think ProjectName only affects the sort order inside Visual Studio so similar projects are sorted together visually. I'd recommend aligning at least the `TargetName` (or confirming the CI pattern isn't looking for the plural) here and then filing a follow on task to make the test libraries named more consistently. _Originally posted by @miniksa in https://github.com/_render_node/MDIzOlB1bGxSZXF1ZXN0UmV2aWV3VGhyZWFkMTc1MTU5OTM4OnYy/pull_request_review_threads/discussion_
claunia added the Issue-TaskProduct-MetaArea-CodeHealth labels 2026-01-30 22:19:57 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#1239