Deprecate assign_to matcher. #183

Closed
drapergeek opened this Issue Nov 2, 2012 · 5 comments

Projects

None yet

5 participants

@drapergeek
Member

No description provided.

@brycesenz

Is this matcher going to be replaced with anything? It's actually incredibly helpful, especially in making sure that variable assignments do/don't take place in certain contexts.

@mxie mxie closed this Mar 25, 2013
@mxie
Member
mxie commented Mar 25, 2013

Closing this because we have included this matcher to be deprecated via #253.

@jarl-dk
jarl-dk commented May 10, 2013

What is the rationale behind this? And what is the recommended replacement?

@mxie
Member
mxie commented May 10, 2013

@jarl-dk We have a blog post up talk about the changes in 2.0: http://robots.thoughtbot.com/post/47031676783/shoulda-matchers-2-0. Hope that helps!

@ejalbos
ejalbos commented Jan 23, 2014

The referenced blog post only says why you did it and that you think a different kind of test is more appropriate.
However, lots of us like doing this very localized kinda test and find it very useful, so not offering another path (beyond 'do it differently') for replacing this ripped out functionality isn't very helpful at all.

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