ModelState.IsValid always true when testing Controller in Asp.Net MVC Web Api

Question

I have tried to make this work and made many google/stackoverflow searches with no luck at all.

I have a simple Model:

public class MovieModel
{
    public string Id { get; set; }

    [Required]
    [StringLength(100)]
    public string Name { get; set; }
}

A method in the controller:

// POST: api/Movies
public IHttpActionResult Post([FromBody]MovieModel movieModel)
{
    if (ModelState.IsValid)
    {
        //Code
    }
}

And a test method (is an integration test, but the same would happen in unit tests):

[TestMethod]
public void MoviesController_Post_Without_Name()
{
    // Arrange
    var model = new MovieModel();
    model.Name = "";

    // Act
    var result = controller.Post(model);

    // Assert
    Assert.IsInstanceOfType(result, typeof(InvalidModelStateResult));
    Assert.AreEqual(6, controller.Get().Count());
}

Despite the fact that the model is clearly invalid it always evaluates the IsValid property to true.

I tried many approaches so far without success.

score 12 · Answer 1 · edited Jun 01 '16 at 02:01

Thanks to this site, I found out the solution:

private void SimulateValidation(object model)
{
    // mimic the behaviour of the model binder which is responsible for Validating the Model
    var validationContext = new ValidationContext(model, null, null);
    var validationResults = new List<ValidationResult>();
    Validator.TryValidateObject(model, validationContext, validationResults, true);
    foreach (var validationResult in validationResults)
    {
        this.controller.ModelState.AddModelError(validationResult.MemberNames.First(), validationResult.ErrorMessage);
    }
}

And including one line in the test method like this:

public void MoviesController_Post_Without_Name()
{
    // Arrange
    var model = new MovieModel();
    model.Name = "";

    // Act
    SimulateValidation(model);
    var result = controller.Post(model);

    // Assert
    Assert.IsInstanceOfType(result, typeof(InvalidModelStateResult));
    Assert.AreEqual(6, controller.Get().Count());
}

Hope that helps someone, it would have saved me some hours hunting the web.

Worked for Core 2.0, for nested just do `SimulateValidation(model); SimulateValidation(model.child);` — ono2012, Nov 02 '17 at 02:11

score 9 · Accepted Answer · answered Jun 01 '16 at 03:44

9

Your solution probably works, but a better way is using ApiController.Validate method.

public void MoviesController_Post_Without_Name()
{
    // Arrange
    var model = new MovieModel();
    model.Name = "";

    // Act
    controller.Validate(model);   //<---- use the built-in method
    var result = controller.Post(model);

    // Assert
    Assert.IsInstanceOfType(result, typeof(InvalidModelStateResult));
    Assert.AreEqual(6, controller.Get().Count());
}

answered Jun 01 '16 at 03:44

Cheng Chen

42,509
16
113
174

I tried this .Validate method before, but it doesn't exist. I'm using MVC 5. – André Baptista Jun 01 '16 at 11:39
The microsoft reference shows the Validate method, for version 5 of MVC , but it simply don't work, neither Intellisense nor build works. – André Baptista Jun 01 '16 at 12:28
@AndréBaptista Why are you talking about MVC? `ApiController` is a Web API class, and is found in `System.Web.Http`. Ensure you added the correct NuGet packages to the test project. – Federico Dipuma Jun 01 '16 at 12:35
Yes, I know it sounds weird, but I checked out everything: - The nuget package (Microsoft.AspNet.WebApi.Core) is installed and is 5.0.0 version, both in Web and Test projects; - The dll System.Web.Http is referenced and is 5.0.0 version on both the web project and the test project; - the using directive is there; - my controller inherits from ApiController; When I click "Go to definition" (F12 key) over ApiController it shows the method list, and there isn't neither Validate nor TryValidate methods. – André Baptista Jun 01 '16 at 13:29

score 1 · Answer 3 · answered Apr 04 '19 at 16:21

On WebAPI 5.2.7:

[TestMethod]
public void MoviesController_Post_Without_Name()()
{
    // Arrange
    var model = new MovieModel();
    model.Name = "";

    controller.Request = new HttpRequestMessage();
    controller.Configuration = new HttpConfiguration();
    controller.Validate(model);

    // Act
    var result = controller.Post(model);

    // Assert
    ...

This article helped me: https://www.c-sharpcorner.com/article/unit-testing-controllers-in-web-api/

score 0 · Answer 4 · answered Jun 21 '18 at 00:16

This worked for me:

public MyResultData Post([FromBody] MyQueryData queryData)
{
    if (!this.Request.Properties.ContainsKey("MS_HttpConfiguration")) 
    {
        this.Request.Properties["MS_HttpConfiguration"] = new HttpConfiguration();
    }

    this.Validate(queryData);

    if (ModelState.IsValid)
    {
        DoSomething();
    }
}

Also check out this question: Validate fails in unit tests

score 0 · Answer 5 · answered Oct 07 '21 at 02:10

0

Manually add a model error:

// Arrange
controller.ModelState.AddModelError("fakeError", "fakeMessage");

// Act
var result = controller.Post(model);

answered Oct 07 '21 at 02:10

the dick

63
1
5

ModelState.IsValid always true when testing Controller in Asp.Net MVC Web Api

5 Answers5

Linked

Related