carterdte_smtp_filter 0.0.3.6 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7055df6b6f82a4aad397d0fbf3ca159a0dc16d91
-  data.tar.gz: 8a86125592c46b870b02d3fe4afcd2cfea7370e5
+  metadata.gz: 48b021452547b95cf8e39e280ba9313cfbf42b95
+  data.tar.gz: f48def6d7ee510c4b5433e5ff0e994fa50f8e22e
 SHA512:
-  metadata.gz: 32c766d0b421c68ad68553d0ee19584cc07c50fd8ed8ede6fbdf9d4025d3c7366953dbf7c1ae01ff0e487b1836f56c9a4a2e0cf342bcce41c03cff726926438f
-  data.tar.gz: 80b13d792cc64598012d976ff721603fb5324be5fea6db46606bee4d4f841cb59b109007d34a0323336efc4ed2c45e37fbfd0fc82ae26a7a879ee056d35bb3bd
+  metadata.gz: b436782a20003d01854a65b571da4781b76468b8f64d28c9bbc014a11cf7811daaeae108f9901c2ba65fc07e807db232fbc02262dd43f4b0267ebe6f9ace0133
+  data.tar.gz: 236b3d79c940ae603b9e3fc59099b844849616138bb490a0701a45c7a2889884c1486199e19f916ac6b117bc3794d4708a9f38c8df1438ec3b17895a95c9a27a

data/bin/carterdte_smtp_filter

@@ -32,6 +32,22 @@ if options[:config]
   logger = CarterdteSmtpFilter.logger
   logger.info("#{Time.now}: Starting CarterDte SMTP Filter v: #{CarterdteSmtpFilter::VERSION}")
 
+  # Create Spool directory
+  CarterdteSmtpFilter::Spool.directory_setup
+  
+  # Create a new Spool Manager
+  spool_manager = CarterdteSmtpFilter::SpoolManager.new
+  
+  # Create a Rufus Scheduler
+  # https://github.com/jmettraux/rufus-scheduler
+  
+  scheduler = Rufus::Scheduler.new
+  
+  # Process de Queue directory every 10 seconds
+  scheduler.every '10s' do
+    spool_manager.async.perform
+  end
+
   # Create a new server instance
   server = CarterdteSmtpFilter::SmtpServer.new()
 
@@ -42,6 +58,7 @@ if options[:config]
   sleep 1
 
   # Run forever and ever
+  scheduler.join
   server.join
 
   # setup exit code

data/carterdte_smtp_filter.gemspec

@@ -24,6 +24,8 @@ Gem::Specification.new do |spec|
   spec.add_dependency "json", "~> 1.8"
   spec.add_dependency "bundler", "~> 1.6"
   spec.add_dependency "nokogiri", "~> 1.6"
+  spec.add_dependency 'sucker_punch', '~> 1.0'
+  spec.add_dependency 'rufus-scheduler', '~> 3.0'
   
   spec.add_development_dependency "rake", "~> 10.4"
   spec.add_development_dependency "guard", "~> 2.12"

data/lib/carterdte_smtp_filter.rb

@@ -6,6 +6,9 @@ require 'time'
 require 'date'
 require 'logger'
 require 'rest-client'
+require 'sucker_punch'
+require 'fileutils'
+require 'rufus-scheduler'
 
 require "carterdte_smtp_filter/version"
 require "carterdte_smtp_filter/config"
@@ -13,6 +16,8 @@ require "carterdte_smtp_filter/smtp_server"
 require "carterdte_smtp_filter/message"
 require "carterdte_smtp_filter/dte"
 require "carterdte_smtp_filter/api_client"
+require "carterdte_smtp_filter/spool"
+require "carterdte_smtp_filter/spool_manager"
 
 module CarterdteSmtpFilter
   

data/lib/carterdte_smtp_filter/api_client.rb

@@ -1,42 +1,64 @@
 module CarterdteSmtpFilter
   
