From eb6b6e58c2566ea951f77898087fdd96b236b78a Mon Sep 17 00:00:00 2001 From: Pavel Krymets Date: Fri, 7 Sep 2018 10:12:38 -0700 Subject: [PATCH] Reenable tests that were throwing in Program.Main (#1379) --- build/dependencies.props | 56 +++++++++---------- .../Common.FunctionalTests/AppOfflineTests.cs | 2 +- .../InProcess/StartupTests.cs | 18 ++++++ .../StartupExceptionWebSite/Program.cs | 2 + 4 files changed, 49 insertions(+), 29 deletions(-) diff --git a/build/dependencies.props b/build/dependencies.props index 5418510639..3377ec3b0a 100644 --- a/build/dependencies.props +++ b/build/dependencies.props @@ -5,38 +5,38 @@ 0.10.13 2.2.0-preview1-20180821.1 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 0.6.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 0.6.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 15.6.82 15.6.82 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 - 2.2.0-preview2-35143 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 + 2.2.0-preview3-35179 2.0.9 - 2.1.2 - 2.2.0-preview1-26618-02 + 2.1.3 + 2.2.0-preview2-26905-02 1.0.1 - 2.2.0-preview2-35143 + 2.2.0-preview3-35179 15.6.1 11.1.0 2.0.3 diff --git a/test/Common.FunctionalTests/AppOfflineTests.cs b/test/Common.FunctionalTests/AppOfflineTests.cs index 53afd34504..41bf8874d9 100644 --- a/test/Common.FunctionalTests/AppOfflineTests.cs +++ b/test/Common.FunctionalTests/AppOfflineTests.cs @@ -81,7 +81,7 @@ namespace Microsoft.AspNetCore.Server.IIS.FunctionalTests DeletePublishOutput(deploymentResult); } - [ConditionalFact(Skip = "https://github.com/aspnet/IISIntegration/issues/933")] + [ConditionalFact] public async Task AppOfflineDroppedWhileSiteFailedToStartInRequestHandler_SiteStops_InProcess() { var deploymentResult = await DeployApp(HostingModel.InProcess); diff --git a/test/IISExpress.FunctionalTests/InProcess/StartupTests.cs b/test/IISExpress.FunctionalTests/InProcess/StartupTests.cs index 4c288ef90e..0dc4e1acd6 100644 --- a/test/IISExpress.FunctionalTests/InProcess/StartupTests.cs +++ b/test/IISExpress.FunctionalTests/InProcess/StartupTests.cs @@ -145,6 +145,24 @@ namespace Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests EventLogHelpers.InProcessFailedToStart(deploymentResult, "CLR worker thread exited prematurely"), EventLogHelpers.InProcessThreadException(deploymentResult, ".*?Application is running inside IIS process but is not configured to use IIS server")); } + + [ConditionalFact] + public async Task LogsStartupExceptionExitError() + { + var deploymentParameters = _fixture.GetBaseDeploymentParameters(_fixture.StartupExceptionWebsite, publish: true); + deploymentParameters.TransformArguments((a, _) => $"{a} Throw"); + + var deploymentResult = await DeployAsync(deploymentParameters); + + var response = await deploymentResult.HttpClient.GetAsync("/"); + Assert.Equal(HttpStatusCode.InternalServerError, response.StatusCode); + + StopServer(); + + EventLogHelpers.VerifyEventLogEvents(deploymentResult, + EventLogHelpers.InProcessFailedToStart(deploymentResult, "CLR worker thread exited prematurely"), + EventLogHelpers.InProcessThreadException(deploymentResult, ", exception code = '0xe0434352'")); + } [ConditionalFact] public async Task LogsUnexpectedThreadExitError() diff --git a/test/WebSites/StartupExceptionWebSite/Program.cs b/test/WebSites/StartupExceptionWebSite/Program.cs index 70c0ac0108..95bac44503 100644 --- a/test/WebSites/StartupExceptionWebSite/Program.cs +++ b/test/WebSites/StartupExceptionWebSite/Program.cs @@ -44,6 +44,8 @@ namespace IISTestSite case "Hang": Thread.Sleep(Timeout.Infinite); break; + case "Throw": + throw new InvalidOperationException("Program.Main exception"); case "HangOnStop": var host = new WebHostBuilder() .UseIIS()