Drop staff restriction in beta and staging
because it was preventing testing of signups.
This commit is contained in:
@@ -3,7 +3,6 @@ module Authorization
|
||||
|
||||
included do
|
||||
before_action :ensure_can_access_account, if: -> { Current.account.present? && authenticated? }
|
||||
before_action :ensure_only_staff_can_access_non_production_remote_environments, if: :authenticated?
|
||||
end
|
||||
|
||||
class_methods do
|
||||
@@ -30,10 +29,6 @@ module Authorization
|
||||
redirect_to session_menu_url(script_name: nil) if Current.user.blank? || !Current.user.active?
|
||||
end
|
||||
|
||||
def ensure_only_staff_can_access_non_production_remote_environments
|
||||
head :forbidden unless Rails.env.local? || Rails.env.production? || Current.identity.staff?
|
||||
end
|
||||
|
||||
def redirect_existing_user
|
||||
redirect_to root_path if Current.user
|
||||
end
|
||||
|
||||
@@ -1,34 +0,0 @@
|
||||
require "test_helper"
|
||||
|
||||
class NonProductionRemoteAccessTest < ActionDispatch::IntegrationTest
|
||||
test "staff can access in staging environment" do
|
||||
sign_in_as :david
|
||||
|
||||
Rails.stubs(:env).returns(ActiveSupport::EnvironmentInquirer.new("staging"))
|
||||
get cards_path
|
||||
assert_response :success
|
||||
end
|
||||
|
||||
test "non-staff cannot access in staging environment" do
|
||||
sign_in_as :jz
|
||||
|
||||
Rails.stubs(:env).returns(ActiveSupport::EnvironmentInquirer.new("staging"))
|
||||
get cards_path
|
||||
assert_response :forbidden
|
||||
end
|
||||
|
||||
test "non-staff can access in production environment" do
|
||||
sign_in_as :jz
|
||||
|
||||
Rails.stubs(:env).returns(ActiveSupport::EnvironmentInquirer.new("production"))
|
||||
get cards_path
|
||||
assert_response :success
|
||||
end
|
||||
|
||||
test "non-staff can access in local environment" do
|
||||
sign_in_as :jz
|
||||
|
||||
get cards_path
|
||||
assert_response :success
|
||||
end
|
||||
end
|
||||
Reference in New Issue
Block a user