whitelist_mail_proxy 0.4.1 → 0.4.2

data/Rakefile

@@ -22,19 +22,8 @@ Rake::RDocTask.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-require "rubygems"
 require "rake/gempackagetask"
-require "rake/rdoctask"
 
-require "rake/testtask"
-Rake::TestTask.new do |t|
-  t.libs << "test"
-  t.test_files = FileList["test/**/*_test.rb"]
-  t.verbose = true
-end
-
-
-task :default => ["test"]
 
 # This builds the actual gem. For details of what all these options
 # mean, and other ones you can add, check the documentation here:
@@ -45,7 +34,7 @@ spec = Gem::Specification.new do |s|
 
   # Change these as appropriate
   s.name              = "whitelist_mail_proxy"
-  s.version           = "0.4.1"
+  s.version           = "0.4.2"
   s.summary           = "A thin proxy for Mail and ActionMailer to enable whitelisting"
   s.author            = "Matthew Rudy Jacobs"
   s.email             = "MatthewRudyJacobs@gmail.com"
@@ -56,7 +45,7 @@ spec = Gem::Specification.new do |s|
   s.rdoc_options      = %w(--main README)
 
   # Add any extra files to include in the gem
-  s.files             = %w(MIT-LICENSE Rakefile README) + Dir.glob("{test,lib/**/*}")
+  s.files             = %w(MIT-LICENSE Rakefile README) + Dir.glob("{test,lib}/**/*")
   s.require_paths     = ["lib"]
 
   # If you want to depend on other gems, add them here, along with any

data/test/test_helper.rb

@@ -0,0 +1,3 @@
+require 'rubygems'
+require 'test/unit'
+require 'active_support'

data/test/whitelist_mail_proxy_test.rb

@@ -0,0 +1,104 @@
+require 'test_helper'
+require File.dirname(__FILE__)+"/../lib/whitelist_mail_proxy"
+
+class WhitelistMailProxyTest < ActiveSupport::TestCase
+  
+  test "extract_email_address - just an email" do
+    assert_extracted "murple@murple.com", "murple@murple.com"
+  end
+  
+  test "extract_email_address - with a trailing space" do
+    assert_extracted "murple@murple.com", "murple@murple.com  "
+  end
+  
+  test "extract_email_address - with some magic" do
+    assert_extracted "murple+somemagic@murple.com", "murple+somemagic@murple.com"
+  end
+  
+  test "extract_email_address - with a quoted name" do
+    full = %("Purple Murple" <murple@murple.com>)
+    assert_extracted "murple@murple.com", full
+  end
+  
+  test "extract_email_address - with a non-quoted name" do
+    full = %(Purple Murple <murple@murple.com>)
+    assert_extracted "murple@murple.com", full
+  end
+  
+  test "extract_email_address - without a space between the name and email" do
+    full = %("Purple Murple"<murple@murple.com>)
+    assert_extracted "murple@murple.com", full
+  end
+  
+  test "extract_email_domain" do
+    assert_domain "murple.com", "murple@murple.com"
+  end
+  
+  test "extract_email_domain - with quoted name" do
+    full = %("Purple Murple" <murple@murple.com>)
+    assert_domain "murple.com", full
+  end
+  
+  test "block_recipient - by regexp" do
+    @proxy = WhitelistMailProxy.new(:delivery_method => :test, :regexp => /purple/)
+    
+    # matching the email address
+    assert !@proxy.block_recipient?("purple@murple.com")
+    assert  @proxy.block_recipient?("murple@murple.com")
+    
+    # but also the name
+    assert  @proxy.block_recipient?(%("Purple Murple" <murple@murple.com>))
+    assert !@proxy.block_recipient?(%("purple Murple" <murple@murple.com>)) # case sensitive
+    assert  @proxy.block_recipient?(%("Murple Murple" <murple@murple.com>))
+  end
+  
+  test "block_recipient - by single domain" do
+    @proxy = WhitelistMailProxy.new(:delivery_method => :test, :domain => "murple.com")
+    
+    # matching the domain
+    assert !@proxy.block_recipient?("murple@murple.com")
+    assert  @proxy.block_recipient?("murple@murpoo.com")
+    
+    # regardless of name
+    assert !@proxy.block_recipient?(%("Purple Murple" <murple@murple.com>))
+    assert  @proxy.block_recipient?(%("Purple Murple" <murple@murpoo.com>))
+  end
+  
+  test "block_recipient - multiple domains" do
+    @proxy = WhitelistMailProxy.new(:delivery_method => :test, :domain => ["murple.com", "murpoo.com"])
+    
+    # matching the domain
+    assert !@proxy.block_recipient?("murple@murple.com")
+    assert !@proxy.block_recipient?("murple@murpoo.com")
+    assert  @proxy.block_recipient?("murple@somethingelse.com")
+    
+    # regardless of name
+    assert !@proxy.block_recipient?(%("Purple Murple" <murple@murple.com>))
+    assert !@proxy.block_recipient?(%("Purple Murple" <murple@murpoo.com>))
+    assert  @proxy.block_recipient?(%("Purple Murple" <murple@somethingelse.com>))
+  end
+  
+  test "raises an exception if not given a delivery method" do
+    assert_raise(WhitelistMailProxy::SettingsError) do
+      WhitelistMailProxy.new(:domain => ["murple.com", "murpoo.com"])
+    end
+  end
+  
+  test "raises an exception if not given a regexp or a domain" do
+    assert_raise(WhitelistMailProxy::SettingsError) do
+      WhitelistMailProxy.new(:delivery_method => :test)
+    end
+    assert WhitelistMailProxy.new(:delivery_method => :test, :regexp => /something/)
+    assert WhitelistMailProxy.new(:delivery_method => :test, :domain => ["something"])
+    assert WhitelistMailProxy.new(:delivery_method => :test, :domain => []) # it doesnt care if its empty
+  end
+  
+  def assert_extracted(extracted, full)
+    assert_equal extracted, WhitelistMailProxy.extract_email_address(full)
+  end
+  
+  def assert_domain(domain, full)
+    assert_equal domain, WhitelistMailProxy.extract_email_domain(full)
+  end
+  
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: whitelist_mail_proxy
 version: !ruby/object:Gem::Version 
-  hash: 13
-  prerelease: false
+  hash: 11
+  prerelease: 
   segments: 
   - 0
   - 4
-  - 1
-  version: 0.4.1
+  - 2
+  version: 0.4.2
 platform: ruby
 authors: 
 - Matthew Rudy Jacobs
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-08 00:00:00 +08:00
+date: 2011-01-09 00:00:00 +08:00
 default_executable: 
 dependencies: []
 
@@ -31,6 +31,8 @@ files:
 - MIT-LICENSE
 - Rakefile
 - README
+- test/test_helper.rb
+- test/whitelist_mail_proxy_test.rb
 - lib/whitelist_mail_proxy.rb
 has_rdoc: true
 homepage: http://github.com/matthewrudy/whitelist-mail-proxy
@@ -63,7 +65,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.4.1
 signing_key: 
 specification_version: 3
 summary: A thin proxy for Mail and ActionMailer to enable whitelisting