Improve MSTEST0001 message

[gao-artur]

When building a solution with quite logging (-v q) we get a bunch of MSTEST0001 warnings without a reference to the project name. In a solution with dozens of test projects, it's very difficult to find out the projects to fix

CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)
CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)
CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)
CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)
CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)

Please add a reported project name to the error message.

[Youssef1313]

@baronfel @rainersigwald This is a warning from Roslyn analyzer. Is it expected from MSBuild point of view that the project name/path isn't shown after the CSC warning?

[baronfel]

I'd be curious to see the binlogs that generate this diagnostic - if it's coming from a Roslyn analyzer then those should inherently belong to a project, so I'm interesting in what's doing the reporting of the diagnostic itself.

[rainersigwald]

My first thought is that the terminal logger uses indentation grouping on this sort of warning, rather than the old suffix project--I agree that a binlog would be super helpful.

[gao-artur]

Here is a new project created from the MSTest template without MSTestSettings.cs file + the binlog. Built with

dotnet build -bl -v q

TestProject11.zip

[gao-artur]

Interestingly, in the binlog the message contains the file path

CSC warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001) [C:\temp\TestProject11\TestProject11\TestProject11.csproj]

[gao-artur]

@rainersigwald, you were right. With a terminal logger disabled, the path is shown in the message

dotnet build -v q --tl:off

CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001) [C:\temp\TestProject11\TestProject11\TestProject11.cs
proj]

[baronfel]

I think this might actually be something that we need Roslyn to dig into a bit. For comparison to the analyzer-reported diagnostic, here's a simple Warning I added to your test project in non-quiet mode:

➜  TestProject11 dotnet build
Restore complete (0.4s)
  TestProject11 net9.0 succeeded with 2 warning(s) (0.2s) → bin/Debug/net9.0/TestProject11.dll
    CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)
    /Users/chethusk/Downloads/TestProject11/TestProject11.csproj(12,5): warning FAKE001: This is a fake diagnostic message for testing purposes.

Build succeeded with 2 warning(s) in 0.8s

and quiet mode:

➜  TestProject11 dotnet build -v q
CSC : warning MSTEST0001: Explicitly enable or disable tests parallelization (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0001)
/Users/chethusk/Downloads/TestProject11/TestProject11.csproj(12,5): warning FAKE001: This is a fake diagnostic message for testing purposes.

It seems like the diagnostics reported from the Csc task get the CSC 'file' segment both in the binlog viewer and in the terminal logger renderer. Since we're just interpreting the results of individual Tasks calling LogWarning/LogError/etc here, I suspect that the Csc Task implementation needs to change a bit in order to give a user a more reasonable display.