about summary refs log tree commit diff
path: root/spec/models/concerns
diff options
context:
space:
mode:
authorRenato "Lond" Cerqueira <renato@lond.com.br>2018-08-29 21:13:49 +0200
committerEugen Rochko <eugen@zeonfederated.com>2018-08-29 21:13:49 +0200
commit5b2b493a908cbea55096f7f028c306f6270e3b00 (patch)
tree6792309266a0051789401168586d5ce9b03cdb9a /spec/models/concerns
parentcc26fd71acdcd0ef1c0992273a07d755af2b6108 (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/models/concerns')
-rw-r--r--spec/models/concerns/remotable_spec.rb13
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)