bodyparts 0.1.7 → 0.1.8

data/VERSION

@@ -1 +1 @@
-0.1.7
+0.1.8

data/bodyparts.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{bodyparts}
-  s.version = "0.1.7"
+  s.version = "0.1.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Max Ogden"]

data/lib/bodyparts.rb

@@ -11,7 +11,7 @@ class BodyParts
   
   def self.extract_tmail_attributes(tmail_object)
     if mail_encoding = tmail_object['content_transfer_encoding']
-      content_encoding = mail_encoding.to_s
+      content_encoding = mail_encoding.to_s.downcase
     else
       content_encoding = "not known"
     end
@@ -50,10 +50,11 @@ class BodyParts
       else raise "You must pass in either a TMail or Mail object or raw email source text"
     end
     
-    body = mail_attributes[:body]
-    
-    if mail_attributes[:content_encoding] == 'base64'
-      body = Base64.decode64 body
+    raw_body = mail_attributes[:body]
+    body = case mail_attributes[:content_encoding]
+      when "base64" then Base64.decode64 raw_body
+      when "quoted-printable" then Mail::Encodings::QuotedPrintable.decode raw_body
+      else raw_body
     end
 
     matches = rules.map {|rule| body.match(rule[:reply_delimiter])}.compact!

data/spec/bodyparts_spec.rb

@@ -41,11 +41,15 @@ describe "BodyParts" do
     BodyParts.find_reply_in(message)[:new_message].should == with_attachment[:reply_text]
   end
   
-  [TMail::Mail, Mail].each do |mail_type|
-    it "should correctly parse base64 encoded #{mail_type} messages" do
-      base64_encoded = FakeMessage.fake_emails[:base64_encoded]
-      message = FakeMessage.new_mail(mail_type, base64_encoded[:headers])
-      BodyParts.find_reply_in(message)[:new_message].should == base64_encoded[:reply_text]
-    end
+  describe "encodings" do  
+    [TMail::Mail, Mail].each do |mail_type|
+      ["base64", "Quoted-printable"].each do |encoding|
+        it "should correctly parse #{encoding} encoded #{mail_type} messages" do
+          encoded_message = FakeMessage.fake_emails[encoding.to_sym]
+          message = FakeMessage.new_mail(mail_type, encoded_message[:headers])
+          BodyParts.find_reply_in(message)[:new_message].strip.should == encoded_message[:reply_text]
+        end
+      end
+    end      
   end
 end

data/spec/emails.yml

@@ -140,11 +140,20 @@
     "email_with_attachment"
   :reply_text:
     HELLO
-:base64_encoded:
+:base64:
   :reply_text:
     INSTAIN REPLY. DO WAY
   :headers:
     content_transfer_encoding:
       base64
     body:
-      SU5TVEFJTiBSRVBMWS4gRE8gV0FZ\n
+      SU5TVEFJTiBSRVBMWS4gRE8gV0FZ\n
+:"Quoted-printable":
+  :reply_text:
+    To make your upload take a significantly less amount of time, you can use Windows Movie Maker to shrink your video before uploading it.
+  :headers:
+    content_transfer_encoding:
+      Quoted-printable
+    body: |
+      To make your upload take a significantly less amount of time, you can use W=
+      indows Movie Maker to shrink your video before uploading it.=20

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 7
-  version: 0.1.7
+  - 8
+  version: 0.1.8
 platform: ruby
 authors: 
 - Max Ogden