-  module ApiClient
+  class ApiClient
     require 'rest-client'
+    include SuckerPunch::Job
     
-    def self.logger
+    def logger
       CarterdteSmtpFilter.logger
     end
+    
+    def perform(message)
+      push message if valid?(message)
+    end
    
-    def self.push(message)
-      post({payload: message})  
+    def push(message, url = nil)
+      response = post({payload: message.to_json, url: url})
+      return response if response
+      message.enqueue
+    end
+    
+    def valid?(message)
+      return false if CarterdteSmtpFilter::Config::testing
+      is_json?(message.to_json)
+    end
+    
+    def is_json?(message)
+      begin
+        result = JSON.parse message
+      rescue Exception => e
+        result = false
+      end
+      result.is_a? Hash
     end
    
-    def self.api_user
+    def api_user
       CarterdteSmtpFilter::Config::api_user
     end
     
-    def self.api_host
+    def api_host
       CarterdteSmtpFilter::Config::api_host
     end
     
-    def self.api_password
+    def api_password
       CarterdteSmtpFilter::Config::api_password
     end
     
-    def self.post(opts = {})
-      url = opts[:url] || "https://#{CarterdteSmtpFilter::Config::api_host}/dtes"
+    def post(opts = {})
+      protocol = CarterdteSmtpFilter::Config::use_https ? "https" : "http"
+      url = opts[:url] || "#{protocol}://#{CarterdteSmtpFilter::Config::api_host}/messages"
       payload = opts[:payload] || {}
       begin
-        # We make sure we are sending JSON
-        JSON.parse payload
-        logger.info("Post #{payload} to #{url}")
         resource = RestClient::Resource.new url, api_user, api_password
-        return if CarterdteSmtpFilter::Config::testing
+        logger.debug("Post #{payload} to #{url}") if CarterdteSmtpFilter::Config::debug
         response = resource.post payload, :content_type => :json, :accept => :json, :verify_ssl => OpenSSL::SSL::VERIFY_NONE
         logger.info("Api response #{response}")
       rescue Exception => e
-        logger.error("#{e} - #{url}")
-        response = false
+        logger.error("#{e} #{e.http_code} - #{url}")
+        
+        # Esto significa que se trato de enviar un mensaje duplicado
+        # Por lo tanto no se debe considerar como un fallo de envio
+        response = e.http_code == 422 ? true : false
       end
       response
     end

data/lib/carterdte_smtp_filter/config.rb

@@ -2,17 +2,32 @@ module CarterdteSmtpFilter
   
   module Config
     attr_accessor :config
-    OPTIONS = %w(bind_address bind_port return_host return_port elasticsearch_host elasticsearch_port max_connections debug api_user api_password api_host log_file stand_alone testing)
+    
+    SPOOL_DIR = "/var/spool/carter_smtp_filter"
+    
+    def self.spool_dir
+      @config["spool_dir"] || SPOOL_DIR
+    end
     
     def self.parse(file = nil)
       @config = YAML.load_file(file)
     end
     
-    OPTIONS.each do |op|
-      self.class.instance_eval do
-        define_method(op) {@config[op].nil? ? false : @config[op].to_s}
-      end
+    def self.method_missing(m, *args, &block)
+      option = m.to_s
+      return get_value(option) unless /=$/.match(option)
+      set_value(option, args[0])
+    end
+    
+    def self.get_value(option)
+      return false unless @config[option]
+      @config[option].to_s
+    end
+    
+    def self.set_value(option, value)
+      @config[option.to_sym] = value
     end
+    
         
   end
   

data/lib/carterdte_smtp_filter/dte.rb

@@ -76,8 +76,7 @@ module CarterdteSmtpFilter
     end
     
     def fecha_emision
-      time_stamp = get_data "TmstFirmaEnv" if envio?
-      time_stamp = get_data "TmstFirmaResp" if respuesta?
+      time_stamp = get_data "FchEmis"
       Time.parse(time_stamp).to_date
     end
     

