Pytanie o semantykę speców (czyli czy więcej it'ów czy też mniej)

Zielona szkoła
1

Będzie kolejny spam.
Zaraz wychodzę z roboty ale zastanawiam mnie jedna rzecz.

Mianowicie jak mam spec kontrolera:

[code=ruby] describe “with invalid params” do
it “assigns a newly created but unsaved profile as @profile” do
# Trigger the behavior that occurs when invalid params are submitted
Profile.any_instance.stub(:save).and_return(false)
post :create, :profile => {}
assigns(:profile).should be_a_new(Profile)
end

  it "re-renders the 'new' template" do
    # Trigger the behavior that occurs when invalid params are submitted                                                                                                                                      
    Profile.any_instance.stub(:save).and_return(false)
    post :create, :profile => {}
    response.should render_template("new")
  end
end

end[/code]
To czy nie lepiej by to było napisać

describe "with invalid params" do it "assigns a newly created but unsaved profile as @profile and re-renders the 'new' template" do # Trigger the behavior that occurs when invalid params are submitted Profile.any_instance.stub(:save).and_return(false) post :create, :profile => {} assigns(:profile).should be_a_new(Profile) response.should render_template("new") end end end
Chodzi mi o wyrzucenie jednej asercji do drugiego IT’a.
A wy jak to robicie?

2

Teoretycznie bardziej koszerne jest pierwsze podejście.

Praktycznie… Już musisz sam zdecydować. :slight_smile:

3

Imo najlepiej, jeśli jest jedno “should” na jedno “it”. Wtedy łatwo dojść, gdzie wystąpił błąd.

4

[quote=filiptepper]Teoretycznie bardziej koszerne jest pierwsze podejście.

Praktycznie… Już musisz sam zdecydować. :-)[/quote]
Dobrze ujęte.

5

Z blogów wynika że powinnyśmy stawiać na… jak to ujął filiptepper ‘koszerność’


http://eggsonbread.com/2010/03/28/my-rspec-best-practices-and-tips/
[credit Bandzarewicz]

6

A jak wiele zła czyni się robiąc to w ten sposób?

describe "with invalid params" do before do # Trigger the behavior that occurs when invalid params are submitted Profile.any_instance.stub(:save).and_return(false) post :create, :profile => {} end it "assigns a newly created but unsaved profile as @profile" do assigns(:profile).should be_a_new(Profile) end it "re-renders the 'new' template" do response.should render_template("new") end end