diff options
Diffstat (limited to 'spec/workers/personal_access_tokens/expired_notification_worker_spec.rb')
-rw-r--r-- | spec/workers/personal_access_tokens/expired_notification_worker_spec.rb | 28 |
1 files changed, 6 insertions, 22 deletions
diff --git a/spec/workers/personal_access_tokens/expired_notification_worker_spec.rb b/spec/workers/personal_access_tokens/expired_notification_worker_spec.rb index 676a419553f..3ff67f47523 100644 --- a/spec/workers/personal_access_tokens/expired_notification_worker_spec.rb +++ b/spec/workers/personal_access_tokens/expired_notification_worker_spec.rb @@ -9,32 +9,16 @@ RSpec.describe PersonalAccessTokens::ExpiredNotificationWorker, type: :worker do context 'when a token has expired' do let(:expired_today) { create(:personal_access_token, expires_at: Date.current) } - context 'when feature is enabled' do - it 'uses notification service to send email to the user' do - expect_next_instance_of(NotificationService) do |notification_service| - expect(notification_service).to receive(:access_token_expired).with(expired_today.user) - end - - worker.perform + it 'uses notification service to send email to the user' do + expect_next_instance_of(NotificationService) do |notification_service| + expect(notification_service).to receive(:access_token_expired).with(expired_today.user) end - it 'updates notified column' do - expect { worker.perform }.to change { expired_today.reload.after_expiry_notification_delivered }.from(false).to(true) - end + worker.perform end - context 'when feature is disabled' do - before do - stub_feature_flags(expired_pat_email_notification: false) - end - - it 'does not update notified column' do - expect { worker.perform }.not_to change { expired_today.reload.after_expiry_notification_delivered } - end - - it 'does not trigger email' do - expect { worker.perform }.not_to change { ActionMailer::Base.deliveries.count } - end + it 'updates notified column' do + expect { worker.perform }.to change { expired_today.reload.after_expiry_notification_delivered }.from(false).to(true) end end |