fat_free_crm 0.11.1 → 0.11.2

data/spec/controllers/tasks_controller_spec.rb

@@ -8,7 +8,6 @@ describe TasksController do
   end
 
   def produce_tasks(user, view)
-    #~ Time.zone = 'UTC'
     settings = (view != "completed" ? Setting.task_bucket : Setting.task_completed)
 
     settings.inject({}) do | hash, due |
@@ -61,6 +60,7 @@ describe TasksController do
     end
 
     TASK_STATUSES.each do |view|
+
       it "should expose all tasks as @tasks and render [index] template for #{view} view" do
         @tasks = produce_tasks(@current_user, view)
 
@@ -117,10 +117,6 @@ describe TasksController do
   describe "responding to GET show" do
 
     TASK_STATUSES.each do |view|
-      it "should render tasks index for #{view} view (since a task doesn't have landing page)" do
-        get :show, :id => 42, :view => view
-        response.should render_template("tasks/index")
-      end
 
       it "should render the requested task as JSON for #{view} view" do
         Task.stub_chain(:tracked_by, :find).and_return(task = mock("Task"))

data/spec/controllers/users_controller_spec.rb

@@ -74,7 +74,7 @@ describe UsersController do
 
     describe "if user is allowed to sign up" do
       it "should expose a new user as @user and render [new] template" do
-        controller.should_receive(:can_signup?).and_return(true)
+        @controller.should_receive(:can_signup?).and_return(true)
         @user = FactoryGirl.build(:user)
         User.stub!(:new).and_return(@user)
 
@@ -86,7 +86,7 @@ describe UsersController do
 
     describe "if user is not allowed to sign up" do
       it "should redirect to login_path" do
-        controller.should_receive(:can_signup?).and_return(false)
+        @controller.should_receive(:can_signup?).and_return(false)
 
         get :new
         response.should redirect_to(login_path)
@@ -139,7 +139,6 @@ describe UsersController do
         flash[:notice].should =~ /approval/
         response.should redirect_to(login_path)
       end
-
     end
 
     describe "with invalid params" do
@@ -152,7 +151,6 @@ describe UsersController do
         response.should render_template("users/new")
       end
     end
-
   end
 
   # PUT /users/1
@@ -168,12 +166,12 @@ describe UsersController do
 
       it "should update user information and render [update] template" do
         xhr :put, :update, :id => @user.id, :user => { :first_name => "Billy", :last_name => "Bones" }
-        @user.reload.first_name.should == "Billy"
+        @user.reload
+        @user.first_name.should == "Billy"
         @user.last_name.should == "Bones"
         assigns[:user].should == @user
         response.should render_template("users/update")
       end
-
     end
 
     describe "with invalid params" do
@@ -184,9 +182,7 @@ describe UsersController do
         assigns[:user].should == @user
         response.should render_template("users/update")
       end
-
     end
-
   end
 
   # DELETE /users/1
@@ -247,7 +243,7 @@ describe UsersController do
     end
 
     it "should save the user avatar if it was successfully uploaded and resized" do
-      @image = fixture_file_upload("/rails.png", "image/png")
+      @image = fixture_file_upload('/rails.png', 'image/png')
 
       xhr :put, :upload_avatar, :id => @user.id, :avatar => { :image => @image }
       @user.avatar.should_not == nil

data/spec/factories/subscription_factories.rb

@@ -0,0 +1,6 @@
+# Read about factories at https://github.com/thoughtbot/factory_girl
+
+FactoryGirl.define do
+  factory :subscription do
+  end
+end

data/spec/lib/mail_processor/base_spec.rb

