Run tests against .NET 7

[Shane32]

• Few changes to samples so they compile with updated analysis rules
• No changes to main codebase
• No additional targeted TFMs for main codebase

[sungam3r]

Will review shortly. I'm fighting with email all these days.

[sungam3r]

net7 here and net7.0 in some other places. Let's use net7 in all projects.

[Shane32]

Using net7.0 as this is the standard and, when using net7, VS adds new tests to unusable location in test browser

• https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-new-sdk-templates
• https://learn.microsoft.com/en-us/dotnet/standard/frameworks

[sungam3r]

VS adds new tests to unusable location in test browser

Could you share a screenshot?

[sungam3r]

I think that I'm fine switching to net7.0 instead of net7 although I am used to the second.

[Shane32]

[Shane32]

Attempting to run the new test from the test browser will fail because when VS compiles the code, it will appear under the "net7" tfm and so will not run.

With the targetframework set to net7.0, the new test is added to the proper section in the first place, and so it can be selected and run immediately.

[Shane32]

This is not new; it has done this for a long time. I've just never bothered to fix it.

[sungam3r]

And after rebuild everything becomes as it should?

[Shane32]

yes

[Shane32]

So long as we use the recommended TFMs (net7.0, etc) then it works in the first place.

[codecov-commenter]

Codecov Report

Merging #954 (eead476) into master (c83ceac) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #954   +/-   ##
=======================================
  Coverage   95.11%   95.11%           
=======================================
  Files          42       42           
  Lines        2087     2087           
  Branches      359      359           
=======================================
  Hits         1985     1985           
  Misses         59       59           
  Partials       43       43           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[Shane32]

@sungam3r ok to merge?

[sungam3r]

ok