mail_daemon 0.0.15 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: e4359f022c6ceae588ca63c6ce25580d5989a40b
-  data.tar.gz: d750731afde3a6fefc52517724a199e0c78a338d
+SHA256:
+  metadata.gz: 5dc49957f5e88b2e212563de19073c524e516a99e7e5717c2aaeaf23b77c6485
+  data.tar.gz: 59ebc8efd0fa48cf99502c237d78f48f2105ef5e747bd978909cf3318a65618b
 SHA512:
-  metadata.gz: 8cd2ce921482159cdc350c559af5e5291f0f856d6b9ef7adcab0d8eb4614af8ccb70210a2a5f50eaf6d6f70d57ce1976e571fb9e5bd02bd81d5d60c00bfc0be9
-  data.tar.gz: ca643bd953856b41407ad048267d57a81b58f1efa638c6f5da4b210424351a79818377e939b82b3a39938f09db74c050839e682cf2d07ac59a253206ec66d287
+  metadata.gz: 68e487850c8a5d3eac0922110a16fdd6a24b7671272198e11810bb95708fa7bda62c7316f729d62000a862fce2e4c07489021eb17e5e2061e153a93531c33a97
+  data.tar.gz: bf1c436e16959004738bb9ede0fedb2cb553ec32c5282dc4a09771788017076819b774080e27afc877bd5715885a5578a2316ef3cfb1a1ca25ed8f54f09149bf

data/.rspec

@@ -0,0 +1,3 @@
+--require spec_helper
+--format documentation
+--color

data/.ruby-version

@@ -1 +1 @@
-ruby-2.2.3
+3.3.6

data/lib/mail_daemon/email_body_parser.rb

@@ -19,8 +19,8 @@ module MailDaemon
       if exit_code == 0
         result
       else
-        puts result if @options[:debug]
-        raise error
+        # @options is not available in class method context, so skip debug output
+        raise error || "Process exited with code #{exit_code}"
       end
     end
 

data/lib/mail_daemon/helpers.rb

@@ -4,7 +4,8 @@ module MailDaemon
       @options = options.inject({}){|memo,(k,v)| memo[k.to_sym] = v; memo}
     end
     def reload_options(options)
-      @options.merge!(options)
+      normalized = options.inject({}){|memo,(k,v)| memo[k.to_sym] = v; memo}
+      @options.merge!(normalized)
     end
     def default_option(name, value)
       @options[name.to_s.to_sym] = value unless @options.has_key?(name.to_s.to_sym)

data/lib/mail_daemon/imap/connection.rb

@@ -11,6 +11,7 @@ module MailDaemon
       def initialize(options, &block)
         setup_options(options)
 
+
         required_option [:host, :username, :password]
 
         default_option :port, 143
@@ -18,10 +19,11 @@ module MailDaemon
         default_option :ssl, false
         default_option :start_tls, false
         default_option :ssl_options, false
-        default_option :sleep_time, 10000
+        default_option :sleep_time, 30
         default_option :imap_timeout, 60
         default_option :idle_recycle_time, 29 * SECONDS_IN_MINUTES
 
+
         @notify_status_block = block
 
         puts "initializing new imap connection for #{@options[:username]}:#{@options[:password]}@#{@options[:host]}:#{@options[:port]} " if @options[:debug]
@@ -152,7 +154,7 @@ module MailDaemon
       end
 
       def running?
-        @imap && !@imap.disconnected?
+        !@imap.nil? && !@imap.disconnected?
       end
 
       private

data/lib/mail_daemon/imap/watcher.rb

@@ -30,6 +30,7 @@ module MailDaemon
       end
 
       def running?
+        return false unless @connection
         @connection.running?
       end
 

data/lib/mail_daemon/imap_watcher.rb

@@ -1,3 +1,6 @@
+require 'redis'
+require 'mysql2'
+require_relative 'encryption'
 
 module MailDaemon
   class ImapWatcher
@@ -10,9 +13,9 @@ module MailDaemon
       raise "MYSQL_HOST environment variable is required" unless ENV["MYSQL_HOST"]
       raise "MYSQL_DATABASE environment variable is required" unless ENV["MYSQL_DATABASE"]
       raise "MYSQL_USERNAME environment variable is required" unless ENV["MYSQL_USERNAME"]
