cb902732be
[Tech] Retirer la colonne obsolète 'hidden_at' des dossiers (part 1)
219 lines
9.4 KiB
Ruby
219 lines
9.4 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
describe Expired::UsersDeletionService do
|
|
let(:last_signed_in_not_expired) { (Expired::INACTIVE_USER_RETATION_IN_YEAR - 1).years.ago }
|
|
let(:last_signed_in_expired) { (Expired::INACTIVE_USER_RETATION_IN_YEAR + 1).years.ago }
|
|
let(:before_close_to_expiration) { nil }
|
|
let(:notified_close_to_expiration) { (Expired::REMAINING_WEEKS_BEFORE_EXPIRATION - 1).weeks.ago }
|
|
let(:due_close_to_expiration) { (Expired::REMAINING_WEEKS_BEFORE_EXPIRATION + 1).weeks.ago }
|
|
let(:mail_double) do
|
|
dbl = double()
|
|
expect(dbl).to receive(:deliver_later).with(wait: 0)
|
|
dbl
|
|
end
|
|
|
|
before { user && dossier }
|
|
|
|
describe '#process_expired' do
|
|
subject { Expired::UsersDeletionService.new.process_expired }
|
|
|
|
context 'when user is expirable and have a dossier' do
|
|
let(:dossier) { create(:dossier, user:, created_at: last_signed_in_expired) }
|
|
|
|
context 'when user was not notified' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired, inactive_close_to_expiration_notice_sent_at: before_close_to_expiration) }
|
|
|
|
it 'update user.inactive_close_to_expiration_notice_sent_at ' do
|
|
expect(UserMailer).to receive(:notify_inactive_close_to_deletion).with(user).and_return(mail_double)
|
|
expect { subject }
|
|
.to change { user.reload.inactive_close_to_expiration_notice_sent_at }
|
|
.from(nil).to(anything)
|
|
end
|
|
end
|
|
|
|
context 'user has been notified 1 week ago' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired, inactive_close_to_expiration_notice_sent_at: notified_close_to_expiration) }
|
|
|
|
it 'do nothing' do
|
|
expect { subject }.not_to change { Dossier.count }
|
|
expect { user.reload }.not_to raise_error
|
|
end
|
|
end
|
|
|
|
context 'user has been notified 3 weeks ago' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired, inactive_close_to_expiration_notice_sent_at: due_close_to_expiration) }
|
|
|
|
it 'destroys user and dossier' do
|
|
expect { subject }.to change { Dossier.count }.by(-1)
|
|
expect { user.reload }.to raise_error(ActiveRecord::RecordNotFound)
|
|
end
|
|
|
|
context 'when dossier brouillon' do
|
|
let(:dossier) { create(:dossier, :brouillon, user:, created_at: last_signed_in_expired) }
|
|
it 'destroys user and dossier' do
|
|
expect { subject }.to change { Dossier.count }.by(-1)
|
|
expect { user.reload }.to raise_error(ActiveRecord::RecordNotFound)
|
|
end
|
|
end
|
|
|
|
context 'when dossier en_construction' do
|
|
let(:dossier) { create(:dossier, :en_construction, user:, created_at: last_signed_in_expired) }
|
|
it 'destroys user and dossier' do
|
|
expect { subject }.to change { Dossier.count }.by(-1)
|
|
expect { user.reload }.to raise_error(ActiveRecord::RecordNotFound)
|
|
end
|
|
end
|
|
|
|
context 'when dossier en_instruction' do
|
|
let(:dossier) { create(:dossier, :en_instruction, user:, created_at: last_signed_in_expired) }
|
|
it 'does not do anything' do
|
|
expect { subject }.not_to change { Dossier.count }
|
|
expect { user.reload }.not_to raise_error
|
|
end
|
|
end
|
|
|
|
context 'when dossier termine' do
|
|
let(:dossier) { create(:dossier, :accepte, user:, created_at: last_signed_in_expired) }
|
|
it 'marks dossier as hidden by user due to user_removal and remove user' do
|
|
expect { subject }.to change { dossier.reload.hidden_by_user_at }.from(nil).to(anything)
|
|
expect { user.reload }.to raise_error(ActiveRecord::RecordNotFound)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'when user is expirable but does not have a dossier' do
|
|
let(:dossier) { nil }
|
|
|
|
context 'when user was not notified' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired, inactive_close_to_expiration_notice_sent_at: before_close_to_expiration) }
|
|
|
|
it 'update user.inactive_close_to_expiration_notice_sent_at ' do
|
|
expect(UserMailer).to receive(:notify_inactive_close_to_deletion).with(user).and_return(mail_double)
|
|
expect { subject }
|
|
.to change { user.reload.inactive_close_to_expiration_notice_sent_at }
|
|
.from(nil).to(anything)
|
|
end
|
|
end
|
|
|
|
context 'when user has been notified 1 week ago' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired, inactive_close_to_expiration_notice_sent_at: notified_close_to_expiration) }
|
|
|
|
it 'do nothing' do
|
|
expect { subject }.not_to change { Dossier.count }
|
|
expect { user.reload }.not_to raise_error
|
|
end
|
|
end
|
|
|
|
context 'when user has been notified 3 weeks ago' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired, inactive_close_to_expiration_notice_sent_at: due_close_to_expiration) }
|
|
|
|
it 'destroys user and dossier' do
|
|
subject
|
|
expect { user.reload }.to raise_error(ActiveRecord::RecordNotFound)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#expired_users_without_dossiers' do
|
|
let(:dossier) { nil }
|
|
subject { Expired::UsersDeletionService.new.send(:expired_users_without_dossiers) }
|
|
|
|
context 'when user last_sign_in_at is 1 year ago and has no dossier' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_not_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user last_sign_in_at is 3 year ago and has no dossier' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired) }
|
|
it { is_expected.to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has an expert' do
|
|
let(:user) { create(:user, expert: create(:expert), last_sign_in_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has an instructeur' do
|
|
let(:user) { create(:user, instructeur: create(:instructeur), last_sign_in_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has an admin' do
|
|
let(:user) { create(:user, administrateur: administrateurs(:default_admin), last_sign_in_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired but have a dossier' do
|
|
let(:user) { users(:default_user_admin).tap { _1.update(last_sign_in_at: last_signed_in_expired) } }
|
|
let(:dossier) { create(:dossier, :brouillon, user:, created_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
end
|
|
|
|
describe '#expired_users_with_dossiers' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_expired) }
|
|
let(:dossier) { create(:dossier, :brouillon, user:, created_at: last_signed_in_expired) }
|
|
subject { Expired::UsersDeletionService.new.send(:expired_users_with_dossiers) }
|
|
|
|
context 'when user is not expired' do
|
|
let(:user) { create(:user, last_sign_in_at: last_signed_in_not_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has a dossier brouillon' do
|
|
let(:dossier) { create(:dossier, :brouillon, user:, created_at: last_signed_in_expired) }
|
|
it { is_expected.to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has a many dossier brouillon' do
|
|
before do
|
|
create(:dossier, :brouillon, user:, created_at: last_signed_in_expired)
|
|
create(:dossier, :brouillon, user:, created_at: last_signed_in_expired)
|
|
end
|
|
it { is_expected.to eq([user]) }
|
|
end
|
|
|
|
context 'when user is expired and has a dossier en_construction' do
|
|
let(:dossier) { create(:dossier, :en_construction, user:, created_at: last_signed_in_expired) }
|
|
it { is_expected.to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has a dossier en_instruction' do
|
|
let(:dossier) { create(:dossier, :en_instruction, user:, created_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has a dossier en_instruction plus another one brouillon' do
|
|
before do
|
|
create(:dossier, :en_instruction, user:, created_at: last_signed_in_expired)
|
|
create(:dossier, :brouillon, user:, created_at: last_signed_in_expired)
|
|
end
|
|
it { is_expected.to eq([]) }
|
|
end
|
|
|
|
context 'when user is expired and has a dossier termine' do
|
|
let(:dossier) { create(:dossier, :accepte, user:, created_at: last_signed_in_expired) }
|
|
it { is_expected.to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has an expert' do
|
|
let(:dossier) { create(:dossier, user:, created_at: last_signed_in_expired) }
|
|
let(:user) { create(:user, expert: create(:expert), last_sign_in_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has an instructeur' do
|
|
let(:dossier) { create(:dossier, user:, created_at: last_signed_in_expired) }
|
|
let(:user) { create(:user, instructeur: create(:instructeur), last_sign_in_at: last_signed_in_expired) }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
|
|
context 'when user is expired and has an admin' do
|
|
let(:dossier) { create(:dossier, user:, created_at: last_signed_in_expired) }
|
|
let(:user) { users(:default_user_admin).tap { _1.update(last_sign_in_at: last_signed_in_expired) } }
|
|
it { is_expected.not_to include(user) }
|
|
end
|
|
end
|
|
end
|