Improve resource management in TestWorkspaceFixture

Remove hard-coded TS check for RaiseDiagnosticsRemovedForDocument

Remove CompilationWithAnalyzers.IsDiagnosticAnalyzerSuppressed checks…

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19619.10

Assert that ParsingTests.N nodes are not missing, fix up tests that didn't assume this.

Merge feature branch features/editorconfig-generated-code into master

* Fix nullable annotations for ValueSource

* Refactor WeakValueSource

* Replace OptionalValueSource<T> with ValueSource<Optional<T>>

Add missing back compat overload

Copy any CommentTrivia to simple using statements 

Fixes #39960

Remove null from Diagnostic.Arguments

Co-Authored-By: NJoey Robichaud <joseph.robichaud@microsoft.com>

Nullable progress: InternalUtilities and Diagnostics

This changes the implementation of `DiagnosticInfo` to use an empty
array in all cases for arguments vs. a mix of empty arrays and `null`
for the empty case. This allows us to ensure the `Diagnostic.Arguments`
is always a non-null value.

This makes adopting nullability higher in the stack significantly
simpler as there are assumptions this is non-null in places.

Remove unnecessary allocations in 'keyword highlighting'

Disabled by default analyzers, which are enabled via .editorconfig are not executing in the IDE diagnostic service. This is due to us skipping live analysis for analyzers disabled by default OR disabled via compilation options as a performance optimization. However, this check is no longer correct as analyzers can be enabled/disabled for individual documents through .editorconfig files. So we no longer perform this check and pessimistically assume analyzer is not suppressed and let the core analyzer driver in the compiler layer handle skipping redundant analysis callbacks.

Remove some nullable helper methods

Add feature to simplify interpolation expressions when possible

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19617.1