我开始使用AutoFixture http://autofixture.codeplex.com/因为我的单元测试因大量数据设置而变得臃肿。与编写单元测试相比，我花在设置数据上的时间更多。这是我的初始单元测试的示例(示例取自 DDD 蓝皮书的 cargo 应用程序示例)

[Test]
public void should_create_instance_with_correct_ctor_parameters()
{
    var carrierMovements = new List<CarrierMovement>();

    var deparureUnLocode1 = new UnLocode("AB44D");
    var departureLocation1 = new Location(deparureUnLocode1, "HAMBOURG");
    var arrivalUnLocode1 = new UnLocode("XX44D");
    var arrivalLocation1 = new Location(arrivalUnLocode1, "TUNIS");
    var departureDate1 = new DateTime(2010, 3, 15);
    var arrivalDate1 = new DateTime(2010, 5, 12);

    var carrierMovement1 = new CarrierMovement(departureLocation1, arrivalLocation1, departureDate1, arrivalDate1);

    var deparureUnLocode2 = new UnLocode("CXRET");
    var departureLocation2 = new Location(deparureUnLocode2, "GDANSK");
    var arrivalUnLocode2 = new UnLocode("ZEZD4");
    var arrivalLocation2 = new Location(arrivalUnLocode2, "LE HAVRE");
    var departureDate2 = new DateTime(2010, 3, 18);
    var arrivalDate2 = new DateTime(2010, 3, 31);

    var carrierMovement2 = new CarrierMovement(departureLocation2, arrivalLocation2, departureDate2, arrivalDate2);

    carrierMovements.Add(carrierMovement1);
    carrierMovements.Add(carrierMovement2);

    new Schedule(carrierMovements).ShouldNotBeNull();
}

这是我尝试使用 AutoFixture 重构它的方法

[Test]
public void should_create_instance_with_correct_ctor_parameters_AutoFixture()
{
    var fixture = new Fixture();

    fixture.Register(() => new UnLocode(UnLocodeString()));

    var departureLoc = fixture.CreateAnonymous<Location>();
    var arrivalLoc = fixture.CreateAnonymous<Location>();
    var departureDateTime = fixture.CreateAnonymous<DateTime>();
    var arrivalDateTime = fixture.CreateAnonymous<DateTime>();

    fixture.Register<Location, Location, DateTime, DateTime, CarrierMovement>(
        (departure, arrival, departureTime, arrivalTime) => new CarrierMovement(departureLoc, arrivalLoc, departureDateTime, arrivalDateTime));

    var carrierMovements = fixture.CreateMany<CarrierMovement>(50).ToList();

    fixture.Register<List<CarrierMovement>, Schedule>((carrierM) => new Schedule(carrierMovements));

    var schedule = fixture.CreateAnonymous<Schedule>();

    schedule.ShouldNotBeNull();
}

private static string UnLocodeString()
{
    var stringBuilder = new StringBuilder();

    for (int i = 0; i < 5; i++)
        stringBuilder.Append(GetRandomUpperCaseCharacter(i));

    return stringBuilder.ToString();
}

private static char GetRandomUpperCaseCharacter(int seed)
{
    return ((char)((short)'A' + new Random(seed).Next(26)));
}

我想知道是否有更好的方法来重构它。想做得更短更容易。

最佳答案

您的初步尝试看起来不错，但至少有几件事您可以稍微简化。

首先，你应该能够减少这个:

fixture.Register<Location, Location, DateTime, DateTime, CarrierMovement>(
    (departure, arrival, departureTime, arrivalTime) =>
        new CarrierMovement(departureLoc, arrivalLoc, departureDateTime, arrivalDateTime));

为此:

fixture.Register<Location, Location, DateTime, DateTime, CarrierMovement>(
    () => new CarrierMovement(departureLoc, arrivalLoc, departureDateTime, arrivalDateTime));

因为您没有使用那些其他变量。但是，这实质上将 CarrierMovement 的任何创建都锁定为使用相同的四个值。虽然每个创建的 CarrierMovement 都是一个单独的实例，但它们都将共享相同的四个值，我想知道这是否是您的意思？

同上，而不是

fixture.Register<List<CarrierMovement>, Schedule>((carrierM) =>
    new Schedule(carrierMovements));

你可以写

fixture.Register(() => new Schedule(carrierMovements));

因为您不使用 carrierM多变的。由于 Func 的返回类型，类型推断将确定您正在注册一个 Schedule。

但是，假设 Schedule 构造函数如下所示:

public Schedule(IEnumerable<CarrierMovement> carrierMovements)

您可以直接注册 carrierMovements像这样:

fixture.Register<IEnumerable<CarrierMovement>>(carrierMovements);

这会导致 AutoFixture 自动正确解析 Schedule。这种方法更易于维护，因为它允许您在将来向 Schedule 构造函数添加参数而不会破坏测试(只要 AutoFixture 可以解析参数类型)。

但是，在这种情况下我们可以做得更好，因为我们并没有真正使用 carrierMovements除了注册之外的任何变量。我们真正需要做的只是告诉 AutoFixture 如何创建 IEnumerable<CarrierMovement> 的实例。 .如果您不关心数字 50(您不应该关心)，我们甚至可以像这样使用 Method Group 语法:

fixture.Register(fixture.CreateMany<CarrierMovement>);

注意缺少方法调用括号:我们正在注册一个 Func，并且由于 CreateMany<T>方法返回 IEnumerable<T>类型推断负责其余的工作。

不过，这些都是细节。在更高的层次上，您可能想要考虑根本不注册 CarrierMovement。假设这个构造函数:

public CarrierMovement(Location departureLocation,
    Location arrivalLocation,
    DateTime departureTime,
    DateTime arrivalTime)

autofixture 应该能够自己解决。

它将为每个 departmentLocation 和 arrivalLocation 创建一个新的 Location 实例，但这与您在原始测试中手动执行的操作没有什么不同。

说到时间，AutoFixture 默认使用 DateTime.Now ，这至少保证了到达时间永远不会早于出发时间。然而，它们很可能是相同的，但如果这是一个问题，你总是可以注册一个自动递增函数。

考虑到这些因素，这里有一个替代方案:

public void should_create_instance_with_correct_ctor_parameters_AutoFixture()
{
    var fixture = new Fixture();

    fixture.Register(() => new UnLocode(UnLocodeString()));

    fixture.Register(fixture.CreateMany<CarrierMovement>);

    var schedule = fixture.CreateAnonymous<Schedule>();

    schedule.ShouldNotBeNull();
}

解决 IList<CarrierMovement> 的问题你需要注册它。这是一种方法:

fixture.Register<IList<CarrierMovement>>(() =>
    fixture.CreateMany<CarrierMovement>().ToList());

但是，既然你问了，我的意思是 Schedule 构造函数看起来像这样:

public Schedule(IList<CarrierMovement> carrierMovements)

我真的认为您应该重新考虑更改该 API 以获取 IEnumerable<Carriemovement> .从 API 设计的角度来看，通过任何成员(包括构造函数)提供集合意味着允许该成员修改集合(例如，通过调用它的 Add、Remove 和 Clear 方法)。这不是您期望的构造函数的行为，所以不要允许它。

AutoFixture 将自动为所有 Location 生成新值上面示例中的对象，但由于 CPU 的速度，DateTime 的后续实例可能是相同的。

如果您想要增加 DateTimes，您可以编写一个小类，在每次调用时增加返回的 DateTime。我会将该类的实现留给感兴趣的读者，但您可以像这样注册它:

var dtg = new DateTimeGenerator();
fixture.Register(dtg.Next);

假设这个 API(再次注意上面的方法组语法):

public class DateTimeGenerator
{
    public DateTime Next();
}

关于c# - AutoFixture重构，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/2622334/