如何对拦截器进行单元测试？

Question

我想为拦截 Loggable 基类（实现 ILoggable）的拦截器编写一些单元测试。
Loggable 基类没有可调用的方法，仅用于由日志记录工具进行初始化。
据我了解，我应该：

1. 模拟 ILoggable 和 ILogger
2. 初始化日志记录工具
3. 在其上注册我的拦截器
4. 调用模拟的 ILoggable 的某些

方法问题是我的 ILoggable 接口没有可调用的方法，因此不会拦截任何内容。
在这里正确的做法是什么？
我应该手动模拟 ILoggable 并添加一个存根方法来调用吗？
另外，我也应该嘲笑容器吗？

我正在使用 Moq 和 NUnit。
编辑：
这是我的拦截器实现供参考：

public class LoggingWithDebugInterceptor : IInterceptor
{
    #region IInterceptor Members

    public void Intercept(IInvocation invocation)
    {
        var invocationLogMessage = new InvocationLogMessage(invocation);

        ILoggable loggable = invocation.InvocationTarget as ILoggable;

        if (loggable == null)
            throw new InterceptionFailureException(invocation, string.Format("Class {0} does not implement ILoggable.", invocationLogMessage.InvocationSource));

        loggable.Logger.DebugFormat("Method {0} called with arguments {1}", invocationLogMessage.InvokedMethod, invocationLogMessage.Arguments);

        Stopwatch stopwatch = new Stopwatch();
        try
        {
            stopwatch.Start();
            invocation.Proceed();
            stopwatch.Stop();
        }
        catch (Exception e)
        {
            loggable.Logger.ErrorFormat(e, "An exception occured in {0} while calling method {1} with arguments {2}", invocationLogMessage.InvocationSource, invocationLogMessage.InvokedMethod, invocationLogMessage.Arguments);
            throw;
        }
        finally
        {
            loggable.Logger.DebugFormat("Method {0} returned with value {1} and took exactly {2} to run.", invocationLogMessage.InvokedMethod, invocation.ReturnValue, stopwatch.Elapsed);
        }
    }

    #endregion IInterceptor Members
}

Answer 1

如果只是拦截器使用类上的 Logger 属性，那为什么还要在那里呢？你也可以把它放在拦截器上。 （就像 Ayende 在他的帖子中解释的那样）。

除此之外 - 拦截器只是一个与接口交互的类 - 一切都高度可测试。

Answer 2

我同意 Krzysztof 的观点，如果您希望通过 AOP 添加日志记录，那么有关日志记录的责任和实现细节应该对调用者透明。因此，它是拦截器可以拥有的东西。我将尝试概述如何测试这一点。

如果我正确地理解了问题，您的 ILoggable 实际上只是一个注释类的命名容器，以便拦截器可以确定是否应该执行日志记录。它公开包含记录器的属性。 （这样做的缺点是该类仍然需要配置 Logger。）

public interface ILoggable
{
     ILogger { get; set; }
}

测试拦截器应该是一个直接的过程。我看到您提出的唯一挑战是如何手动构造IIn Vocation 输入参数，使其类似于运行时数据。我建议您使用经典的基于状态的验证来测试它，而不是尝试通过模拟等来重现这一点：创建一个使用您的拦截器的代理并验证您的日志是否反映了您的期望。

这可能看起来需要更多工作，但它提供了一个非常好的示例，说明拦截器如何独立于代码库的其他部分工作。您团队中的其他开发人员可以从中受益，因为他们可以参考此示例作为学习工具。

public class TypeThatSupportsLogging : ILoggable
{
     public ILogger { get; set; }

     public virtual void MethodToIntercept()
     {
     }

     public void MethodWithoutLogging()
     {
     }
}

public class TestLogger : ILogger
{
     private StringBuilder _output;

     public TestLogger()
     {
        _output = new StringBuilder();
     }

     public void DebugFormat(string message, params object[] args)
     {
        _output.AppendFormat(message, args);
     }

     public string Output
     {
        get { return _output.ToString(); }
     }
}

[TestFixture]
public class LoggingWithDebugInterceptorTests
{
     protected TypeThatSupportsLogging Input;
     protected LoggingWithDebugInterceptor Subject;
     protected ILogger Log;         

     [Setup]
     public void Setup()
     {
         // create your interceptor
         Subject = new LoggingWithDebugInterceptor();

         // create your proxy
         var generator = new Castle.DynamicProxy.ProxyGenerator();
         Input = generator.CreateClassProxy<TypeThatSupportLogging>( Subject );

         // setup the logger
         Log = new TestLogger();
         Input.Logger = Log;
     }

     [Test]
     public void DemonstrateThatTheInterceptorLogsInformationAboutVirtualMethods()
     {
          // act
          Input.MethodToIntercept();

          // assert
          StringAssert.Contains("MethodToIntercept", Log.Output);
     }

     [Test]
     public void DemonstrateNonVirtualMethodsAreNotLogged()
     {
          // act
          Input.MethodWithoutLogging();

          // assert
          Assert.AreEqual(String.Empty, Log.Output);
     }
}

Answer 3

没有方法吗？你在测试什么？

就个人而言，这听起来太过分了。我意识到 TDD 和代码覆盖率是教条，但如果您模拟一个没有方法的接口，并证明模拟框架按照您的指示执行操作，那么您真正证明了什么？

这里还有另一个误导：日志记录是面向方面编程的“hello world”。你为什么不记录拦截器/方面？如果您这样做，那么您的所有类就没有理由实现 ILoggable ；您可以用声明性的日志记录功能来装饰它们。我认为这是一种侵入性较小的设计，并且更好地利用了拦截器。