2

在我的网上商店应用程序中，我有一个“购物车”类，可以添加、删除和计算商品的总价值。数据模型如下：1 个项目包含 1 个产品和 1 个运输。产品有价格字段，运输有成本字段。这是购物车类代码： 

public class CartLine
{
    public Item Item { get; set; }
    public int Quantity { get; set; }
}

public class Cart
{
    private List<CartLine> lineCollection = new List<CartLine>();

    // methods: 
    // Add(Item item, int quantity) 
    // Remove(Item item)

    public decimal ComputeTotalProductValue()
    {
        return lineCollection.Sum(l => l.Item.Product.Price*l.Quantity);
    }

    // methods with the same principle: 
    // ComputeTotalShippingValue()
    // ComputeOveralValue()
}

这是我的单元测试（当然不起作用）： 

[TestMethod]
    public void Can_Compute_Total_Values()
    {
        // Arrange 
        var itemMock = new Mock<IItemsRepository>();
        itemMock.Setup(i => i.GetItems()).Returns(new[]
            {
                new Item { ItemId = 1, ProductId = 1, ShippingId = 1 },
                new Item { ItemId = 2, ProductId = 2, ShippingId = 2 }
            });

        var productMock = new Mock<IProductRepository>();
        productMock.Setup(p => p.GetProducts()).Returns(new[]
            {
                new Product { ProductId = 1, Price = 10 },
                new Product { ProductId = 2, Price = 20 }
            });

        var shippingMock = new Mock<IShippingRepository>();
        shippingMock.Setup(s => s.GetShippings()).Returns(new[]
            {
                new Shipping { ShippingId = 1, Cost = 2 },
                new Shipping { ShippingId = 2, Cost = 5 }
            });

        var item1 = itemMock.Object.GetItems().ToArray()[0];
        var item2 = itemMock.Object.GetItems().ToArray()[1];    

        var target = new Cart();

        //Act
        target.Add(item1, 2);
        target.Add(item2, 4);

        decimal totalProduct = target.ComputeTotalProductValue();
        decimal totalShipping = target.ComputeTotalShippingValue();
        decimal overalSum = target.ComputeOveralValue();

        // Assert
        Assert.AreEqual(totalProduct, 100);
        Assert.AreEqual(totalShipping, 24);
        Assert.AreEqual(overalSum, 124);
    }
}

该问题可能与将项目绑定到产品和运输有关。我怎样才能做到这一点？

提前致谢！

4

2 回答 2

2

这里的根本问题是有关如何持久化模型的详细信息已泄漏到模型类中，因此使用模型的类（例如 Cart 使用 Item）有一定的义务了解您的持久层，它们不应该需要去做。这是太多的耦合，这就是让你的测试变得困难的原因。

考虑重新定义Item并添加ItemDescription这样的：

public class Item
{
   public Product Product {get;set;}
   public Shipping Shipping {get;set;}
}

public class ItemDescription
{
   public int ProductId {get;set;}
   public int ShippingId {get;set;}
}

这使您可以很容易地测试 Cart 和其他使用您的Item类的类。如果您需要，我可以添加一个示例，但我猜这对您来说很明显。如果由于某种原因您无法更改，请将Item这种方法颠倒过来并创建一个FullyConsitituedItem类供您Cart使用。

这里的代价是你必须创建另一个Item从ItemDescriptions 构造 s 的类，但实际上你无论如何都会这样做，它只会在整个地方零碎完成。添加ItemDescription使其明确，这更好，更具可读性。当然更可测试。

于 2012-12-29T15:08:29.993 回答
0

您需要在测试方法中直接分配Product和Shipping。item1item2

见代码示例：

[TestMethod]
public void Can_Compute_Total_Values()
{
    // Arrange 
    var item1 = new Item {
        Product = new Product { Price = 10 },
        Shipping = new Shipping { Cost = 2 }
    };

    var item2 = new Item {
        Product = new Product { Price = 20 },
        Shipping = new Shipping { Cost = 5 }
    };

    var target = new Cart();

    //Act
    target.Add(item1, 2);
    target.Add(item2, 4);

    decimal totalProduct = target.ComputeTotalProductValue();

    // Assert
    Assert.AreEqual(totalProduct, 100);

    Console.ReadLine();
}

正如评论中提到的那样，没有必要模拟存储库，因为它们没有在正在测试的方法中使用。此方法更希望这样做Product并且Shipping已经初始化。
所以你需要手动准备正确的初始数据进行测试:)。

于 2013-01-01T15:55:54.717 回答