Skip to content

Backport: Fix empty template rendering for Rails 4.2.5.1 #1542

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

Closed
wants to merge 1 commit into from

Conversation

zimuliu
Copy link

@zimuliu zimuliu commented Jan 30, 2016

Backport from #1535

Refactor the template rendering overrides in controller specs to be more
compatible with the Action View public api. Previously EmptyTemplatePathSetDecorator
overrode the find_all method to wrap a path set. However ActionView::PathSet
is not public api so instead create a subclass of ActionView::FileSystemResolver
to return an empty template and treat view_paths as an array of string-like objects.

Closes rspec#1532.

Refactor the template rendering overrides in controller specs to be more
compatible with the Action View public api. Previously `EmptyTemplatePathSetDecorator`
overrode the `find_all` method to wrap a path set. However `ActionView::PathSet`
is not public api so instead create a subclass of `ActionView::FileSystemResolver`
to return an empty template and treat `view_paths` as an array of string-like objects.
@JonRowe
Copy link
Member

JonRowe commented Jan 31, 2016

We don't maintain older versions of RSpec. This has been released already please run bundle update rspec-rails to gain this fix.

@JonRowe JonRowe closed this Jan 31, 2016
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.

3 participants