avatar 0.0.4 → 0.0.5

data/test/lib/paperclip/test/test_integration.rb

@@ -0,0 +1,76 @@
+require 'test/helper.rb'
+
+class PaperclipTest < Test::Unit::TestCase
+  context "A model with an attachment" do
+    setup do
+      rebuild_model :styles => { :large => "300x300>",
+                                 :medium => "100x100",
+                                 :thumb => ["32x32#", :gif] },
+                    :default_style => :medium,
+                    :url => "/:attachment/:class/:style/:id/:basename.:extension",
+                    :path => ":rails_root/tmp/:attachment/:class/:style/:id/:basename.:extension"
+    end
+
+    should "integrate" do
+      @dummy     = Dummy.new
+      @file      = File.new(File.join(FIXTURES_DIR, "5k.png"))
+      @bad_file  = File.new(File.join(FIXTURES_DIR, "bad.png"))
+
+      assert @dummy.avatar = @file
+      assert @dummy.valid?
+      assert @dummy.save
+
+      [["100x15", nil],
+       ["434x66", :original],
+       ["300x46", :large],
+       ["100x15", :medium],
+       ["32x32", :thumb]].each do |geo, style|
+        cmd = %Q[identify -format "%wx%h" #{@dummy.avatar.to_io(style).path}]
+        assert_equal geo, `#{cmd}`.chomp, cmd
+      end
+
+      saved_paths = [:thumb, :medium, :large, :original].collect{|s| @dummy.avatar.to_io(s).path }
+
+      @d2 = Dummy.find(@dummy.id)
+      assert_equal "100x15", `identify -format "%wx%h" #{@dummy.avatar.to_io.path}`.chomp
+      assert_equal "434x66", `identify -format "%wx%h" #{@dummy.avatar.to_io(:original).path}`.chomp
+      assert_equal "300x46", `identify -format "%wx%h" #{@d2.avatar.to_io(:large).path}`.chomp
+      assert_equal "100x15", `identify -format "%wx%h" #{@d2.avatar.to_io(:medium).path}`.chomp
+      assert_equal "32x32",  `identify -format "%wx%h" #{@d2.avatar.to_io(:thumb).path}`.chomp
+
+      @dummy.avatar = nil
+      assert_nil @dummy.avatar_file_name
+      assert @dummy.valid?
+      assert @dummy.save
+
+      saved_paths.each do |p|
+        assert ! File.exists?(p)
+      end
+
+      @d2 = Dummy.find(@dummy.id)
+      assert_nil @d2.avatar_file_name
+
+      @d2.avatar = @bad_file
+      assert ! @d2.valid?
+      @d2.avatar = nil
+      assert @d2.valid?
+
+      Dummy.validates_attachment_presence :avatar
+      @d3 = Dummy.find(@d2.id)
+      @d3.avatar = @file
+      assert   @d3.valid?
+      @d3.avatar = @bad_file
+      assert ! @d3.valid?
+      @d3.avatar = nil
+      assert ! @d3.valid?
+
+      @dummy.avatar = @file
+      assert @dummy.save
+      @dummy.avatar = nil
+      assert_nil @dummy.avatar_file_name
+      @dummy.reload
+      assert_equal "5k.png", @dummy.avatar_file_name
+    end
+  end
+end
+

data/test/lib/paperclip/test/test_iostream.rb

@@ -0,0 +1,60 @@
+require 'rubygems'
+require 'test/unit'
+require 'stringio'
+require 'tempfile'
+require 'shoulda'
+
+require File.join(File.dirname(__FILE__), '..', 'lib', 'paperclip', 'iostream.rb')
+
+class IOStreamTest < Test::Unit::TestCase
+  context "IOStream" do
+    should "be included in IO, File, Tempfile, and StringIO" do
+      [IO, File, Tempfile, StringIO].each do |klass|
+        assert klass.included_modules.include?(IOStream), "Not in #{klass}"
+      end
+    end
+  end
+
+  context "A file" do
+    setup do
+      @file = File.new(File.join(File.dirname(__FILE__), "fixtures", "5k.png"))
+    end
+
+    context "that is sent #stream_to" do
+
+      [["/tmp/iostream.string.test", File],
+       [Tempfile.new('iostream.test'), Tempfile]].each do |args|
+
+        context "and given a #{args[0].class.to_s}" do
+          setup do
+            assert @result = @file.stream_to(args[0])
+          end
+
+          should "return a #{args[1].to_s}" do
+            assert @result.is_a?(args[1])
+          end
+
+          should "contain the same data as the original file" do
+            @file.rewind; @result.rewind
+            assert_equal @file.read, @result.read
+          end
+        end
+      end
+    end
+
+    context "that is sent #to_tempfile" do
+      setup do
+        assert @tempfile = @file.to_tempfile
+      end
+
+      should "convert it to a Tempfile" do
+        assert @tempfile.is_a?(Tempfile)
+      end
+
+      should "have the Tempfile contain the same data as the file" do
+        @file.rewind; @tempfile.rewind
+        assert_equal @file.read, @tempfile.read
+      end
+    end
+  end
+end

data/test/lib/paperclip/test/test_paperclip.rb

@@ -0,0 +1,83 @@
+require 'test/helper.rb'
+
+class PaperclipTest < Test::Unit::TestCase
+  context "An ActiveRecord model with an 'avatar' attachment" do
+    setup do
+      rebuild_model
+      @file = File.new(File.join(FIXTURES_DIR, "5k.png"))
+    end
+
+    context "that is attr_protected" do
+      setup do
+        Dummy.class_eval do
+          attr_protected :avatar
+        end
+        @dummy = Dummy.new
+      end
+
+      should "not assign the avatar on mass-set" do
+        @dummy.logger.expects(:debug)
+
+        @dummy.attributes = { :other => "I'm set!",
+                              :avatar => @file }
+        
+        assert_equal "I'm set!", @dummy.other
+        assert ! @dummy.avatar?
+      end
+
+      should "still allow assigment on normal set" do
+        @dummy.logger.expects(:debug).times(0)
+
+        @dummy.other  = "I'm set!"
+        @dummy.avatar = @file
+        
+        assert_equal "I'm set!", @dummy.other
+        assert @dummy.avatar?
+      end
+    end
+
+    should "have an #avatar method" do
+      assert Dummy.new.respond_to?(:avatar)
+    end
+
+    should "have an #avatar= method" do
+      assert Dummy.new.respond_to?(:avatar=)
+    end
+
+    [[:presence,   nil,               "5k.png", nil],
+     [:size,       {:in => 1..10240}, "5k.png", "12k.png"]].each do |args|
+      context "with #{args[0]} validations" do
+        setup do
+          Dummy.class_eval do
+            send(*[:"validates_attachment_#{args[0]}", :avatar, args[1]].compact)
+          end
+          @dummy = Dummy.new
+        end
+
+        context "and a valid file" do
+          setup do
+            @file = args[2] && File.new(File.join(FIXTURES_DIR, args[2]))
+          end
+
+          should "not have any errors" do
+            @dummy.avatar = @file
+            assert @dummy.avatar.valid?
+            assert_equal 0, @dummy.avatar.errors.length
+          end
+        end
+
+        context "and an invalid file" do
+          setup do
+            @file = args[3] && File.new(File.join(FIXTURES_DIR, args[3]))
+          end
+
+          should "have errors" do
+            @dummy.avatar = @file
+            assert ! @dummy.avatar.valid?
+            assert_equal 1, @dummy.avatar.errors.length
+          end
+        end
+      end
+    end
+  end
+end

data/test/lib/paperclip/test/test_thumbnail.rb

@@ -0,0 +1,76 @@
+require 'rubygems'
+require 'test/unit'
+require 'shoulda'
+require 'mocha'
+require 'tempfile'
+
+require File.join(File.dirname(__FILE__), '..', 'lib', 'paperclip', 'geometry.rb')
+require File.join(File.dirname(__FILE__), '..', 'lib', 'paperclip', 'thumbnail.rb')
+
+class ThumbnailTest < Test::Unit::TestCase
+
+  context "A Paperclip Tempfile" do
+    setup do
+      @tempfile = Paperclip::Tempfile.new("file.jpg")
+    end
+
+    should "have its path contain a real extension" do
+      assert_equal ".jpg", File.extname(@tempfile.path)
+    end
+
+    should "be a real Tempfile" do
+      assert @tempfile.is_a?(::Tempfile)
+    end
+  end
+
+  context "Another Paperclip Tempfile" do
+    setup do
+      @tempfile = Paperclip::Tempfile.new("file")
+    end
+
+    should "not have an extension if not given one" do
+      assert_equal "", File.extname(@tempfile.path)
+    end
+
+    should "still be a real Tempfile" do
+      assert @tempfile.is_a?(::Tempfile)
+    end
+  end
+
+  context "An image" do
+    setup do
+      @file = File.new(File.join(File.dirname(__FILE__), "fixtures", "5k.png"))
+    end
+
+    context "being thumbnailed at 100x50 with cropping" do
+      setup do
+        @thumb = Paperclip::Thumbnail.new(@file, "100x50#")
+      end
+
+      should "report its correct current and target geometries" do
+        assert_equal "100x50#", @thumb.target_geometry.to_s
+        assert_equal "434x66", @thumb.current_geometry.to_s
+      end
+
+      should "report its correct format" do
+        assert_nil @thumb.format
+      end
+
+      should "have whiny_thumbnails turned on by default" do
+        assert @thumb.whiny_thumbnails
+      end
+
+      should "send the right command to convert when sent #make" do
+        @thumb.expects(:system).with do |arg|
+          arg.match %r{convert\s+"#{File.expand_path(@thumb.file.path)}"\s+-scale\s+\"x50\"\s+-crop\s+\"100x50\+114\+0\"\s+\+repage\s+".*?"}
+        end
+        @thumb.make
+      end
+
+      should "create the thumbnail when sent #make" do
+        dst = @thumb.make
+        assert_match /100x50/, `identify #{dst.path}`
+      end
+    end
+  end
+end

data/test/lib/schema.rb

@@ -1,7 +1,20 @@
 ActiveRecord::Schema.define :version => 0 do
-  create_table :users, :force => true do |t|
+  create_table :file_column_users, :force => true do |t|
     t.column :name,  :text, :nil => false
     t.column :email, :text, :nil => false
     t.column :avatar, :string
+    t.column :icon, :string
+  end
+  create_table :paperclip_users, :force => true do |t|
+    t.column :name,  :text, :nil => false
+    t.column :email, :text, :nil => false
+    
+    t.column :avatar_file_name, :string
+    t.column :avatar_content_type, :string
+    t.column :avatar_file_size, :integer
+    
+    t.column :icon_file_name, :string
+    t.column :icon_content_type, :string
+    t.column :icon_file_size, :integer
   end
 end

data/test/test_file_column_source.rb

@@ -1,8 +1,10 @@
+$: << File.expand_path(File.join(File.dirname(__FILE__), ['lib', 'file_column', 'lib']))
 require File.join(File.dirname(__FILE__), 'test_helper.rb')
+require File.join(File.dirname(__FILE__), ['lib', 'file_column', 'init'])
 require 'file_column'
 require 'avatar/source/file_column_source'
 
-class User < ActiveRecord::Base
+class FileColumnUser < ActiveRecord::Base
   file_column :avatar
   file_column :icon
 end
@@ -12,9 +14,9 @@ class TestFileColumnSource < Test::Unit::TestCase
   def setup
     @source = Avatar::Source::FileColumnSource.new
     png = File.new(File.join(File.dirname(__FILE__), ['lib', 'user_suit.png']))
-    @user_with_avatar = User.create!(:email => 'joe@example.com', :avatar => png)
-    @user_with_icon = User.create!(:email => 'terry@example.com', :icon => png)
-    @user_without_avatar = User.create!(:email => 'sue@example.com')
+    @user_with_avatar = FileColumnUser.create!(:email => 'joe@example.com', :avatar => png)
+    @user_with_icon = FileColumnUser.create!(:email => 'terry@example.com', :icon => png)
+    @user_without_avatar = FileColumnUser.create!(:email => 'sue@example.com')
   end
   
   def test_avatar_url_is_nil_if_person_is_nil
@@ -26,11 +28,11 @@ class TestFileColumnSource < Test::Unit::TestCase
   end
   
   def test_avatar_url_for_person_with_avatar
-    assert_equal "/user/avatar/#{@user_with_avatar.id}/0/user_suit.png", @source.avatar_url_for(@user_with_avatar)
+    assert_equal "/file_column_user/avatar/#{@user_with_avatar.id}/0/user_suit.png", @source.avatar_url_for(@user_with_avatar)
   end
   
   def test_avatar_url_for_person_with_icon_and_custom_file_column_field
-    assert_equal "/user/icon/#{@user_with_icon.id}/0/user_suit.png", @source.avatar_url_for(@user_with_icon, :file_column_field => :icon)
+    assert_equal "/file_column_user/icon/#{@user_with_icon.id}/0/user_suit.png", @source.avatar_url_for(@user_with_icon, :file_column_field => :icon)
   end
   
 end

data/test/test_gravatar_source.rb

@@ -71,4 +71,9 @@ class TestGravatarSource < Test::Unit::TestCase
     assert_equal "http://www.gravatar.com/avatar/#{@email_hash}", @source.avatar_url_for(@gary, :r => 'EVIL')
   end
   
+  def test_default_always_at_end
+    assert_equal "http://www.gravatar.com/avatar/#{@email_hash}?rating=any&default=http://gonzo.com",  @source.avatar_url_for(@gary, :gravatar_rating => :any, :gravatar_default_url => 'http://gonzo.com')
+    assert_equal "http://www.gravatar.com/avatar/#{@email_hash}?size=110&default=http://gonzo.com",  @source.avatar_url_for(@gary, :gravatar_size => 110, :gravatar_default_url => 'http://gonzo.com')
+  end
+  
 end

data/test/test_helper.rb

@@ -7,8 +7,13 @@ require 'active_record'
 require 'active_support'
 require 'action_controller'
 require 'action_view'
-$: << File.expand_path(File.join(File.dirname(__FILE__), ['lib', 'file_column', 'lib']))
-require File.join(File.dirname(__FILE__), ['lib', 'file_column', 'init'])
+
+#$: << File.expand_path(File.join(File.dirname(__FILE__), ['lib', 'file_column', 'lib']))
+#$: << File.expand_path(File.join(File.dirname(__FILE__), ['lib', 'paperclip', 'lib']))
+
+#require File.join(File.dirname(__FILE__), ['lib', 'file_column', 'init'])
+#require File.join(File.dirname(__FILE__), ['lib', 'paperclip', 'init'])
+
 require File.join(File.dirname(__FILE__), ['lib', 'database'])
 require File.join(File.dirname(__FILE__), ['lib', 'schema'])
 require File.join(File.dirname(__FILE__), ['..', 'lib', 'avatar'])

data/test/test_paperclip_source.rb

@@ -0,0 +1,52 @@
+$: << File.expand_path(File.join(File.dirname(__FILE__), ['lib', 'paperclip', 'lib']))
+require File.join(File.dirname(__FILE__), 'test_helper.rb')
+require File.join(File.dirname(__FILE__), ['lib', 'paperclip', 'init'])
+require 'avatar/source/paperclip_source'
+
+class PaperclipUser < ActiveRecord::Base
+  has_attached_file :avatar, 
+                    :styles => { :medium => "300x300>",
+                                   :thumb  => "100x100>" },
+                    :default_style => :medium,
+                    :path => ":rails_root/public/images/:class/:id/:attachment/:style/:basename.:extension",
+                    :url => "/images/:class/:id/:attachment/:style/:basename.:extension"
+
+  has_attached_file :icon, 
+                    :styles => { :small => "80x80" },
+                    :default_style => :small,
+                    :path => ":rails_root/public/images/:class/:id/:attachment/:style/:basename.:extension",
+                    :url => "/images/:class/:id/:attachment/:style/:basename.:extension"
+end
+
+class TestPaperclipSource < Test::Unit::TestCase
+  
+  def setup
+    @source = Avatar::Source::PaperclipSource.new
+    png = File.new(File.join(File.dirname(__FILE__), ['lib', 'user_suit.png']))
+    assert File.exists?(png.path)
+    @user_with_avatar = PaperclipUser.create!(:email => 'clarence@example.com', :avatar => png)
+    @user_with_icon = PaperclipUser.create!(:email => 'bobbi@example.com', :icon => png)
+    @user_without_avatar = PaperclipUser.create!(:email => 'brunhilde@example.com')
+  end
+  
+  def test_avatar_url_is_nil_if_person_is_nil
+    assert_nil @source.avatar_url_for(nil)
+  end
+  
+  def test_avatar_url_is_nil_if_person_has_no_avatar
+    assert_nil @source.avatar_url_for(@user_without_avatar)
+  end
+  
+  def test_avatar_url_for_person_with_avatar
+    assert_equal "/images/paperclipusers/#{@user_with_avatar.id}/avatars/medium/user_suit.png", @source.avatar_url_for(@user_with_avatar)
+  end
+
+  def test_avatar_url_is_nil_for_invalid_size
+    assert_nil @source.avatar_url_for(@user_with_avatar, :paperclip_style => :not_a_valid_style)
+  end
+  
+  def test_avatar_url_for_person_with_icon_and_custom_file_column_field
+    assert_equal "/images/paperclipusers/#{@user_with_icon.id}/icons/small/user_suit.png", @source.avatar_url_for(@user_with_icon, :paperclip_field => :icon)
+  end
+  
+end

data/test/test_rails_asset_source_wrapper.rb

@@ -0,0 +1,37 @@
+require File.dirname(__FILE__) + '/test_helper.rb'
+require 'avatar/source/wrapper/rails_asset_source_wrapper'
+require 'avatar/source/static_url_source'
+require 'action_controller/base'
+
+class TestRailsAssetSourceWrapper < Test::Unit::TestCase
+  
+  def setup
+    ActionController::Base.asset_host = 'http://test.com'
+    @source = Avatar::Source::StaticUrlSource.new('/')
+    @wrapper = Avatar::Source::Wrapper::RailsAssetSourceWrapper.new(@source)
+  end
+  
+  def test_returns_nil_if_underlying_source_returns_nil
+    @source.instance_eval { def avatar_url_for(*args); nil; end }
+    assert_nil @wrapper.avatar_url_for(:a_person)
+  end
+  
+  def test_does_not_change_fully_qualified_uri
+    @source.url = 'http://example.com/images/avatar.png'
+    assert_equal 'http://example.com/images/avatar.png', @wrapper.avatar_url_for(3)
+  end
+  
+  def test_uses_asset_host
+    @source.url = '/images/avatar.png'
+    assert_equal 'http://test.com/images/avatar.png', @wrapper.avatar_url_for(4)
+  end
+  
+  def test_error_if_cannot_generate_full_uri
+    ActionController::Base.asset_host = ''
+    @source.url = '/images/avatar.png'
+    assert_raise(RuntimeError) {
+      @wrapper.avatar_url_for(4)
+    }
+  end
+  
+end