Converted samples and test projects to run on netcoreapp2.0

This commit is contained in:
Kiran Challa 2017-03-23 00:28:49 -07:00 committed by Kiran Challa
parent 60f456dc7e
commit 0586fd136f
17 changed files with 78 additions and 25 deletions

View File

@ -3,6 +3,7 @@
<AspNetCoreVersion>1.2.0-*</AspNetCoreVersion> <AspNetCoreVersion>1.2.0-*</AspNetCoreVersion>
<CoreFxVersion>4.3.0</CoreFxVersion> <CoreFxVersion>4.3.0</CoreFxVersion>
<NetStandardImplicitPackageVersion>1.6.1</NetStandardImplicitPackageVersion> <NetStandardImplicitPackageVersion>1.6.1</NetStandardImplicitPackageVersion>
<RuntimeFrameworkVersion>2.0.0-*</RuntimeFrameworkVersion>
<TestSdkVersion>15.0.0</TestSdkVersion> <TestSdkVersion>15.0.0</TestSdkVersion>
<XunitVersion>2.2.0</XunitVersion> <XunitVersion>2.2.0</XunitVersion>
</PropertyGroup> </PropertyGroup>

View File

@ -3,7 +3,7 @@
<Import Project="..\..\build\dependencies.props" /> <Import Project="..\..\build\dependencies.props" />
<PropertyGroup> <PropertyGroup>
<TargetFrameworks>net451;netcoreapp1.1</TargetFrameworks> <TargetFrameworks>net451;netcoreapp2.0</TargetFrameworks>
<OutputType>Exe</OutputType> <OutputType>Exe</OutputType>
<ServerGarbageCollection>true</ServerGarbageCollection> <ServerGarbageCollection>true</ServerGarbageCollection>
</PropertyGroup> </PropertyGroup>

View File

@ -3,7 +3,7 @@
<Import Project="..\..\build\dependencies.props" /> <Import Project="..\..\build\dependencies.props" />
<PropertyGroup> <PropertyGroup>
<TargetFrameworks>net451;netcoreapp1.1</TargetFrameworks> <TargetFrameworks>net451;netcoreapp2.0</TargetFrameworks>
<OutputType>Exe</OutputType> <OutputType>Exe</OutputType>
<ServerGarbageCollection>true</ServerGarbageCollection> <ServerGarbageCollection>true</ServerGarbageCollection>
</PropertyGroup> </PropertyGroup>

View File

