Skip to content
This repository

New should_receive syntax #214

Closed
laserlemon opened this Issue February 23, 2013 · 1 comment

2 participants

Steve Richert Myron Marston
Steve Richert

Now that the preferred RSpec syntax is:

expect(2 + 2).to eql(4)

rather than:

(2 + 2).should eq(4)

, should this also change:

user.should_receive(:update_attributes).with(name: "Steve")

to:

expect(user).to receive(:update_attributes).with(name: "Steve")

?

I like the new expect syntax but feel funny every time I use should_receive because it doesn't feel consistent. I just wanted to open up the conversation. Thank you!

Myron Marston

There's an ongoing discussion about this in rspec/rspec-mocks#153. The short answer is that we're already planning on doing this but haven't gotten around to it yet.

Myron Marston myronmarston closed this February 23, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.