From e8043f92e5436d7e327cf4ff223c5caece5e9b46 Mon Sep 17 00:00:00 2001 From: Sornakumar Sundararajan Date: Fri, 28 Aug 2015 10:52:48 -0700 Subject: [PATCH] Issue #2757 - Adding Order to exception filters so that they are executed in the same order irrespective of the platform. --- test/Microsoft.AspNet.Mvc.FunctionalTests/FiltersTest.cs | 4 +--- test/WebSites/FiltersWebSite/Controllers/HomeController.cs | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/test/Microsoft.AspNet.Mvc.FunctionalTests/FiltersTest.cs b/test/Microsoft.AspNet.Mvc.FunctionalTests/FiltersTest.cs index b24c1e3949..a31b1e3c47 100644 --- a/test/Microsoft.AspNet.Mvc.FunctionalTests/FiltersTest.cs +++ b/test/Microsoft.AspNet.Mvc.FunctionalTests/FiltersTest.cs @@ -506,9 +506,7 @@ namespace Microsoft.AspNet.Mvc.FunctionalTests // Action has two Exception filters. // Verifies that the second Exception Filter was not executed. - [ConditionalTheory] - // https://github.com/aspnet/Mvc/issues/2757 - [FrameworkSkipCondition(RuntimeFrameworks.Mono)] + [Fact] public async Task ExceptionFilterShortCircuitsAnotherExceptionFilter() { // Arrange diff --git a/test/WebSites/FiltersWebSite/Controllers/HomeController.cs b/test/WebSites/FiltersWebSite/Controllers/HomeController.cs index 8c17c7ffba..108cb062d4 100644 --- a/test/WebSites/FiltersWebSite/Controllers/HomeController.cs +++ b/test/WebSites/FiltersWebSite/Controllers/HomeController.cs @@ -77,7 +77,7 @@ namespace FiltersWebSite.Controllers } [HandleInvalidOperationExceptionFilter] - [ShortCircuitExceptionFilter] + [ShortCircuitExceptionFilter(Order=12)] public IActionResult ThrowRandomExcpetion() { throw new InvalidOperationException("Controller threw.");