Getting "undefined local variable or method `executor_around_each_request'" after edge rails upgrade
jasnow opened this issue · 12 comments
This commit to "rails": rails/rails@5046d1c appears to have affected the rails-controller-testing gem.
Steps to reproduce:
- Clone my repo, https://github.com/jasnow/show-me-the-foodedge .
- Run "rake" to see a clean run before change.
- Run "bundle" to get a new version of edge rails.
- Run "rake" again and see the following errors in this gist:
Thanks.
@ghiculescu, @byroot: Ya'll might be able to help with this.
Could you add a backtrace too? I'll have a look tomorrow and that would help a lot.
Thanks. I’ll have a look at this tomorrow.
I'll have a look tomorrow and that would help a lot.
Race is on 🚀
Could you add a backtrace too? I'll have a look tomorrow and that would help a lot.
@byroot: Is not the backtrace inside the above gist? If not, how do I create it. Thanks
My bad, I only skimmed the issue as I was to go to bed and missed it.
Ok, so you use RSpec which probably explains it: https://github.com/jasnow/show-me-the-foodedge/blob/2cb663714dd0bd0c4ab9edc78aadd3f15c70190b/spec/controllers/locations_controller_spec.rb
Really unsure how to fix this, I'm really unfamiliar with how RSpec does these things.
@ghiculescu I suppose we can make it a global flag instead of a class_attribute
, I kinda liked it that way but...
I don’t know rspec either. Maybe accessing the class method instead of the instance method would work here 🤔
You can test by changing the Gemfile to
gem 'rails', git: 'https://github.com/ghiculescu/rails', ref: "39cde506c66b565cf8b4d889dcdb61204aabd6c4"
Added
gem 'rails', git: 'https://github.com/ghiculescu/rails'
and it ran cleanly. Will close this issue when fix makes it into edge rails. THANKS!