问对于改进BDD/TDD的这种风格，有什么建议吗？
EN

Stack Overflow用户

提问于 2010-05-21 16:48:19

回答 2查看 241关注 0票数 3

我正在对我们的单元测试规范进行设置，如下所示

当行为X在场景Y中发生时的SUT规范，假设这件事和我做X时的另一件事。那它应该能做到。而且它也应该做到..。

我将GivenThat的每个步骤都封装在操作中……有没有反馈说，把动作分开是好是坏，还是让GivenThat清晰的更好方法？

        /// <summary>
        /// Given a product is setup for injection
        ///     And Product Image Factory Is Stubbed();
        ///     And Product Size Is Stubbed();
        ///     And Drawing Scale Is Stubbed();
        ///     And Product Type Is Stubbed();
        /// </summary>
        protected override void GivenThat()
        {
            base.GivenThat();

            Action givenThatAProductIsSetupforInjection = () =>
              {
                  var randomGenerator = new RandomGenerator();

                  this.Position = randomGenerator.Generate<Point>();

                  this.Product = new Diffuser
                                     {
                                         Size =
                                             new RectangularProductSize(
                                             2.Inches()),
                                         Position = this.Position,
                                         ProductType =
                                             Dep<IProductType>()
                                     };
              };

            Action andProductImageFactoryIsStubbed = () => Dep<IProductBitmapImageFactory>().Stub(f => f.GetInstance(Dep<IProductType>())).Return(ExpectedBitmapImage);

            Action andProductSizeIsStubbed = () =>
                 {
                     Stub<IDisplacementProduct, IProductSize>(p => p.Size);

                     var productBounds = new ProductBounds(Width.Feet(), Height.Feet());

                     Dep<IProductSize>().Stub(s => s.Bounds).Return(productBounds);
                 };

            Action andDrawingScaleIsStubbed = () => Dep<IDrawingScale>().Stub(s => s.PixelsPerFoot).Return(PixelsPerFoot);

            Action andProductTypeIsStubbed = () => Stub<IDisplacementProduct, IProductType>(p => p.ProductType);

            givenThatAProductIsSetupforInjection();
            andProductImageFactoryIsStubbed();
            andProductSizeIsStubbed();
            andDrawingScaleIsStubbed();
            andProductTypeIsStubbed();
        }

unit-testing

tdd

bdd

回答 2

Stack Overflow用户

回答已采纳

发布于 2010-05-21 17:11:11

将它们放在单独的方法中，这样您就可以在其他给定函数中组合它们。另外，使用下划线代替空格(而不是驼峰式大小写)。另外，创建一个接受Action委托参数的方法Given_that。

protected void Given_that(params Action[] preconditions)
{
    foreach (var action in preconditions)
    {
        action();
    }
}

...

protected void a_product_is_set_up_for_injection()
{
    ...
}

protected void product_image_factory_is_stubbed()
{
    ...
}

etc...

...

Given_that(a_product_is_set_up_for_injection,
           product_image_factory_is_stubbed,
           product_size_is_stubbed,
           drawing_scale_is_stubbed,
           product_type_is_stubbed);

也就是说，我认为您的前提条件的命名不是BDD。它们本质上是非常技术性的，并不表示业务需求。如果您要告诉非程序员您正在测试什么，您可能不会说“该产品是用于注入的存根”。你更可能会说

Given a displacement product
    that is a two inch rectangular diffuser
    that has a random position
    that has a bitmap
    that has a size bounded by feet
    that has the expected pixels per foot

现在，您可以编写几乎不重复的“给定”方法：

protected [the type of your test class] Given(params Action given)
{
    given();
    return this;
}

protected void That(params Action[] preconditions)
{
    foreach (var precondition in preconditions)
    {
        precondition();
    }
}

Given(a_displacement_product)
    .That(is_a_two_inch_rectangular_diffuser,
          has_a_random_position,
          has_a_bitmap,
          has_a_size_bounded_by_feet,
          has_the_expected_pixels_per_foot);

票数 2

Stack Overflow用户

发布于 2010-05-24 05:14:10

在不同的方法中组合你的Givens、When和Thens是一个好主意，这就是SpecFlow (http://www.specflow.org)的方式。因此，如果想要一些自动化来创建那些无聊的重复管道，我真的会推荐使用像SpecFlow这样的工具。作为奖励，你会得到一个很好的报告工具:)

让你的代码更流畅的另一个选择是创建一个小的BDD基类。看看Jonas Follesoe在GitHub上写的BDD DSL：http://gist.github.com/406014；

public abstract class BDD<T> where T : BDD<T>
{
    protected T Given { get { return (T)this; } }
    protected T And { get { return (T)this; } }
    protected T When { get { return (T)this; } }
    protected T Then { get { return (T)this; } }
}