f4e08ef30d
Fixes the banned IP check not being applied when sending the HEAD request for is_cloudflare?. Also fixes the `#size` method not using the uncached url (which meant the bookmarklet could report the wrong filesize on artstation uploads).
74 lines
2.7 KiB
Ruby
74 lines
2.7 KiB
Ruby
require 'test_helper'
|
|
|
|
module Downloads
|
|
class FileTest < ActiveSupport::TestCase
|
|
context "A post download" do
|
|
setup do
|
|
@source = "http://www.google.com/intl/en_ALL/images/logo.gif"
|
|
@download = Downloads::File.new(@source)
|
|
end
|
|
|
|
context "for a banned IP" do
|
|
should "not try to download the file" do
|
|
Resolv.expects(:getaddress).returns("127.0.0.1")
|
|
assert_raise(Downloads::File::Error) { Downloads::File.new("http://evil.com").download! }
|
|
end
|
|
|
|
should "not try to fetch the size" do
|
|
Resolv.expects(:getaddress).returns("127.0.0.1")
|
|
assert_raise(Downloads::File::Error) { Downloads::File.new("http://evil.com").size }
|
|
end
|
|
|
|
should "not follow redirects to banned IPs" do
|
|
url = "http://httpbin.org/redirect-to?url=http://127.0.0.1"
|
|
stub_request(:get, url).to_return(status: 301, headers: { "Location": "http://127.0.0.1" })
|
|
|
|
assert_raise(Downloads::File::Error) { Downloads::File.new(url).download! }
|
|
end
|
|
|
|
should "not follow redirects that resolve to a banned IP" do
|
|
url = "http://httpbin.org/redirect-to?url=http://127.0.0.1.nip.io"
|
|
stub_request(:get, url).to_return(status: 301, headers: { "Location": "http://127.0.0.1.xip.io" })
|
|
Resolv.expects(:getaddress).returns("127.0.0.1")
|
|
|
|
assert_raise(Downloads::File::Error) { Downloads::File.new(url).download! }
|
|
end
|
|
|
|
should "not send a HEAD request when checking for cloudflare" do
|
|
Resolv.expects(:getaddress).with("www.google.com").returns("127.0.0.1")
|
|
assert_raise(Downloads::File::Error) { @download.is_cloudflare? }
|
|
end
|
|
end
|
|
|
|
context "that fails" do
|
|
should "retry three times before giving up" do
|
|
HTTParty.expects(:get).times(3).raises(Errno::ETIMEDOUT)
|
|
assert_raises(Errno::ETIMEDOUT) { @download.download! }
|
|
end
|
|
|
|
should "return an uncorrupted file on the second try" do
|
|
bomb = stub("bomb")
|
|
bomb.expects(:size).raises(IOError)
|
|
resp = stub("resp", success?: true)
|
|
|
|
HTTParty.expects(:get).twice.multiple_yields("a", bomb, "b", "c").then.multiple_yields("a", "b", "c").returns(resp)
|
|
tempfile, _ = @download.download!
|
|
|
|
assert_equal("abc", tempfile.read)
|
|
end
|
|
end
|
|
|
|
should "throw an exception when the file is larger than the maximum" do
|
|
assert_raise(Downloads::File::Error) do
|
|
@download.download!(max_size: 1)
|
|
end
|
|
end
|
|
|
|
should "store the file in the tempfile path" do
|
|
tempfile, strategy = @download.download!
|
|
assert_operator(tempfile.size, :>, 0, "should have data")
|
|
end
|
|
end
|
|
end
|
|
end
|