-
Notifications
You must be signed in to change notification settings - Fork 48
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
undefined local variable or method 'session' #6
Comments
FYI @request.session seems to work, so maybe is just a shortcut missing or something |
And it also seems I can't access the named routes. Here's a spec to demonstrate the issues I'm having:
|
I just encountered this problem after upgrading rspec-rails from 0.0.5 to 0.1.1 . Is there a workaround other than prefixing named routes with |
Bump |
Still happens today |
@PikachuEXE Please paste the complete stacktrace. |
@apotonick
|
Still an issue with 0.3? |
Let me check next week |
Can't check since I am unable to upgrade to cells 4 yet |
What keeps you from updating? I know the pain, my friend, we just upgraded an app to Rails 3.2 haha. |
The app has too many "old" cells
|
I have this bug too. I try to sign_in a user if it is logged in with CASino and it tries to edit its account,. class RegistrationsController < Devise::RegistrationsController
include CASino::SessionsHelper
before_action :ensure_signed_in, only: [:edit]
skip_before_filter :authenticate_scope!, only: [:edit]
def edit
if current_user
Class.new.extend(Devise::Controllers::Helpers).sign_in(:devise_users, DeviseUser.find(current_user.id))
end
super
end
end Full trace:
|
Your breaking spec file would be helpful to see. I have to warn you, though, I don't use Rspec and Devise so it might take a while. |
This doesn't seem to be related to cells? |
session variable is not available on Cell specs by design, or am I doing something wrong?
The text was updated successfully, but these errors were encountered: