paperclip 3.5.2 → 5.2.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +7 -0
- data/.codeclimate.yml +17 -0
- data/.gitignore +0 -6
- data/.hound.yml +1055 -0
- data/.rubocop.yml +1 -0
- data/.travis.yml +19 -13
- data/Appraisals +4 -16
- data/CONTRIBUTING.md +29 -13
- data/Gemfile +10 -7
- data/LICENSE +1 -3
- data/NEWS +226 -23
- data/README.md +494 -152
- 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 +0 -2
- data/lib/generators/paperclip/templates/paperclip_migration.rb.erb +1 -1
- data/lib/paperclip/attachment.rb +160 -47
- data/lib/paperclip/attachment_registry.rb +4 -1
- data/lib/paperclip/callbacks.rb +13 -1
- data/lib/paperclip/content_type_detector.rb +26 -24
- 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 +10 -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 +14 -10
- data/lib/paperclip/interpolations/plural_cache.rb +6 -5
- data/lib/paperclip/interpolations.rb +27 -14
- data/lib/paperclip/io_adapters/abstract_adapter.rb +28 -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 +8 -8
- 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 +89 -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 +47 -22
- data/lib/paperclip/storage/s3.rb +144 -73
- data/lib/paperclip/style.rb +8 -2
- data/lib/paperclip/tempfile_factory.rb +6 -4
- data/lib/paperclip/thumbnail.rb +26 -14
- data/lib/paperclip/url_generator.rb +25 -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 +12 -3
- data/lib/paperclip/version.rb +3 -1
- data/lib/paperclip.rb +31 -11
- data/lib/tasks/paperclip.rake +34 -5
- data/paperclip.gemspec +18 -17
- data/shoulda_macros/paperclip.rb +13 -3
- data/{test → spec}/database.yml +0 -0
- 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} +524 -400
- data/{test/content_type_detector_test.rb → spec/paperclip/content_type_detector_spec.rb} +17 -19
- data/{test/file_command_content_type_detector_test.rb → spec/paperclip/file_command_content_type_detector_spec.rb} +7 -6
- 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} +141 -133
- data/{test/interpolations_test.rb → spec/paperclip/interpolations_spec.rb} +70 -46
- data/spec/paperclip/io_adapters/abstract_adapter_spec.rb +101 -0
- data/{test/io_adapters/attachment_adapter_test.rb → spec/paperclip/io_adapters/attachment_adapter_spec.rb} +38 -34
- 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/spec/paperclip/io_adapters/file_adapter_spec.rb +131 -0
- 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 +79 -0
- data/spec/paperclip/meta_class_spec.rb +30 -0
- data/spec/paperclip/paperclip_missing_attachment_styles_spec.rb +84 -0
- data/{test/paperclip_test.rb → spec/paperclip/paperclip_spec.rb} +46 -71
- 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} +5 -5
- 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 +561 -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} +150 -131
- 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/{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/spec/support/fixtures/empty.html +1 -0
- data/spec/support/fixtures/empty.xlsx +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/spec/support/fixtures/spaced file.jpg +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/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_interpolator.rb +0 -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 +262 -297
- 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 -77
- data/test/filename_cleaner_test.rb +0 -14
- data/test/generator_test.rb +0 -80
- data/test/geometry_detector_test.rb +0 -24
- data/test/has_attached_file_test.rb +0 -125
- data/test/helper.rb +0 -215
- data/test/io_adapters/abstract_adapter_test.rb +0 -58
- data/test/io_adapters/data_uri_adapter_test.rb +0 -67
- data/test/io_adapters/empty_string_adapter_test.rb +0 -17
- data/test/io_adapters/file_adapter_test.rb +0 -119
- 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/plural_cache_test.rb +0 -36
- data/test/schema_test.rb +0 -200
- data/test/storage/fog_test.rb +0 -453
- data/test/storage/s3_live_test.rb +0 -179
- data/test/storage/s3_test.rb +0 -1348
- data/test/style_test.rb +0 -213
- data/test/support/mock_model.rb +0 -2
- data/test/tempfile_factory_test.rb +0 -13
- data/test/url_generator_test.rb +0 -187
- data/test/validators/attachment_content_type_validator_test.rb +0 -323
- data/test/validators_test.rb +0 -32
data/RUNNING_TESTS.md
DELETED
data/cucumber/paperclip_steps.rb
DELETED
@@ -1,6 +0,0 @@
|
|
1
|
-
When /^I attach an? "([^\"]*)" "([^\"]*)" file to an? "([^\"]*)" on S3$/ do |attachment, extension, model|
|
2
|
-
stub_paperclip_s3(model, attachment, extension)
|
3
|
-
attach_file attachment,
|
4
|
-
"features/support/paperclip/#{model.gsub(" ", "_").underscore}/#{attachment}.#{extension}"
|
5
|
-
end
|
6
|
-
|
data/gemfiles/3.0.gemfile
DELETED
@@ -1,11 +0,0 @@
|
|
1
|
-
# This file was generated by Appraisal
|
2
|
-
|
3
|
-
source "https://rubygems.org"
|
4
|
-
|
5
|
-
gem "jruby-openssl", :platform=>:jruby
|
6
|
-
gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
|
7
|
-
gem "sqlite3", :platform=>:ruby
|
8
|
-
gem "rails", "~> 3.0.15"
|
9
|
-
gem "paperclip", :path=>"../"
|
10
|
-
|
11
|
-
gemspec :path=>"../"
|
data/gemfiles/3.1.gemfile
DELETED
@@ -1,11 +0,0 @@
|
|
1
|
-
# This file was generated by Appraisal
|
2
|
-
|
3
|
-
source "https://rubygems.org"
|
4
|
-
|
5
|
-
gem "jruby-openssl", :platform=>:jruby
|
6
|
-
gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
|
7
|
-
gem "sqlite3", :platform=>:ruby
|
8
|
-
gem "rails", "~> 3.1.6"
|
9
|
-
gem "paperclip", :path=>"../"
|
10
|
-
|
11
|
-
gemspec :path=>"../"
|
data/gemfiles/3.2.gemfile
DELETED
@@ -1,11 +0,0 @@
|
|
1
|
-
# This file was generated by Appraisal
|
2
|
-
|
3
|
-
source "https://rubygems.org"
|
4
|
-
|
5
|
-
gem "jruby-openssl", :platform=>:jruby
|
6
|
-
gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
|
7
|
-
gem "sqlite3", :platform=>:ruby
|
8
|
-
gem "rails", "~> 3.2.6"
|
9
|
-
gem "paperclip", :path=>"../"
|
10
|
-
|
11
|
-
gemspec :path=>"../"
|
data/gemfiles/4.0.gemfile
DELETED
@@ -1,11 +0,0 @@
|
|
1
|
-
# This file was generated by Appraisal
|
2
|
-
|
3
|
-
source "https://rubygems.org"
|
4
|
-
|
5
|
-
gem "jruby-openssl", :platform=>:jruby
|
6
|
-
gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
|
7
|
-
gem "sqlite3", :platform=>:ruby
|
8
|
-
gem "rails", "~> 4.0.0"
|
9
|
-
gem "paperclip", :path=>"../"
|
10
|
-
|
11
|
-
gemspec :path=>"../"
|
@@ -1,12 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class AttachmentDefinitionsTest < Test::Unit::TestCase
|
4
|
-
should 'return all of the attachments on the class' do
|
5
|
-
reset_class "Dummy"
|
6
|
-
Dummy.has_attached_file :avatar, {:path => "abc"}
|
7
|
-
Dummy.has_attached_file :other_attachment, {:url => "123"}
|
8
|
-
expected = {:avatar => {:path => "abc"}, :other_attachment => {:url => "123"}}
|
9
|
-
|
10
|
-
assert_equal expected, Dummy.attachment_definitions
|
11
|
-
end
|
12
|
-
end
|
@@ -1,77 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
require 'paperclip/attachment_registry'
|
3
|
-
|
4
|
-
class AttachmentRegistryTest < Test::Unit::TestCase
|
5
|
-
def setup
|
6
|
-
Paperclip::AttachmentRegistry.clear
|
7
|
-
end
|
8
|
-
|
9
|
-
context '.names_for' do
|
10
|
-
should 'include attachment names for the given class' do
|
11
|
-
foo = Class.new
|
12
|
-
Paperclip::AttachmentRegistry.register(foo, :avatar, {})
|
13
|
-
|
14
|
-
assert_equal [:avatar], Paperclip::AttachmentRegistry.names_for(foo)
|
15
|
-
end
|
16
|
-
|
17
|
-
should 'not include attachment names for other classes' do
|
18
|
-
foo = Class.new
|
19
|
-
bar = Class.new
|
20
|
-
Paperclip::AttachmentRegistry.register(foo, :avatar, {})
|
21
|
-
Paperclip::AttachmentRegistry.register(bar, :lover, {})
|
22
|
-
|
23
|
-
assert_equal [:lover], Paperclip::AttachmentRegistry.names_for(bar)
|
24
|
-
end
|
25
|
-
|
26
|
-
should 'produce the empty array for a missing key' do
|
27
|
-
assert_empty Paperclip::AttachmentRegistry.names_for(Class.new)
|
28
|
-
end
|
29
|
-
end
|
30
|
-
|
31
|
-
context '.each_definition' do
|
32
|
-
should 'call the block with the class, attachment name, and options' do
|
33
|
-
foo = Class.new
|
34
|
-
expected_accumulations = [
|
35
|
-
[foo, :avatar, { yo: 'greeting' }],
|
36
|
-
[foo, :greeter, { ciao: 'greeting' }]
|
37
|
-
]
|
38
|
-
expected_accumulations.each do |args|
|
39
|
-
Paperclip::AttachmentRegistry.register(*args)
|
40
|
-
end
|
41
|
-
accumulations = []
|
42
|
-
|
43
|
-
Paperclip::AttachmentRegistry.each_definition do |*args|
|
44
|
-
accumulations << args
|
45
|
-
end
|
46
|
-
|
47
|
-
assert_equal expected_accumulations, accumulations
|
48
|
-
end
|
49
|
-
end
|
50
|
-
|
51
|
-
context '.definitions_for' do
|
52
|
-
should 'produce the attachment name and options' do
|
53
|
-
expected_definitions = {
|
54
|
-
avatar: { yo: 'greeting' },
|
55
|
-
greeter: { ciao: 'greeting' }
|
56
|
-
}
|
57
|
-
foo = Class.new
|
58
|
-
Paperclip::AttachmentRegistry.register(foo, :avatar, { yo: 'greeting' })
|
59
|
-
Paperclip::AttachmentRegistry.register(foo, :greeter, { ciao: 'greeting' })
|
60
|
-
|
61
|
-
definitions = Paperclip::AttachmentRegistry.definitions_for(foo)
|
62
|
-
|
63
|
-
assert_equal expected_definitions, definitions
|
64
|
-
end
|
65
|
-
end
|
66
|
-
|
67
|
-
context '.clear' do
|
68
|
-
should 'remove all of the existing attachment definitions' do
|
69
|
-
foo = Class.new
|
70
|
-
Paperclip::AttachmentRegistry.register(foo, :greeter, { ciao: 'greeting' })
|
71
|
-
|
72
|
-
Paperclip::AttachmentRegistry.clear
|
73
|
-
|
74
|
-
assert_empty Paperclip::AttachmentRegistry.names_for(foo)
|
75
|
-
end
|
76
|
-
end
|
77
|
-
end
|
@@ -1,14 +0,0 @@
|
|
1
|
-
# encoding: utf-8
|
2
|
-
require './test/helper'
|
3
|
-
|
4
|
-
class FilenameCleanerTest < Test::Unit::TestCase
|
5
|
-
should 'convert invalid characters to underscores' do
|
6
|
-
cleaner = Paperclip::FilenameCleaner.new(/[aeiou]/)
|
7
|
-
assert_equal "b_s_b_ll", cleaner.call("baseball")
|
8
|
-
end
|
9
|
-
|
10
|
-
should 'not convert anything if the character regex is nil' do
|
11
|
-
cleaner = Paperclip::FilenameCleaner.new(nil)
|
12
|
-
assert_equal "baseball", cleaner.call("baseball")
|
13
|
-
end
|
14
|
-
end
|
data/test/generator_test.rb
DELETED
@@ -1,80 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
require 'rails/generators'
|
3
|
-
require 'generators/paperclip/paperclip_generator'
|
4
|
-
|
5
|
-
class GeneratorTest < Rails::Generators::TestCase
|
6
|
-
tests PaperclipGenerator
|
7
|
-
destination File.expand_path("../tmp", File.dirname(__FILE__))
|
8
|
-
setup :prepare_destination
|
9
|
-
|
10
|
-
context 'running migration' do
|
11
|
-
context 'with single attachment name' do
|
12
|
-
setup do
|
13
|
-
run_generator %w(user avatar)
|
14
|
-
end
|
15
|
-
|
16
|
-
should 'create a correct migration file' do
|
17
|
-
assert_migration 'db/migrate/add_attachment_avatar_to_users.rb' do |migration|
|
18
|
-
assert_match /class AddAttachmentAvatarToUsers/, migration
|
19
|
-
|
20
|
-
assert_class_method :up, migration do |up|
|
21
|
-
expected = <<-migration
|
22
|
-
change_table :users do |t|
|
23
|
-
t.attachment :avatar
|
24
|
-
end
|
25
|
-
migration
|
26
|
-
|
27
|
-
assert_equal expected.squish, up.squish
|
28
|
-
end
|
29
|
-
|
30
|
-
assert_class_method :down, migration do |down|
|
31
|
-
expected = <<-migration
|
32
|
-
drop_attached_file :users, :avatar
|
33
|
-
migration
|
34
|
-
|
35
|
-
assert_equal expected.squish, down.squish
|
36
|
-
end
|
37
|
-
end
|
38
|
-
end
|
39
|
-
end
|
40
|
-
|
41
|
-
context 'with multiple attachment names' do
|
42
|
-
setup do
|
43
|
-
run_generator %w(user avatar photo)
|
44
|
-
end
|
45
|
-
|
46
|
-
should 'create a correct migration file' do
|
47
|
-
assert_migration 'db/migrate/add_attachment_avatar_photo_to_users.rb' do |migration|
|
48
|
-
assert_match /class AddAttachmentAvatarPhotoToUsers/, migration
|
49
|
-
|
50
|
-
assert_class_method :up, migration do |up|
|
51
|
-
expected = <<-migration
|
52
|
-
change_table :users do |t|
|
53
|
-
t.attachment :avatar
|
54
|
-
t.attachment :photo
|
55
|
-
end
|
56
|
-
migration
|
57
|
-
|
58
|
-
assert_equal expected.squish, up.squish
|
59
|
-
end
|
60
|
-
|
61
|
-
assert_class_method :down, migration do |down|
|
62
|
-
expected = <<-migration
|
63
|
-
drop_attached_file :users, :avatar
|
64
|
-
drop_attached_file :users, :photo
|
65
|
-
migration
|
66
|
-
|
67
|
-
assert_equal expected.squish, down.squish
|
68
|
-
end
|
69
|
-
end
|
70
|
-
end
|
71
|
-
end
|
72
|
-
|
73
|
-
context 'without required arguments' do
|
74
|
-
should 'not create the migration' do
|
75
|
-
silence_stream(STDERR) { run_generator %w() }
|
76
|
-
assert_no_migration 'db/migrate/add_attachment_avatar_to_users.rb'
|
77
|
-
end
|
78
|
-
end
|
79
|
-
end
|
80
|
-
end
|
@@ -1,24 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
|
3
|
-
class GeometryDetectorTest < Test::Unit::TestCase
|
4
|
-
should 'identify an image and extract its dimensions' do
|
5
|
-
Paperclip::GeometryParser.stubs(:new).with("434x66,").returns(stub(:make => :correct))
|
6
|
-
file = fixture_file("5k.png")
|
7
|
-
factory = Paperclip::GeometryDetector.new(file)
|
8
|
-
|
9
|
-
output = factory.make
|
10
|
-
|
11
|
-
assert_equal :correct, output
|
12
|
-
end
|
13
|
-
|
14
|
-
should 'identify an image and extract its dimensions and orientation' do
|
15
|
-
Paperclip::GeometryParser.stubs(:new).with("300x200,6").returns(stub(:make => :correct))
|
16
|
-
file = fixture_file("rotated.jpg")
|
17
|
-
factory = Paperclip::GeometryDetector.new(file)
|
18
|
-
|
19
|
-
output = factory.make
|
20
|
-
|
21
|
-
assert_equal :correct, output
|
22
|
-
end
|
23
|
-
end
|
24
|
-
|
@@ -1,125 +0,0 @@
|
|
1
|
-
require './test/helper'
|
2
|
-
require 'paperclip/has_attached_file'
|
3
|
-
|
4
|
-
class HasAttachedFileTest < Test::Unit::TestCase
|
5
|
-
context '#define_on' do
|
6
|
-
should 'define a setter on the class object' do
|
7
|
-
assert_adding_attachment('avatar').defines_method('avatar=')
|
8
|
-
end
|
9
|
-
|
10
|
-
should 'define a getter on the class object' do
|
11
|
-
assert_adding_attachment('avatar').defines_method('avatar')
|
12
|
-
end
|
13
|
-
|
14
|
-
should 'define a query on the class object' do
|
15
|
-
assert_adding_attachment('avatar').defines_method('avatar?')
|
16
|
-
end
|
17
|
-
|
18
|
-
should 'define a method on the class to get all of its attachments' do
|
19
|
-
assert_adding_attachment('avatar').defines_class_method('attachment_definitions')
|
20
|
-
end
|
21
|
-
|
22
|
-
should 'flush errors as part of validations' do
|
23
|
-
assert_adding_attachment('avatar').defines_validation
|
24
|
-
end
|
25
|
-
|
26
|
-
should 'register the attachment with Paperclip::AttachmentRegistry' do
|
27
|
-
assert_adding_attachment('avatar').registers_attachment
|
28
|
-
end
|
29
|
-
|
30
|
-
should 'define an after_save callback' do
|
31
|
-
assert_adding_attachment('avatar').defines_callback('after_save')
|
32
|
-
end
|
33
|
-
|
34
|
-
should 'define a before_destroy callback' do
|
35
|
-
assert_adding_attachment('avatar').defines_callback('before_destroy')
|
36
|
-
end
|
37
|
-
|
38
|
-
should 'define an after_destroy callback' do
|
39
|
-
assert_adding_attachment('avatar').defines_callback('after_destroy')
|
40
|
-
end
|
41
|
-
|
42
|
-
should 'define the Paperclip-specific callbacks' do
|
43
|
-
assert_adding_attachment('avatar').defines_callback('define_paperclip_callbacks')
|
44
|
-
end
|
45
|
-
end
|
46
|
-
|
47
|
-
private
|
48
|
-
|
49
|
-
def assert_adding_attachment(attachment_name)
|
50
|
-
AttachmentAdder.new(attachment_name)
|
51
|
-
end
|
52
|
-
|
53
|
-
class AttachmentAdder
|
54
|
-
include Mocha::API
|
55
|
-
include Test::Unit::Assertions
|
56
|
-
|
57
|
-
def initialize(attachment_name)
|
58
|
-
@attachment_name = attachment_name
|
59
|
-
end
|
60
|
-
|
61
|
-
def defines_method(method_name)
|
62
|
-
a_class = stub_class
|
63
|
-
|
64
|
-
Paperclip::HasAttachedFile.define_on(a_class, @attachment_name, {})
|
65
|
-
|
66
|
-
assert_received(a_class, :define_method) do |expect|
|
67
|
-
expect.with(method_name)
|
68
|
-
end
|
69
|
-
end
|
70
|
-
|
71
|
-
def defines_class_method(method_name)
|
72
|
-
a_class = stub_class
|
73
|
-
a_class.class.stubs(:define_method)
|
74
|
-
|
75
|
-
Paperclip::HasAttachedFile.define_on(a_class, @attachment_name, {})
|
76
|
-
|
77
|
-
assert_received(a_class, :extend) do |expect|
|
78
|
-
expect.with(Paperclip::HasAttachedFile::ClassMethods)
|
79
|
-
end
|
80
|
-
end
|
81
|
-
|
82
|
-
def defines_validation
|
83
|
-
a_class = stub_class
|
84
|
-
|
85
|
-
Paperclip::HasAttachedFile.define_on(a_class, @attachment_name, {})
|
86
|
-
|
87
|
-
assert_received(a_class, :validates_each) do |expect|
|
88
|
-
expect.with(@attachment_name)
|
89
|
-
end
|
90
|
-
end
|
91
|
-
|
92
|
-
def registers_attachment
|
93
|
-
a_class = stub_class
|
94
|
-
Paperclip::AttachmentRegistry.stubs(:register)
|
95
|
-
|
96
|
-
Paperclip::HasAttachedFile.define_on(a_class, @attachment_name, {size: 1})
|
97
|
-
|
98
|
-
assert_received(Paperclip::AttachmentRegistry, :register) do |expect|
|
99
|
-
expect.with(a_class, @attachment_name, {size: 1})
|
100
|
-
end
|
101
|
-
end
|
102
|
-
|
103
|
-
def defines_callback(callback_name)
|
104
|
-
a_class = stub_class
|
105
|
-
|
106
|
-
Paperclip::HasAttachedFile.define_on(a_class, @attachment_name, {})
|
107
|
-
|
108
|
-
assert_received(a_class, callback_name.to_sym)
|
109
|
-
end
|
110
|
-
|
111
|
-
private
|
112
|
-
|
113
|
-
def stub_class
|
114
|
-
stub('class',
|
115
|
-
validates_each: nil,
|
116
|
-
define_method: nil,
|
117
|
-
after_save: nil,
|
118
|
-
before_destroy: nil,
|
119
|
-
after_destroy: nil,
|
120
|
-
define_paperclip_callbacks: nil,
|
121
|
-
extend: nil,
|
122
|
-
name: 'Billy')
|
123
|
-
end
|
124
|
-
end
|
125
|
-
end
|
data/test/helper.rb
DELETED
@@ -1,215 +0,0 @@
|
|
1
|
-
require 'rubygems'
|
2
|
-
require 'tempfile'
|
3
|
-
require 'pathname'
|
4
|
-
require 'test/unit'
|
5
|
-
require 'active_record'
|
6
|
-
require 'active_record/version'
|
7
|
-
require 'active_support'
|
8
|
-
require 'active_support/core_ext'
|
9
|
-
require 'shoulda'
|
10
|
-
require 'mocha/setup'
|
11
|
-
require 'bourne'
|
12
|
-
require 'shoulda/context'
|
13
|
-
require 'mime/types'
|
14
|
-
require 'pathname'
|
15
|
-
require 'ostruct'
|
16
|
-
|
17
|
-
begin
|
18
|
-
require 'pry'
|
19
|
-
rescue LoadError
|
20
|
-
# Pry is not available, just ignore.
|
21
|
-
end
|
22
|
-
|
23
|
-
puts "Testing against version #{ActiveRecord::VERSION::STRING}"
|
24
|
-
|
25
|
-
`ruby -e 'exit 0'` # Prime $? with a value.
|
26
|
-
|
27
|
-
begin
|
28
|
-
require 'ruby-debug'
|
29
|
-
rescue LoadError => e
|
30
|
-
puts "debugger disabled"
|
31
|
-
end
|
32
|
-
|
33
|
-
ROOT = Pathname(File.expand_path(File.join(File.dirname(__FILE__), '..')))
|
34
|
-
|
35
|
-
class Test::Unit::TestCase
|
36
|
-
def setup
|
37
|
-
silence_warnings do
|
38
|
-
Object.const_set(:Rails, stub('Rails'))
|
39
|
-
Rails.stubs(:root).returns(Pathname.new(ROOT).join('tmp'))
|
40
|
-
Rails.stubs(:env).returns('test')
|
41
|
-
Rails.stubs(:const_defined?).with(:Railtie).returns(false)
|
42
|
-
end
|
43
|
-
end
|
44
|
-
end
|
45
|
-
|
46
|
-
$LOAD_PATH << File.join(ROOT, 'lib')
|
47
|
-
$LOAD_PATH << File.join(ROOT, 'lib', 'paperclip')
|
48
|
-
|
49
|
-
require File.join(ROOT, 'lib', 'paperclip.rb')
|
50
|
-
|
51
|
-
require './shoulda_macros/paperclip'
|
52
|
-
|
53
|
-
FIXTURES_DIR = File.join(File.dirname(__FILE__), "fixtures")
|
54
|
-
config = YAML::load(IO.read(File.dirname(__FILE__) + '/database.yml'))
|
55
|
-
ActiveRecord::Base.logger = Logger.new(File.dirname(__FILE__) + "/debug.log")
|
56
|
-
ActiveRecord::Base.establish_connection(config['test'])
|
57
|
-
Paperclip.options[:logger] = ActiveRecord::Base.logger
|
58
|
-
|
59
|
-
def using_protected_attributes?
|
60
|
-
ActiveRecord::VERSION::MAJOR < 4
|
61
|
-
end
|
62
|
-
|
63
|
-
def require_everything_in_directory(directory_name)
|
64
|
-
Dir[File.join(File.dirname(__FILE__), directory_name, '*')].each do |f|
|
65
|
-
require f
|
66
|
-
end
|
67
|
-
end
|
68
|
-
|
69
|
-
require_everything_in_directory('support')
|
70
|
-
|
71
|
-
def reset_class class_name
|
72
|
-
ActiveRecord::Base.send(:include, Paperclip::Glue)
|
73
|
-
Object.send(:remove_const, class_name) rescue nil
|
74
|
-
klass = Object.const_set(class_name, Class.new(ActiveRecord::Base))
|
75
|
-
|
76
|
-
klass.class_eval do
|
77
|
-
include Paperclip::Glue
|
78
|
-
end
|
79
|
-
|
80
|
-
klass.reset_column_information
|
81
|
-
klass.connection_pool.clear_table_cache!(klass.table_name) if klass.connection_pool.respond_to?(:clear_table_cache!)
|
82
|
-
klass.connection.schema_cache.clear_table_cache!(klass.table_name) if klass.connection.respond_to?(:schema_cache)
|
83
|
-
klass
|
84
|
-
end
|
85
|
-
|
86
|
-
def reset_table table_name, &block
|
87
|
-
block ||= lambda { |table| true }
|
88
|
-
ActiveRecord::Base.connection.create_table :dummies, {:force => true}, &block
|
89
|
-
end
|
90
|
-
|
91
|
-
def modify_table table_name, &block
|
92
|
-
ActiveRecord::Base.connection.change_table :dummies, &block
|
93
|
-
end
|
94
|
-
|
95
|
-
def rebuild_model options = {}
|
96
|
-
ActiveRecord::Base.connection.create_table :dummies, :force => true do |table|
|
97
|
-
table.column :title, :string
|
98
|
-
table.column :other, :string
|
99
|
-
table.column :avatar_file_name, :string
|
100
|
-
table.column :avatar_content_type, :string
|
101
|
-
table.column :avatar_file_size, :integer
|
102
|
-
table.column :avatar_updated_at, :datetime
|
103
|
-
table.column :avatar_fingerprint, :string
|
104
|
-
end
|
105
|
-
rebuild_class options
|
106
|
-
end
|
107
|
-
|
108
|
-
def rebuild_class options = {}
|
109
|
-
reset_class("Dummy").tap do |klass|
|
110
|
-
klass.has_attached_file :avatar, options
|
111
|
-
Paperclip.reset_duplicate_clash_check!
|
112
|
-
end
|
113
|
-
end
|
114
|
-
|
115
|
-
def rebuild_meta_class_of obj, options = {}
|
116
|
-
(class << obj; self; end).tap do |metaklass|
|
117
|
-
metaklass.has_attached_file :avatar, options
|
118
|
-
Paperclip.reset_duplicate_clash_check!
|
119
|
-
end
|
120
|
-
end
|
121
|
-
|
122
|
-
class FakeModel
|
123
|
-
attr_accessor :avatar_file_name,
|
124
|
-
:avatar_file_size,
|
125
|
-
:avatar_updated_at,
|
126
|
-
:avatar_content_type,
|
127
|
-
:avatar_fingerprint,
|
128
|
-
:id
|
129
|
-
|
130
|
-
def errors
|
131
|
-
@errors ||= []
|
132
|
-
end
|
133
|
-
|
134
|
-
def run_paperclip_callbacks name, *args
|
135
|
-
end
|
136
|
-
|
137
|
-
def valid?
|
138
|
-
errors.empty?
|
139
|
-
end
|
140
|
-
end
|
141
|
-
|
142
|
-
def attachment(options={})
|
143
|
-
Paperclip::Attachment.new(:avatar, FakeModel.new, options)
|
144
|
-
end
|
145
|
-
|
146
|
-
def silence_warnings
|
147
|
-
old_verbose, $VERBOSE = $VERBOSE, nil
|
148
|
-
yield
|
149
|
-
ensure
|
150
|
-
$VERBOSE = old_verbose
|
151
|
-
end
|
152
|
-
|
153
|
-
def should_accept_dummy_class
|
154
|
-
should "accept the class" do
|
155
|
-
assert_accepts @matcher, @dummy_class
|
156
|
-
end
|
157
|
-
|
158
|
-
should "accept an instance of that class" do
|
159
|
-
assert_accepts @matcher, @dummy_class.new
|
160
|
-
end
|
161
|
-
end
|
162
|
-
|
163
|
-
def should_reject_dummy_class
|
164
|
-
should "reject the class" do
|
165
|
-
assert_rejects @matcher, @dummy_class
|
166
|
-
end
|
167
|
-
|
168
|
-
should "reject an instance of that class" do
|
169
|
-
assert_rejects @matcher, @dummy_class.new
|
170
|
-
end
|
171
|
-
end
|
172
|
-
|
173
|
-
def with_exitstatus_returning(code)
|
174
|
-
saved_exitstatus = $?.nil? ? 0 : $?.exitstatus
|
175
|
-
begin
|
176
|
-
`ruby -e 'exit #{code.to_i}'`
|
177
|
-
yield
|
178
|
-
ensure
|
179
|
-
`ruby -e 'exit #{saved_exitstatus.to_i}'`
|
180
|
-
end
|
181
|
-
end
|
182
|
-
|
183
|
-
def fixture_file(filename)
|
184
|
-
File.join(File.dirname(__FILE__), 'fixtures', filename)
|
185
|
-
end
|
186
|
-
|
187
|
-
def assert_success_response(url)
|
188
|
-
Net::HTTP.get_response(URI.parse(url)) do |response|
|
189
|
-
assert_equal "200", response.code,
|
190
|
-
"Expected HTTP response code 200, got #{response.code}"
|
191
|
-
end
|
192
|
-
end
|
193
|
-
|
194
|
-
def assert_not_found_response(url)
|
195
|
-
Net::HTTP.get_response(URI.parse(url)) do |response|
|
196
|
-
assert_equal "404", response.code,
|
197
|
-
"Expected HTTP response code 404, got #{response.code}"
|
198
|
-
end
|
199
|
-
end
|
200
|
-
|
201
|
-
def assert_file_exists(path)
|
202
|
-
assert File.exists?(path), %(Expect "#{path}" to be exists.)
|
203
|
-
end
|
204
|
-
|
205
|
-
def assert_file_not_exists(path)
|
206
|
-
assert !File.exists?(path), %(Expect "#{path}" to not exists.)
|
207
|
-
end
|
208
|
-
|
209
|
-
def assert_frame_dimensions(range, frames)
|
210
|
-
frames.each_with_index do |frame, frame_index|
|
211
|
-
frame.split('x').each_with_index do |dimension, dimension_index |
|
212
|
-
assert range.include?(dimension.to_i), "Frame #{frame_index}[#{dimension_index}] should have been within #{range.inspect}, but was #{dimension}"
|
213
|
-
end
|
214
|
-
end
|
215
|
-
end
|
@@ -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
|
-
[: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
|