@@ -0,0 +1,164 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+require File.dirname(__FILE__) + '/sample_emails/dropbox'
+
+require "fat_free_crm/mail_processor/base"
+
+describe FatFreeCRM::MailProcessor::Base do
+  include MockIMAP
+
+  before do
+    @mock_address = "base-mail-processor@example.com"
+  end
+
+  before(:each) do
+    @crawler = FatFreeCRM::MailProcessor::Base.new
+    # MailProcessor::Base doesn't load any settings by default
+    @crawler.instance_variable_set "@settings", {
+      :server   => "example.com",
+      :port     => "123",
+      :ssl      => true,
+      :address  => "test@example.com",
+      :user     => "test@example.com",
+      :password => "123"
+    }
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Connecting to the IMAP server" do
+    it "should connect to the IMAP server and login as user, and select folder" do
+      mock_imap
+      @imap.should_receive(:login).once.with(@settings[:user], @settings[:password])
+      @imap.should_receive(:select).once.with(@settings[:scan_folder])
+      @crawler.send(:connect!)
+    end
+
+    it "should connect to the IMAP server, login as user, but not select folder when requested so" do
+      mock_imap
+      @imap.should_receive(:login).once.with(@settings[:user], @settings[:password])
+      @imap.should_not_receive(:select).with(@settings[:scan_folder])
+      @crawler.send(:connect!, :setup => true)
+    end
+
+    it "should raise the error if connection fails" do
+      Net::IMAP.should_receive(:new).and_raise(SocketError) # No mocks this time! :-)
+      @crawler.send(:connect!).should == nil
+    end
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Disconnecting from the IMAP server" do
+    it "should logout and diconnect" do
+      mock_connect
+      mock_disconnect
+      @imap.should_receive(:logout).once
+      @imap.should_receive(:disconnect).once
+
+      @crawler.send(:connect!)
+      @crawler.send(:disconnect!)
+    end
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Discarding a message" do
+    before(:each) do
+      mock_connect
+      @uid = mock
+      @crawler.send(:connect!)
+    end
+
+    it "should copy message to invalid folder if it's set and flag the message as deleted" do
+      @settings[:move_invalid_to_folder] = "invalid"
+      @imap.should_receive(:uid_copy).once.with(@uid, @settings[:move_invalid_to_folder])
+      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Deleted])
+      @crawler.send(:discard, @uid)
+    end
+
+    it "should not copy message to invalid folder if it's not set and flag the message as deleted" do
+      @settings[:move_invalid_to_folder] = nil
+      @imap.should_not_receive(:uid_copy)
+      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Deleted])
+      @crawler.send(:discard, @uid)
+    end
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Archiving a message" do
+    before(:each) do
+      mock_connect
+      @uid = mock
+      @crawler.send(:connect!)
+    end
+
+    it "should copy message to archive folder if it's set and flag the message as seen" do
+      @settings[:move_to_folder] = "processed"
+      @imap.should_receive(:uid_copy).once.with(@uid, @settings[:move_to_folder])
+      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Seen])
+      @crawler.send(:archive, @uid)
+    end
+
+    it "should not copy message to archive folder if it's not set and flag the message as seen" do
+      @settings[:move_to_folder] = nil
+      @imap.should_not_receive(:uid_copy)
+      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Seen])
+      @crawler.send(:archive, @uid)
+    end
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Validating email" do
+    before(:each) do
+      @email = mock
+    end
+
+    it "should be valid email if its contents type is text/plain" do
+      @email.stub!(:content_type).and_return("text/plain")
+      @crawler.send(:is_valid?, @email).should == true
+    end
+
+    it "should be invalid email if its contents type is not text/plain" do
+      @email.stub!(:content_type).and_return("text/html")
+      @crawler.send(:is_valid?, @email).should == false
+    end
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Finding email sender among users" do
+    before(:each) do
+      @from = [ "Aaron@Example.Com", "Ben@Example.com" ]
+      @email = mock
+      @email.stub!(:from).and_return(@from)
+    end
+
+    it "should find non-suspended user that matches From: field" do
+      @user = FactoryGirl.create(:user, :email => @from.first, :suspended_at => nil)
+      @crawler.send(:sent_from_known_user?, @email).should == true
+      @crawler.instance_variable_get("@sender").should == @user
+    end
+
+    it "should not find user if his email doesn't match From: field" do
+      FactoryGirl.create(:user, :email => "nobody@example.com")
+      @crawler.send(:sent_from_known_user?, @email).should == false
+      @crawler.instance_variable_get("@sender").should == nil
+    end
+
+    it "should not find user if his email matches From: field but is suspended" do
+      FactoryGirl.create(:user, :email => @from.first, :suspended_at => Time.now)
+      @crawler.send(:sent_from_known_user?, @email).should == false
+      @crawler.instance_variable_get("@sender").should == nil
+    end
+
+    #------------------------------------------------------------------------------
+    describe "Extracting plain text body" do
+
+      it "should extract text from multipart text/plain" do
+        text = @crawler.send(:plain_text_body, Mail.new(DROPBOX_EMAILS[:plain]))
+        text.should be_present
+      end
+
+      it "should extract text and strip tags from multipart text/html" do
+        text = @crawler.send(:plain_text_body, Mail.new(DROPBOX_EMAILS[:multipart]))
+        text.should eql('Hello,')
+      end
+    end
+  end
+end

