bin/mail_receiver.rb bin/mail_receiver_ctl.rb config/mail.sample.yml spec/fixtures/emails/red spec/fixtures/emails/red_address_two_lines spec/fixtures/emails/red_address_two_lines_sig spec/fixtures/emails/red_mixedcase_subject spec/fixtures/emails/red_no_body spec/fixtures/emails/red_no_subject spec/fixtures/emails/red_uppercase_subject @@ -1,4 +1,21 @@ class Location < ActiveRecord::Base + def self.existing_address(location) + location.send :attach_geocode + # find others with the same address + Location.first(:conditions => {:street => location.street, :city => location.city, :state => location.state, :zip => location.zip}) + end + + def self.new_from_email(source) + attrs, email = {}, TMail::Mail.parse(source) + attrs[:signs] = email.subject.blank? ? '' : email.subject.downcase.strip.titleize + attrs[:street] = parse_address(email.body) + new(attrs) + end + + def self.parse_address(body) + body.split("\n\n").first + end + SIGN_OPTIONS = { 'Blue' => 'Obama & Democrats', 'Red' => 'McCain & Republicans', @@ -50,14 +67,8 @@ class Location < ActiveRecord::Base def self.state_count count(:select => 'DISTINCT(state)') end - -private - def self.existing_address(location) - location.send :attach_geocode - # find others with the same address - Location.first(:conditions => {:street => location.street, :city => location.city, :state => location.state, :zip => location.zip}) - end +private def check_geocode_precision attach_geocode errors.add_to_base "We can't find a precise enough address match." if geocode.blank? || geocode.precision != 'address' app/models/location.rb @@ -1,17 +1,48 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper') describe Location do - before(:each) do - @valid_attributes = { - :street => "value for street", - :city => "value for city", - :state => "value for state", - :zip => "value for zip", - :signs => "Blue" - } - end - - it "should create a new instance given valid attributes" do - Location.create!(@valid_attributes) + describe "#new_from_email" do + it "should be subject" do + location = Location.new_from_email(email_fixture(:red_no_subject)) + location.should have_error_on(:signs) + end + + it "should require body" do + location = Location.new_from_email(email_fixture(:red_no_body)) + location.valid? + location.errors.full_messages.should include("We can't find a precise enough address match.") + end + + it "should set street to email body" do + location = Location.new_from_email(email_fixture(:red)) + location.street.should == '1600 Pennsylvania Ave. Washington, D.C.' + end + + it "should set sign equal to subject" do + location = Location.new_from_email(email_fixture(:red)) + location.signs.should == 'Red' + end + + it "should work with mixed case subject" do + location = Location.new_from_email(email_fixture(:red_mixedcase_subject)) + location.signs.should == 'Red' + end + + it "should work with upper case subject" do + location = Location.new_from_email(email_fixture(:red_uppercase_subject)) + location.signs.should == 'Red' + end + + it "should work with multi line body" do + location = Location.new_from_email(email_fixture(:red_address_two_lines)) + location.valid? + location.to_location.to_s.should == "1600 Pennsylvania Ave Nw\nWashington, DC 20006" + end + + it "should work with multi line body and email signature" do + location = Location.new_from_email(email_fixture(:red_address_two_lines_sig)) + location.valid? + location.to_location.to_s.should == "1600 Pennsylvania Ave Nw\nWashington, DC 20006" + end end end spec/models/location_spec.rb @@ -45,3 +45,7 @@ Spec::Runner.configure do |config| # # For more information take a look at Spec::Example::Configuration and Spec::Runner end + +def email_fixture(name) + File.read(File.join(File.dirname(__FILE__), 'fixtures', 'emails', name.to_s)) +end \ No newline at end of file spec/spec_helper.rb 51d2e57accc841ef4048e1ea76413e36c484890b 4870b7aee4107494c5f93c3ea40151f6cd29a64f Daniel Morrison daniel@collectiveidea.com http://github.com/collectiveidea/yardvote.com/commit/948a7b6bf78a81eed50668472ddd2789f6aa8775 948a7b6bf78a81eed50668472ddd2789f6aa8775 2008-10-23T14:34:48-07:00 2008-10-23T14:34:48-07:00 Merge branch 'master' of git://github.com/jnunemaker/yardvote.com 1bd6396f283354baa2fdf93d7bdf7fcbb4aa3651 Daniel Morrison daniel@collectiveidea.com