assign value to multiple variables.. when calling a method

Question

I have something like this in my RSpec file (it works fine for me):

    before do
      @attr = attributes_for(:album)      
      post :create, album: @attr
    end
    it { expect(Album.exists?(@attr)).to be_true }
    it { expect(response).to redirect_to(assigns[:album]) }

Is it possible to make it without attributes_for(:album) in a separate line and @attr variable ? Or maybe even other better way..

Get rid of @attr temp. variable. I believe it can be done more.. Ruby :) — pawel7318
– pawel7318, Commented Mar 31, 2014 at 19:45

Kirti Thorat · Accepted Answer · 2014-03-31 19:46:42Z

4

No need of before block or @attr variable, you could change the example to something like this:

it "creates a album in the database" do
   expect { post :create, album: attributes_for(:album) }.to change(Album, :count).by(1)
end

edited Mar 31, 2014 at 19:46

answered Mar 31, 2014 at 19:41

Kirti Thorat♦

53.1k9 gold badges105 silver badges108 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

engineersmnky Over a year ago

yes but then how does he retrieve that Album in the test to make sure it was created correctly?

pawel7318 Over a year ago

I saw that way somewhere too but.. is it good enough to assume that it was saved correctly ? and I have more it tests there so I would like to keep it in before block.

Kirti Thorat Over a year ago

@pawel7318 Yes, it is good enough to test the record was saved correctly. And it is normal way of testing controllers create method.

Kirti Thorat Over a year ago

@pawel7318 As for the more tests goes, lets discuss this on chat as I need to understand few things chat.stackoverflow.com/rooms/48530/ror

Collectives™ on Stack Overflow

assign value to multiple variables.. when calling a method

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related