Fix tests involving attachment urls
This commit is contained in:
parent
785a09b326
commit
c6326bfa77
10 changed files with 14 additions and 13 deletions
|
@ -209,7 +209,6 @@ describe API::V2::GraphqlController do
|
||||||
checksum
|
checksum
|
||||||
byteSize
|
byteSize
|
||||||
contentType
|
contentType
|
||||||
url
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
avis {
|
avis {
|
||||||
|
@ -270,8 +269,7 @@ describe API::V2::GraphqlController do
|
||||||
filename: commentaire.piece_jointe.filename.to_s,
|
filename: commentaire.piece_jointe.filename.to_s,
|
||||||
contentType: commentaire.piece_jointe.content_type,
|
contentType: commentaire.piece_jointe.content_type,
|
||||||
checksum: commentaire.piece_jointe.checksum,
|
checksum: commentaire.piece_jointe.checksum,
|
||||||
byteSize: commentaire.piece_jointe.byte_size,
|
byteSize: commentaire.piece_jointe.byte_size
|
||||||
url: Rails.application.routes.url_helpers.url_for(commentaire.piece_jointe)
|
|
||||||
},
|
},
|
||||||
email: commentaire.email
|
email: commentaire.email
|
||||||
}
|
}
|
||||||
|
|
|
@ -17,9 +17,9 @@ describe Instructeurs::DossiersController, type: :controller do
|
||||||
context 'when a dossier has an attestation' do
|
context 'when a dossier has an attestation' do
|
||||||
let(:dossier) { create(:dossier, :accepte, attestation: create(:attestation, :with_pdf), procedure: procedure) }
|
let(:dossier) { create(:dossier, :accepte, attestation: create(:attestation, :with_pdf), procedure: procedure) }
|
||||||
|
|
||||||
it 'redirects to attestation pdf' do
|
it 'redirects to a service tmp_url' do
|
||||||
get :attestation, params: { procedure_id: procedure.id, dossier_id: dossier.id }
|
get :attestation, params: { procedure_id: procedure.id, dossier_id: dossier.id }
|
||||||
expect(response).to redirect_to(dossier.attestation.pdf_url.gsub('http://localhost:3000', ''))
|
expect(response.location).to match '/rails/active_storage/disk/'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -14,7 +14,7 @@ describe NewAdministrateur::MailTemplatesController, type: :controller do
|
||||||
it { expect(response).to have_http_status(:ok) }
|
it { expect(response).to have_http_status(:ok) }
|
||||||
|
|
||||||
it 'displays the procedure logo' do
|
it 'displays the procedure logo' do
|
||||||
expect(response.body).to have_css("img[src*='#{procedure.logo_url}']")
|
expect(response.body).to have_css("img[src*='/rails/active_storage/blobs/']")
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'displays the action buttons' do
|
it 'displays the action buttons' do
|
||||||
|
|
|
@ -145,7 +145,7 @@ describe Users::DossiersController, type: :controller do
|
||||||
|
|
||||||
it 'redirects to attestation pdf' do
|
it 'redirects to attestation pdf' do
|
||||||
get :attestation, params: { id: dossier.id }
|
get :attestation, params: { id: dossier.id }
|
||||||
expect(response).to redirect_to(dossier.attestation.pdf_url.gsub('http://localhost:3000', ''))
|
expect(response.location).to match '/rails/active_storage/disk/'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -9,12 +9,12 @@ describe Champs::PieceJustificativeChamp do
|
||||||
|
|
||||||
context 'when file is safe' do
|
context 'when file is safe' do
|
||||||
let(:status) { ActiveStorage::VirusScanner::SAFE }
|
let(:status) { ActiveStorage::VirusScanner::SAFE }
|
||||||
it { is_expected.to include("/rails/active_storage/blobs/") }
|
it { is_expected.to include("/rails/active_storage/disk/") }
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'when file is not scanned' do
|
context 'when file is not scanned' do
|
||||||
let(:status) { ActiveStorage::VirusScanner::PENDING }
|
let(:status) { ActiveStorage::VirusScanner::PENDING }
|
||||||
it { is_expected.to include("/rails/active_storage/blobs/") }
|
it { is_expected.to include("/rails/active_storage/disk/") }
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'when file is infected' do
|
context 'when file is infected' do
|
||||||
|
|
|
@ -14,7 +14,7 @@ describe ChampSerializer do
|
||||||
end
|
end
|
||||||
after { champ.piece_justificative_file.purge }
|
after { champ.piece_justificative_file.purge }
|
||||||
|
|
||||||
it { is_expected.to include(value: url_for(champ.piece_justificative_file)) }
|
it { expect(subject[:value]).to match('/rails/active_storage/disk/') }
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'when type champ is not piece justificative' do
|
context 'when type champ is not piece justificative' do
|
||||||
|
|
|
@ -79,13 +79,14 @@ describe DossierSerializer do
|
||||||
],
|
],
|
||||||
pieces_justificatives: [
|
pieces_justificatives: [
|
||||||
{
|
{
|
||||||
"content_url" => champ_pj.for_api,
|
"content_url" => subject[:pieces_justificatives][0]["content_url"],
|
||||||
"created_at" => champ_pj.created_at.in_time_zone('UTC').iso8601(3),
|
"created_at" => champ_pj.created_at.in_time_zone('UTC').iso8601(3),
|
||||||
"type_de_piece_justificative_id" => original_pj_id,
|
"type_de_piece_justificative_id" => original_pj_id,
|
||||||
"user" => a_hash_including("id" => dossier.user.id)
|
"user" => a_hash_including("id" => dossier.user.id)
|
||||||
}
|
}
|
||||||
]
|
]
|
||||||
)
|
)
|
||||||
|
expect(subject[:pieces_justificatives][0]["content_url"]).to match('/rails/active_storage/disk/')
|
||||||
end
|
end
|
||||||
|
|
||||||
it "does not expose the PJ as a champ" do
|
it "does not expose the PJ as a champ" do
|
||||||
|
|
|
@ -150,6 +150,8 @@ RSpec.configure do |config|
|
||||||
Typhoeus::Expectation.clear
|
Typhoeus::Expectation.clear
|
||||||
|
|
||||||
ActionMailer::Base.deliveries.clear
|
ActionMailer::Base.deliveries.clear
|
||||||
|
|
||||||
|
ActiveStorage::Current.host = 'http://test.host'
|
||||||
}
|
}
|
||||||
|
|
||||||
RSpec::Matchers.define :have_same_attributes_as do |expected, options|
|
RSpec::Matchers.define :have_same_attributes_as do |expected, options|
|
||||||
|
|
|
@ -103,7 +103,7 @@ describe 'instructeurs/dossiers/state_button.html.haml', type: :view do
|
||||||
|
|
||||||
it 'allows to download the justificatif' do
|
it 'allows to download the justificatif' do
|
||||||
expect(rendered).to have_dropdown_item('Justificatif')
|
expect(rendered).to have_dropdown_item('Justificatif')
|
||||||
expect(rendered).to have_link(href: url_for(dossier.justificatif_motivation.attachment.blob))
|
expect(response).to have_css("a[href*='/rails/active_storage/blobs/']", text: dossier.justificatif_motivation.attachment.filename.to_s)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -17,7 +17,7 @@ describe 'users/dossiers/brouillon.html.haml', type: :view do
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'affiche un lien vers la notice' do
|
it 'affiche un lien vers la notice' do
|
||||||
expect(rendered).to have_link("Guide de la démarche", href: url_for(procedure.notice))
|
expect(response).to have_css("a[href*='/rails/active_storage/blobs/']", text: "Guide de la démarche")
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'affiche les boutons de validation' do
|
it 'affiche les boutons de validation' do
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue