paperclip 3.5.4 → 5.3.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.codeclimate.yml +17 -0
- data/.gitignore +0 -6
- data/.hound.yml +1055 -0
- data/.rubocop.yml +1 -0
- data/.travis.yml +17 -20
- data/Appraisals +4 -16
- data/CONTRIBUTING.md +29 -13
- data/Gemfile +11 -3
- data/LICENSE +1 -3
- data/NEWS +241 -49
- data/README.md +471 -166
- data/RELEASING.md +17 -0
- data/Rakefile +6 -8
- data/UPGRADING +12 -9
- data/features/basic_integration.feature +27 -8
- data/features/migration.feature +0 -24
- data/features/step_definitions/attachment_steps.rb +36 -28
- data/features/step_definitions/html_steps.rb +2 -2
- data/features/step_definitions/rails_steps.rb +68 -37
- data/features/step_definitions/s3_steps.rb +2 -2
- data/features/step_definitions/web_steps.rb +1 -103
- data/features/support/env.rb +3 -2
- data/features/support/file_helpers.rb +2 -2
- data/features/support/fixtures/gemfile.txt +1 -1
- data/features/support/paths.rb +1 -1
- data/features/support/rails.rb +2 -25
- data/gemfiles/4.2.gemfile +17 -0
- data/gemfiles/5.0.gemfile +17 -0
- data/lib/generators/paperclip/paperclip_generator.rb +9 -3
- data/lib/generators/paperclip/templates/paperclip_migration.rb.erb +2 -2
- data/lib/paperclip/attachment.rb +151 -46
- data/lib/paperclip/attachment_registry.rb +3 -2
- data/lib/paperclip/callbacks.rb +13 -1
- data/lib/paperclip/content_type_detector.rb +26 -22
- data/lib/paperclip/errors.rb +8 -1
- data/lib/paperclip/file_command_content_type_detector.rb +6 -8
- data/lib/paperclip/geometry_detector_factory.rb +5 -3
- data/lib/paperclip/geometry_parser_factory.rb +1 -1
- data/lib/paperclip/glue.rb +1 -1
- data/lib/paperclip/has_attached_file.rb +17 -1
- data/lib/paperclip/helpers.rb +15 -11
- data/lib/paperclip/interpolations/plural_cache.rb +6 -5
- data/lib/paperclip/interpolations.rb +26 -13
- data/lib/paperclip/io_adapters/abstract_adapter.rb +32 -4
- data/lib/paperclip/io_adapters/attachment_adapter.rb +13 -8
- data/lib/paperclip/io_adapters/data_uri_adapter.rb +11 -16
- data/lib/paperclip/io_adapters/empty_string_adapter.rb +5 -4
- data/lib/paperclip/io_adapters/file_adapter.rb +12 -6
- data/lib/paperclip/io_adapters/http_url_proxy_adapter.rb +7 -7
- data/lib/paperclip/io_adapters/identity_adapter.rb +12 -6
- data/lib/paperclip/io_adapters/nil_adapter.rb +8 -5
- data/lib/paperclip/io_adapters/registry.rb +6 -2
- data/lib/paperclip/io_adapters/stringio_adapter.rb +15 -16
- data/lib/paperclip/io_adapters/uploaded_file_adapter.rb +10 -6
- data/lib/paperclip/io_adapters/uri_adapter.rb +41 -19
- data/lib/paperclip/locales/en.yml +1 -0
- data/lib/paperclip/matchers/have_attached_file_matcher.rb +2 -1
- data/lib/paperclip/matchers/validate_attachment_content_type_matcher.rb +4 -4
- data/lib/paperclip/matchers/validate_attachment_presence_matcher.rb +2 -1
- data/lib/paperclip/matchers/validate_attachment_size_matcher.rb +2 -1
- data/lib/paperclip/media_type_spoof_detector.rb +90 -0
- data/lib/paperclip/processor.rb +5 -41
- data/lib/paperclip/processor_helpers.rb +50 -0
- data/lib/paperclip/rails_environment.rb +25 -0
- data/lib/paperclip/schema.rb +9 -7
- data/lib/paperclip/storage/filesystem.rb +14 -3
- data/lib/paperclip/storage/fog.rb +37 -19
- data/lib/paperclip/storage/s3.rb +129 -69
- data/lib/paperclip/style.rb +8 -2
- data/lib/paperclip/tempfile_factory.rb +5 -1
- data/lib/paperclip/thumbnail.rb +30 -18
- data/lib/paperclip/url_generator.rb +26 -14
- data/lib/paperclip/validators/attachment_content_type_validator.rb +4 -0
- data/lib/paperclip/validators/attachment_file_name_validator.rb +80 -0
- data/lib/paperclip/validators/attachment_file_type_ignorance_validator.rb +29 -0
- data/lib/paperclip/validators/attachment_presence_validator.rb +4 -0
- data/lib/paperclip/validators/attachment_size_validator.rb +5 -3
- data/lib/paperclip/validators/media_type_spoof_detection_validator.rb +27 -0
- data/lib/paperclip/validators.rb +11 -4
- data/lib/paperclip/version.rb +3 -1
- data/lib/paperclip.rb +30 -11
- data/lib/tasks/paperclip.rake +34 -5
- data/paperclip.gemspec +21 -16
- data/shoulda_macros/paperclip.rb +0 -1
- data/spec/paperclip/attachment_definitions_spec.rb +13 -0
- data/{test/attachment_processing_test.rb → spec/paperclip/attachment_processing_spec.rb} +17 -20
- data/spec/paperclip/attachment_registry_spec.rb +158 -0
- data/{test/attachment_test.rb → spec/paperclip/attachment_spec.rb} +502 -407
- data/{test/content_type_detector_test.rb → spec/paperclip/content_type_detector_spec.rb} +17 -20
- data/spec/paperclip/file_command_content_type_detector_spec.rb +40 -0
- data/spec/paperclip/filename_cleaner_spec.rb +14 -0
- data/spec/paperclip/geometry_detector_spec.rb +39 -0
- data/{test/geometry_parser_test.rb → spec/paperclip/geometry_parser_spec.rb} +27 -27
- data/{test/geometry_test.rb → spec/paperclip/geometry_spec.rb} +50 -52
- data/spec/paperclip/glue_spec.rb +44 -0
- data/spec/paperclip/has_attached_file_spec.rb +158 -0
- data/{test/integration_test.rb → spec/paperclip/integration_spec.rb} +137 -128
- data/{test/interpolations_test.rb → spec/paperclip/interpolations_spec.rb} +70 -46
- data/spec/paperclip/io_adapters/abstract_adapter_spec.rb +132 -0
- data/{test/io_adapters/attachment_adapter_test.rb → spec/paperclip/io_adapters/attachment_adapter_spec.rb} +33 -32
- data/spec/paperclip/io_adapters/data_uri_adapter_spec.rb +89 -0
- data/spec/paperclip/io_adapters/empty_string_adapter_spec.rb +17 -0
- data/{test/io_adapters/file_adapter_test.rb → spec/paperclip/io_adapters/file_adapter_spec.rb} +38 -42
- data/spec/paperclip/io_adapters/http_url_proxy_adapter_spec.rb +121 -0
- data/spec/paperclip/io_adapters/identity_adapter_spec.rb +8 -0
- data/{test/io_adapters/nil_adapter_test.rb → spec/paperclip/io_adapters/nil_adapter_spec.rb} +7 -7
- data/{test/io_adapters/registry_test.rb → spec/paperclip/io_adapters/registry_spec.rb} +12 -9
- data/{test/io_adapters/stringio_adapter_test.rb → spec/paperclip/io_adapters/stringio_adapter_spec.rb} +21 -18
- data/{test/io_adapters/uploaded_file_adapter_test.rb → spec/paperclip/io_adapters/uploaded_file_adapter_spec.rb} +46 -46
- data/spec/paperclip/io_adapters/uri_adapter_spec.rb +172 -0
- data/spec/paperclip/matchers/have_attached_file_matcher_spec.rb +19 -0
- data/spec/paperclip/matchers/validate_attachment_content_type_matcher_spec.rb +109 -0
- data/spec/paperclip/matchers/validate_attachment_presence_matcher_spec.rb +69 -0
- data/spec/paperclip/matchers/validate_attachment_size_matcher_spec.rb +88 -0
- data/spec/paperclip/media_type_spoof_detector_spec.rb +94 -0
- data/spec/paperclip/meta_class_spec.rb +30 -0
- data/spec/paperclip/paperclip_missing_attachment_styles_spec.rb +84 -0
- data/spec/paperclip/paperclip_spec.rb +192 -0
- data/spec/paperclip/plural_cache_spec.rb +37 -0
- data/spec/paperclip/processor_helpers_spec.rb +57 -0
- data/{test/processor_test.rb → spec/paperclip/processor_spec.rb} +7 -7
- data/spec/paperclip/rails_environment_spec.rb +33 -0
- data/{test/rake_test.rb → spec/paperclip/rake_spec.rb} +15 -15
- data/spec/paperclip/schema_spec.rb +248 -0
- data/{test/storage/filesystem_test.rb → spec/paperclip/storage/filesystem_spec.rb} +18 -18
- data/spec/paperclip/storage/fog_spec.rb +566 -0
- data/spec/paperclip/storage/s3_live_spec.rb +188 -0
- data/spec/paperclip/storage/s3_spec.rb +1693 -0
- data/spec/paperclip/style_spec.rb +255 -0
- data/spec/paperclip/tempfile_factory_spec.rb +33 -0
- data/spec/paperclip/tempfile_spec.rb +35 -0
- data/{test/thumbnail_test.rb → spec/paperclip/thumbnail_spec.rb} +153 -134
- data/spec/paperclip/url_generator_spec.rb +222 -0
- data/spec/paperclip/validators/attachment_content_type_validator_spec.rb +322 -0
- data/spec/paperclip/validators/attachment_file_name_validator_spec.rb +160 -0
- data/{test/validators/attachment_presence_validator_test.rb → spec/paperclip/validators/attachment_presence_validator_spec.rb} +20 -20
- data/{test/validators/attachment_size_validator_test.rb → spec/paperclip/validators/attachment_size_validator_spec.rb} +77 -64
- data/spec/paperclip/validators/media_type_spoof_detection_validator_spec.rb +52 -0
- data/spec/paperclip/validators_spec.rb +164 -0
- data/spec/spec_helper.rb +47 -0
- data/spec/support/assertions.rb +82 -0
- data/spec/support/conditional_filter_helper.rb +5 -0
- data/spec/support/fake_model.rb +25 -0
- data/spec/support/fake_rails.rb +12 -0
- data/spec/support/fixtures/empty.html +1 -0
- data/spec/support/fixtures/empty.xlsx +0 -0
- data/spec/support/fixtures/spaced file.jpg +0 -0
- data/spec/support/matchers/accept.rb +5 -0
- data/spec/support/matchers/exist.rb +5 -0
- data/spec/support/matchers/have_column.rb +23 -0
- data/{test → spec}/support/mock_attachment.rb +2 -0
- data/{test → spec}/support/mock_url_generator_builder.rb +2 -2
- data/spec/support/model_reconstruction.rb +68 -0
- data/spec/support/reporting.rb +11 -0
- data/spec/support/test_data.rb +13 -0
- data/spec/support/version_helper.rb +9 -0
- metadata +348 -225
- data/RUNNING_TESTS.md +0 -4
- data/cucumber/paperclip_steps.rb +0 -6
- data/gemfiles/3.0.gemfile +0 -11
- data/gemfiles/3.1.gemfile +0 -11
- data/gemfiles/3.2.gemfile +0 -11
- data/gemfiles/4.0.gemfile +0 -11
- data/test/attachment_definitions_test.rb +0 -12
- data/test/attachment_registry_test.rb +0 -88
- data/test/file_command_content_type_detector_test.rb +0 -27
- data/test/filename_cleaner_test.rb +0 -14
- data/test/generator_test.rb +0 -84
- data/test/geometry_detector_test.rb +0 -24
- data/test/has_attached_file_test.rb +0 -125
- data/test/helper.rb +0 -232
- data/test/io_adapters/abstract_adapter_test.rb +0 -58
- data/test/io_adapters/data_uri_adapter_test.rb +0 -74
- data/test/io_adapters/empty_string_adapter_test.rb +0 -18
- data/test/io_adapters/http_url_proxy_adapter_test.rb +0 -102
- data/test/io_adapters/identity_adapter_test.rb +0 -8
- data/test/io_adapters/uri_adapter_test.rb +0 -102
- data/test/matchers/have_attached_file_matcher_test.rb +0 -24
- data/test/matchers/validate_attachment_content_type_matcher_test.rb +0 -110
- data/test/matchers/validate_attachment_presence_matcher_test.rb +0 -69
- data/test/matchers/validate_attachment_size_matcher_test.rb +0 -86
- data/test/meta_class_test.rb +0 -32
- data/test/paperclip_missing_attachment_styles_test.rb +0 -90
- data/test/paperclip_test.rb +0 -217
- data/test/plural_cache_test.rb +0 -36
- data/test/schema_test.rb +0 -200
- data/test/storage/fog_test.rb +0 -473
- data/test/storage/s3_live_test.rb +0 -179
- data/test/storage/s3_test.rb +0 -1356
- data/test/style_test.rb +0 -213
- data/test/support/mock_model.rb +0 -2
- data/test/tempfile_factory_test.rb +0 -17
- data/test/url_generator_test.rb +0 -187
- data/test/validators/attachment_content_type_validator_test.rb +0 -324
- data/test/validators_test.rb +0 -61
- /data/{test → spec}/database.yml +0 -0
- /data/{test → spec/support}/fixtures/12k.png +0 -0
- /data/{test → spec/support}/fixtures/50x50.png +0 -0
- /data/{test → spec/support}/fixtures/5k.png +0 -0
- /data/{test → spec/support}/fixtures/animated +0 -0
- /data/{test → spec/support}/fixtures/animated.gif +0 -0
- /data/{test → spec/support}/fixtures/animated.unknown +0 -0
- /data/{test → spec/support}/fixtures/bad.png +0 -0
- /data/{test → spec/support}/fixtures/fog.yml +0 -0
- /data/{test → spec/support}/fixtures/rotated.jpg +0 -0
- /data/{test → spec/support}/fixtures/s3.yml +0 -0
- /data/{test → spec/support}/fixtures/spaced file.png +0 -0
- /data/{test → spec/support}/fixtures/text.txt +0 -0
- /data/{test → spec/support}/fixtures/twopage.pdf +0 -0
- /data/{test → spec/support}/fixtures/uppercase.PNG +0 -0
- /data/{test → spec}/support/mock_interpolator.rb +0 -0
@@ -1,58 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class AbstractAdapterTest < Test::Unit::TestCase
|
4
|
-
class TestAdapter < Paperclip::AbstractAdapter
|
5
|
-
attr_accessor :tempfile
|
6
|
-
|
7
|
-
def content_type
|
8
|
-
Paperclip::ContentTypeDetector.new(path).detect
|
9
|
-
end
|
10
|
-
end
|
11
|
-
|
12
|
-
context "content type from file command" do
|
13
|
-
setup do
|
14
|
-
@adapter = TestAdapter.new
|
15
|
-
@adapter.stubs(:path).returns("image.png")
|
16
|
-
end
|
17
|
-
|
18
|
-
should "return the content type without newline" do
|
19
|
-
assert_equal "image/png", @adapter.content_type
|
20
|
-
end
|
21
|
-
end
|
22
|
-
|
23
|
-
context "nil?" do
|
24
|
-
should "return false" do
|
25
|
-
assert !TestAdapter.new.nil?
|
26
|
-
end
|
27
|
-
end
|
28
|
-
|
29
|
-
context "delegation" do
|
30
|
-
setup do
|
31
|
-
@adapter = TestAdapter.new
|
32
|
-
@adapter.tempfile = stub("Tempfile")
|
33
|
-
end
|
34
|
-
|
35
|
-
[:binmode, :binmode?, :close, :close!, :closed?, :eof?, :path, :rewind, :unlink].each do |method|
|
36
|
-
should "delegate #{method} to @tempfile" do
|
37
|
-
@adapter.tempfile.stubs(method)
|
38
|
-
@adapter.public_send(method)
|
39
|
-
assert_received @adapter.tempfile, method
|
40
|
-
end
|
41
|
-
end
|
42
|
-
end
|
43
|
-
|
44
|
-
should 'get rid of slashes and colons in filenames' do
|
45
|
-
@adapter = TestAdapter.new
|
46
|
-
@adapter.original_filename = "awesome/file:name.png"
|
47
|
-
|
48
|
-
assert_equal "awesome_file_name.png", @adapter.original_filename
|
49
|
-
end
|
50
|
-
|
51
|
-
should 'be an assignment' do
|
52
|
-
assert TestAdapter.new.assignment?
|
53
|
-
end
|
54
|
-
|
55
|
-
should 'not be nil' do
|
56
|
-
assert !TestAdapter.new.nil?
|
57
|
-
end
|
58
|
-
end
|
@@ -1,74 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class DataUriAdapterTest < Test::Unit::TestCase
|
4
|
-
|
5
|
-
def teardown
|
6
|
-
if @subject
|
7
|
-
@subject.close
|
8
|
-
end
|
9
|
-
end
|
10
|
-
|
11
|
-
context "a new instance" do
|
12
|
-
setup do
|
13
|
-
@contents = "data:image/png;base64,#{original_base64_content}"
|
14
|
-
@subject = Paperclip.io_adapters.for(@contents)
|
15
|
-
end
|
16
|
-
|
17
|
-
should "return a file name" do
|
18
|
-
assert_equal "base64.txt", @subject.original_filename
|
19
|
-
end
|
20
|
-
|
21
|
-
should "return a content type" do
|
22
|
-
assert_equal "image/png", @subject.content_type
|
23
|
-
end
|
24
|
-
|
25
|
-
should "return the size of the data" do
|
26
|
-
assert_equal 4456, @subject.size
|
27
|
-
end
|
28
|
-
|
29
|
-
should "generate a correct MD5 hash of the contents" do
|
30
|
-
assert_equal(
|
31
|
-
Digest::MD5.hexdigest(Base64.decode64(original_base64_content)),
|
32
|
-
@subject.fingerprint
|
33
|
-
)
|
34
|
-
end
|
35
|
-
|
36
|
-
should "generate correct fingerprint after read" do
|
37
|
-
fingerprint = Digest::MD5.hexdigest(@subject.read)
|
38
|
-
assert_equal fingerprint, @subject.fingerprint
|
39
|
-
end
|
40
|
-
|
41
|
-
should "generate same fingerprint" do
|
42
|
-
assert_equal @subject.fingerprint, @subject.fingerprint
|
43
|
-
end
|
44
|
-
|
45
|
-
should 'accept a content_type' do
|
46
|
-
@subject.content_type = 'image/png'
|
47
|
-
assert_equal 'image/png', @subject.content_type
|
48
|
-
end
|
49
|
-
|
50
|
-
should 'accept an original_filename' do
|
51
|
-
@subject.original_filename = 'image.png'
|
52
|
-
assert_equal 'image.png', @subject.original_filename
|
53
|
-
end
|
54
|
-
|
55
|
-
should "not generate filenames that include restricted characters" do
|
56
|
-
@subject.original_filename = 'image:restricted.png'
|
57
|
-
assert_equal 'image_restricted.png', @subject.original_filename
|
58
|
-
end
|
59
|
-
|
60
|
-
should "not generate paths that include restricted characters" do
|
61
|
-
@subject.original_filename = 'image:restricted.png'
|
62
|
-
assert_no_match /:/, @subject.path
|
63
|
-
end
|
64
|
-
|
65
|
-
end
|
66
|
-
|
67
|
-
def original_base64_content
|
68
|
-
Base64.encode64(original_file_contents)
|
69
|
-
end
|
70
|
-
|
71
|
-
def original_file_contents
|
72
|
-
@original_file_contents ||= File.read(fixture_file('5k.png'))
|
73
|
-
end
|
74
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class EmptyStringAdapterTest < Test::Unit::TestCase
|
4
|
-
|
5
|
-
context 'a new instance' do
|
6
|
-
setup do
|
7
|
-
@subject = Paperclip.io_adapters.for('')
|
8
|
-
end
|
9
|
-
|
10
|
-
should "return false for a call to nil?" do
|
11
|
-
assert !@subject.nil?
|
12
|
-
end
|
13
|
-
|
14
|
-
should 'return false for a call to assignment?' do
|
15
|
-
assert !@subject.assignment?
|
16
|
-
end
|
17
|
-
end
|
18
|
-
end
|
@@ -1,102 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class HttpUrlProxyAdapterTest < Test::Unit::TestCase
|
4
|
-
context "a new instance" do
|
5
|
-
setup do
|
6
|
-
@open_return = StringIO.new("xxx")
|
7
|
-
@open_return.stubs(:content_type).returns("image/png")
|
8
|
-
Paperclip::HttpUrlProxyAdapter.any_instance.stubs(:download_content).returns(@open_return)
|
9
|
-
@url = "http://thoughtbot.com/images/thoughtbot-logo.png"
|
10
|
-
@subject = Paperclip.io_adapters.for(@url)
|
11
|
-
end
|
12
|
-
|
13
|
-
teardown do
|
14
|
-
@subject.close
|
15
|
-
end
|
16
|
-
|
17
|
-
should "return a file name" do
|
18
|
-
assert_equal "thoughtbot-logo.png", @subject.original_filename
|
19
|
-
end
|
20
|
-
|
21
|
-
should 'close open handle after reading' do
|
22
|
-
assert_equal true, @open_return.closed?
|
23
|
-
end
|
24
|
-
|
25
|
-
should "return a content type" do
|
26
|
-
assert_equal "image/png", @subject.content_type
|
27
|
-
end
|
28
|
-
|
29
|
-
should "return the size of the data" do
|
30
|
-
assert_equal @open_return.size, @subject.size
|
31
|
-
end
|
32
|
-
|
33
|
-
should "generate an MD5 hash of the contents" do
|
34
|
-
assert_equal Digest::MD5.hexdigest("xxx"), @subject.fingerprint
|
35
|
-
end
|
36
|
-
|
37
|
-
should "generate correct fingerprint after read" do
|
38
|
-
fingerprint = Digest::MD5.hexdigest(@subject.read)
|
39
|
-
assert_equal fingerprint, @subject.fingerprint
|
40
|
-
end
|
41
|
-
|
42
|
-
should "generate same fingerprint" do
|
43
|
-
assert_equal @subject.fingerprint, @subject.fingerprint
|
44
|
-
end
|
45
|
-
|
46
|
-
should "return the data contained in the StringIO" do
|
47
|
-
assert_equal "xxx", @subject.read
|
48
|
-
end
|
49
|
-
|
50
|
-
should 'accept a content_type' do
|
51
|
-
@subject.content_type = 'image/png'
|
52
|
-
assert_equal 'image/png', @subject.content_type
|
53
|
-
end
|
54
|
-
|
55
|
-
should 'accept an original_filename' do
|
56
|
-
@subject.original_filename = 'image.png'
|
57
|
-
assert_equal 'image.png', @subject.original_filename
|
58
|
-
end
|
59
|
-
end
|
60
|
-
|
61
|
-
context "a url with query params" do
|
62
|
-
setup do
|
63
|
-
Paperclip::HttpUrlProxyAdapter.any_instance.stubs(:download_content).returns(StringIO.new("x"))
|
64
|
-
@url = "https://github.com/thoughtbot/paperclip?file=test"
|
65
|
-
@subject = Paperclip.io_adapters.for(@url)
|
66
|
-
end
|
67
|
-
|
68
|
-
teardown do
|
69
|
-
@subject.close
|
70
|
-
end
|
71
|
-
|
72
|
-
should "return a file name" do
|
73
|
-
assert_equal "paperclip", @subject.original_filename
|
74
|
-
end
|
75
|
-
end
|
76
|
-
|
77
|
-
context "a url with restricted characters in the filename" do
|
78
|
-
setup do
|
79
|
-
Paperclip::HttpUrlProxyAdapter.any_instance.stubs(:download_content).returns(StringIO.new("x"))
|
80
|
-
@url = "https://github.com/thoughtbot/paper:clip.jpg"
|
81
|
-
@subject = Paperclip.io_adapters.for(@url)
|
82
|
-
end
|
83
|
-
|
84
|
-
teardown do
|
85
|
-
begin
|
86
|
-
@subject.close
|
87
|
-
rescue Exception => e
|
88
|
-
binding.pry
|
89
|
-
true
|
90
|
-
end
|
91
|
-
end
|
92
|
-
|
93
|
-
should "not generate filenames that include restricted characters" do
|
94
|
-
assert_equal "paper_clip.jpg", @subject.original_filename
|
95
|
-
end
|
96
|
-
|
97
|
-
should "not generate paths that include restricted characters" do
|
98
|
-
assert_no_match /:/, @subject.path
|
99
|
-
end
|
100
|
-
end
|
101
|
-
|
102
|
-
end
|
@@ -1,102 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class UriProxyTest < Test::Unit::TestCase
|
4
|
-
context "a new instance" do
|
5
|
-
setup do
|
6
|
-
@open_return = StringIO.new("xxx")
|
7
|
-
@open_return.stubs(:content_type).returns("image/png")
|
8
|
-
Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(@open_return)
|
9
|
-
@uri = URI.parse("http://thoughtbot.com/images/thoughtbot-logo.png")
|
10
|
-
@subject = Paperclip.io_adapters.for(@uri)
|
11
|
-
end
|
12
|
-
|
13
|
-
should "return a file name" do
|
14
|
-
assert_equal "thoughtbot-logo.png", @subject.original_filename
|
15
|
-
end
|
16
|
-
|
17
|
-
should 'close open handle after reading' do
|
18
|
-
assert_equal true, @open_return.closed?
|
19
|
-
end
|
20
|
-
|
21
|
-
should "return a content type" do
|
22
|
-
assert_equal "image/png", @subject.content_type
|
23
|
-
end
|
24
|
-
|
25
|
-
should "return the size of the data" do
|
26
|
-
assert_equal @open_return.size, @subject.size
|
27
|
-
end
|
28
|
-
|
29
|
-
should "generate an MD5 hash of the contents" do
|
30
|
-
assert_equal Digest::MD5.hexdigest("xxx"), @subject.fingerprint
|
31
|
-
end
|
32
|
-
|
33
|
-
should "generate correct fingerprint after read" do
|
34
|
-
fingerprint = Digest::MD5.hexdigest(@subject.read)
|
35
|
-
assert_equal fingerprint, @subject.fingerprint
|
36
|
-
end
|
37
|
-
|
38
|
-
should "generate same fingerprint" do
|
39
|
-
assert_equal @subject.fingerprint, @subject.fingerprint
|
40
|
-
end
|
41
|
-
|
42
|
-
should "return the data contained in the StringIO" do
|
43
|
-
assert_equal "xxx", @subject.read
|
44
|
-
end
|
45
|
-
|
46
|
-
should 'accept a content_type' do
|
47
|
-
@subject.content_type = 'image/png'
|
48
|
-
assert_equal 'image/png', @subject.content_type
|
49
|
-
end
|
50
|
-
|
51
|
-
should 'accept an orgiginal_filename' do
|
52
|
-
@subject.original_filename = 'image.png'
|
53
|
-
assert_equal 'image.png', @subject.original_filename
|
54
|
-
end
|
55
|
-
|
56
|
-
end
|
57
|
-
|
58
|
-
context "a directory index url" do
|
59
|
-
setup do
|
60
|
-
Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(StringIO.new("xxx"))
|
61
|
-
@uri = URI.parse("http://thoughtbot.com")
|
62
|
-
@subject = Paperclip.io_adapters.for(@uri)
|
63
|
-
end
|
64
|
-
|
65
|
-
should "return a file name" do
|
66
|
-
assert_equal "index.html", @subject.original_filename
|
67
|
-
end
|
68
|
-
|
69
|
-
should "return a content type" do
|
70
|
-
assert_equal "text/html", @subject.content_type
|
71
|
-
end
|
72
|
-
end
|
73
|
-
|
74
|
-
context "a url with query params" do
|
75
|
-
setup do
|
76
|
-
Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(StringIO.new("xxx"))
|
77
|
-
@uri = URI.parse("https://github.com/thoughtbot/paperclip?file=test")
|
78
|
-
@subject = Paperclip.io_adapters.for(@uri)
|
79
|
-
end
|
80
|
-
|
81
|
-
should "return a file name" do
|
82
|
-
assert_equal "paperclip", @subject.original_filename
|
83
|
-
end
|
84
|
-
end
|
85
|
-
|
86
|
-
context "a url with restricted characters in the filename" do
|
87
|
-
setup do
|
88
|
-
Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(StringIO.new("xxx"))
|
89
|
-
@uri = URI.parse("https://github.com/thoughtbot/paper:clip.jpg")
|
90
|
-
@subject = Paperclip.io_adapters.for(@uri)
|
91
|
-
end
|
92
|
-
|
93
|
-
should "not generate filenames that include restricted characters" do
|
94
|
-
assert_equal "paper_clip.jpg", @subject.original_filename
|
95
|
-
end
|
96
|
-
|
97
|
-
should "not generate paths that include restricted characters" do
|
98
|
-
assert_no_match /:/, @subject.path
|
99
|
-
end
|
100
|
-
end
|
101
|
-
|
102
|
-
end
|
@@ -1,24 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class HaveAttachedFileMatcherTest < Test::Unit::TestCase
|
4
|
-
context "have_attached_file" do
|
5
|
-
setup do
|
6
|
-
@dummy_class = reset_class "Dummy"
|
7
|
-
reset_table "dummies"
|
8
|
-
@matcher = self.class.have_attached_file(:avatar)
|
9
|
-
end
|
10
|
-
|
11
|
-
context "given a class with no attachment" do
|
12
|
-
should_reject_dummy_class
|
13
|
-
end
|
14
|
-
|
15
|
-
context "given a class with an attachment" do
|
16
|
-
setup do
|
17
|
-
modify_table("dummies"){|d| d.string :avatar_file_name }
|
18
|
-
@dummy_class.has_attached_file :avatar
|
19
|
-
end
|
20
|
-
|
21
|
-
should_accept_dummy_class
|
22
|
-
end
|
23
|
-
end
|
24
|
-
end
|
@@ -1,110 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class ValidateAttachmentContentTypeMatcherTest < Test::Unit::TestCase
|
4
|
-
context "validate_attachment_content_type" do
|
5
|
-
setup do
|
6
|
-
reset_table("dummies") do |d|
|
7
|
-
d.string :title
|
8
|
-
d.string :avatar_file_name
|
9
|
-
d.string :avatar_content_type
|
10
|
-
end
|
11
|
-
@dummy_class = reset_class "Dummy"
|
12
|
-
@dummy_class.has_attached_file :avatar
|
13
|
-
@matcher = self.class.validate_attachment_content_type(:avatar).
|
14
|
-
allowing(%w(image/png image/jpeg)).
|
15
|
-
rejecting(%w(audio/mp3 application/octet-stream))
|
16
|
-
end
|
17
|
-
|
18
|
-
context "given a class with no validation" do
|
19
|
-
should_reject_dummy_class
|
20
|
-
end
|
21
|
-
|
22
|
-
context "given a class with a validation that doesn't match" do
|
23
|
-
setup do
|
24
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{audio/.*}
|
25
|
-
end
|
26
|
-
|
27
|
-
should_reject_dummy_class
|
28
|
-
end
|
29
|
-
|
30
|
-
context "given a class with a matching validation" do
|
31
|
-
setup do
|
32
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
|
33
|
-
end
|
34
|
-
|
35
|
-
should_accept_dummy_class
|
36
|
-
end
|
37
|
-
|
38
|
-
context "given a class with other validations but matching types" do
|
39
|
-
setup do
|
40
|
-
@dummy_class.validates_presence_of :title
|
41
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
|
42
|
-
end
|
43
|
-
|
44
|
-
should_accept_dummy_class
|
45
|
-
end
|
46
|
-
|
47
|
-
context "given a class that matches and a matcher that only specifies 'allowing'" do
|
48
|
-
setup do
|
49
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
|
50
|
-
@matcher = self.class.validate_attachment_content_type(:avatar).
|
51
|
-
allowing(%w(image/png image/jpeg))
|
52
|
-
end
|
53
|
-
|
54
|
-
should_accept_dummy_class
|
55
|
-
end
|
56
|
-
|
57
|
-
context "given a class that does not match and a matcher that only specifies 'allowing'" do
|
58
|
-
setup do
|
59
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{audio/.*}
|
60
|
-
@matcher = self.class.validate_attachment_content_type(:avatar).
|
61
|
-
allowing(%w(image/png image/jpeg))
|
62
|
-
end
|
63
|
-
|
64
|
-
should_reject_dummy_class
|
65
|
-
end
|
66
|
-
|
67
|
-
context "given a class that matches and a matcher that only specifies 'rejecting'" do
|
68
|
-
setup do
|
69
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
|
70
|
-
@matcher = self.class.validate_attachment_content_type(:avatar).
|
71
|
-
rejecting(%w(audio/mp3 application/octet-stream))
|
72
|
-
end
|
73
|
-
|
74
|
-
should_accept_dummy_class
|
75
|
-
end
|
76
|
-
|
77
|
-
context "given a class that does not match and a matcher that only specifies 'rejecting'" do
|
78
|
-
setup do
|
79
|
-
@dummy_class.validates_attachment_content_type :avatar, :content_type => %r{audio/.*}
|
80
|
-
@matcher = self.class.validate_attachment_content_type(:avatar).
|
81
|
-
rejecting(%w(audio/mp3 application/octet-stream))
|
82
|
-
end
|
83
|
-
|
84
|
-
should_reject_dummy_class
|
85
|
-
end
|
86
|
-
|
87
|
-
context "using an :if to control the validation" do
|
88
|
-
setup do
|
89
|
-
@dummy_class.class_eval do
|
90
|
-
validates_attachment_content_type :avatar, :content_type => %r{image/*} , :if => :go
|
91
|
-
attr_accessor :go
|
92
|
-
end
|
93
|
-
@matcher = self.class.validate_attachment_content_type(:avatar).
|
94
|
-
allowing(%w(image/png image/jpeg)).
|
95
|
-
rejecting(%w(audio/mp3 application/octet-stream))
|
96
|
-
@dummy = @dummy_class.new
|
97
|
-
end
|
98
|
-
|
99
|
-
should "run the validation if the control is true" do
|
100
|
-
@dummy.go = true
|
101
|
-
assert_accepts @matcher, @dummy
|
102
|
-
end
|
103
|
-
|
104
|
-
should "not run the validation if the control is false" do
|
105
|
-
@dummy.go = false
|
106
|
-
assert_rejects @matcher, @dummy
|
107
|
-
end
|
108
|
-
end
|
109
|
-
end
|
110
|
-
end
|
@@ -1,69 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class ValidateAttachmentPresenceMatcherTest < Test::Unit::TestCase
|
4
|
-
context "validate_attachment_presence" do
|
5
|
-
setup do
|
6
|
-
reset_table("dummies") do |d|
|
7
|
-
d.string :avatar_file_name
|
8
|
-
end
|
9
|
-
@dummy_class = reset_class "Dummy"
|
10
|
-
@dummy_class.has_attached_file :avatar
|
11
|
-
@matcher = self.class.validate_attachment_presence(:avatar)
|
12
|
-
end
|
13
|
-
|
14
|
-
context "given a class with no validation" do
|
15
|
-
should_reject_dummy_class
|
16
|
-
end
|
17
|
-
|
18
|
-
context "given a class with a matching validation" do
|
19
|
-
setup do
|
20
|
-
@dummy_class.validates_attachment_presence :avatar
|
21
|
-
end
|
22
|
-
|
23
|
-
should_accept_dummy_class
|
24
|
-
end
|
25
|
-
|
26
|
-
context "given an instance with other attachment validations" do
|
27
|
-
setup do
|
28
|
-
reset_table("dummies") do |d|
|
29
|
-
d.string :avatar_file_name
|
30
|
-
d.string :avatar_content_type
|
31
|
-
end
|
32
|
-
|
33
|
-
@dummy_class.class_eval do
|
34
|
-
validates_attachment_presence :avatar
|
35
|
-
validates_attachment_content_type :avatar, :content_type => 'image/gif'
|
36
|
-
end
|
37
|
-
|
38
|
-
@dummy = @dummy_class.new
|
39
|
-
@matcher = self.class.validate_attachment_presence(:avatar)
|
40
|
-
end
|
41
|
-
|
42
|
-
should "it should validate properly" do
|
43
|
-
@dummy.avatar = File.new fixture_file('5k.png')
|
44
|
-
assert_accepts @matcher, @dummy
|
45
|
-
end
|
46
|
-
end
|
47
|
-
|
48
|
-
context "using an :if to control the validation" do
|
49
|
-
setup do
|
50
|
-
@dummy_class.class_eval do
|
51
|
-
validates_attachment_presence :avatar, :if => :go
|
52
|
-
attr_accessor :go
|
53
|
-
end
|
54
|
-
@dummy = @dummy_class.new
|
55
|
-
@dummy.avatar = nil
|
56
|
-
end
|
57
|
-
|
58
|
-
should "run the validation if the control is true" do
|
59
|
-
@dummy.go = true
|
60
|
-
assert_accepts @matcher, @dummy
|
61
|
-
end
|
62
|
-
|
63
|
-
should "not run the validation if the control is false" do
|
64
|
-
@dummy.go = false
|
65
|
-
assert_rejects @matcher, @dummy
|
66
|
-
end
|
67
|
-
end
|
68
|
-
end
|
69
|
-
end
|
@@ -1,86 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class ValidateAttachmentSizeMatcherTest < Test::Unit::TestCase
|
4
|
-
context "validate_attachment_size" do
|
5
|
-
setup do
|
6
|
-
reset_table("dummies") do |d|
|
7
|
-
d.string :avatar_file_name
|
8
|
-
d.integer :avatar_file_size
|
9
|
-
end
|
10
|
-
@dummy_class = reset_class "Dummy"
|
11
|
-
@dummy_class.has_attached_file :avatar
|
12
|
-
end
|
13
|
-
|
14
|
-
context "of limited size" do
|
15
|
-
setup{ @matcher = self.class.validate_attachment_size(:avatar).in(256..1024) }
|
16
|
-
|
17
|
-
context "given a class with no validation" do
|
18
|
-
should_reject_dummy_class
|
19
|
-
end
|
20
|
-
|
21
|
-
context "given a class with a validation that's too high" do
|
22
|
-
setup { @dummy_class.validates_attachment_size :avatar, :in => 256..2048 }
|
23
|
-
should_reject_dummy_class
|
24
|
-
end
|
25
|
-
|
26
|
-
context "given a class with a validation that's too low" do
|
27
|
-
setup { @dummy_class.validates_attachment_size :avatar, :in => 0..1024 }
|
28
|
-
should_reject_dummy_class
|
29
|
-
end
|
30
|
-
|
31
|
-
context "given a class with a validation that matches" do
|
32
|
-
setup { @dummy_class.validates_attachment_size :avatar, :in => 256..1024 }
|
33
|
-
should_accept_dummy_class
|
34
|
-
end
|
35
|
-
end
|
36
|
-
|
37
|
-
context "allowing anything" do
|
38
|
-
setup{ @matcher = self.class.validate_attachment_size(:avatar) }
|
39
|
-
|
40
|
-
context "given a class with an upper limit" do
|
41
|
-
setup { @dummy_class.validates_attachment_size :avatar, :less_than => 1 }
|
42
|
-
should_accept_dummy_class
|
43
|
-
end
|
44
|
-
|
45
|
-
context "given a class with a lower limit" do
|
46
|
-
setup { @dummy_class.validates_attachment_size :avatar, :greater_than => 1 }
|
47
|
-
should_accept_dummy_class
|
48
|
-
end
|
49
|
-
end
|
50
|
-
|
51
|
-
context "using an :if to control the validation" do
|
52
|
-
setup do
|
53
|
-
@dummy_class.class_eval do
|
54
|
-
validates_attachment_size :avatar, :greater_than => 1024, :if => :go
|
55
|
-
attr_accessor :go
|
56
|
-
end
|
57
|
-
@dummy = @dummy_class.new
|
58
|
-
@matcher = self.class.validate_attachment_size(:avatar).greater_than(1024)
|
59
|
-
end
|
60
|
-
|
61
|
-
should "run the validation if the control is true" do
|
62
|
-
@dummy.go = true
|
63
|
-
assert_accepts @matcher, @dummy
|
64
|
-
end
|
65
|
-
|
66
|
-
should "not run the validation if the control is false" do
|
67
|
-
@dummy.go = false
|
68
|
-
assert_rejects @matcher, @dummy
|
69
|
-
end
|
70
|
-
end
|
71
|
-
|
72
|
-
context "post processing" do
|
73
|
-
setup do
|
74
|
-
@dummy_class.validates_attachment_size :avatar, :greater_than => 1024
|
75
|
-
|
76
|
-
@dummy = @dummy_class.new
|
77
|
-
@matcher = self.class.validate_attachment_size(:avatar).greater_than(1024)
|
78
|
-
end
|
79
|
-
|
80
|
-
should "be skipped" do
|
81
|
-
@dummy.avatar.expects(:post_process).never
|
82
|
-
assert_accepts @matcher, @dummy
|
83
|
-
end
|
84
|
-
end
|
85
|
-
end
|
86
|
-
end
|
data/test/meta_class_test.rb
DELETED
@@ -1,32 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class MetaClassTest < Test::Unit::TestCase
|
4
|
-
context "A meta-class of dummy" do
|
5
|
-
setup do
|
6
|
-
rebuild_model
|
7
|
-
@file = File.new(fixture_file("5k.png"), 'rb')
|
8
|
-
end
|
9
|
-
|
10
|
-
teardown { @file.close }
|
11
|
-
|
12
|
-
should "be able to use Paperclip like a normal class" do
|
13
|
-
reset_class("Dummy")
|
14
|
-
@dummy = Dummy.new
|
15
|
-
|
16
|
-
assert_nothing_raised do
|
17
|
-
rebuild_meta_class_of(@dummy)
|
18
|
-
end
|
19
|
-
end
|
20
|
-
|
21
|
-
should "work like any other instance" do
|
22
|
-
reset_class("Dummy")
|
23
|
-
@dummy = Dummy.new
|
24
|
-
rebuild_meta_class_of(@dummy)
|
25
|
-
|
26
|
-
assert_nothing_raised do
|
27
|
-
@dummy.avatar = @file
|
28
|
-
end
|
29
|
-
assert @dummy.save
|
30
|
-
end
|
31
|
-
end
|
32
|
-
end
|