Unit testing/How to assert a non-mocked method was called/C#/Rhino Mocks/NUnit

Question

I have a MailService which permit me to send Email which implement the following Interface.

public interface IMailService
  {
    bool SendRegisteringEmail(RegisterModel registerModel);
    bool SendMail(MailMessage mailMessage);
    MailMessage CreateRegisteringMailMessage(RegisterModel registerModel);

    IAppSettingsRepository AppSettingsRepository { get; set; }

    ISmtpClient SmtpClient { get; set; }
  }

The function SendRegisteringEmail should call CreateRegisteringMailMessage then give the return MailMessage to the SendMail function and SendRegisteringEmail should return the return Boolean of SendMail.

I'm using NUnit and Rhino Mocks to do my test, I'm new to testing (1week) and I'm practicing TDD (At least I try). My problem is I don't know how to assert that CreateRegisteringMailMessage was called when I call SendRegisteringEmail, because MailService isn't a Mocked Object. Here is my test:

[Test]
public void SendRegisteringEmail_ShouldCallCreateRegisteringMailMessage()
{
  //Arrange
  //MailMessage mailMessage = new MailMessage();
  IMailService mailService = new MailService { AppSettingsRepository = CreateAppSettingsMockReposotory() };
  //ISmtpClient smtpClientStub = MockRepository.GenerateStub<ISmtpClient>();
  //mailService.SmtpClient = smtpClientStub;
  //smtpClientStub.Stub(f => f.Send(mailMessage)).Return(true);


  //Act
  mailService.SendRegisteringEmail(ValidRegisterModel);

  //Assert
  mailService.AssertWasCalled(f => f.CreateRegisteringMailMessage(ValidRegisterModel));
}

I get the following error when I launch my test: FtpWeb.Tests.MailServiceTests.SendRegisteringEmail_ShouldCallCreateRegisteringMailMessage: System.InvalidOperationException : The object 'FtpWeb.Models.MailService' is not a mocked object.

I understand why I'm getting this error but now how to test my call. Since it's the same object I can't mock it to test it. If anybody can give me some lead to resolve this.

Thanks.

Answer 1

The methods CreateRegisteringMailMessage and SendMail seem on a lower abstraction level than SendRegisteringEmail. You could consider creating a higher level class containing SendRegisteringEmail, and this class would use IMailService, which you could mock and assert as usual.

If the first solution is not an option, then you should treat SendRegisteringEmail as a whole, so effectively you have to test the side effects of both CreateRegisteringMailMessage and SendMail in a single test (a bit of a code smell, hence my suggestion to extract another level of indirection) - see Jon's answer.