diff options
| author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-09-08 18:03:17 -0300 |
|---|---|---|
| committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-09-08 18:04:45 -0300 |
| commit | fb110381481fe79fa2921d08a60d343d3ea2b363 (patch) | |
| tree | 7fafaaa38bf6fe0ccf68736cde10a878c683c98d /actionview/lib/action_view/test_case.rb | |
| parent | 8683396dd8c31c70ba9332586ee05889305fdde6 (diff) | |
| download | rails-fb110381481fe79fa2921d08a60d343d3ea2b363.tar.gz rails-fb110381481fe79fa2921d08a60d343d3ea2b363.tar.bz2 rails-fb110381481fe79fa2921d08a60d343d3ea2b363.zip | |
Do not memoize document_root_element in view tests
Memoizing will not make possible to assert the output of the view if it
is changed after the first assert_select call
Related with plataformatec/simple_form#1130 and rails/rails-dom-testing#15
Diffstat (limited to 'actionview/lib/action_view/test_case.rb')
| -rw-r--r-- | actionview/lib/action_view/test_case.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/actionview/lib/action_view/test_case.rb b/actionview/lib/action_view/test_case.rb index 7edfc436a6..af34d2ce5a 100644 --- a/actionview/lib/action_view/test_case.rb +++ b/actionview/lib/action_view/test_case.rb @@ -158,8 +158,7 @@ module ActionView # Need to experiment if this priority is the best one: rendered => output_buffer def document_root_element - @html_document ||= Nokogiri::HTML::Document.parse(@rendered.blank? ? @output_buffer : @rendered) - @html_document.root + Nokogiri::HTML::Document.parse(@rendered.blank? ? @output_buffer : @rendered).root end def say_no_to_protect_against_forgery! |