diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2020-06-30 23:58:02 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-06-30 23:58:02 +0200 |
commit | 7aaf2b44ec698fd4f20b927fcac7edc0394a2647 (patch) | |
tree | 036b96777e1f525b2b76aab36204dee30cfa3345 /spec/models/concerns | |
parent | 65506bac3f3fe233b5b7b3241020bd74eb5c9259 (diff) |
Fix remote files not using Content-Type header, streaming (#14184)
Diffstat (limited to 'spec/models/concerns')
-rw-r--r-- | spec/models/concerns/remotable_spec.rb | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/spec/models/concerns/remotable_spec.rb b/spec/models/concerns/remotable_spec.rb index 2e6c8a9c6..9cc849ded 100644 --- a/spec/models/concerns/remotable_spec.rb +++ b/spec/models/concerns/remotable_spec.rb @@ -162,19 +162,15 @@ RSpec.describe Remotable do let(:headers) { { 'content-disposition' => file } } it 'assigns file' do - string_io = StringIO.new('') - extname = '.txt' - basename = '0123456789abcdef' + response_with_limit = ResponseWithLimit.new(nil, 0) - allow(SecureRandom).to receive(:hex).and_return(basename) - allow(StringIO).to receive(:new).with(anything).and_return(string_io) + allow(ResponseWithLimit).to receive(:new).with(anything, anything).and_return(response_with_limit) expect(foo).to receive(:public_send).with("download_#{hoge}!", url) foo.hoge_remote_url = url - expect(foo).to receive(:public_send).with("#{hoge}=", string_io) - expect(foo).to receive(:public_send).with("#{hoge}_file_name=", basename + extname) + expect(foo).to receive(:public_send).with("#{hoge}=", response_with_limit) foo.download_hoge!(url) end |