data/spec/lib/mail_processor/comment_replies_spec.rb

@@ -0,0 +1,63 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+require "fat_free_crm/mail_processor/comment_replies"
+
+describe FatFreeCRM::MailProcessor::CommentReplies do
+  include MockIMAP
+
+  before do
+    @mock_address = "crm-comment@example.com"
+  end
+
+  before(:each) do
+    @crawler = FatFreeCRM::MailProcessor::CommentReplies.new
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Processing new emails" do
+    before do
+      FactoryGirl.create(:user, :email => "aaron@example.com")
+    end
+
+    before(:each) do
+      mock_connect
+      mock_disconnect
+    end
+
+    it "should attach a new comment to a contact" do
+      @contact = FactoryGirl.create(:contact)
+      comment_reply = "This is a new comment reply via email"
+
+      mail = Mail.new :from    => "Aaron Assembler <aaron@example.com>",
+                      :to      => "FFCRM Comments <crm-commment@example.com>",
+                      :subject => "[contact:#{@contact.id}] Test Contact",
+                      :body    => comment_reply
+      mock_message mail.to_s
+
+      @crawler.should_receive(:archive).once
+      @crawler.should_not_receive(:with_recipients)
+      @crawler.run
+
+      @contact.comments.size.should == 1
+      @contact.comments.first.comment.should == comment_reply
+    end
+
+    it "should attach a new comment to an opportunity, using the 'op' shortcut in subject" do
+      @opportunity = FactoryGirl.create(:opportunity)
+      comment_reply = "This is a new comment reply via email"
+
+      mail = Mail.new :from    => "Aaron Assembler <aaron@example.com>",
+                      :to      => "FFCRM Comments <crm-commment@example.com>",
+                      :subject => "[op:#{@opportunity.id}] Test Opportunity",
+                      :body    => comment_reply
+      mock_message mail.to_s
+
+      @crawler.should_receive(:archive).once
+      @crawler.should_not_receive(:with_recipients)
+      @crawler.run
+
+      @opportunity.comments.size.should == 1
+      @opportunity.comments.first.comment.should == comment_reply
+    end
+  end
+end

data/spec/lib/{dropbox_spec.rb → mail_processor/dropbox_spec.rb}

@@ -1,166 +1,14 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require File.dirname(__FILE__) + '/dropbox/email_samples'
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+require File.dirname(__FILE__) + '/sample_emails/dropbox'
 
-require "fat_free_crm/dropbox"
+require "fat_free_crm/mail_processor/dropbox"
 
