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
Remove ActiveSupport::TestCase#pending, use skip
instead
#4575
Remove ActiveSupport::TestCase#pending, use skip
instead
#4575
Conversation
Remove ActiveSupport::TestCase#pending, use `skip` instead
This needs to be deprecated on 3-2-stable. Otherwise, tests just break when upgrading an app and you have to Google to find out why and how to fix. |
@jeremy ok. @rafaelfranca are you already working on this, confirm? |
Confirm. I need to discuss it better, but it is almost done. |
Ok, if you need anything, just ping me later. Thanks! At. On Friday, May 18, 2012 at 7:14 PM, Rafael Mendonça França wrote:
|
I was reviewing my old list of issues and found this one, and I did the changes in this branch, let me know what you think. |
👍 |
Properly deprecate #pending from AS::TestCase. This has been previously removed from master, and is now back with a deprecation instead, to avoid people having tests breaking when upgrading an app. Please check #4575 for more background.
Done, thanks! |
The
pending
method nowadays works basically delegating toskip
, so I believe there is no real need to have it. We should just start usingskip
directly.