Update TestHostApp to always copy TerminalApp.LocalTests.dll #6228

Open
opened 2026-01-31 00:32:55 +00:00 by claunia · 0 comments
Owner

Originally created by @zadjii-msft on GitHub (Jan 31, 2020).

Since I just used a copy rule for the TestHostApp project's dependency on the LocalTests_TerminalApp project, if you just change the test code, the TestHostApp build won't auto pick-up the updated TerminalApp.LocalTests.dll. This is annoying if youre iterating on the local tests, since you can't just build TestHostApp to get the updated LocalTests.

The rule that's bad:

    <!-- Copy our test code from LocalTests_TerminalApp into this directory -->
    <Copy SourceFiles="$(_TestBinRoot)\LocalTests_TerminalApp\TerminalApp.LocalTests.dll"
          DestinationFiles="$(TargetDir)\TerminalApp.LocalTests.dll" />

As a workaround, you have to manually

copy bin\x64\Debug\LocalTests_TerminalApp\TerminalApp.LocalTests.dll x64\Debug\TestHostApp\ /y

from the solution root

Originally created by @zadjii-msft on GitHub (Jan 31, 2020). Since I just used a copy rule for the `TestHostApp` project's dependency on the `LocalTests_TerminalApp` project, if you just change the test code, the `TestHostApp` build won't auto pick-up the updated `TerminalApp.LocalTests.dll`. This is annoying if youre iterating on the local tests, since you can't just build `TestHostApp` to get the updated LocalTests. The rule that's bad: ```xml <!-- Copy our test code from LocalTests_TerminalApp into this directory --> <Copy SourceFiles="$(_TestBinRoot)\LocalTests_TerminalApp\TerminalApp.LocalTests.dll" DestinationFiles="$(TargetDir)\TerminalApp.LocalTests.dll" /> ``` As a workaround, you have to manually `copy bin\x64\Debug\LocalTests_TerminalApp\TerminalApp.LocalTests.dll x64\Debug\TestHostApp\ /y` from the solution root
claunia added the Issue-TaskNeeds-Tag-FixProduct-TerminalArea-CodeHealth labels 2026-01-31 00:32:55 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#6228