Make :respect_data_method default to true with capybara/rails #793
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm writing an upgrade guide, and I find myself telling people "Capybara 2.0 will break your test suite. To stop it from breaking your test suite, enable
:respect_data_method
." This is something that we'll basically have to tell all users, or they'll get tripped up.Instead, I'd suggest that we enable the right behavior by default, whenever
capybara/rails
is required.Ignoring data-method in the RackTest driver is very reasonable when we're not testing Rails apps, for what it's worth. But in Rails apps, people will generally want to honor data-method.