@ -104,8 +104,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys
{ {
#if NET452 #if NET452
var handler = new WebRequestHandler(); var handler = new WebRequestHandler();
#else #elif NETCOREAPP2_0
var handler = new WinHttpHandler(); var handler = new WinHttpHandler();
#else
#error Target framework needs to be updated
#endif #endif
handler.ServerCertificateValidationCallback = (a, b, c, d) => true; handler.ServerCertificateValidationCallback = (a, b, c, d) => true;
if (cert != null) if (cert != null)
@ -122,8 +124,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys
{ {
#if NET452 #if NET452
var handler = new WebRequestHandler(); var handler = new WebRequestHandler();
#else #elif NETCOREAPP2_0
var handler = new WinHttpHandler(); var handler = new WinHttpHandler();
#else
#error Target framework needs to be updated
#endif #endif
handler.ServerCertificateValidationCallback = (a, b, c, d) => true; handler.ServerCertificateValidationCallback = (a, b, c, d) => true;
using (HttpClient client = new HttpClient(handler)) using (HttpClient client = new HttpClient(handler))

View File

@ -111,8 +111,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
{ {
#if NET452 #if NET452
WebRequestHandler handler = new WebRequestHandler(); WebRequestHandler handler = new WebRequestHandler();
#else #elif NETCOREAPP2_0
WinHttpHandler handler = new WinHttpHandler(); WinHttpHandler handler = new WinHttpHandler();
#else
#error Target framework needs to be updated
#endif #endif
handler.ServerCertificateValidationCallback = (a, b, c, d) => true; handler.ServerCertificateValidationCallback = (a, b, c, d) => true;
if (cert != null) if (cert != null)
@ -129,8 +131,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
{ {
#if NET452 #if NET452
WebRequestHandler handler = new WebRequestHandler(); WebRequestHandler handler = new WebRequestHandler();
#else #elif NETCOREAPP2_0
WinHttpHandler handler = new WinHttpHandler(); WinHttpHandler handler = new WinHttpHandler();
#else
#error Target framework needs to be updated
#endif #endif
handler.ServerCertificateValidationCallback = (a, b, c, d) => true; handler.ServerCertificateValidationCallback = (a, b, c, d) => true;
using (HttpClient client = new HttpClient(handler)) using (HttpClient client = new HttpClient(handler))

View File

@ -72,6 +72,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
Assert.Equal("Hello World", response); Assert.Equal("Hello World", response);
} }
} }
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
[ConditionalFact] [ConditionalFact]

View File

@ -100,8 +100,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
var stream = context.Response.Body; var stream = context.Response.Body;
#if NET452 #if NET452
stream.EndWrite(stream.BeginWrite(new byte[10], 0, 10, null, null)); stream.EndWrite(stream.BeginWrite(new byte[10], 0, 10, null, null));
#else #elif NETCOREAPP2_0
await stream.WriteAsync(new byte[10], 0, 10); await stream.WriteAsync(new byte[10], 0, 10);
#else
#error Target framework needs to be updated
#endif #endif
stream.Write(new byte[10], 0, 10); stream.Write(new byte[10], 0, 10);
await stream.WriteAsync(new byte[10], 0, 10); await stream.WriteAsync(new byte[10], 0, 10);
@ -129,11 +131,14 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
var context = await server.AcceptAsync(Utilities.DefaultTimeout); var context = await server.AcceptAsync(Utilities.DefaultTimeout);
context.Response.Headers["Content-lenGth"] = " 20 "; context.Response.Headers["Content-lenGth"] = " 20 ";
context.Dispose(); context.Dispose();
#if !NETCOREAPP1_1 #if NET452
// HttpClient retries the request because it didn't get a response. // HttpClient retries the request because it didn't get a response.
context = await server.AcceptAsync(Utilities.DefaultTimeout); context = await server.AcceptAsync(Utilities.DefaultTimeout);
context.Response.Headers["Content-lenGth"] = " 20 "; context.Response.Headers["Content-lenGth"] = " 20 ";
context.Dispose(); context.Dispose();
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
await Assert.ThrowsAsync<HttpRequestException>(() => responseTask); await Assert.ThrowsAsync<HttpRequestException>(() => responseTask);
} }
@ -291,6 +296,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
writeTask = context.Response.Body.WriteAsync(new byte[10], 0, 10, cts.Token); writeTask = context.Response.Body.WriteAsync(new byte[10], 0, 10, cts.Token);
Assert.True(writeTask.IsCanceled); Assert.True(writeTask.IsCanceled);
context.Dispose(); context.Dispose();
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
await Assert.ThrowsAsync<HttpRequestException>(() => responseTask); await Assert.ThrowsAsync<HttpRequestException>(() => responseTask);
} }
@ -320,6 +328,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
writeTask = context.Response.Body.WriteAsync(new byte[10], 0, 10, cts.Token); writeTask = context.Response.Body.WriteAsync(new byte[10], 0, 10, cts.Token);
Assert.True(writeTask.IsCanceled); Assert.True(writeTask.IsCanceled);
context.Dispose(); context.Dispose();
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
await Assert.ThrowsAsync<HttpRequestException>(() => responseTask); await Assert.ThrowsAsync<HttpRequestException>(() => responseTask);
} }

View File

@ -221,10 +221,12 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
Assert.Equal(0, response.ContentLength); Assert.Equal(0, response.ContentLength);
Assert.NotNull(response.Headers["Date"]); Assert.NotNull(response.Headers["Date"]);
Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]); Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]);
#if NETCOREAPP1_1 // WebHeaderCollection.GetValues() not available in CoreCLR. #if NETCOREAPP2_0 // WebHeaderCollection.GetValues() not available in CoreCLR.
Assert.Equal("custom1, and custom2, custom3", response.Headers["WWW-Authenticate"]); Assert.Equal("custom1, and custom2, custom3", response.Headers["WWW-Authenticate"]);
#else #elif NET452
Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("WWW-Authenticate")); Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("WWW-Authenticate"));
#else
#error Target framework needs to be updated
#endif #endif
} }
} }
@ -250,10 +252,12 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
Assert.Equal(0, response.ContentLength); Assert.Equal(0, response.ContentLength);
Assert.NotNull(response.Headers["Date"]); Assert.NotNull(response.Headers["Date"]);
Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]); Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]);
#if NETCOREAPP1_1 // WebHeaderCollection.GetValues() not available in CoreCLR. #if NETCOREAPP2_0 // WebHeaderCollection.GetValues() not available in CoreCLR.
Assert.Equal("custom1, and custom2, custom3", response.Headers["Custom-Header1"]); Assert.Equal("custom1, and custom2, custom3", response.Headers["Custom-Header1"]);
#else #elif NET452
Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("Custom-Header1")); Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("Custom-Header1"));
#else
#error Target framework needs to be updated
#endif #endif
} }
} }

