diff options
author | Renato "Lond" Cerqueira <renato@lond.com.br> | 2018-08-29 21:13:49 +0200 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2018-08-29 21:13:49 +0200 |
commit | 5b2b493a908cbea55096f7f028c306f6270e3b00 (patch) | |
tree | 6792309266a0051789401168586d5ce9b03cdb9a /spec | |
parent | cc26fd71acdcd0ef1c0992273a07d755af2b6108 (diff) |
Fix nil host in remotable (#8508)
Host can be nil in urls like 'https:https://example.com/path/file.png'
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/concerns/remotable_spec.rb | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/spec/models/concerns/remotable_spec.rb b/spec/models/concerns/remotable_spec.rb index b39233739..a4289cc45 100644 --- a/spec/models/concerns/remotable_spec.rb +++ b/spec/models/concerns/remotable_spec.rb @@ -88,7 +88,18 @@ RSpec.describe Remotable do context 'parsed_url.host is empty' do it 'makes no request' do - parsed_url = double(scheme: 'https', host: double(empty?: true)) + parsed_url = double(scheme: 'https', host: double(blank?: true)) + allow(Addressable::URI).to receive_message_chain(:parse, :normalize) + .with(url).with(no_args).and_return(parsed_url) + + foo.hoge_remote_url = url + expect(request).not_to have_been_requested + end + end + + context 'parsed_url.host is nil' do + it 'makes no request' do + parsed_url = Addressable::URI.parse('https:https://example.com/path/file.png') allow(Addressable::URI).to receive_message_chain(:parse, :normalize) .with(url).with(no_args).and_return(parsed_url) |