-      raise "MYSQL_PASSWORD environment variable is required" unless ENV["MYSQL_PASSWORD"]
-
-      ENV["MYSQL_PASSWORD"] = "" unless ENV["MYSQL_PASSWORD"]
+      # MYSQL_PASSWORD can be empty string, but not nil
+      raise "MYSQL_PASSWORD environment variable is required" if ENV["MYSQL_PASSWORD"].nil?
+      
       ENV["MYSQL_PORT"] = "3306"
 
       redis_url = URI.parse(ENV["REDIS_URL"])
@@ -34,7 +37,7 @@ module MailDaemon
     def setup_watchers
       # load uptodate config
 
-      watchers = []
+      @watchers = []
       mysql_client do |mysql|
         statement = mysql.prepare("select ea.*, a.nickname from case_blocks_email_accounts ea join case_blocks_accounts a on a.id = ea.account_id where ea.imap_enabled=1")
         result = statement.execute()
@@ -42,8 +45,8 @@ module MailDaemon
           ssl = row["imap_ssl"]==1 ? {:verify_mode => "none"} : false
           decrypted_password = Encryption.new.decrypt(row["imap_encrypted_password"])
           begin
-            watcher = Imap::Connection.new({:email => row["imap_username"], :host => row["imap_host"], :port => row["imap_port"], :password => decrypted_password, :ssl => ssl, :start_tls => row["imap_start_tls"]==1, :name => row["imap_folder_name"], :search_command => row["imap_search_command"], :message_count => row["imap_messages_processed"], :last_delivered_at => row["imap_last_delivered_at"], :delivery_method=>"sidekiq", :delivery_options=>{:redis_url => ENV["REDIS_URL"], :queue => "email_handler", :worker=>"EmailHandlerWorker"}})
-            watchers << watcher
+            watcher = Imap::Connection.new(@options.merge({:email => row["imap_username"], :host => row["imap_host"], :port => row["imap_port"], :password => decrypted_password, :ssl => ssl, :start_tls => row["imap_start_tls"]==1, :name => row["imap_folder_name"], :search_command => row["imap_search_command"], :message_count => row["imap_messages_processed"], :last_delivered_at => row["imap_last_delivered_at"], :delivery_method=>"sidekiq", :delivery_options=>{:redis_url => ENV["REDIS_URL"], :queue => "email_handler", :worker=>"EmailHandlerWorker"}}))
+            @watchers << watcher
           rescue => e
             puts "Unable to create IMAP4 connection for #{row['nickname']}/#{row['name']}. #{e.message}"
           end
@@ -53,15 +56,15 @@ module MailDaemon
     end
 
     def start
-      watchers.each do |watcher|
-        watcher.watch do |message|
+      @watchers.each do |watcher|
+        watcher.wait_for_messages do |message|
           ap message
         end
       end
     end
 
     def stop
-      watchers.map{|watcher| watcher.stop! }
+      @watchers.map{|watcher| watcher.disconnect }
     end
 
     def restart

data/lib/mail_daemon/version.rb

@@ -1,3 +1,3 @@
 module MailDaemon
-  VERSION = "0.0.15"
+  VERSION = "1.0.0"
 end

data/lib/mail_daemon.rb

@@ -6,6 +6,7 @@ require_relative 'mail_daemon/imap/statuses'
 require_relative 'mail_daemon/imap/connection'
 require_relative 'mail_daemon/imap/watcher'
 require_relative 'mail_daemon/encryption'
+require_relative 'mail_daemon/imap_watcher'
 require 'sinatra/base'
 require 'json'
 require 'open3'
@@ -40,7 +41,8 @@ module MailDaemon
     end
 
     def running?
-      @watchers && @watchers.detect{|watcher| watcher.running?}
+      return false unless @watchers
+      @watchers.any?{|watcher| watcher.running?}
     end
 
     # Signal catching
@@ -63,7 +65,7 @@ module MailDaemon
         mailbox[:ssl_options] = {:verify_mode => OpenSSL::SSL::VERIFY_NONE} if mailbox[:ssl]
 
         puts "Setting up watcher for #{mailbox[:username]}" if @options[:debug]
-        @watchers << MailDaemon::Imap::Watcher.new(mailbox.merge(:debug => @options[:debug]))
+        @watchers << MailDaemon::Imap::Watcher.new(@options.merge(mailbox))
       end
       @threads = []
       @watchers.each do |watcher|

data/mail_daemon.gemspec

@@ -18,11 +18,18 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.6"
+  spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "awesome_print"
+  spec.add_development_dependency "rspec", "~> 3.12"
+  spec.add_development_dependency "rspec-mocks", "~> 3.12"
 
