Moq.Setup Выражение типа "System.Web.Mvc.ActionResult" нельзя использовать для типа возвращаемого значения "System.Web.Mvc.ActionResult".

Question

Я работаю над некоторым унаследованным кодом (def: непроверенный код - некоторые хорошо спроектированы, некоторые нет) и пытаюсь разработать некоторые тесты, чтобы подтвердить, что недавние изменения сделали то, что они ожидали, и т. Д. Я столкнулся с проблемой, в которой я пытаюсь заставить метод с блоком try{catch} для выдачи исключения с использованием Moq. Когда я пытаюсь запустить тест, он завершается неудачно во время вызова mock.Setup с System.ArgumentException "Выражение типа" System.Web.Mvc.ActionResult "не может использоваться для возвращаемого типа" System.Web.Mvc.ActionResult "".

Базовая настройка кода:

Интерфейс для FilterController...

public interface IFilterController
{
    ActionResult DeleteFilter(string reportFilter, bool customReport = true);
}

Класс FilterController...

public class FilterController : BaseController, IFilterController
{
    public FilterController(
        IServiceFactory serviceFactory,
        IAwsServiceFactory awsServiceFactory,
        IReportServiceFactory reportServiceFactory,
        IAzureServiceFactory azureServiceFactory)
        : base(typeof(FilterController), serviceFactory, awsServiceFactory, reportServiceFactory, azureServiceFactory)
    {
    }

     // method under test
    public ActionResult (string reportFilter, bool customReport = true) {
        try {
             // NOTE: I have trimmed down the actual code in the try block significantly for brevity - I should be able to hook onto something here as a way to mock something throwing an exception
             var customReportFilterService = _serviceFactory.CreateCustomReportFilterService();
            var emailReportSettingService = _serviceFactory.CreateEmailReportSettingService();
            string message = string.Empty;
            JsonReturnType type = JsonReturnType.DisplayMessage; // an enum

            var filter = customReportFilterService.GetReportFilterByHash(SessionHelper.User.CustomerId, reportFilter, initLinkedProjects: true);
            return JsonActionResult(type, ajaxMessage: message, redirectTo: filter == null ? null : string.Format("Report/{0}", filter.ReportName));
        }
        catch (Exception ex)
        {
            return JsonActionResult(JsonReturnType.Error, ajaxMessage: "There was an error in deleting the filter.");
        }
    }         
}

Класс BaseController...

public class BaseController : Controller
{
    private readonly ProgressController _progressController;
    protected IServiceFactory _serviceFactory;
    protected IAwsServiceFactory _awsServiceFactory;
    protected IReportServiceFactory _reportServiceFactory;
    protected IAzureServiceFactory _azureServiceFactory;
    protected IApplicationSettingService _applicationSettingService;
    protected IReportMonitorService _reportMonitorService;
    protected ISymmetricAlgorithmProvider HiddenEncrypter { get; set; }
    private Stopwatch _watch;
    private bool _timePageEnabled;
    private bool _maintenance;
    private int _pageLoadThreshold;
    private readonly ILog Logger;

    public BaseController(Type type, IServiceFactory serviceFactory, IAwsServiceFactory awsServiceFactory, IReportServiceFactory reportServiceFactory, IAzureServiceFactory azureServiceFactory)
    {
        Logger = LogManager.GetLogger(type);
        _progressController = new ProgressController();
        _serviceFactory = serviceFactory;
        _awsServiceFactory = awsServiceFactory;
        _reportServiceFactory = reportServiceFactory;
        _azureServiceFactory = azureServiceFactory;
        _applicationSettingService = _serviceFactory.CreateApplicationSettingService();
        _reportMonitorService = _serviceFactory.CreateReportMonitorService();

        _watch = new Stopwatch();
        _timePageEnabled = _applicationSettingService.ReadApplicationSettingFromCache<bool>(CC.Data.Model.Constants.ApplicationSettings.CheckSlowPageLoad, true);
        _pageLoadThreshold = _applicationSettingService.ReadApplicationSettingFromCache<int>(CC.Data.Model.Constants.ApplicationSettings.PageLoadThreshold, 120);
        _maintenance = _applicationSettingService.MaintenanceMode();
    }

