为什么 RhinoMocks 在我的测试中抛出 InvalidOperationException？

Question

这是我目前正在进行的测试：
（根据 Lees 的回答编辑）

[Test]
public void AddLockClassWithNullNameShouldCallInsertOnSessionWithEmptyString()
{
    LockClass lockClass = new LockClass { Id = ValidId, Name = null };

    using ( mockRepository.Record() ) {
        sessionFactory.CreateSession();
        LastCall.Return( session );

        session.InsertWithId( lockClass );
        LastCall.Return( lockClass );

        session.Commit();
        session.Dispose();
    }

    using ( mockRepository.Playback() ) {
        controller = new LockClassPanelController( sessionFactory );
        controller.AddLockClass( lockClass.Id, string.Empty );
    }

    mockRepository.VerifyAll();
}

运行测试结果为：

Test 'Test.Unit.Controllers.LockClassPanelControllerTests.AddLockWithNullNameClassShouldCallInsertOnSessionWithEmptyString' failed:
 System.InvalidOperationException : The operation is invalid because of the current object state. (translated from german, dunno if thats the original english wording)
 at System.Reflection.RuntimeMethodInfo.GetGenericMethodDefinition()
 at Rhino.Mocks.MethodRecorders.UnorderedMethodRecorder.MethodsEquals(MethodInfo method, ProxyMethodExpectationTriplet triplet)
 at Rhino.Mocks.MethodRecorders.UnorderedMethodRecorder.GetAllExpectationsForProxyAndMethod(Object proxy, MethodInfo method)
 at Rhino.Mocks.MethodRecorders.UnorderedMethodRecorder.CalcExpectedAndActual.Calculate(Object proxy, MethodInfo method, Object[] args)
 at Rhino.Mocks.MethodRecorders.UnorderedMethodRecorder.CalcExpectedAndActual..ctor(UnorderedMethodRecorder parent, Object proxy, MethodInfo method, Object[] args)
 at Rhino.Mocks.MethodRecorders.UnorderedMethodRecorder.UnexpectedMethodCall(IInvocation invocation, Object proxy, MethodInfo method, Object[] args)
 at Rhino.Mocks.MethodRecorders.UnorderedMethodRecorder.DoGetRecordedExpectation(IInvocation invocation, Object proxy, MethodInfo method, Object[] args)
 at Rhino.Mocks.MethodRecorders.MethodRecorderBase.GetRecordedExpectation(IInvocation invocation, Object proxy, MethodInfo method, Object[] args)
 at Rhino.Mocks.Impl.ReplayMockState.DoMethodCall(IInvocation invocation, MethodInfo method, Object[] args)
 at Rhino.Mocks.Impl.ReplayMockState.MethodCall(IInvocation invocation, MethodInfo method, Object[] args)
 at Rhino.Mocks.MockRepository.MethodCall(IInvocation invocation, Object proxy, MethodInfo method, Object[] args)
 at Rhino.Mocks.Impl.RhinoInterceptor.Intercept(IInvocation invocation)
 at Castle.DynamicProxy.AbstractInvocation.Proceed()
 at ISessionProxy2762dfaac4274133bc97e10d4e5c35d0.InsertWithId[TEntity](TEntity entity)
 Controllers\LockClassPanelController.cs(20,0): at Artea.Service.Controllers.LockClassPanelController.AddLockClass(Int32 id, String name)
 Unit\Controllers\LockClassPanelControllerTests.cs(80,0): at Test.Unit.Controllers.LockClassPanelControllerTests.AddLockWithNullNameClassShouldCallInsertOnSessionWithEmptyString()

有什么想法吗？

编辑：
我刚刚发现，如果更改方法的第一行，它就可以正常工作：（

LockClass lockClass = new LockClass { Id = ValidId, Name = string.Empty };

string.Empty 而不是 null） 但是测试应该检查如果 Name 属性为 null 会发生什么，因此将 Name 设为除 null 以外的任何其他值都不会很有帮助。

编辑：
该代码实际上没有测试我想测试的内容。该方法的第一行应该是

LockClass lockClass = new LockClass { Id = ValidId, Name = string.Empty };

Thats myected object。 LockClass 是一个 DTO，仅包含上面一行中初始化的两个属性以及必需的 Equals 内容。
行为行应该是

controller.AddLockClass( lockClass.Id, null );

[SetUp] 创建所有模拟对象。我想要测试的是，如果用户通过 GUI 手势创建一个 LockClass 对象（GUI 在控制器上调用 AddLockClass），其中名称为 null，则控制器会创建一个名称为空的对象。还有其他方法可以做到这一点，但现在必须采用这种方式。更改后的代码确实有效（例如，Rhino 不会呕吐）。我仍然保留这个问题，因为很好奇为什么 Rhino 不喜欢原始代码。

要使其完整：

private const int ValidId = 4711;
private const int InvalidId = 0;
private MockRepository mockRepository;
private ISessionFactory sessionFactory;
private ISession session;
private LockClassPanelController controller;

[SetUp]
public void Setup()
{
    mockRepository = new MockRepository();
    sessionFactory = mockRepository.StrictMock<ISessionFactory>();
    session = mockRepository.StrictMock<ISession>();
}

编辑：

public void AddLockClass( int id, string name )
{
    if ( id != 0 ) {
        using ( var session = sessionFactory.CreateSession() ) {
            session.InsertWithId( new LockClass { Id = id, Name = name } );
            session.Commit();
        }
        LoadLockClasses();
        view.Initialize();
    }
}

Answer 1

我认为您需要将测试的“act”部分包含在播放范围内：

using(mockRepository.Playback())
{
    controller = new LockClassPanelController( sessionFactory ); 
    controller.AddLockClass( lockClass.Id, string.Empty );
}

mockRepository.VerifyAll();

Answer 2

Rhino Mocks 为您提供正确的异常。在 AddLockClass 中，您有以下行：

session.InsertWithId( new LockClass { Id = id, Name = ( name ?? string.Empty ) } );

它清楚地表明，如果您使用 null 名称参数调用此方法，则在创建 LockClass 实例时，它仍将使用空字符串。因此，正如您在问题的第一次编辑中注意到的那样，您在测试中应该期望的 LockClass 是：

LockClass lockClass = new LockClass { Id = ValidId, Name = string.Empty };

而不是带有 null 的版本。如果您需要进一步说明，请告诉我。