DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

spec cleaning

Browse source
This commit is contained in:
simon lehericey 2021-10-06 16:30:51 +02:00
parent 750b2b4f58
commit 2f470b25aa
2 changed files with 6 additions and 33 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
spec/controllers/france_connect/particulier_controller_spec.rb
View file

@ -79,7 +79,6 @@ describe FranceConnect::ParticulierController, type: :controller do
context 'when france_connect_particulier_id does not exist in database' do
it { expect { subject }.to change { FranceConnectInformation.count }.by(1) }
it { expect { subject }.to change { FranceConnectInformation.count }.by(1) }
describe 'FranceConnectInformation attributs' do
let(:stored_fci) { FranceConnectInformation.last }

38
spec/features/france_connect/france_connect_particulier_spec.rb
View file

@ -21,17 +21,13 @@ feature 'France Connect Particulier Connexion' do
end
context 'when user is on login page' do
before do
visit new_user_session_path
end
before { visit new_user_session_path }
scenario 'link to France Connect is present' do
expect(page).to have_css('.france-connect-login-button')
end
context 'and click on france connect link' do
let(:code) { 'plop' }
context 'when authentification is ok' do
before do
allow_any_instance_of(FranceConnectParticulierClient).to receive(:authorization_uri).and_return(france_connect_particulier_callback_path(code: code))
@ -39,44 +35,22 @@ feature 'France Connect Particulier Connexion' do
end
context 'when is the first connexion' do
let(:france_connect_information) do
build(:france_connect_information,
france_connect_particulier_id: france_connect_particulier_id,
given_name: given_name,
family_name: family_name,
birthdate: birthdate,
birthplace: birthplace,
gender: gender,
email_france_connect: email)
end
let(:france_connect_information) { build(:france_connect_information, user_info) }
before do
page.find('.france-connect-login-button').click
end
before { page.find('.france-connect-login-button').click }
scenario 'he is redirected to user dossiers page' do
expect(page).to have_content('Dossiers')
expect(User.find_by(email: email)).not_to be nil
end
end
context 'when is not the first connexion' do
let!(:france_connect_information) do
create(:france_connect_information,
:with_user,
france_connect_particulier_id: france_connect_particulier_id,
given_name: given_name,
family_name: family_name,
birthdate: birthdate,
birthplace: birthplace,
gender: gender,
email_france_connect: email,
created_at: Time.zone.parse('12/12/2012'),
updated_at: Time.zone.parse('12/12/2012'))
create(:france_connect_information, :with_user, user_info.merge(created_at: Time.zone.parse('12/12/2012'), updated_at: Time.zone.parse('12/12/2012')))
end
before do
page.find('.france-connect-login-button').click
end
before { page.find('.france-connect-login-button').click }
scenario 'he is redirected to user dossiers page' do
expect(page).to have_content('Dossiers')