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

Removing xhr and xml_http_request was non-passive. #44

Closed
mjhenkes opened this issue Dec 5, 2018 · 5 comments
Closed

Removing xhr and xml_http_request was non-passive. #44

mjhenkes opened this issue Dec 5, 2018 · 5 comments

Comments

@mjhenkes
Copy link

mjhenkes commented Dec 5, 2018

Removing xhr and xml_http_request was non-passive and should have been a major version bump.

Tests that were passing using rails-controller-testing 1.0.2 no longer pass with the 1.0.3 release. I'm all for removing the depreciation warning, but it should have been done with a major version bump to give consumers time to upgrade.

Specifically the tests fail with the error:

Failure/Error: http_verbs.push('xhr', 'xml_http_request') if ActionPack.version < Gem::Version.new('5.1')

NameError:
  uninitialized constant Rails::Controller::Testing::Integration::ActionPack

PR removing xhr and xml_http_request:
#32

@mjhenkes
Copy link
Author

mjhenkes commented Dec 5, 2018

On second glance my issue may just be an error in rails-contoller-testing, although removing xhr and xml_http_request is still non-passive :)

@rafaelfranca
Copy link
Member

They were not removed. For all rails versions that support them they are still there. For Rails version that doesn't support them they were removed, but it was not working anyway since that Rails version don't have that methods.

@rafaelfranca
Copy link
Member

1.0.4 released.

@mjhenkes
Copy link
Author

mjhenkes commented Dec 5, 2018

@rafaelfranca thank you for the quick response :)

@rafaelfranca
Copy link
Member

Thank you for reporting, in the end it was a oversight in my side.

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

No branches or pull requests

2 participants