Skip to content

Commit ff85fdd

Browse files
committed
Update checking feature_daily_submission_emails_enabled flag in tests
The FeatureService stubbing didn't work with the new feature flag that had been added, so set the feature flag using RSpec metadata instead.
1 parent dad21c3 commit ff85fdd

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

spec/requests/forms/daily_submission_batch_controller_spec.rb

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,12 @@
11
require "rails_helper"
22

3-
RSpec.describe Forms::DailySubmissionBatchController, type: :request do
3+
RSpec.describe Forms::DailySubmissionBatchController, :feature_daily_submission_emails_enabled, type: :request do
44
let(:form) { create(:form, :live, send_daily_submission_batch: send_daily_submission_batch_original_value) }
55
let(:send_daily_submission_batch_original_value) { false }
66
let(:current_user) { standard_user }
77
let(:group) { create(:group, organisation: standard_user.organisation) }
8-
let(:daily_submission_emails_enabled) { true }
98

109
before do
11-
allow(FeatureService).to receive(:enabled?).with(:daily_submission_emails_enabled).and_return(daily_submission_emails_enabled)
12-
1310
Membership.create!(group_id: group.id, user: standard_user, added_by: standard_user)
1411
GroupForm.create!(form_id: form.id, group_id: group.id)
1512

@@ -37,9 +34,7 @@
3734
end
3835
end
3936

40-
context "when the feature flag is disabled" do
41-
let(:daily_submission_emails_enabled) { false }
42-
37+
context "when the feature flag is disabled", feature_daily_submission_emails_enabled: false do
4338
it "returns 404" do
4439
expect(response).to have_http_status(:not_found)
4540
end

0 commit comments

Comments
 (0)