-describe "IMAP Dropbox" do
-  before(:each) do
-    @crawler = FatFreeCRM::Dropbox.new
-    @crawler.stub!("expunge!").and_return(true)
-  end
-
-  def mock_imap
-    @imap = mock
-    @settings = @crawler.instance_variable_get("@settings")
-    @settings[:address] = "dropbox@example.com"
-    Net::IMAP.stub!(:new).with(@settings[:server], @settings[:port], @settings[:ssl]).and_return(@imap)
-  end
-
-  def mock_connect
-    mock_imap
-    @imap.stub!(:login).and_return(true)
-    @imap.stub!(:select).and_return(true)
-  end
-
-  def mock_disconnect
-    @imap.stub!(:disconnected?).and_return(false)
-    @imap.stub!(:logout).and_return(true)
-    @imap.stub!(:disconnect).and_return(true)
-  end
-
-  def mock_message(body = EMAIL[:plain])
-    @fetch_data = mock
-    @fetch_data.stub!(:attr).and_return("RFC822" => body)
-    @imap.stub!(:uid_search).and_return([ :uid ])
-    @imap.stub!(:uid_fetch).and_return([ @fetch_data ])
-    @imap.stub!(:uid_copy).and_return(true)
-    @imap.stub!(:uid_store).and_return(true)
-    body
-  end
-
-  #------------------------------------------------------------------------------
-  describe "Connecting to the IMAP server" do
-    it "should connect to the IMAP server and login as user, and select folder" do
-      mock_imap
-      @imap.should_receive(:login).once.with(@settings[:user], @settings[:password])
-      @imap.should_receive(:select).once.with(@settings[:scan_folder])
-      @crawler.send(:connect!)
-    end
-
-    it "should connect to the IMAP server, login as user, but not select folder when requested so" do
-      mock_imap
-      @imap.should_receive(:login).once.with(@settings[:user], @settings[:password])
-      @imap.should_not_receive(:select).with(@settings[:scan_folder])
-      @crawler.send(:connect!, :setup => true)
-    end
-
-    it "should raise the error if connection fails" do
-      Net::IMAP.should_receive(:new).and_raise(SocketError) # No mocks this time! :-)
-      @crawler.send(:connect!).should == nil
-    end
-  end
-
-  #------------------------------------------------------------------------------
-  describe "Disconnecting from the IMAP server" do
-    it "should logout and diconnect" do
-      mock_connect
-      mock_disconnect
-      @imap.should_receive(:logout).once
-      @imap.should_receive(:disconnect).once
-
-      @crawler.send(:connect!)
-      @crawler.send(:disconnect!)
-    end
-  end
-
-  #------------------------------------------------------------------------------
-  describe "Discarding a message" do
-    before(:each) do
-      mock_connect
-      @uid = mock
-      @crawler.send(:connect!)
-    end
-
-    it "should copy message to invalid folder if it's set and flag the message as deleted" do
-      @settings[:move_invalid_to_folder] = "invalid"
-      @imap.should_receive(:uid_copy).once.with(@uid, @settings[:move_invalid_to_folder])
-      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Deleted])
-      @crawler.send(:discard, @uid)
-    end
-
-    it "should not copy message to invalid folder if it's not set and flag the message as deleted" do
-      @settings[:move_invalid_to_folder] = nil
-      @imap.should_not_receive(:uid_copy)
-      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Deleted])
-      @crawler.send(:discard, @uid)
-    end
-  end
-
-  #------------------------------------------------------------------------------
-  describe "Archiving a message" do
-    before(:each) do
-      mock_connect
-      @uid = mock
-      @crawler.send(:connect!)
-    end
-
-    it "should copy message to archive folder if it's set and flag the message as seen" do
-      @settings[:move_to_folder] = "processed"
-      @imap.should_receive(:uid_copy).once.with(@uid, @settings[:move_to_folder])
-      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Seen])
-      @crawler.send(:archive, @uid)
-    end
-
-    it "should not copy message to archive folder if it's not set and flag the message as seen" do
-      @settings[:move_to_folder] = nil
-      @imap.should_not_receive(:uid_copy)
-      @imap.should_receive(:uid_store).once.with(@uid, "+FLAGS", [:Seen])
-      @crawler.send(:archive, @uid)
-    end
-  end
-
-  #------------------------------------------------------------------------------
-  describe "Validating email" do
-    before(:each) do
-      @email = mock
-    end
+describe FatFreeCRM::MailProcessor::Dropbox do
+  include MockIMAP
 
-    it "should be valid email if its contents type is text/plain" do
-      @email.stub!(:content_type).and_return("text/plain")
-      @crawler.send(:is_valid?, @email).should == true
-    end
-
-    it "should be invalid email if its contents type is not text/plain" do
-      @email.stub!(:content_type).and_return("text/html")
-      @crawler.send(:is_valid?, @email).should == false
-    end
-  end
-
-  #------------------------------------------------------------------------------
-  describe "Finding email sender among users" do
-    before(:each) do
-      @from = [ "Aaron@Example.Com", "Ben@Example.com" ]
-      @email = mock
-      @email.stub!(:from).and_return(@from)
-    end
-
-    it "should find non-suspended user that matches From: field" do
-      @user = FactoryGirl.create(:user, :email => @from.first, :suspended_at => nil)
-      @crawler.send(:sent_from_known_user?, @email).should == true
-      @crawler.instance_variable_get("@sender").should == @user
-    end
-
-    it "should not find user if his email doesn't match From: field" do
-      FactoryGirl.create(:user, :email => "nobody@example.com")
-      @crawler.send(:sent_from_known_user?, @email).should == false
-      @crawler.instance_variable_get("@sender").should == nil
-    end
-
-    it "should not find user if his email matches From: field but is suspended" do
-      FactoryGirl.create(:user, :email => @from.first, :suspended_at => Time.now)
-      @crawler.send(:sent_from_known_user?, @email).should == false
-      @crawler.instance_variable_get("@sender").should == nil
-    end
+  before(:each) do
+    @mock_address = "dropbox@example.com"
+    @crawler = FatFreeCRM::MailProcessor::Dropbox.new
   end
 
   #------------------------------------------------------------------------------
@@ -168,7 +16,7 @@ describe "IMAP Dropbox" do
     before(:each) do
       mock_connect
       mock_disconnect
-      mock_message
+      mock_message(DROPBOX_EMAILS[:plain])
     end
 
     it "should discard a message if it's invalid" do
@@ -203,7 +51,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should find the named asset and attach the email message" do
-      mock_message(EMAIL[:first_line])
+      mock_message(DROPBOX_EMAILS[:first_line])
       @campaign = FactoryGirl.create(:campaign, :name => "Got milk!?")
       @crawler.should_receive(:archive).once
       @crawler.should_not_receive(:with_recipients)
@@ -214,7 +62,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should create the named asset and attach the email message" do
-      mock_message(EMAIL[:first_line])
+      mock_message(DROPBOX_EMAILS[:first_line])
       @crawler.should_receive(:archive).once
       @crawler.should_not_receive(:with_recipients)
       @crawler.run
@@ -226,7 +74,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should find the lead and attach the email message" do
-      mock_message(EMAIL[:first_line_lead])
+      mock_message(DROPBOX_EMAILS[:first_line_lead])
       @lead = FactoryGirl.create(:lead, :first_name => "Cindy", :last_name => "Cluster")
       @crawler.should_receive(:archive).once
       @crawler.should_not_receive(:with_recipients)
@@ -237,7 +85,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should create the lead and attach the email message" do
-      mock_message(EMAIL[:first_line_lead])
+      mock_message(DROPBOX_EMAILS[:first_line_lead])
       @crawler.should_receive(:archive).once
       @crawler.should_not_receive(:with_recipients)
       @crawler.run
@@ -250,7 +98,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should find the contact and attach the email message" do
-      mock_message(EMAIL[:first_line_contact])
+      mock_message(DROPBOX_EMAILS[:first_line_contact])
       @contact = FactoryGirl.create(:contact, :first_name => "Cindy", :last_name => "Cluster")
       @crawler.should_receive(:archive).once
       @crawler.should_not_receive(:with_recipients)
@@ -261,7 +109,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should create the contact and attach the email message" do
-      mock_message(EMAIL[:first_line_contact])
+      mock_message(DROPBOX_EMAILS[:first_line_contact])
       @crawler.should_receive(:archive).once
       @crawler.should_not_receive(:with_recipients)
       @crawler.run
@@ -273,7 +121,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should move on if first line has no keyword" do
-      mock_message(EMAIL[:plain])
+      mock_message(DROPBOX_EMAILS[:plain])
       @crawler.should_receive(:with_recipients).twice
       @crawler.should_receive(:with_forwarded_recipient).twice
       @crawler.run
@@ -281,11 +129,11 @@ describe "IMAP Dropbox" do
   end
 
   #------------------------------------------------------------------------------
-  describe "Pipieline: processing recipients (To: recipient, Bcc: dropbox)" do
+  describe "Pipeline: processing recipients (To: recipient, Bcc: dropbox)" do
     before(:each) do
       mock_connect
       mock_disconnect
-      mock_message(EMAIL[:plain])
+      mock_message(DROPBOX_EMAILS[:plain])
       FactoryGirl.create(:user, :email => "aaron@example.com")
     end
 
@@ -299,20 +147,23 @@ describe "IMAP Dropbox" do
       @lead.emails.first.mediator.should == @lead
     end
 
-    it "should move on if asset recipients did not match" do
-      @crawler.should_receive(:with_recipients).twice
-      @crawler.should_receive(:with_forwarded_recipient).twice
-      @crawler.run
+    it "should create the asset and attach the email message" do
+      @crawler.should_receive(:archive).once
+      lambda { @crawler.run }.should change(Contact, :count).by(1)
+
+      @contact = Contact.last
+      @contact.emails.size.should == 1
+      @contact.emails.first.mediator.should == @contact
     end
   end
 
   #------------------------------------------------------------------------------
-  describe "Pipieline: processing forwarded recipient (To: dropbox)" do
+  describe "Pipeline: processing forwarded recipient (To: dropbox)" do
     before(:each) do
       mock_connect
       mock_disconnect
       FactoryGirl.create(:user, :email => "aaron@example.com")
-      mock_message(EMAIL[:forwarded])
+      mock_message(DROPBOX_EMAILS[:forwarded])
     end
 
     it "should find the asset and attach the email message" do
@@ -352,7 +203,33 @@ describe "IMAP Dropbox" do
   end
 
   #------------------------------------------------------------------------------
-  describe "Pipieline: creating recipient if s/he was not found" do
+  describe "Pipeline: processing forwarded recipient from email sent to dropbox alias address" do
+    before(:each) do
+      @mock_address = "dropbox-alias-address@example.com"
+      mock_connect
+      mock_disconnect
+
+      @settings = @crawler.instance_variable_get("@settings")
+      @settings[:address_aliases] = ["dropbox@example.com"]
+
+      FactoryGirl.create(:user, :email => "aaron@example.com")
+      mock_message(DROPBOX_EMAILS[:forwarded])
+    end
+
+    it "should not match the dropbox email address if routed to an alias" do
+      @lead = FactoryGirl.create(:lead, :email => "ben@example.com", :access => "Public")
+      @lead_dropbox = FactoryGirl.create(:lead, :email => "dropbox@example.com", :access => "Public")
+
+      @crawler.should_receive(:archive).once
+      @crawler.run
+
+      @lead_dropbox.emails.size.should == 0
+      @lead.emails.size.should == 1
+    end
+  end
+
+  #------------------------------------------------------------------------------
+  describe "Pipeline: creating recipient if s/he was not found" do
     before(:each) do
       mock_connect
       mock_disconnect
@@ -360,7 +237,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should create a contact from the email recipient (To: recipient, Bcc: dropbox)" do
-      mock_message(EMAIL[:plain])
+      mock_message(DROPBOX_EMAILS[:plain])
       @crawler.should_receive(:archive).once
       @crawler.run
 
@@ -371,7 +248,7 @@ describe "IMAP Dropbox" do
     end
 
     it "should create a contact from the forwarded email (To: dropbox)" do
-      mock_message(EMAIL[:forwarded])
+      mock_message(DROPBOX_EMAILS[:forwarded])
       @crawler.should_receive(:archive).once
       @crawler.run
 
@@ -383,7 +260,24 @@ describe "IMAP Dropbox" do
   end
 
   #------------------------------------------------------------------------------
+  describe "Extracting body" do
+    before do
+      @dropbox = FatFreeCRM::MailProcessor::Dropbox.new
+    end
+
+    it "should extract text from multipart text/plain" do
+      text = @dropbox.send(:plain_text_body, Mail.new(DROPBOX_EMAILS[:plain]))
+      text.should be_present
+    end
 
+    it "should extract text and strip tags from multipart text/html" do
+      text = @dropbox.send(:plain_text_body, Mail.new(DROPBOX_EMAILS[:html]))
+      text.should be_present
+      text.should_not match(/<\/?[^>]*>/)
+    end
+  end
+
+  #------------------------------------------------------------------------------
   describe "Default values" do
 
     describe "'access'" do
@@ -404,8 +298,6 @@ describe "IMAP Dropbox" do
       end
 
     end
-
   end
-
 end