View File

@ -371,6 +371,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
writeTask = context.Response.SendFileAsync(AbsoluteFilePath, 0, null, cts.Token); writeTask = context.Response.SendFileAsync(AbsoluteFilePath, 0, null, cts.Token);
Assert.True(writeTask.IsCanceled); Assert.True(writeTask.IsCanceled);
context.Dispose(); context.Dispose();
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
await Assert.ThrowsAsync<HttpRequestException>(() => responseTask); await Assert.ThrowsAsync<HttpRequestException>(() => responseTask);
} }
@ -400,6 +403,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
writeTask = context.Response.SendFileAsync(AbsoluteFilePath, 0, null, cts.Token); writeTask = context.Response.SendFileAsync(AbsoluteFilePath, 0, null, cts.Token);
Assert.True(writeTask.IsCanceled); Assert.True(writeTask.IsCanceled);
context.Dispose(); context.Dispose();
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
await Assert.ThrowsAsync<HttpRequestException>(() => responseTask); await Assert.ThrowsAsync<HttpRequestException>(() => responseTask);
} }

View File

@ -186,10 +186,13 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
context.Abort(); context.Abort();
Assert.True(canceled.WaitOne(interval), "Aborted"); Assert.True(canceled.WaitOne(interval), "Aborted");
Assert.True(ct.IsCancellationRequested, "IsCancellationRequested"); Assert.True(ct.IsCancellationRequested, "IsCancellationRequested");
#if !NETCOREAPP1_1 #if NET452
// HttpClient re-tries the request because it doesn't know if the request was received. // HttpClient re-tries the request because it doesn't know if the request was received.
context = await server.AcceptAsync(Utilities.DefaultTimeout); context = await server.AcceptAsync(Utilities.DefaultTimeout);
context.Abort(); context.Abort();
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
await Assert.ThrowsAsync<HttpRequestException>(() => responseTask); await Assert.ThrowsAsync<HttpRequestException>(() => responseTask);
} }

View File

@ -20,6 +20,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
{ {
#if NET452 #if NET452
return !string.IsNullOrEmpty(System.DirectoryServices.ActiveDirectory.Domain.GetComputerDomain().Name); return !string.IsNullOrEmpty(System.DirectoryServices.ActiveDirectory.Domain.GetComputerDomain().Name);
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
} }
catch catch

View File

@ -28,8 +28,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys.Listener
#if NET452 #if NET452
IsWin8orLater = (Environment.OSVersion.Version >= win8Version); IsWin8orLater = (Environment.OSVersion.Version >= win8Version);
#else #elif NETCOREAPP2_0
IsWin8orLater = (new Version(RuntimeEnvironment.OperatingSystemVersion) >= win8Version); IsWin8orLater = (new Version(RuntimeEnvironment.OperatingSystemVersion) >= win8Version);
#else
#error Target framework needs to be updated
#endif #endif
} }

View File

@ -3,8 +3,8 @@
<Import Project="..\..\build\common.props" /> <Import Project="..\..\build\common.props" />
<PropertyGroup> <PropertyGroup>
<TargetFrameworks>netcoreapp1.1;net452</TargetFrameworks> <TargetFrameworks>netcoreapp2.0;net452</TargetFrameworks>
<TargetFrameworks Condition=" '$(OS)' != 'Windows_NT' ">netcoreapp1.1</TargetFrameworks> <TargetFrameworks Condition=" '$(OS)' != 'Windows_NT' ">netcoreapp2.0</TargetFrameworks>
</PropertyGroup> </PropertyGroup>
<ItemGroup> <ItemGroup>
@ -24,7 +24,7 @@
<Reference Include="System.Net.Http.WebRequest" /> <Reference Include="System.Net.Http.WebRequest" />
</ItemGroup> </ItemGroup>
<ItemGroup Condition=" '$(TargetFramework)' == 'netcoreapp1.1' "> <ItemGroup Condition=" '$(TargetFramework)' == 'netcoreapp2.0' ">
<PackageReference Include="System.Net.Http.WinHttpHandler" Version="$(CoreFxVersion)" /> <PackageReference Include="System.Net.Http.WinHttpHandler" Version="$(CoreFxVersion)" />
<PackageReference Include="System.Net.WebSockets.Client" Version="$(CoreFxVersion)" /> <PackageReference Include="System.Net.WebSockets.Client" Version="$(CoreFxVersion)" />
</ItemGroup> </ItemGroup>

View File

