0

我正在阅读这篇博文:

http://clarkware.com/blog/2007/09/08/how-would-you-test-this

最后看到了这段代码:

describe MenuItemsController, 'Creating a new menu item' do

  before do
    @attributes = {'name' => "Enchilada", 'price' => 4.99}
    @menu_item = mock_model(MenuItem)
    MenuItem.should_receive(:new).with(@attributes).once.
      and_return(@menu_item)
  end

  it 'should redirect to index with a notice on successful save' do
    @menu_item.should_receive(:save).with().once.and_return(true)

    post :create, :menu_item => @attributes

    assigns[:menu_item].should be(@menu_item)
    flash[:notice].should_not be(nil)    
    response.should redirect_to(menu_items_url)
  end

  it 'should re-render new template on failed save' do
    @menu_item.should_receive(:save).with().once.and_return(false)

    post :create, :menu_item => @attributes

    assigns[:menu_item].should be(@menu_item)
    flash[:notice].should be(nil)    
    response.should be_success
    response.should render_template('new')
  end

end

我的印象是最好将每个测试(应该、断言、期望)放在它自己的“它”块中。这段代码放了几个。

是的,它使代码更易于阅读,但是如果例如 line: flash[:notice].should_not be(nil)failed 那么您的结果不会直接指向该测试,对吗?

这里有什么建议?每个测试单独或捆绑一些以帮助可读性?

尼尔

4

1 回答 1

0

一般来说,最好的方法是每个测试描述只放置一个断言,但它仍在讨论中,如下所示:http: //betterspecs.org/#single

在某些情况下,我同意expect同一测试中的许多 s 可能有用,但是,例如,如果第一个expect失败,那么 rspec 将停止运行该测试并只说这个失败。

于 2013-09-16T13:41:36.433 回答