The recommendation tests can be run in parallel but were in an assembly which
had many other tests that could not. Splitting out into a separate assembly
which is designed for parallel runs.

Changed some tests from blocking waits to be properly async.

This closes #6028, and #6001

that intentionally slowed down the code to help detect race conditions
Fixes #4337

Ensure that flow analysis calls VisitRvalue instead of Visit for expressions in order to enforce proper state transition.

Fixes #5992.

If logical expression is not visited with VisitRvalue method, current state remains splitted, which causes a NullReferenceException when the next expression is visited.

Update TestRunner.cs to properly create the parent directory instead of a directory with the file name.

Fixes #6002

Essentially, the problem is that the type character should never be considered as part of the parameter name. This was happening in two places:

1. Comparison of parameter names in code model events. Because the type character was being included, a Rename event would be incorrectly triggered.
2. ICodeModelService.GetName(...) incorrectly returned the type character for parameters. This resulted in downstream effects of not being able to look up the CodeElement for a node by name.

Updating the other instance of AssertDebugException in RealParserTests to only be used in Debug mode.

It's not guaranteed.

Fixes #5927

Fixes #4221

This change is being tracked by #5962

Fixes #2544.

watson shows null exception on DocumentExtensions.GetLanguageService

guarded that method from null exception

Previously our VSIX dependencies demanded that an exactly-numbered
version was installed. This meant that everything was versioned in
lockstep, but prevents the ability to install a single VSIX that
you built locally atop the base product. For now, let's just allow
any higher version.