Question

I need to setup a policy in base controller that applies to all controller instance, like below:

  public class BaseController : Controller
    {

        private IPolicy Policy;

        public BaseController()
        {
            this.Policy= new Policy(HttpContext);
        }
    }

Within the Policy class, I need to do something like:

   this.httpContextBase.User.

Questions: (Update)

What is the better way to design the BaseController in terms of using HttpContext and Unit test.

Answer 1

What is the correct way to unit test HttpContext?

Absolutely no way. You are using the HttpContext inside the constructor of a controller when this context is still not initialized. Not only that this code cannot be tested but when you run the application it will also crash with NRE. You should never use any HttpContext related stuff in a constructor of a controller.

One possibility is to refactor your code and perform this inside the Initialize method:

public class BaseController : Controller
{
    private IPolicy Policy;

    protected override void Initialize(RequestContext requestContext)
    {
        base.Initialize(requestContext);
        this.Policy = new Policy(HttpContext);
    }
}

---

This being said, that’s not the approach I would recommend. I would recommend you using dependency injection instead of service location which is considered by many as an anti-pattern.

So:

public abstract class BaseController : Controller
{
    protected IPolicy Policy { get; private set; }

    protected BaseController(IPolicy policy)
    {
        Policy = policy;
    }
}

Now, all that’s left is to configure your favourite Dependency Injection framework to inject the correct instance into the constructor. For example with Ninject.Mvc3 this is achieved with a single line of code:

kernel.Bind<IPolicy>().To<Policy>();

Now you can feel more than free to mock this IPolicy in your unit test without even caring about any HttpContext.

For example let’s suppose that you have the following controller that you want to unit test:

public class FooController : BaseController
{
    public FooController(IPolicy policy): base(policy)
    { }

    [Authorize]
    public ActionResult Index()
    {
        Policy.DoSomething();
        return View();
    }
}

Now, all that you need to do is pick up your favorite mock framework (Rhino Mocks in my case) and do the mocking:

[TestMethod]
public void Index_Action_Should_DoSomething_With_The_Policy()
{
    // arrange
    var policyStub = MockRepository.GenerateStub<IPolicy>();
    var sut = new FooController(policyStub);

    // act
    var actual = sut.Index();

    // assert
    Assert.IsInstanceOfType(actual, typeof(ViewResult));
    policyStub.AssertWasCalled(x => x.DoSomething());
}