moved to dotnet-xunit instead of self hosted runner, teamcity appveyor reporters and out files should be back now

Conflicts:
	src/Tests/Tests.csproj

Conflicts:
	build/Clients.Common.targets
	src/Tests/Tests.csproj

Author: Mpdreamz

build/Clients.Common.targets

@@ -14,20 +14,26 @@
     <!-- File version reflects actual version number without prelease since that not allowed in its struct -->
     <FileVersion>$(CurrentAssemblyFileVersion)</FileVersion>
 
-    <!-- 'dotnet xunit' does a build but has no way to pass properties or prevent build so we snoop for TRAVIS here
-    Ideally we handle this in the FAKE script (which we do for 'dotnet build')
-    TODO too lazy to write and test a <CHOOSE>
-    -->
-    <DotNetCoreOnly Condition="'$(TRAVIS)'=='true'">1</DotNetCoreOnly>
-    <DotNetCoreOnly Condition="'$(TRAVIS)'==''"></DotNetCoreOnly>
+    <Authors></Authors>
+    <Company></Company>
+    <NeutralLanguage></NeutralLanguage>
+    <AssemblyTitle></AssemblyTitle>
+    <Description></Description>
+    <Copyright></Copyright>
+    <DotNetCoreOnly></DotNetCoreOnly>
     <DoSourceLink></DoSourceLink>
+
+
+
+
+
     <SignAssembly>true</SignAssembly>
     <AssemblyOriginatorKeyFile Condition="'$(DotNetCoreOnly)'==''">..\..\build\keys\keypair.snk</AssemblyOriginatorKeyFile>
     <GenerateDocumentationFile Condition="'$(DotNetCoreOnly)'==''">true</GenerateDocumentationFile>
     <NoWarn>1591,1572,1571,1573,1587,1570</NoWarn>
     <Prefer32Bit>false</Prefer32Bit>
     <DefineConstants Condition="'$(TargetFramework)'=='netstandard1.3' OR '$(DotNetCoreOnly)'=='1'">$(DefineConstants);DOTNETCORE</DefineConstants>
-    <DebugType>embedded</DebugType>
+    <DebugType Condition="'$(DotNetCoreOnly)'==''">embedded</DebugType>
     <SourceLink Condition="'$(DoSourceLink)'!=''">$(BaseIntermediateOutputPath)\sl-$(MsBuildProjectName)-$(TargetFramework).json</SourceLink>
     <RepoUri>https://raw.githubusercontent.com/elastic/elasticsearch-net</RepoUri>
   </PropertyGroup>

build/scripts/Testing.fsx

@@ -23,26 +23,26 @@ module Tests =
         setProcessEnvironVar "NEST_INTEGRATION_CLUSTER" clusterFilter
         setProcessEnvironVar "NEST_TEST_FILTER" testFilter
 
-    let private dotnetTest() =
-        let folder = Paths.IncrementalOutputFolder (PrivateProject PrivateProject.Tests) DotNetFramework.NetCoreApp1_1
-        let testPath = sprintf "%s/Tests.dll" folder
-        DotNetCli.RunCommand (fun p -> { p with TimeOut = TimeSpan.FromMinutes(10.) }) (sprintf "%s -- Test" testPath) |> ignore
-
-    let private runTestExeOnDesktopCLR() = 
-        let folder = Paths.IncrementalOutputFolder (PrivateProject PrivateProject.Tests) DotNetFramework.Net46
-        let testRunner = Tooling.BuildTooling(folder @@ "Tests.exe")
-        testRunner.Exec ["Test"; "-parallel"; "-xml"; Paths.Output("TestResults-Desktop-Clr.xml")] |> ignore
-        
+    type MultiTarget = All | One  
+    let private dotnetTest target =
+        CreateDir Paths.BuildOutput
+        let command = 
+            let p = ["xunit"; "-parallel"; "all"; "-xml"; "../.." @@ Paths.Output("TestResults-Desktop-Clr.xml")] 
+            match (target, buildingOnTravis) with 
+            | (_, true) 
+            | (One, _) -> p |> List.append ["-framework"; "netcoreapp1.1"]
+            | _  -> p
+
+        let dotnet = Tooling.BuildTooling("dotnet")
+        dotnet.ExecIn "src/Tests" command |> ignore
+
     let IncrementalTest() = 
         setLocalEnvVars()
