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
fix admin fixture, use admin user, fix redirect #261
fix admin fixture, use admin user, fix redirect #261
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for tackling this one!
We've been trying to move away from fixtures and toward using FactoryBotyfactories instead. This wasn't documented anywhere, so there is no way you could have known.
In any case, if you can update the calls to users(:admin)
to create(:admin_user)
, we should be able to merge this in.
Also, the test failure on this branch has been fixed on the |
@jim gotcha bud, I will get right to it. |
That's strange, I am running the |
@pineappledreams Everything is passing locally for me as well. It might be a CI issue, as we've been having some build instability on Circle recently. I'm going to merge this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making those revisions!
Issue
Fixes #218
Summary
require_admin
check atapp/controllers/admin/base_controller.rb
test/fixtures/users.yml
the admin user is now an actual admin