Commit 5bd6722c authored by Sean Arnold's avatar Sean Arnold

Tweak policy order and fix service spec

parent 63fa46cd
...@@ -11,6 +11,7 @@ RSpec.describe IncidentManagement::OncallSchedulePolicy do ...@@ -11,6 +11,7 @@ RSpec.describe IncidentManagement::OncallSchedulePolicy do
before do before do
stub_feature_flags(oncall_schedules_mvc: project) stub_feature_flags(oncall_schedules_mvc: project)
stub_licensed_features(oncall_schedules: true)
end end
describe 'rules' do describe 'rules' do
...@@ -21,14 +22,14 @@ RSpec.describe IncidentManagement::OncallSchedulePolicy do ...@@ -21,14 +22,14 @@ RSpec.describe IncidentManagement::OncallSchedulePolicy do
project.add_reporter(user) project.add_reporter(user)
end end
it { is_expected.to be_disallowed :read_incident_management_oncall_schedule } it { is_expected.to be_allowed :read_incident_management_oncall_schedule }
context 'licensed feature enabled' do context 'licensed feature disabled' do
before do before do
stub_licensed_features(oncall_schedules: true) stub_licensed_features(oncall_schedules: false)
end end
it { is_expected.to be_allowed :read_incident_management_oncall_schedule } it { is_expected.to be_disallowed :read_incident_management_oncall_schedule }
end end
end end
end end
......
...@@ -13,7 +13,7 @@ RSpec.describe IncidentManagement::OncallRotations::CreateService do ...@@ -13,7 +13,7 @@ RSpec.describe IncidentManagement::OncallRotations::CreateService do
[ [
{ {
user: current_user, user: current_user,
color_palette: 'black', color_palette: 'blue',
color_weight: '500' color_weight: '500'
} }
] ]
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment