Ensure we actually include the workload manifest targets in the package

[danegsta]

Targets weren't getting packaged in the workload manifest nuget

Microsoft Reviewers: Open in CodeFlow

[joperezr]

LGTM, should we add some sort of validation that runs after packaging to ensure the files are there and fail the build if they aren't?

[danegsta]

LGTM, should we add some sort of validation that runs after packaging to ensure the files are there and fail the build if they aren't?

Probably a good idea; I'd vote for doing that in a separate PR though, just so we can get this unblocked for testing ASAP.

[drewnoakes]

LGTM, should we add some sort of validation that runs after packaging to ensure the files are there and fail the build if they aren't?

In the .NET Project System we have a similar test that uses the Verify library to track the contents of our VSIX file, which works really well. The set of expected files is kept in a text file in the repo.

https://github.com/dotnet/project-system/tree/main/tests/Microsoft.VisualStudio.ProjectSystem.Managed.VS.UnitTests/Setup