-  # spec.add_dependency 'mail'
+  spec.add_dependency 'mail'
+  spec.add_dependency 'sinatra'
+  spec.add_dependency 'nokogiri'
+  spec.add_dependency 'sanitize'
+  spec.add_dependency 'redis'
+  spec.add_dependency 'mysql2'
 
 
 end

data/spec/examples.txt

@@ -0,0 +1,147 @@
+example_id                                            | status | run_time        |
+----------------------------------------------------- | ------ | --------------- |
+./spec/mail_daemon/email_body_parser_spec.rb[1:1:1:1] | passed | 0.00157 seconds |
+./spec/mail_daemon/email_body_parser_spec.rb[1:1:1:2] | passed | 0.00544 seconds |
+./spec/mail_daemon/email_body_parser_spec.rb[1:1:2:1] | passed | 0.01017 seconds |
+./spec/mail_daemon/email_body_parser_spec.rb[1:1:3:1] | passed | 0.00037 seconds |
+./spec/mail_daemon/email_body_parser_spec.rb[1:1:3:2] | passed | 0.00037 seconds |
+./spec/mail_daemon/email_body_parser_spec.rb[1:1:3:3] | passed | 0.00074 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:1:1]       | passed | 0.00086 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:1:2]       | passed | 0.00085 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:1:3]       | passed | 0.18024 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:1:4]       | passed | 0.00276 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:1:5]       | passed | 0.00359 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:2:1]       | passed | 0.00182 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:2:2]       | passed | 0.00142 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:3:1]       | passed | 0.0048 seconds  |
+./spec/mail_daemon/email_handler_spec.rb[1:3:2]       | passed | 0.00195 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:3:3]       | passed | 0.00257 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:3:4]       | passed | 0.00129 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:3:5]       | passed | 0.00175 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:4:1]       | passed | 0.00293 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:4:2]       | passed | 0.00132 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:5:1]       | passed | 0.01028 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:5:2]       | passed | 0.03103 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:5:3]       | passed | 0.00524 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:6:1]       | passed | 0.00067 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:6:2]       | passed | 0.00186 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:7:1]       | passed | 0.00566 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:8:1]       | passed | 0.00277 seconds |
+./spec/mail_daemon/email_handler_spec.rb[1:9:1]       | passed | 0.00195 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:1:1]          | passed | 0.00034 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:1:2]          | passed | 0.00036 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:1:3]          | passed | 0.00053 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:1:4]          | passed | 0.00027 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:2:1]          | passed | 0.00052 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:2:2]          | passed | 0.00151 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:2:3]          | passed | 0.00067 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:2:4]          | passed | 0.00037 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:2:5]          | passed | 0.00147 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:3:1]          | passed | 0.00156 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:3:2]          | passed | 0.00183 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:3:3]          | passed | 0.00047 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:3:4]          | passed | 0.0004 seconds  |
+./spec/mail_daemon/encryption_spec.rb[1:3:5]          | passed | 0.00064 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:4:1]          | passed | 0.00035 seconds |
+./spec/mail_daemon/encryption_spec.rb[1:4:2]          | passed | 0.00047 seconds |
+./spec/mail_daemon/handler_spec.rb[1:1:1]             | passed | 0.00269 seconds |
+./spec/mail_daemon/handler_spec.rb[1:1:2]             | passed | 0.0005 seconds  |
+./spec/mail_daemon/handler_spec.rb[1:1:3]             | passed | 0.00066 seconds |
+./spec/mail_daemon/handler_spec.rb[1:1:4]             | passed | 0.00011 seconds |
+./spec/mail_daemon/handler_spec.rb[1:1:5]             | passed | 0.00009 seconds |
+./spec/mail_daemon/handler_spec.rb[1:2:1:1]           | passed | 0.00019 seconds |
+./spec/mail_daemon/handler_spec.rb[1:2:2:1]           | passed | 0.00029 seconds |
+./spec/mail_daemon/handler_spec.rb[1:2:3:1]           | passed | 0.00012 seconds |
+./spec/mail_daemon/handler_spec.rb[1:3:1]             | passed | 0.00058 seconds |
+./spec/mail_daemon/handler_spec.rb[1:4:1]             | passed | 0.00076 seconds |
+./spec/mail_daemon/handler_spec.rb[1:4:2]             | passed | 0.00153 seconds |
+./spec/mail_daemon/handler_spec.rb[1:5:1]             | passed | 0.00045 seconds |
+./spec/mail_daemon/handler_spec.rb[1:6:1]             | passed | 0.10414 seconds |
+./spec/mail_daemon/handler_spec.rb[1:6:2]             | passed | 0.10433 seconds |
+./spec/mail_daemon/handler_spec.rb[1:6:3]             | passed | 0.10226 seconds |
+./spec/mail_daemon/handler_spec.rb[1:6:4]             | passed | 0.10348 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:1:1]             | passed | 0.00014 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:1:2]             | passed | 0.00016 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:1:3]             | passed | 0.00015 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:1:4]             | passed | 0.00008 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:2:1]             | passed | 0.0001 seconds  |
+./spec/mail_daemon/helpers_spec.rb[1:2:2]             | passed | 0.00007 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:2:3]             | passed | 0.00008 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:3:1]             | passed | 0.00007 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:3:2]             | passed | 0.00009 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:3:3]             | passed | 0.00007 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:3:4]             | passed | 0.00008 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:4:1]             | passed | 0.00008 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:4:2]             | passed | 0.00009 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:4:3]             | passed | 0.00009 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:4:4]             | passed | 0.00008 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:4:5]             | passed | 0.0003 seconds  |
+./spec/mail_daemon/helpers_spec.rb[1:4:6]             | passed | 0.00007 seconds |
+./spec/mail_daemon/helpers_spec.rb[1:4:7]             | passed | 0.00007 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:1]     | passed | 0.0001 seconds  |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:2]     | passed | 0.00012 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:3]     | passed | 0.00012 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:4]     | passed | 0.00012 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:5]     | passed | 0.00008 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:6]     | passed | 0.00008 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:7]     | passed | 0.00006 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:8]     | passed | 0.00007 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:1:9]     | passed | 0.0001 seconds  |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:1]     | passed | 0.00042 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:2]     | passed | 0.001 seconds   |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:3]     | passed | 0.00082 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:4]     | passed | 0.00047 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:5]     | passed | 0.00071 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:6]     | passed | 0.00044 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:7]     | passed | 0.00059 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:2:8]     | passed | 0.00057 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:3:1]     | passed | 0.00335 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:3:2]     | passed | 0.00044 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:3:3]     | passed | 0.00098 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:4:1]     | passed | 0.00065 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:4:2]     | passed | 0.00056 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:4:3]     | passed | 0.00104 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:5:1]     | passed | 0.00117 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:5:2]     | passed | 0.0014 seconds  |
+./spec/mail_daemon/imap/connection_spec.rb[1:5:3]     | passed | 0.00083 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:6:1]     | passed | 0.0001 seconds  |
+./spec/mail_daemon/imap/connection_spec.rb[1:6:2]     | passed | 0.00104 seconds |
+./spec/mail_daemon/imap/connection_spec.rb[1:7:1]     | passed | 0.00226 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:1]         | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:2]         | passed | 0.00006 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:3]         | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:4]         | passed | 0.00005 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:5]         | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:6]         | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:7]         | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:8]         | passed | 0.00003 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:9]         | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:10]        | passed | 0.00003 seconds |
+./spec/mail_daemon/imap/statuses_spec.rb[1:11]        | passed | 0.00002 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:1:1]        | passed | 0.0001 seconds  |
+./spec/mail_daemon/imap/watcher_spec.rb[1:2:1]        | passed | 0.1052 seconds  |
+./spec/mail_daemon/imap/watcher_spec.rb[1:2:2]        | passed | 0.10147 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:3:1]        | passed | 0.00031 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:4:1]        | passed | 0.00079 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:5:1]        | passed | 0.00006 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:5:2]        | passed | 0.00006 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:6:1]        | passed | 0.00005 seconds |
+./spec/mail_daemon/imap/watcher_spec.rb[1:7:1]        | passed | 0.00021 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:1]        | passed | 0.00047 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:2]        | passed | 0.00031 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:3]        | passed | 0.0003 seconds  |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:4]        | passed | 0.00058 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:5]        | passed | 0.00037 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:6]        | passed | 0.00148 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:7]        | passed | 0.00074 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:8]        | passed | 0.00112 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:9]        | passed | 0.00106 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:1:10]       | passed | 0.00141 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:2:1]        | passed | 0.00106 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:2:2]        | passed | 0.00111 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:3:1]        | passed | 0.00066 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:4:1]        | passed | 0.00078 seconds |
+./spec/mail_daemon/imap_watcher_spec.rb[1:4:2]        | passed | 0.00312 seconds |
+./spec/mail_daemon/version_spec.rb[1:1]               | passed | 0.00008 seconds |
+./spec/mail_daemon/version_spec.rb[1:2]               | passed | 0.00041 seconds |
+./spec/mail_daemon/version_spec.rb[1:3]               | passed | 0.002 seconds   |

