I am migrating an app from Paperclip to Active Storage and I'd like to
update the controller tests for the download links, which will redirect to
the expiring url in S3. I'm not using the provided url helpers
rails_blob_path and rails_blob_url because we need to have authentication
(see the first warning at
https://edgeguides.rubyonrails.org/active_storage_overview.html#linking-to-files
).
My test looks (more or less) like this:
subject(:download_attachment) { get :download_attachment, params: { id:
message.id } }
let(:message) { create :message }
let(:service_url) { 'https://example.com/document.pdf' }
it 'redirects to file download link' do
allow_any_instance_of(ActiveStorage::Blob).to
receive(:service_url).and_return(service_url)
expect(download_attachment).to redirect_to service_url
end
but I know that `allow_any_instance_of` is discouraged (and deprecated). An
alternative is
ActiveStorage::Current.host = 'https://example.com'
download_attachment
expect(response.location).to match %{https://example.com}
but this also feels a bit hacky.
Neither method is actually checking that the redirect is to the correct url
for the correct attachment so ideally I would like to have
`allow(message.attachment.blob).to
receive(:service_url).and_return(service_url)`. Obviously this isn't
working as the message is getting loaded new in the controller.
Is there a way to do this? Thanks.
Regards,
Joe
--
You received this message because you are subscribed to the Google Groups
"rspec" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to rspec+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/rspec/CAKRXwc2KaZgkMhMn0rbd8Qz8kOTcZuJyiaHrdnTtkbX9B6hdTg%40mail.gmail.com.
