chore: easier to test .gouv / new host using APP_HOST

This commit is contained in:
Colin Darie 2024-03-27 11:04:46 +01:00
parent 033b2fa63d
commit 8c0375fe6f
No known key found for this signature in database
GPG key ID: 8C76CADD40253590
4 changed files with 13 additions and 10 deletions

View file

@ -29,10 +29,9 @@ class ApplicationController < ActionController::Base
Current.user = current_user
Current.browser = browser
# TODO: remove this block when migration to new domain is done
# headers['Host'] instead of request.host to keep the port (ex: localhost:3000)
Current.host = request.headers['Host']
Current.host = request.host_with_port
if Current.host.include?("gouv.fr")
if Current.host.include?(".gouv.fr")
Current.application_name = "demarches.gouv.fr"
Current.contact_email = "contact@demarches.gouv.fr"
Current.application_base_url = "https://demarches.gouv.fr"

View file

@ -932,7 +932,7 @@ fr:
confirmation_mail_sent: "Nous venons de vous envoyer le mail de confirmation, veuillez cliquer sur le lien contenu dans ce mail pour fusionner vos comptes"
invalid_password: "Mauvais mot de passe"
connection_done: "Les comptes FranceConnect et %{application_name} sont à présent fusionnés"
merger_token_expired: "Le délai pour fusionner les comptes FranceConnect et %{application_name} est expirée. Veuillez recommencer la procédure pour vous fusionner les comptes."
merger_token_expired: "Le délai pour fusionner les comptes FranceConnect et %{application_name} est expiré. Veuillez recommencer la procédure pour vous fusionner les comptes."
groupe_gestionnaires:
flash:
alert:

View file

@ -51,7 +51,7 @@ RSpec.describe SwitchDomainBannerComponent, type: :component do
let(:path) { "/admin/procedures" }
it "generate an url to the new domain" do
expect(rendered.to_html).to have_link(APPLICATION_NAME, href: "http://demarches.gouv.fr/admin/procedures")
expect(rendered.to_html).to have_link("demarches.gouv.fr", href: "http://demarches.gouv.fr/admin/procedures")
expect(rendered.to_html).not_to include("window.location")
expect(rendered.to_html).to include("Suivez ce lien")
end

View file

@ -159,7 +159,7 @@ describe FranceConnect::ParticulierController, type: :controller do
RSpec.shared_examples "a method that needs a valid merge token" do
context 'when the merge token is invalid' do
before do
stub_const("APPLICATION_NAME", "demarches-simplifiees.fr")
allow(Current).to receive(:application_name).and_return('demarches-simplifiees.fr')
merge_token
fci.update(merge_token_created_at: 2.years.ago)
end
@ -171,7 +171,7 @@ describe FranceConnect::ParticulierController, type: :controller do
else
expect(subject).to redirect_to root_path
end
expect(flash.alert).to eq('Le délai pour fusionner les comptes FranceConnect et demarches-simplifiees.fr est expirée. Veuillez recommencer la procédure pour vous fusionner les comptes.')
expect(flash.alert).to eq('Le délai pour fusionner les comptes FranceConnect et demarches-simplifiees.fr est expiré. Veuillez recommencer la procédure pour vous fusionner les comptes.')
end
end
end
@ -193,12 +193,12 @@ describe FranceConnect::ParticulierController, type: :controller do
let(:merge_token) { 'i do not exist' }
before do
stub_const("APPLICATION_NAME", "demarches-simplifiees.fr")
allow(Current).to receive(:application_name).and_return('demarches-simplifiees.fr')
end
it do
expect(subject).to redirect_to root_path
expect(flash.alert).to eq("Le délai pour fusionner les comptes FranceConnect et demarches-simplifiees.fr est expirée. Veuillez recommencer la procédure pour vous fusionner les comptes.")
expect(flash.alert).to eq("Le délai pour fusionner les comptes FranceConnect et demarches-simplifiees.fr est expiré. Veuillez recommencer la procédure pour vous fusionner les comptes.")
end
end
end
@ -273,6 +273,10 @@ describe FranceConnect::ParticulierController, type: :controller do
context 'when the merge_token is ok and the user is found' do
subject { post :mail_merge_with_existing_account, params: { email_merge_token: } }
before do
allow(Current).to receive(:application_name).and_return('demarches-simplifiees.fr')
end
let!(:user) { create(:user, email: email, password: 'abcdefgh') }
it 'merges the account, signs in, and delete the merge token' do
@ -283,7 +287,7 @@ describe FranceConnect::ParticulierController, type: :controller do
expect(fci.merge_token).to be_nil
expect(fci.email_merge_token).to be_nil
expect(controller.current_user).to eq(user)
expect(flash[:notice]).to eq("Les comptes FranceConnect et #{APPLICATION_NAME} sont à présent fusionnés")
expect(flash[:notice]).to eq("Les comptes FranceConnect et #{Current.application_name} sont à présent fusionnés")
end
context 'but the targeted user is an instructeur' do