test/unit/gravatarify_base_test.rb in gravatarify-1.2.0 vs test/unit/gravatarify_base_test.rb in gravatarify-1.2.1

@@ -44,12 +44,15 @@
     should "allow different :filetype to be set, like 'gif' or 'png'" do
       assert_equal "#{BELLA_AT_GMAIL}.gif", build_gravatar_url('[e-mail]', :filetype => :gif)
       assert_equal "#{BELLA_AT_GMAIL}.png", build_gravatar_url('[e-mail]', :filetype => :png)
     end
   
-    should "skip :filetype if set to false" do
-      assert_equal "#{BELLA_AT_GMAIL}", build_gravatar_url('[e-mail]', :filetype => false)      
-      assert_equal "#{BELLA_AT_GMAIL_JPG}", build_gravatar_url('[e-mail]', :filetype => nil)      
+    should "skip :filetype if set to false, nil or ''" do
+      assert_equal "#{BELLA_AT_GMAIL}", build_gravatar_url('[e-mail]', :filetype => false)
+      assert_equal "#{BELLA_AT_GMAIL}", build_gravatar_url('[e-mail]', :filetype => nil)
+      assert_equal "#{BELLA_AT_GMAIL}", build_gravatar_url('[e-mail]', :filetype => '')
+      assert_equal "#{BELLA_AT_GMAIL}.foobar", build_gravatar_url('[e-mail]', :filetype => 'foobar')
+      assert_equal "#{BELLA_AT_GMAIL}.gif", build_gravatar_url('[e-mail]', :filetype => :gif)
     end
     
     should "handle Procs as :default, to easily generate default urls based on supplied :size" do
       default = Proc.new { |*args| "http://example.com/gravatar#{args.first[:size] ? '-' + args.first[:size].to_s : ''}.jpg" }
       assert_equal "#{BELLA_AT_GMAIL_JPG}?d=http%3A%2F%2Fexample.com%2Fgravatar.jpg", build_gravatar_url('[e-mail]', :default => default)