ASP.NET Web API xUnit测试：注释FakeItEasy Mock仍生成假数据？

这是FakeItEasy的默认行为导致的：当你创建一个接口的Fake实例（比如你的_userRepository），如果没有为某个方法显式配置返回值，FakeItEasy会自动生成该方法返回类型的“默认Fake对象”，而不是返回null。

具体到你的代码场景： #

1. 你的_userRepository是用FakeItEasy创建的Fake实例（应该是在测试类的初始化代码中，比如var _userRepository = A.Fake<IUserRepository>()，然后注入给_controller）。
2. 当控制器调用_userRepository.GetMembersAsync(currentUser.Email)时，因为你没有显式配置这个方法的返回值（注释掉了Returns代码），FakeItEasy会自动生成一个IEnumerable<MemberDto>类型的Fake集合实例——这个实例不是null，是一个空的Fake集合对象。
3. 控制器拿到这个Fake集合后，正常返回Ok(users)，所以你的所有断言（结果非空、类型正确、返回Ok结果）都能通过，和你显式配置返回Fake集合时的效果一致。

为什么会有这个默认行为？ #

FakeItEasy的设计初衷是减少测试中的重复配置，让你只需要关注那些对测试逻辑有影响的方法返回值。对于返回引用类型的方法，默认返回对应类型的Fake实例；对于值类型，返回默认值（比如int返回0，bool返回false）。

如何改进你的测试？ #

当前的测试只能验证接口返回了Ok结果，但没有验证业务逻辑的正确性，建议补充以下内容：

• 验证依赖方法的调用逻辑：确保GetMembersAsync被传入了正确的参数（比如当前用户的邮箱）：

  // 在Assert阶段添加
  A.CallTo(() => _userRepository.GetMembersAsync(A<string>.That.IsEqualTo(currentUserEmail))).MustHaveHappenedOnceExactly();
  

• 验证返回的内容：如果需要验证返回的用户列表是否符合预期，要显式配置GetMembersAsync的返回值，然后断言返回结果的内容：

  // Arrange阶段
  var expectedUsers = new List<MemberDto> { new MemberDto { Id = 1, Name = "Test" } };
  var currentUser = new AppUser { Email = "test@example.com" };
  A.CallTo(() => _userRepository.GetUserByEmailAsync(A<string>._)).Returns(currentUser);
  A.CallTo(() => _userRepository.GetMembersAsync(currentUser.Email)).Returns(expectedUsers);
  
  // Assert阶段
  var okResult = result.Result as OkObjectResult;
  okResult.Value.Should().BeEquivalentTo(expectedUsers);
  

• 如果需要让方法返回null：可以显式配置：

  A.CallTo(() => _userRepository.GetMembersAsync(A<string>._)).Returns((IEnumerable<MemberDto>)null);
  

内容的提问来源于stack exchange，提问作者Natan