FullyQualifiedName is not escaped correctly

F# allows names quoted with double backticks to contain spaces and other special characters. The CLI can handle this, if those names are quoted, otherwise MSBuild interprets the spaces as separate args.

Example:

[<Fact>]
let ``This is a test method`` () = Assert.True(true)

Expected:

dotnet test --filter "FullyQualifiedName~This is a test method"
#...
#Test Run Succesfull.

Actual:

dotnet test --filter FullyQualifiedName~This is a test method
#MSBUILD : error MSB1008: Only one project can be specified.
#Switch: a
#
#For switch syntax, type "MSBuild /help"

The simplest solution would be to just put quotes around the thing and be done with it. However, quotes are legal characters in such names, so they would have to be escaped. This even allows arbitrary code to be executed, if the user selects that test to be run:

[<Fact>]
let ``; echo "Hello World!";`` () = Assert.True(true)

dotnet test --filter FullyQualifiedName~; echo "Hello World!";
# Build started, please wait...
# ...
# No test is available in ...
#
# Hello World!

I think simply quoting the argument is good enough since most test names don’t contain special characters beyond some spaces and escaping strings correctly would be quite some work.

[Tarmil]

This fix has been reverted by 4ba12de5756414725b1b6369671421e378eb74d1, it’s failing again.

[stefanforsberg]

Fixed with #90