data/lib/carterdte_smtp_filter/message.rb

@@ -4,23 +4,42 @@ module CarterdteSmtpFilter
     
     TMP_MESSAGE_DIR = "/tmp/carterdte_smtp_filter"
     
-    attr_accessor :raw_data, :qid, :email, :response, :dte
+    attr_accessor :raw_data, :return_qid, :email, :response, :dte, :qid
     
-    def initialize(raw_data)
+    def initialize(raw_data, qid = nil)
       set_mail_defaults
       @raw_data = raw_data
       @email = Mail.read_from_string raw_data
       @logger = CarterdteSmtpFilter.logger
       @dte = extract_dte
-      @qid = nil
+      @qid = qid || generate_qid
+      @return_qid = nil
+    end
+    
+    def queue_file
+      queue_sub_dir = qid.split(//).first
+      "#{Config::spool_directory}/#{queue_sub_dir}/#{qid}"
+    end
+    
+    def enqueue
+      @logger.info("Queueing message #{qid} - #{queue_file}")
+      begin
+        File.open("#{queue_file}", 'w') { |file| file.write(@raw_data) }  
+      rescue Exception => e
+        @logger.info("Could not save #{queue_file} - #{e}")
+      end
     end
     
     def extract_dte
       return false unless @email.attachments.any?
-      xml_attachments = @email.attachments.select {|m| m.sub_type == "xml"}
+      xml_attachments = @email.attachments.select {|m| File.extname(m.filename) == ".xml"}
       file = xml_attachments.any? ? xml_attachments.first : false
       return false unless file
-      CarterdteSmtpFilter::Dte.new file.body.decoded
+      Dte.new file.body.decoded
+    end
+    
+    def generate_qid
+      (("A".."F").to_a + (0..9).to_a).sample(11).join("")
     end
     
     def has_dte?
@@ -28,20 +47,21 @@ module CarterdteSmtpFilter
     end
     
     def to_json
-      JSON.generate({
-        to: @email.to,
-        from: @email.from,
+      JSON.generate(message: {
+        to: @email.to.first,
+        from: @email.from.first,
+        qid: @qid,
         message_id: @email.message_id,
         cc: @email.cc,
-        date: @email.date.to_s,
-        qid: qid,
-        dte: JSON.parse(@dte.to_json)
+        sent_date: @email.date.to_s,
+        return_qid: @return_qid,
+        dte_attributes: JSON.parse(@dte.to_json)
         })
     end
     
     def set_mail_defaults
       Mail.defaults do
-        delivery_method :smtp, address: CarterdteSmtpFilter::Config::return_host, port: CarterdteSmtpFilter::Config::return_port, 
+        delivery_method :smtp, address: Config::return_host, port: Config::return_port, 
         return_response: true, enable_starttls_auto: false
       end
     end
@@ -50,7 +70,7 @@ module CarterdteSmtpFilter
       return false unless response.status == "250"
       # We suppose the string is like "250 2.0.0 Ok: queued as J5D0AWOR4F8\n"
       return false unless /Ok: queued as/.match response.string
-      @qid = response.string.split(/\s+/).last
+      @return_qid = response.string.split(/\s+/).last
     end
     
     def save_tmp

data/lib/carterdte_smtp_filter/smtp_server.rb

@@ -10,9 +10,13 @@ module CarterdteSmtpFilter
       super
     end
     
+    def carterdte_logger
+      CarterdteSmtpFilter.logger
+    end
+    
     def on_message_data_event(ctx)
       message = CarterdteSmtpFilter::Message.new(ctx[:message][:data])
-
+      carterdte_logger.info("Processing message: #{message.qid}")
       # Save a copy when testing
       # /tmp/carterdte_smtp_filter/<message-id>
       message.save_tmp if CarterdteSmtpFilter::Config::testing
@@ -22,17 +26,18 @@ module CarterdteSmtpFilter
       message.return_email unless CarterdteSmtpFilter::Config::stand_alone
       
       # We send it to CarterDte App
-      CarterdteSmtpFilter::ApiClient.push message.to_json if message.has_dte?
+      @logger.debug("Message DTE: #{message.dte}") if CarterdteSmtpFilter::Config::debug
+      CarterdteSmtpFilter::ApiClient.new.async.perform message if message.has_dte?
     end
     
     # get event before Connection
     def on_connect_event(ctx)
-      CarterdteSmtpFilter.logger.info("Connection from #{ctx[:server][:remote_ip]}")
+      carterdte_logger.info("Connection from #{ctx[:server][:remote_ip]}")
     end
 
     # get event before DISONNECT
     def on_disconnect_event(ctx)
-      CarterdteSmtpFilter.logger.info("Disconnect from #{ctx[:server][:remote_ip]}")
+      carterdte_logger.info("Disconnect from #{ctx[:server][:remote_ip]}")
     end
     
   end

data/lib/carterdte_smtp_filter/spool.rb

@@ -0,0 +1,29 @@
+module CarterdteSmtpFilter
+  
+  module Spool
+    
+    
+    def self.directory_setup
+      build_spool
+      build_spool_structure
+    end
+    
+    def self.spool_directory
+      CarterdteSmtpFilter::Config::spool_directory
+    end
+    
+    def self.build_spool
+      FileUtils.mkdir_p spool_directory
+    end
+    
+    def self.build_spool_structure
+      (("A".."F").to_a + (0..9).to_a).each do |d|
+        base_dir = "#{spool_directory}/#{d.to_s}"
+        FileUtils.mkdir_p base_dir
+      end
+    end
+    
+  end
+  
+  
+end

data/lib/carterdte_smtp_filter/spool_manager.rb

@@ -0,0 +1,38 @@
+module CarterdteSmtpFilter
+  
+  class SpoolManager
+    include SuckerPunch::Job
+    
+    def logger
+      CarterdteSmtpFilter.logger
+    end
+    
+    def perform()
+      logger.info("Checking queued files...")
+      queued_files.each do |qf|
+        resend_message qf
+      end
+    end
+    
+    def resend_message(file_path)
+      qid = queue_id file_path
+      logger.info("Resending #{qid} - #{file_path}")
+      message = CarterdteSmtpFilter::Message.new(File.read(file_path), qid)
+      FileUtils.rm file_path
+      CarterdteSmtpFilter::ApiClient.new.async.perform message
+      message
+    end
+    
+    def queued_files
+      Dir.glob("#{Config::spool_directory}/[A-F0-9]/*")
+    end
+    
+    def queue_id(file_path)
+      File.basename file_path
+    end
+        
+  end
+  
+end
+    
+    

data/lib/carterdte_smtp_filter/version.rb

@@ -1,3 +1,3 @@
 module CarterdteSmtpFilter
-  VERSION = "0.0.3.6"
+  VERSION = "0.0.4"
 end

data/test/fake_api.rb

@@ -30,7 +30,7 @@ class FakeApi < Sinatra::Base
   end
 
   
-  post '/dtes' do
+  post '/messages' do
     request.body.rewind
     data = JSON.parse request.body.read
     status = 200
@@ -38,6 +38,11 @@ class FakeApi < Sinatra::Base
     JSON.generate(data)
   end
   
+  post '/messages/uniq' do
+    status 422
+    ""
+  end
+  
   private
 
   def authenticate(env)

data/test/fixtures/config.yml

@@ -1,5 +1,5 @@
 ---
-#debug: true
+debug: true
 bind_address: 127.0.0.1
 bind_port: 30024
 return_host: 127.0.0.1
@@ -10,4 +10,6 @@ elasticsearch_port: 9200
 api_user: pbruna@example.com
 api_password: 123456
 api_host: "api.dte.zboxapp.com"
-log_file: "./test/tmp/carterdte_smtp_filter.log"
+log_file: "./test/tmp/carterdte_smtp_filter.log"
+spool_directory: "./test/tmp/spool"
+use_https: true

data/test/test_api_client.rb

@@ -1,7 +1,7 @@
 require 'test_helper'
 
 class TestApiClient < Minitest::Test
-  
+  require 'sucker_punch/testing/inline'
 
   def setup
     CarterdteSmtpFilter::Config.parse("./test/fixtures/config.yml")
@@ -11,7 +11,7 @@ class TestApiClient < Minitest::Test
   end
   
   def test_push_should_return_false_if_message_is_not_json
-    response = CarterdteSmtpFilter::ApiClient.push "hola"
+    response = CarterdteSmtpFilter::ApiClient.new.async.perform "hola"
     assert(!response, "Failure message.")
   end
     
@@ -22,22 +22,36 @@ class TestApiClient < Minitest::Test
   end
   
   def test_post_should_workout_unauthorized
-    response = CarterdteSmtpFilter::ApiClient.post({url: "#{@api_url}/api/v1/denied" })
+    response = CarterdteSmtpFilter::ApiClient.new.post({url: "#{@api_url}/api/v1/denied" })
     assert(!response, "Failure message.")
   end
   
   def test_post_should_workout_application_error
-    response = CarterdteSmtpFilter::ApiClient.post({url: "#{@api_url}/api/v1/app_error" })
+    response = CarterdteSmtpFilter::ApiClient.new.post({url: "#{@api_url}/api/v1/app_error" })
     assert(!response, "Failure message.")
   end
   
   def test_post_should_return_json
-    hash = {dte_type: 33, msg_type: "envio"}
-    json = JSON.generate hash
-    response = CarterdteSmtpFilter::ApiClient.post({payload: json})
+    raw_mail = File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+    message = CarterdteSmtpFilter::Message.new raw_mail
+    response = CarterdteSmtpFilter::ApiClient.new.async.perform(message)
     new_hash = JSON.parse response
-    assert_equal("123456", new_hash["id"])
-    assert_equal(hash[:dte_type], new_hash["dte_type"])
+    # assert_equal("123456", new_hash["id"])
+    # assert_equal(hash[:dte_type], new_hash["dte_type"])
+  end
+  
+  def test_post_duplicated_message_should_return_true
+    response = CarterdteSmtpFilter::ApiClient.new.post({url: "#{@api_url}/messages/uniq" })
+    assert(response, "Failure message.")
   end
+  
+  def test_should_save_queue_file_if_negative_response
+    CarterdteSmtpFilter::Spool.directory_setup
+    raw_mail = File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+    message = CarterdteSmtpFilter::Message.new raw_mail
+    CarterdteSmtpFilter::ApiClient.new.push(message, "#{@api_url}/api/v1/app_error")
+    assert(File.file?("#{message.queue_file}"), "Failure message.")
+  end
+  
     
 end

data/test/test_config.rb

@@ -2,19 +2,30 @@ require 'test_helper'
 
 class TestConfig < Minitest::Test
   
+  def setup
+    @config = YAML.load_file("./test/fixtures/config.yml")
+    @config.each do |k,v|
+      @config[k] = v.to_s
+    end
+    CarterdteSmtpFilter::Config.parse("./test/fixtures/config.yml")
+  end
+  
   def test_should_raise_not_file_if_file_not_exists
     assert_raises(Errno::ENOENT) { CarterdteSmtpFilter::Config.parse("/tmp/file") }
   end
   
   def test_should_return_config_values
-    CarterdteSmtpFilter::Config.parse("./test/fixtures/config.yml")
-    assert_equal("127.0.0.1", CarterdteSmtpFilter::Config::bind_address)
-    assert_equal("127.0.0.1", CarterdteSmtpFilter::Config::return_host)
-    assert_equal("30025", CarterdteSmtpFilter::Config::return_port)
-    assert_equal("pbruna@example.com", CarterdteSmtpFilter::Config::api_user)
-    assert_equal("123456", CarterdteSmtpFilter::Config::api_password)
-    assert_equal("api.dte.zboxapp.com", CarterdteSmtpFilter::Config::api_host)
-    assert_equal("./test/tmp/carterdte_smtp_filter.log", CarterdteSmtpFilter::Config::log_file)
+    assert_equal(@config["bind_address"], CarterdteSmtpFilter::Config::bind_address)
+    assert_equal(@config["return_host"], CarterdteSmtpFilter::Config::return_host)
+    assert_equal(@config["return_port"], CarterdteSmtpFilter::Config::return_port)
+    assert_equal(@config["api_user"], CarterdteSmtpFilter::Config::api_user)
+    assert_equal(@config["api_password"], CarterdteSmtpFilter::Config::api_password)
+    assert_equal(@config["api_host"], CarterdteSmtpFilter::Config::api_host)
+    assert_equal(@config["log_file"], CarterdteSmtpFilter::Config::log_file)
+  end
+  
+  def test_should_return_spool_dir_from_config_file_or_default_if_is_not_defined
+    assert_equal("/var/spool/carter_smtp_filter", CarterdteSmtpFilter::Config::spool_dir)
   end
   
 end

data/test/test_dte.rb

@@ -66,9 +66,9 @@ class TestDte < Minitest::Test
   end
   
   def test_dte_fecha_emision
-    date_envio = Time.parse("2015-05-09T11:10:10").to_date
-    date_resultado = Time.parse("2015-05-09T12:30:11").to_date
-    date_problema = Time.parse("2015-05-12T01:03:30").to_date
+    date_envio = Date.parse("2015-05-09")
+    date_resultado = Date.parse("2015-05-11")
+    date_problema = Date.parse("2015-05-11")
     assert_equal(date_resultado, @dte_resultado.fecha_emision)
     assert_equal(date_envio, @dte_envio.fecha_emision)
     assert_equal(date_problema, @dte_problema.fecha_emision)

data/test/test_helper.rb

@@ -12,10 +12,10 @@ require 'fake_api'
 Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new # spec-like progress
 
 # Only log Celluloid errors
-#Celluloid.logger.level = Logger::ERROR
+Celluloid.logger.level = Logger::ERROR
 
 
-#CarterDteSmtp::Config.parse("./test/fixtures/config.yml")
+CarterdteSmtpFilter::Config.parse("./test/fixtures/config.yml")
 
 def check_imap()
   require "mail"
@@ -74,7 +74,7 @@ class ReturnSmtp < MidiSmtpServer::Smtpd
   attr_accessor :mail
 
   def start
-    @logger = Logger.new('/dev/null')
+    @logger = CarterdteSmtpFilter::Config::debug ? CarterdteSmtpFilter.logger : Logger.new("/dev/null")
     @port = CarterdteSmtpFilter::Config::return_port
     @host = CarterdteSmtpFilter::Config::return_host
     super

data/test/test_message.rb

@@ -18,6 +18,13 @@ class TestMessage < Minitest::Test
     @return_stmp.stop
   end
   
+  def test_message_generate_qid_should_generate_a_valid_qid_in_hex_value
+    raw_mail = File.open("./test/fixtures/mail.tmp", "rb").read
+    message = CarterdteSmtpFilter::Message.new raw_mail
+    qid = message.generate_qid
+    assert(qid =~ /^[0-9A-F]+$/ , "Failure message.")
+  end
+  
   def test_message_should_save_server_response_when_returning_email
     raw_mail = File.open("./test/fixtures/mail.tmp", "rb").read
     message = CarterdteSmtpFilter::Message.new raw_mail
@@ -29,17 +36,18 @@ class TestMessage < Minitest::Test
     raw_mail = File.open("./test/fixtures/mail_with_dte.eml", "rb").read
     message = CarterdteSmtpFilter::Message.new raw_mail
     message.return_email
-    assert_equal(message.response.string.split(/\s+/).last, message.qid)  
+    assert_equal(message.response.string.split(/\s+/).last, message.return_qid)  
   end
   
   def test_message_to_json_should_return_json_object_with_message_metada
     message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
     assert(JSON.parse(message.to_json), "No JSON")
     json = JSON.parse(message.to_json)
-    assert_equal(message.email.to, json["to"])
-    assert_equal(message.email.from, json["from"])
-    assert_equal(message.email.date.to_s, json["date"])
-    assert_equal("96529310-8", json["dte"]["rut_emisor"])
+    assert_equal(message.email.to.first, json["message"]["to"])
+    assert_equal(message.email.from.first, json["message"]["from"])
+    assert_equal(message.email.date.to_s, json["message"]["sent_date"])
+    assert_equal("96529310-8", json["message"]["dte_attributes"]["rut_emisor"])
+    assert(json["message"]["qid"])
   end
 
   def test_extract_dte_should_return_false_if_no_attachments
@@ -69,5 +77,18 @@ class TestMessage < Minitest::Test
     assert(File.file?("/tmp/carterdte_smtp_filter/#{message.email.message_id}.eml"), "Failure message.")
   end
   
+  def test_message_queue_file_should_return_the_full_path_of_the_queue_file
+    message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+    full_path = "#{CarterdteSmtpFilter::Config::spool_directory}/#{message.qid.split(//).first}/#{message.qid}"
+    assert_equal(full_path, message.queue_file)
+  end  
+  
+  def test_message_enqueue_must_create_a_file_with_qid_as_name
+    CarterdteSmtpFilter::Spool.directory_setup
+    message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+    message.enqueue
+    assert(File.file?("#{message.queue_file}"), "Failure message.")
+  end
+  
     
 end

data/test/test_spool.rb

@@ -0,0 +1,42 @@
+require 'test_helper'
+
+class TestSpool < Minitest::Test
+  
+  def setup
+    CarterdteSmtpFilter::Config.parse("./test/fixtures/config.yml")
+    @directory = CarterdteSmtpFilter::Config::spool_directory
+  end
+  
+  def teardown
+    FileUtils.rm_r @directory if File.directory? @directory
+  end
+  
+  def test_should_create_spool_directory
+    CarterdteSmtpFilter::Spool.build_spool
+    assert(File.directory?(@directory), "No se creo el spool")
+  end
+  
+  def test_should_not_creatre_the_spool_directory_if_already_exists
+    FileUtils.mkdir_p @directory
+    assert(CarterdteSmtpFilter::Spool.build_spool)
+  end
+  
+  def test_should_create_the_inside_structure
+    CarterdteSmtpFilter::Spool.build_spool_structure
+    (("A".."F").to_a + (0..9).to_a).each do |d|
+      base_dir = "#{@directory}/#{d.to_s}"
+      assert(File.directory?(base_dir), "#{base_dir} no dir" )
+    end
+  end
+  
+  def test_setup_spool_should_do_everything
+    CarterdteSmtpFilter::Spool.directory_setup
+    (("A".."F").to_a + (0..9).to_a).each do |d|
+      base_dir = "#{@directory}/#{d.to_s}"
+      assert(File.directory?(base_dir), "#{base_dir} no dir" )
+    end
+  end
+  
+  
+  
+end

data/test/test_spool_manager.rb

@@ -0,0 +1,66 @@
+require 'test_helper'
+
+class TestSpoolManager < Minitest::Test
+  require 'sucker_punch/testing/inline'
+  
+  def setup
+    @api_host = CarterdteSmtpFilter::Config::api_host
+    @api_url = "http://#{@api_host}"
+    stub_request(:any, /#{@api_host}/).to_rack(FakeApi)
+    CarterdteSmtpFilter::Config.parse("./test/fixtures/config.yml")
+    CarterdteSmtpFilter::Spool.directory_setup
+    @directory = CarterdteSmtpFilter::Config::spool_directory
+    @spool_manager = CarterdteSmtpFilter::SpoolManager.new
+  end
+  
+  def teardown
+    FileUtils.rm_r @directory if File.directory? @directory
+  end
+
+  def test_queued_files_should_return_an_array_queue_files
+    assert_equal(Array, @spool_manager.queued_files.class)
+  end
+  
+  def test_queued_files_elements_should_be_file_names_paths
+    messages = []
+    5.times do 
+      message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+      message.enqueue
+      messages << message
+    end
+    queued_files = @spool_manager.queued_files
+    assert(queued_files.include?(messages.sample.queue_file), "Failure message.")
+  end
+  
+  def test_resend_message_should_resend_the_message_and_erase_the_file
+    message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+    message.enqueue
+    queued_file = @spool_manager.queued_files.first
+    r_message = @spool_manager.resend_message(queued_file)
+    assert(r_message, "Failure message.")
+    assert(!File.file?("#{message.queue_file}"), "Failure message.")
+  end
+  
+  def test_resend_message_should_resend_the_message_and_erase_the_file
+    message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+    message.enqueue
+    queued_file = @spool_manager.queued_files.first
+    r_message = @spool_manager.resend_message(queued_file)
+    assert(r_message, "Failure message.")
+    assert(!File.file?("#{message.queue_file}"), "Failure message.")
+    assert(message.qid == r_message.qid, "Failure message.")
+  end
+  
+  def test_sucker_punch_must_do_all_of_this
+    messages = []
+    5.times do 
+      message = CarterdteSmtpFilter::Message.new File.open("./test/fixtures/mail_with_dte.eml", "rb").read
+      message.enqueue
+      messages << message
+    end
+    @spool_manager.async.perform
+    assert_equal([], @spool_manager.queued_files)
+  end
+
+
+end

metadata

@@ -1,181 +1,209 @@
 --- !ruby/object:Gem::Specification
 name: carterdte_smtp_filter
 version: !ruby/object:Gem::Version
-  version: 0.0.3.6
+  version: 0.0.4
 platform: ruby
 authors:
 - Patricio Bruna
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-12 00:00:00.000000000 Z
+date: 2015-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mail
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.6'
 - !ruby/object:Gem::Dependency
   name: midi-smtp-server
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rest-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.7'
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.8'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
+- !ruby/object:Gem::Dependency
+  name: sucker_punch
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: rufus-scheduler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '10.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '10.4'
 - !ruby/object:Gem::Dependency
   name: guard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.12'
 - !ruby/object:Gem::Dependency
   name: guard-minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.4'
 - !ruby/object:Gem::Dependency
   name: minitest-reporters
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
   name: sinatra
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.20'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.20'
 description: Description - Postfix SMTP Filter to parse DTE files to CarterDTE Platform
@@ -186,7 +214,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -199,6 +227,8 @@ files:
 - lib/carterdte_smtp_filter/dte.rb
 - lib/carterdte_smtp_filter/message.rb
 - lib/carterdte_smtp_filter/smtp_server.rb
+- lib/carterdte_smtp_filter/spool.rb
+- lib/carterdte_smtp_filter/spool_manager.rb
 - lib/carterdte_smtp_filter/version.rb
 - o-ri/bundle
 - o-ri/bundler
@@ -222,6 +252,8 @@ files:
 - test/test_helper.rb
 - test/test_message.rb
 - test/test_smtp_server.rb
+- test/test_spool.rb
+- test/test_spool_manager.rb
 homepage: https://github.com/ZBoxApp/carterdte_smtp_filter
 licenses:
 - MIT
@@ -232,17 +264,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.2
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Postfix SMTP Filter to parse DTE files to CarterDTE Platform
@@ -267,4 +299,6 @@ test_files:
 - test/test_helper.rb
 - test/test_message.rb
 - test/test_smtp_server.rb
+- test/test_spool.rb
+- test/test_spool_manager.rb
 has_rdoc: