rails/rails-controller-testing

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:

  1. Clone my repo, https://github.com/jasnow/show-me-the-foodedge .
  2. Run "rake" to see a clean run before change.
  3. Run "bundle" to get a new version of edge rails.
  4. 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!