Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to Unit Test with ActionResult<T>?

I have a xUnit test like:

[Fact]
public async void GetLocationsCountAsync_WhenCalled_ReturnsLocationsCount()
{
    _locationsService.Setup(s => s.GetLocationsCountAsync("123")).ReturnsAsync(10);
    var controller = new LocationsController(_locationsService.Object, null)
    {
        ControllerContext = { HttpContext = SetupHttpContext().Object }
    };
    var actionResult = await controller.GetLocationsCountAsync();
    actionResult.Value.Should().Be(10);
    VerifyAll();
}

Source is

/// <summary>
/// Get the current number of locations for a user.
/// </summary>
/// <returns>A <see cref="int"></see>.</returns>
/// <response code="200">The current number of locations.</response>
[HttpGet]
[Route("count")]
public async Task<ActionResult<int>> GetLocationsCountAsync()
{
    return Ok(await _locations.GetLocationsCountAsync(User.APropertyOfTheUser()));
}

The value of the result is null, causing my test to fail, but if you look at ActionResult.Result.Value (an internal property) it contains the expected resolved value.

See the following screen capture of the debugger. enter image description here

How do I get the actionResult.Value to populate in a unit test?

like image 247
Richard Collette Avatar asked Jul 24 '18 01:07

Richard Collette


People also ask

Should you write unit tests for controllers?

A unit test for a controller is pointless. You test your controllers implicitly via regression tests (using Selenium for example). You should TDD all the components/objects the controller uses and keep the controllers as thin as possible.


6 Answers

At run time your original code under test would still work because of the implicit conversion.

But based on the provided debugger image it looks like the test was asserting on the wrong property of the result.

So while changing the method under test allowed the test to pass, it would have worked when run live either way

ActioResult<TValue> has two properties that are set depending on what is returned from the action that uses it.

/// <summary>
/// Gets the <see cref="ActionResult"/>.
/// </summary>
public ActionResult Result { get; }

/// <summary>
/// Gets the value.
/// </summary>
public TValue Value { get; }

Source

So when the controller action returned using Ok() it would set the ActionResult<int>.Result property of the action result via implicit conversion.

public static implicit operator ActionResult<TValue>(ActionResult result)
{
    return new ActionResult<TValue>(result);
}

But the test was asserting the Value property (refer to image in OP), which in this case was not being set.

Without having to modify the code under test to satisfy the test it could have accessed the Result property and make assertions on that value

[Fact]
public async Task GetLocationsCountAsync_WhenCalled_ReturnsLocationsCount() {
    //Arrange
    _locationsService
        .Setup(_ => _.GetLocationsCountAsync(It.IsAny<string>()))
        .ReturnsAsync(10);
    var controller = new LocationsController(_locationsService.Object, null) {
        ControllerContext = { HttpContext = SetupHttpContext().Object }
    };

    //Act
    var actionResult = await controller.GetLocationsCountAsync();

    //Assert
    var result = actionResult.Result as OkObjectResult;
    result.Should().NotBeNull();
    result.Value.Should().Be(10);

    VerifyAll();
}
like image 84
Nkosi Avatar answered Oct 25 '22 03:10

Nkosi


The problem lies in the confusing interface of ActionResult<T> that was never designed to be used by us humans. As stated in other answers, ActionResult<T> has either its Result or Value property set but not both. When you return an OkObjectResult the framework populates the Result property. When you return an object the framework populates the Value property.

I created the following simple helper for my test library to help me test the return values when I am using OkObjectResult Ok() or other results inheriting from ObjectResult

private static T GetObjectResultContent<T>(ActionResult<T> result)
{
    return (T) ((ObjectResult) result.Result).Value;
}

This lets me do the following:

var actionResult = await controller.Get("foobar");
Assert.IsType<OkObjectResult>(actionResult.Result);
var resultObject = GetObjectResultContent<ObjectType>(actionResult);
Assert.Equal("foobar", resultObject.Id);
like image 38
Otto Teinonen Avatar answered Oct 25 '22 02:10

Otto Teinonen


The issue is wrapping it in Ok. If you return the object itself, Value is populated correctly.

If you look at Microsoft's examples in the docs, they only use the controller methods for non-default responses like NotFound:

[HttpGet("{id}")]
[ProducesResponseType(StatusCodes.Status404NotFound)]
public ActionResult<Product> GetById(int id)
{
    if (!_repository.TryGetProduct(id, out var product))
    {
        return NotFound();
    }

    return product;
}
like image 20
Eric Eskildsen Avatar answered Oct 25 '22 03:10

Eric Eskildsen


As a continuation to the answer provided by @Nkosi and @Otto Teinonen, there is an implicit opertator to convert to/from Value/Result

Also, ActionResult<TValue> implement the interface IConvertToActionResult

  public sealed class ActionResult<TValue> : IConvertToActionResult

and provide Convert method that handle both Result and Value based on their null values, see the source and return the non null value. Either Result or Value is set.

If the Controler didn't return Ok(value), using OkObjectResult may return null and cause test fail, like:

 var result = actionResult.Result as OkObjectResult; //may be null

The next extension method handle both Result and Value and return the typed object T

public static T GetObjectResult<T>(this ActionResult<T> result)
        {
            if (result.Result != null)
                return (T)((ObjectResult)result.Result).Value;
            return result.Value;            
        }

Test case

[Test]
public async Task Test()
{
    var controller = new ProductsController(Repo);
    var result = await controller.GetProduct(1);
    await Repo.Received().GetProduct(1);
     //result.Result.Should().BeOfType<OkObjectResult>();  // may fail if controller didn't return Ok(value)
     result.GetObjectResult().ProductId.Should().Equals(1);
 }
like image 36
M.Hassan Avatar answered Oct 25 '22 02:10

M.Hassan


Another method would be to serialize and then desalize in json type:

[Fact]
public void Test1()
{
    var controller = new ProductController();
    var result = controller.Products();
    var okResult = Assert.IsType<OkObjectResult>(result);
    if (okResult.Value == null) return;
    var json = JsonConvert.SerializeObject(okResult.Value);
    var values = JsonConvert.DeserializeObject<Product>(json);
    if (values != null) Assert.Equal(1, values.Id);
}

private class Product
{
    public int Id { get; set; }
    public string Name { get; set; }
}

[Route("api/product")]
[ApiController]
public class ProductController : ControllerBase
{
    [HttpGet]
    public IActionResult Products() => Ok(new { Id = 1, Name = "Smartphone" });
}
like image 36
LucioPerez Avatar answered Oct 25 '22 01:10

LucioPerez


Building on other answers, I think this is what you want...

var actionResult = await controller.GetLocationsCountAsync();
var okResult = Assert.IsType<OkObjectResult>(actionResult.Result);
var value = Assert.IsType<int>(okResult.Value);
Assert.Equal(10, value);

The desired .Value is inside the OkObjectResult

My return type was IEnumerable<MyClass> so I used:

var values = Assert.IsAssignableFrom<IEnumerable<MyClass>>(okResult.Value);
like image 28
Peter L Avatar answered Oct 25 '22 03:10

Peter L