data/spec/mail_daemon/email_body_parser_spec.rb

@@ -0,0 +1,114 @@
+require 'spec_helper'
+
+RSpec.describe MailDaemon::EmailBodyParser do
+  describe '.parse' do
+    let(:text) { 'Sample email body text' }
+    let(:type) { 'text/plain' }
+
+    context 'when python script succeeds' do
+      it 'returns the parsed result' do
+        expected_result = 'Parsed body'
+        stdin = StringIO.new
+        stdout = StringIO.new(expected_result)
+        stderr = StringIO.new
+        # Process::Status can be compared to 0 (compares exitstatus)
+        process_status = double('Process::Status')
+        allow(process_status).to receive(:==).with(0).and_return(true)
+        wait_thr = instance_double(Thread, value: process_status)
+
+        # Open3.popen3 returns an array when called without a block
+        allow(Open3).to receive(:popen3).with('python', '/Users/stewartmckee/code/talon/test.py') do
+          [stdin, stdout, stderr, wait_thr]
+        end
+
+        result = described_class.parse(text, type)
+        expect(result).to eq(expected_result)
+      end
+
+      it 'writes JSON to stdin' do
+        json_input = { body: text, type: type }.to_json
+        stdin = StringIO.new
+        stdout = StringIO.new('result')
+        stderr = StringIO.new
+        process_status = double('Process::Status')
+        allow(process_status).to receive(:==).with(0).and_return(true)
+        wait_thr = instance_double(Thread, value: process_status)
+
+        allow(Open3).to receive(:popen3) do
+          [stdin, stdout, stderr, wait_thr]
+        end
+
+        described_class.parse(text, type)
+        expect(stdin.string).to eq(json_input)
+      end
+    end
+
+    context 'when python script fails' do
+      it 'raises an error with stderr message' do
+        error_message = 'Python script error'
+        stdin = StringIO.new
+        stdout = StringIO.new('')
+        stderr = StringIO.new(error_message)
+        process_status = double('Process::Status')
+        allow(process_status).to receive(:==).with(0).and_return(false)
+        wait_thr = instance_double(Thread, value: process_status)
+
+        allow(Open3).to receive(:popen3) do
+          [stdin, stdout, stderr, wait_thr]
+        end
+
+        expect { described_class.parse(text, type) }.to raise_error(RuntimeError, error_message)
+      end
+    end
+
+    context 'with different content types' do
+      it 'passes text/plain type' do
+        stdin = StringIO.new
+        stdout = StringIO.new('result')
+        stderr = StringIO.new
+        process_status = double('Process::Status')
+        allow(process_status).to receive(:==).with(0).and_return(true)
+        wait_thr = instance_double(Thread, value: process_status)
+
+        allow(Open3).to receive(:popen3) do
+          [stdin, stdout, stderr, wait_thr]
+        end
+
+        described_class.parse(text, 'text/plain')
+        expect(JSON.parse(stdin.string)['type']).to eq('text/plain')
+      end
+
+      it 'passes text/html type' do
+        stdin = StringIO.new
+        stdout = StringIO.new('result')
+        stderr = StringIO.new
+        process_status = double('Process::Status')
+        allow(process_status).to receive(:==).with(0).and_return(true)
+        wait_thr = instance_double(Thread, value: process_status)
+
+        allow(Open3).to receive(:popen3) do
+          [stdin, stdout, stderr, wait_thr]
+        end
+
+        described_class.parse(text, 'text/html')
+        expect(JSON.parse(stdin.string)['type']).to eq('text/html')
+      end
+
+      it 'defaults to text/plain when type not specified' do
+        stdin = StringIO.new
+        stdout = StringIO.new('result')
+        stderr = StringIO.new
+        process_status = double('Process::Status')
+        allow(process_status).to receive(:==).with(0).and_return(true)
+        wait_thr = instance_double(Thread, value: process_status)
+
+        allow(Open3).to receive(:popen3) do
+          [stdin, stdout, stderr, wait_thr]
+        end
+
+        described_class.parse(text)
+        expect(JSON.parse(stdin.string)['type']).to eq('text/plain')
+      end
+    end
+  end
+end