-        match buildingOnTravis with
-        | false -> runTestExeOnDesktopCLR() 
-        | true -> dotnetTest() 
+        dotnetTest One 
 
     let RunUnitTests() = 
         setLocalEnvVars()
-        dotnetTest()
-        runTestExeOnDesktopCLR()
+        dotnetTest All
 
     let RunIntegrationTests() =
         setLocalEnvVars()
@@ -54,6 +54,4 @@ module Tests =
 
         for esVersion in esVersions do
             setProcessEnvironVar "NEST_INTEGRATION_VERSION" esVersion
-            runTestExeOnDesktopCLR()
-            //TODO enable integration testing on .net CORE
-            //dotnetTest()
+            dotnetTest One |> ignore

build/scripts/Tooling.fsx

@@ -49,11 +49,11 @@ module Tooling =
         ()
 
 
-    let execProcessWithTimeout proc arguments timeout = 
+    let execProcessWithTimeout proc arguments timeout workingDir = 
         let args = arguments |> String.concat " "
         ExecProcess (fun info ->
             info.FileName <- proc
-            info.WorkingDirectory <- "."
+            info.WorkingDirectory <- workingDir
             info.Arguments <- args
         ) timeout
 
@@ -69,12 +69,14 @@ module Tooling =
 
     let private defaultTimeout = TimeSpan.FromMinutes 15.0
 
-    let execProcess proc arguments =
-        let exitCode = execProcessWithTimeout proc arguments defaultTimeout
+    let execProcessInDirectory proc arguments workingDir =
+        let exitCode = execProcessWithTimeout proc arguments defaultTimeout workingDir
         match exitCode with
         | 0 -> exitCode
         | _ -> failwithf "Calling %s resulted in unexpected exitCode %i" proc exitCode 
 
+    let execProcess proc arguments = execProcessInDirectory proc arguments "."
+
 
     let execProcessAndReturnMessages proc arguments =
         execProcessWithTimeoutAndReturnMessages proc arguments defaultTimeout
@@ -94,6 +96,7 @@ module Tooling =
     type BuildTooling(path) =
         member this.Path = path
         member this.Exec arguments = execProcess this.Path arguments
+        member this.ExecIn workingDirectory arguments = execProcessInDirectory this.Path arguments workingDirectory
 
     type DotTraceTool = {
         Name:string;
@@ -152,7 +155,7 @@ module Tooling =
             this.ExecWithTimeout arguments (TimeSpan.FromMinutes 30.)
 
         member this.ExecWithTimeout arguments timeout =
-            let result = execProcessWithTimeout exe arguments timeout
+            let result = execProcessWithTimeout exe arguments timeout "."
             if result <> 0 then failwith (sprintf "Failed to run dotnet tooling for %s args: %A" exe arguments)
 
     let DotNet = new DotNetTooling("dotnet.exe")

src/Tests/Framework/Xunit/TestRunner.cs

@@ -90,10 +90,6 @@ public static void Main(string[] args)
 				var benchmarkSwitcher = new BenchmarkSwitcher(GetBenchmarkTypes());
 				benchmarkSwitcher.Run(arguments);
 			}
-			else
-			{
-				TestRunner.Run(arguments);
-			}
 		}
 
 #if !DOTNETCORE

src/Tests/Program.cs

@@ -9,13 +9,15 @@
     <VersionSuffix>alpha</VersionSuffix>
     <DefineConstants Condition="'$(TargetFramework)'=='netcoreapp1.1' OR '$(DotNetCoreOnly)'=='1'">$(DefineConstants);DOTNETCORE</DefineConstants>
   </PropertyGroup>
+  <ItemGroup>
+    <DotNetCliToolReference Include="dotnet-xunit" Version="2.3.0-beta1-build3642" />
+  </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\Nest\Nest.csproj" />
     <PackageReference Include="Bogus" Version="11.0.5" />
     <PackageReference Include="Newtonsoft.Json" Version="10.0.1" />
     <PackageReference Include="FluentAssertions" Version="4.19.2" />
-    <PackageReference Include="xunit" Version="2.2.0" />
-    <PackageReference Include="xunit.runner.utility" Version="2.2.0" />
+    <PackageReference Include="xunit" Version="2.3.0-beta1-build3642" />
     <PackageReference Include="SemanticVersioning" Version="0.7.6" />
     <PackageReference Include="BenchMarkDotNet" Version="0.10.0" />
     <PackageReference Include="DiffPlex" Version="1.4.1" />