Skip to content
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

Handle FrozenError if it is available to 5-1-stable #31525

Merged
merged 1 commit into from
Dec 20, 2017

Conversation

yahonda
Copy link
Member

@yahonda yahonda commented Dec 20, 2017

This pull request backports #31520 to 5-1-stable where the actual failures reported like https://travis-ci.org/rails/rails/jobs/318867883

Summary

This pull request handles FrozenError introduced by Ruby 2.5.
Refer https://bugs.ruby-lang.org/projects/ruby-trunk/repository/revisions/61131

Since FrozenError is a subclass of RuntimeError minitest used by master
branch can handle it, though it would be better to handle FrozenError
explicitly if possible.

FrozenError does not exist in Ruby 2.4 or lower, frozen_error_class
handles which exception is expected to be raised.

This pull request handles `FrozenError` introduced by Ruby 2.5.
Refer https://bugs.ruby-lang.org/projects/ruby-trunk/repository/revisions/61131

Since `FrozenError` is a subclass of `RuntimeError` minitest used by master
branch can handle it, though it would be better to handle `FrozenError`
explicitly if possible.

`FrozenError` does not exist in Ruby 2.4 or lower, `frozen_error_class`
handles which exception is expected to be raised.
@rails-bot
Copy link

r? @kamipo

(@rails-bot has picked a reviewer for you, use r? to override)

@rails-bot
Copy link

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against 5-1-stable. Please double check that you specified the right target!

@kamipo kamipo merged commit cb7347a into rails:5-1-stable Dec 20, 2017
rafaelfranca pushed a commit that referenced this pull request Dec 22, 2017
Handle `FrozenError` if it is available to 5-1-stable
@yahonda yahonda deleted the 5-1-stable_backport_31520 branch December 28, 2017 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants