Add support for testconfig.json entries in MSTest

[engyebrahim]

fix: #3572

// Expected format of the json is: -
//
// "mstest" : {
//  "timeout" : {
//      "assemblyInitialize" : strictly positive int,
//      "assemblyCleanup" : strictly positive int,
//      "classInitialize" : strictly positive int,
//      "classCleanup" : strictly positive int,
//      "testInitialize" : strictly positive int,
//      "testCleanup" : strictly positive int,
//      "test" : strictly positive int
 //      "useCooperativeCancellation" : true/false
//  },
//  "parallelism" : {
//      "enabled": true/false,
//      "workers": positive int,
//      "scope": method/class,
//  },
//  "execution": {
//     "collectSourceInformation": true,
//     "executionApartmentState": "STA",
//     "disableAppDomain": true,
//     "mapInconclusiveToFailed" : true,
//     "mapNotRunnableToFailed" : true,
//     "treatDiscoveryWarningsAsErrors" : true,
//     "considerEmptyDataSourceAsInconclusive" : true,
//     "treatClassAndAssemblyCleanupWarningsAsErrors" : true,
//     "considerFixturesAsSpecialTests" : true,
//  },
//  "deployment" : {
//       "enabled": true / false,
//       "deployTestSourceDependencies": true / false,
//       "deleteDeploymentDirectoryAfterTestRunIsComplete": true / false
//  },
//  "assemblyResolution" : [
//        { "path" : "..." , includeSubDirectories: "true" } ,
//        { "path" : "..." , includeSubDirectories: "true" } ,
//        { "path" : "..." , includeSubDirectories: "true" } ,
//        ...
//     ],
//  ... remaining settings
// }

[engyebrahim]

we have that by default generated

so as runsettings file could have some other settings more than mstest should i change the check to search on the key 'key' and throw on the case of it has it and we has it in configration but other than tat we okay two have two files!

check also visual studio default setting

[engyebrahim]

i didn't use MSTESTv2 key

and for deployment settings it's in onother file/flow and i'm on it to get it from config

[engyebrahim]

Directory pathes check if they need more work because of escaping

[Evangelink]

Why is this needed?

[engyebrahim]

to use the Iconfiguration from the bridge