Skip to content

Commit

Permalink
Remove the leading \n added by textarea on assert_select
Browse files Browse the repository at this point in the history
  • Loading branch information
spastorino committed Mar 30, 2012
1 parent 1ce7545 commit 1141f71
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
2 changes: 2 additions & 0 deletions actionpack/CHANGELOG.md
@@ -1,5 +1,7 @@
## Rails 4.0.0 (unreleased) ##

* Remove the leading \n added by textarea on assert_select. *Santiago Pastorino*

* Changed default value for `config.action_view.embed_authenticity_token_in_remote_forms`
to `false`. This change breaks remote forms that need to work also without javascript,
so if you need such behavior, you can either set it to `true` or explicitly pass
Expand Down
Expand Up @@ -269,6 +269,7 @@ def assert_select(*args, &block)
end
end
text.strip! unless NO_STRIP.include?(match.name)
text.sub!(/\A\n/, '') if match.name == "textarea"
unless match_with.is_a?(Regexp) ? (text =~ match_with) : (text == match_with.to_s)
content_mismatch ||= sprintf("<%s> expected but was\n<%s>.", match_with, text)
true
Expand Down
7 changes: 7 additions & 0 deletions actionpack/test/controller/assert_select_test.rb
Expand Up @@ -131,6 +131,13 @@ def test_equality_of_html
assert_raise(Assertion) { assert_select "pre", :html=>text }
end

def test_strip_textarea
render_html %Q{<textarea>\n\nfoo\n</textarea>}
assert_select "textarea", "\nfoo\n"
render_html %Q{<textarea>\nfoo</textarea>}
assert_select "textarea", "foo"
end

def test_counts
render_html %Q{<div id="1">foo</div><div id="2">foo</div>}
assert_nothing_raised { assert_select "div", 2 }
Expand Down

0 comments on commit 1141f71

Please sign in to comment.