   // System.Web.Mvc.ActionResult type mentioned in error
   public ActionResult JsonActionResult(JsonReturnType returnType, string view = null, string ajaxMessage = null, string redirectTo = null, string target = null, object data = null, string popupTitle = null)
    {
        if (returnType == JsonReturnType.LoadContent)
           _progressController.CompleteProgressCache();

        var serializer = new JavaScriptSerializer();
        serializer.MaxJsonLength = Int32.MaxValue;

        var resultData = new { 
            ReturnType = returnType, 
            HtmlView = view, 
            Message = ajaxMessage, 
            RedirectTo = redirectTo, 
            Target = target, 
            CustomData = data, 
            ProjectId = SessionHelper.ProjectId, 
            PopupTitle = popupTitle,
            MaintenanceMode = _maintenance
        };

        ContentResult result;

        result = new ContentResult
        {
                Content = serializer.Serialize(resultData),
                ContentType = "application/json"
        };

        return result;
    }
}

Класс контроллера...

public abstract class Controller : ControllerBase, IActionFilter, IAuthorizationFilter, IDisposable, IExceptionFilter, IResultFilter {
    // stuff
}

Юнит тестовый класс...

[TestClass]
public class FilterControllerTest
{
    private FilterController filterController;
    private Mock<IFilterController> filterControllerMock;

    private Mock<IServiceFactory> serviceFactoryMock;
    private Mock<IAwsServiceFactory> awsServiceFactoryMock;
    private Mock<IReportServiceFactory> reportServiceFactoryMock;
    private Mock<IAzureServiceFactory> azureServiceFactoryMock;
    private Mock<IApplicationSettingService> applicationSettingServiceMock;

    [ClassInitialize]
    public static void ClassInit(TestContext context)
    {
    }

    [TestInitialize]
    public void Initialize()
    {
        filterControllerMock = new Mock<IFilterController>();
        serviceFactoryMock = new Mock<IServiceFactory>();
        awsServiceFactoryMock = new Mock<IAwsServiceFactory>();
        reportServiceFactoryMock = new Mock<IReportServiceFactory>();
        azureServiceFactoryMock = new Mock<IAzureServiceFactory>();
        applicationSettingServiceMock = new Mock<IApplicationSettingService>();

        serviceFactoryMock
            .Setup(s => s.CreateApplicationSettingService())
            .Returns(applicationSettingServiceMock.Object);

        filterController = new FilterController(
            serviceFactoryMock.Object
            , awsServiceFactoryMock.Object
            , reportServiceFactoryMock.Object
            , azureServiceFactoryMock.Object);
    }

    [TestCleanup]
    public void Cleanup()
    {
    }

    [ExpectedException(typeof(Exception))]
    [TestMethod]
    public void DeleteFilter_ExceptionThrown_IsCaughtAndLoggedAndReturnsActionResultOfError()
    {

        // Arrange
        filterControllerMock
            .Setup(x => x.DeleteFilter(It.IsAny<string>(), It.IsAny<bool>()))
            .Throws(new Exception());

        // Act
        var result = filterController.DeleteFilter("myfilt", false);
    }
}

В конце концов, все, что я хочу сделать, это сделать так, чтобы при вызове DeleteFilter для этого теста выдается ошибка, а затем я могу утверждать, что возвращается из блока catch.

РЕДАКТИРОВАТЬ: в основном обновил сообщение по предложению, чтобы облегчить понимание, где проблема.

Answer 1

Вот уменьшенный пример

Дано

public interface IServiceFactory {
    object GetService(string args);
}

public class MyController : Controller {
    IServiceFactory serviceFactory

    public MyController(IServiceFactory serviceFactory) {
        this.serviceFactory = serviceFactory;
    }

     // method under test
    public ActionResult DeleteFilter(string args) {
        try {
            var model = serviceFactory.GetService(args);
            return View(model);
        } catch(Exception ex) {
           return JsonActionResult(JsonReturnType.Error, ajaxMessage: "There was an error in deleting the filter.");
        }
    }         
}

Вы можете использовать moq в своем тесте, как это

[TestMethod]
public void DeleteFilter_ExceptionThrown_IsCaughtAndLoggedAndReturnsActionResultOfError() {
    // Arrange
    var serviceFactoryMock = new Mock<IServiceFactory>();
    serviceFactoryMock
        .Setup(x => x.GetService(It.IsAny<string>())
        .Throws(new Exception())
        .Verifiable();

    var controller = new MyController(serviceFactoryMock.Object);

    // Act
    var result = controller.DeleteFilter("blah blah");

    //Assert
    serviceFactoryMock.Verify(); // verifies that the setup was invoked
    Assert.IsNotNull(result);
    Assert.IsInstanceOfType(result, typeof(JsonActionResult));
    //...other assertions
}

Так что теперь в примере, когда DeleteFilter вызывается, фабрика фальшивых сервисов вызывается, выдается ошибка, основанная на настройке, и вы можете утверждать, что возвращается из блока catch.