New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create only tha data you need #10

Open
andreareginato opened this Issue Oct 3, 2012 · 2 comments

Comments

Projects
None yet
3 participants
@andreareginato
Collaborator

andreareginato commented Oct 3, 2012

Write your thoughts about the "create only the data you need" best practice.

@eladg

This comment has been minimized.

Show comment
Hide comment
@eladg

eladg Jan 9, 2016

@andreareginato It seems that have / have_at_least / have_at_most were deprecated.

From: http://rspec.info/blog/2013/11/rspec-2-99-and-3-0-betas-have-been-released/

rspec-expectations 2.99.0.beta1 - Full Changelog

Deprecations: Deprecate have, have_at_least and have_at_most. 
You can continue using those matchers through https://github.com/rspec/rspec-collection_matchers, 
or you can rewrite your expectations with something like expect(your_object.size).to eq(num). 
(Hugo Baraúna)

would suggest updating those examples...

eladg commented Jan 9, 2016

@andreareginato It seems that have / have_at_least / have_at_most were deprecated.

From: http://rspec.info/blog/2013/11/rspec-2-99-and-3-0-betas-have-been-released/

rspec-expectations 2.99.0.beta1 - Full Changelog

Deprecations: Deprecate have, have_at_least and have_at_most. 
You can continue using those matchers through https://github.com/rspec/rspec-collection_matchers, 
or you can rewrite your expectations with something like expect(your_object.size).to eq(num). 
(Hugo Baraúna)

would suggest updating those examples...

@onebree

This comment has been minimized.

Show comment
Hide comment
@onebree

onebree Jan 11, 2016

Contributor

@eladg Feel free to submit a pull request with changes. It would be much appreciated.

Contributor

onebree commented Jan 11, 2016

@eladg Feel free to submit a pull request with changes. It would be much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment