我应该如何模拟这个简单的服务层方法？

Question

我的服务层中有以下简单方法。我不确定如何从存储库开始模拟不同的链接部分？

public ICollection<GameFile> FindAllActiveGamesFiles()
{
    return _gameFileRepository // <- this is an IRepository<GameFile>
        .Find() // <-- returns an IQueryable<T> .. in this case, an IQueryable<GameFile>
        .WhereIsActive() // <-- a simple Pipe/Filter; 'where x.IsActive'
        .ToList();
}

我的偏好是使用起订量，但我很高兴看到其他的暗示.. 主要是因为我不是在追求确切的语法答案，而是在理论上的答案。 IE。你只需要模拟IRepository<GameFile> 并设置ToList() 方法.. 等等等等...

这就是我不明白我应该嘲笑的东西。

干杯:)

--- 更新：澄清我的问题

我要测试的是“FindAllActiveGamesFiles()”方法是否有效。因此，我相信我需要模拟 gameFileRepository （这是一个接口）。我不确定这是否是我在这个例子中唯一应该嘲笑的事情。

例如。

[TestMethod]
public void MyTest()
{
    // Arrange.
    Mock<IRepository<GameFile>> mockRepository = new Mock<IRepository<GameFile>>();
    mockRepository.Setup(....).MoreStuffToDo(...); // <-- that's what i'm unsure about.
    IGameFileService = new GameFileService(mockRepository, fakeLoggingService);

    // Act.
    var gameFiles = gameFileService.FindAllActiveGamesFiles();

    // Asserts.
    Assert.IsNotNull(gameFiles);
    CollectionAssert.AllItemsAreNotNull(gameFiles.ToArray());
    // .. and more asserts ///
    // What about expectations? eg. that ToList() was entered/called?
}

Answer 1

听起来您想编写一个测试来展示您的 Linq 语句的覆盖范围。您已经指出您的存储库是一个接口 (IRepository) 并且应该被模拟。您只需要一些示例来说明您的服务层如何正确过滤存储库的内容。

据我所知，这是您的 ServiceLayer。

public class ServiceLayer
{
     private readonly IRepository<GameFile> _gameRepository;

     public SerivceLayer(IRepository<GameFile> repository)
     {
         _gameRepository = repository;
     }

     public IEnumerable<GameFile> FindAllActiveGamesFiles()
     {
         return _gameRepository
                    .Find()  // method we need to mock
                    .Where( gameFile => gameFile.IsActive)
                    .ToList();
     }
}

让我们编写一些测试......（NUnit 和 Moq）

[TestFixture]
public class ServiceLayerFixture
{
      protected IRepository<GameFile> Repository;
      protected ServiceLayer Subject;
      protected ICollection<GameFile> Results;

      [Setup]
      public void Setup()
      {
          // create our mock
          Repository = new Mock<IRepository<GameFile>>().Object;

          // initialize our test subject
          Subject = new ServiceLayer(Repository);
      }

      [Test]
      public void WhenRepositoryDoesNotContainItems_ServiceLayer_ReturnsAnEmptyCollection()
      {
         Mock.Get(Repository)
             .Setup( r => r.Find())
             .Returns( new List<GameFile>().AsQueryable() );

         Results = Subject.FindAllActiveGameFiles();

         Assert.AreEqual(0, Results.Count);
      }

      [Test]
      public void WhenRepositoryDoesNotContainActiveItems_ServiceLayer_ReturnsAnEmptyCollection()
      {
         Mock.Get(Repository)
             .Setup( r => r.Find())
             .Returns( 
                 new List<GameFile>()
                     {
                        new GameFile { IsActive = false },
                        new GameFile { IsActive = false }
                     }.AsQueryable() );

         Results = Subject.FindAllActiveGameFiles();

         Assert.AreEqual(0, Results.Count);            
      }

      [Test]
      public void WhenRepositoryContainActiveItems_ServiceLayer_FiltersItemsAppropriately()
      {
         Mock.Get(Repository)
             .Setup( r => r.Find())
             .Returns( 
                 new List<GameFile>()
                     {
                        new GameFile { IsActive = true },
                        new GameFile { IsActive = false }
                     }.AsQueryable() );

         Results = Subject.FindAllActiveGameFiles();

         Assert.AreEqual(1, Results.Count);            
      }
}

您的代码不足之处在于您可以更优雅地处理 IRepository 中的异常。

所以考虑一下：

[Test]
public void WhenTheRepositoryFails_ServiceLayer_ShouldHandleExceptionsGracefully()
{
    Mock.Get(Repository)
        .Setup( r => r.Find())
        .Throws( new InvalidOperationException() );

    Results = Subject.FindAllActiveGameFiles();

    Assert.AreEqual(0, Results.Count);          
}

或者，也许您想包装异常？

[Test]
[ExpectedException(typeof(GameFileNotFoundException))]
public void WhenTheRepositoryFails_ServiceLayer_ShouldReportCustomError()
{
    Mock.Get(Repository)
        .Setup( r => r.Find())
        .Throws( new InvalidOperationException() );

    Subject.FindAllActiveGameFiles();
}

Answer 2

通过断言从ToList() 返回的值，您似乎正在尝试测试实际方法Find()、WhereIsActive() 和ToList()。如果是这样，您就不需要使用模拟对象。

相反，我会为每个方法编写单独的测试。这是健全的单元测试的关键。如果可以的话，FindAllActiveGamesFiles() 的测试应该避免测试它们。如果可能，您应该只测试FindAllActiveGamesFiles() 中的代码。

接下来，FindAllActiveGamesFiles() 的测试需要为mockRepository 设置期望值。假设 Find() 和 WhereIsActive() 都返回相同的存储库，您将模拟设置为这些方法的返回值。

我不知道你的模拟库的 API，但这是它在伪代码中的样子：

Mock<IRepository<GameFile>> mockRepository = new Mock<IRepository<GameFile>>();
List<GameFile> result = new List<GameFile>();
mockRepository.expect('Find')->willReturn(mockRepository);
mockRepository.expect('WhereIsActive')->willReturn(mockRepository);
mockRepository.expect('ToList')->willReturn(result);
IGameFileService service = new GameFileService(mockRepository, fakeLoggingService);

assertSame(service.FindAllActiveGamesFiles(), result);

如您所见，测试代码比被测试代码更复杂。这是您可能不需要测试的好兆头。