@ -50,7 +50,7 @@ namespace Microsoft.AspNetCore.Server.HttpSys
Assert.Equal("Hello World", response); Assert.Equal("Hello World", response);
} }
} }
#if !NETCOREAPP1_1 #if NET452
[ConditionalFact] [ConditionalFact]
public async Task RequestBody_ReadBeginEnd_Success() public async Task RequestBody_ReadBeginEnd_Success()
{ {
@ -68,6 +68,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys
Assert.Equal("Hello World", response); Assert.Equal("Hello World", response);
} }
} }
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
[ConditionalFact] [ConditionalFact]

View File

@ -89,8 +89,10 @@ namespace Microsoft.AspNetCore.Server.HttpSys
Stream stream = httpContext.Response.Body; Stream stream = httpContext.Response.Body;
#if NET452 #if NET452
stream.EndWrite(stream.BeginWrite(new byte[10], 0, 10, null, null)); stream.EndWrite(stream.BeginWrite(new byte[10], 0, 10, null, null));
#else #elif NETCOREAPP2_0
await stream.WriteAsync(new byte[10], 0, 10); await stream.WriteAsync(new byte[10], 0, 10);
#else
#error Target framework needs to be updated
#endif #endif
stream.Write(new byte[10], 0, 10); stream.Write(new byte[10], 0, 10);
await stream.WriteAsync(new byte[10], 0, 10); await stream.WriteAsync(new byte[10], 0, 10);
@ -249,6 +251,9 @@ namespace Microsoft.AspNetCore.Server.HttpSys
Assert.Equal(new byte[10], await response.Content.ReadAsByteArrayAsync()); Assert.Equal(new byte[10], await response.Content.ReadAsByteArrayAsync());
} }
} }
#elif NETCOREAPP2_0
#else
#error Target framework needs to be updated
#endif #endif
[ConditionalFact] [ConditionalFact]
public async Task ResponseBody_WriteAsync_TriggersOnStarting() public async Task ResponseBody_WriteAsync_TriggersOnStarting()

View File

@ -81,10 +81,12 @@ namespace Microsoft.AspNetCore.Server.HttpSys
Assert.Equal(0, response.ContentLength); Assert.Equal(0, response.ContentLength);
Assert.NotNull(response.Headers["Date"]); Assert.NotNull(response.Headers["Date"]);
Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]); Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]);
#if NETCOREAPP1_1 // WebHeaderCollection.GetValues() not available in CoreCLR. #if NETCOREAPP2_0 // WebHeaderCollection.GetValues() not available in CoreCLR.
Assert.Equal("custom1, and custom2, custom3", response.Headers["WWW-Authenticate"]); Assert.Equal("custom1, and custom2, custom3", response.Headers["WWW-Authenticate"]);
#else #elif NET452
Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("WWW-Authenticate")); Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("WWW-Authenticate"));
#else
#error Target framework needs to be updated
#endif #endif
} }
} }
@ -109,10 +111,12 @@ namespace Microsoft.AspNetCore.Server.HttpSys
Assert.Equal(0, response.ContentLength); Assert.Equal(0, response.ContentLength);
Assert.NotNull(response.Headers["Date"]); Assert.NotNull(response.Headers["Date"]);
Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]); Assert.Equal("Microsoft-HTTPAPI/2.0", response.Headers["Server"]);
#if NETCOREAPP1_1 // WebHeaderCollection.GetValues() not available in CoreCLR. #if NETCOREAPP2_0 // WebHeaderCollection.GetValues() not available in CoreCLR.
Assert.Equal("custom1, and custom2, custom3", response.Headers["Custom-Header1"]); Assert.Equal("custom1, and custom2, custom3", response.Headers["Custom-Header1"]);
#else #elif NET452
Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("Custom-Header1")); Assert.Equal(new string[] { "custom1, and custom2", "custom3" }, response.Headers.GetValues("Custom-Header1"));
#else
#error Target framework needs to be updated
#endif #endif
} }
} }

View File

@ -3,8 +3,8 @@
<Import Project="..\..\build\common.props" /> <Import Project="..\..\build\common.props" />
<PropertyGroup> <PropertyGroup>
<TargetFrameworks>netcoreapp1.1;net452</TargetFrameworks> <TargetFrameworks>netcoreapp2.0;net452</TargetFrameworks>
<TargetFrameworks Condition=" '$(OS)' != 'Windows_NT' ">netcoreapp1.1</TargetFrameworks> <TargetFrameworks Condition=" '$(OS)' != 'Windows_NT' ">netcoreapp2.0</TargetFrameworks>
</PropertyGroup> </PropertyGroup>
<ItemGroup> <ItemGroup>