run Rubocop
This commit is contained in:
parent
06a20d34d2
commit
b3c7faba03
|
@ -1,51 +1,51 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
require 'rails_helper'
|
||||
require "rails_helper"
|
||||
|
||||
describe 'role-constrained routes', type: :request do
|
||||
shared_examples_for 'fails to access route' do
|
||||
it 'fails to access route' do
|
||||
describe "role-constrained routes", type: :request do
|
||||
shared_examples_for "fails to access route" do
|
||||
it "fails to access route" do
|
||||
# 404 = no user found -- we have a fallback route if something could not be matched
|
||||
expect(subject).to eq 404
|
||||
end
|
||||
end
|
||||
|
||||
shared_examples_for 'routes for' do |roles, subject_block, skip_reason: nil|
|
||||
shared_examples_for "routes for" do |roles, subject_block, skip_reason: nil|
|
||||
before { skip(skip_reason) } if skip_reason
|
||||
|
||||
subject(&subject_block)
|
||||
|
||||
context 'not signed in' do
|
||||
include_examples 'fails to access route'
|
||||
context "not signed in" do
|
||||
include_examples "fails to access route"
|
||||
end
|
||||
|
||||
roles.each do |role|
|
||||
context "signed in user without #{role} role" do
|
||||
let(:user) { FactoryBot.create(:user, password: 'test1234') }
|
||||
let(:user) { FactoryBot.create(:user, password: "test1234") }
|
||||
|
||||
before(:each) do
|
||||
post '/sign_in', params: { user: { login: user.email, password: user.password } }
|
||||
post "/sign_in", params: { user: { login: user.email, password: user.password } }
|
||||
end
|
||||
|
||||
include_examples 'fails to access route'
|
||||
include_examples "fails to access route"
|
||||
end
|
||||
|
||||
context "signed in user with #{role} role" do
|
||||
let(:user) { FactoryBot.create(:user, password: 'test1234', roles: [role]) }
|
||||
let(:user) { FactoryBot.create(:user, password: "test1234", roles: [role]) }
|
||||
|
||||
before(:each) do
|
||||
post '/sign_in', params: { user: { login: user.email, password: user.password } }
|
||||
post "/sign_in", params: { user: { login: user.email, password: user.password } }
|
||||
end
|
||||
|
||||
it 'can access route' do
|
||||
it "can access route" do
|
||||
expect(subject).to be_in 200..299
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
it_behaves_like('routes for', [:administrator], -> { get('/justask_admin') })
|
||||
it_behaves_like('routes for', [:administrator], -> { get('/sidekiq') })
|
||||
it_behaves_like('routes for', [:administrator], -> { get('/pghero') }, skip_reason: 'PG::InFailedSqlTransaction due to 5.1 upgrade, works fine outside specs though')
|
||||
it_behaves_like('routes for', %i[administrator moderator], -> { get('/moderation') })
|
||||
it_behaves_like("routes for", [:administrator], -> { get("/justask_admin") })
|
||||
it_behaves_like("routes for", [:administrator], -> { get("/sidekiq") })
|
||||
it_behaves_like("routes for", [:administrator], -> { get("/pghero") }), skip_reason: "PG::InFailedSqlTransaction due to 5.1 upgrade, works fine outside specs though")
|
||||
it_behaves_like("routes for", %i[administrator moderator], -> { get("/moderation") })
|
||||
end
|
||||
|
|
|
@ -1,4 +1,3 @@
|
|||
|
||||
# frozen_string_literal: true
|
||||
|
||||
require "factory_bot_rails"
|
||||
|
|
Loading…
Reference in New Issue