iron_mq 4.0.1 → 4.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ae70cddc4f116d5a8d516a5eed1cd722f221df59
-  data.tar.gz: ba911a0556103b6fa9c4b5407b59fd6966b92661
+  metadata.gz: 60ddd32d17b9bf5b091151e1e26915b75a60bb05
+  data.tar.gz: f21165a64e0001ecfff3e9c2144cec2b0cc9c9d2
 SHA512:
-  metadata.gz: 8e07437588ae4af6e20c4816d23db5a88aed43e39311c329a3871751c96147628bf984044d333f270c5b55a8e67af17922fc7bf5694550198cae89fd5f3fea19
-  data.tar.gz: c4a9adda56a141bcad44756ae830310503410019725aa5590d1b36f9aa9449b21ed0ed03db725ad83e8842ba3360c5e6d1d9de3d96aa2b883f150a4335765c7a
+  metadata.gz: d96d56ac39de492bc36e976b8467101bac942c34cf34af3896dc959d5f732b755d784060671e3ab25b7bd68dfd3a077c185e14fab7c4dd80e708fcff8f55611b
+  data.tar.gz: 0dbc4b37f6344681aec4d18b08fd2148738accd8f0abc0c6965ded1325f074e78d56d466d1035f3df8e7fe7189078cb5b076e79796438040a5c4daebc8466c93

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    iron_mq (4.0.1)
+    iron_mq (4.0.3)
       iron_core (>= 0.5.1)
 
 GEM
@@ -20,9 +20,9 @@ GEM
     net-http-persistent (2.8)
     quicky (0.4.0)
     rake (10.0.3)
-    rest (2.2.0)
-      net-http-persistent
+    rest (2.5.0)
       rest-client (>= 0.3.0)
+      typhoeus (>= 0.5.4)
     rest-client (1.6.7)
       mime-types (>= 1.16)
     test-unit (2.5.4)

data/README.md

@@ -116,6 +116,7 @@ all_queues = ironmq.queues.all  # => [#<IronMQ::Queue:...>, ...]
 
 * `page`: The 0-based page to view. The default is 0.
 * `per_page`: The number of queues to return per page. The default is 30, the maximum is 100.
+* `raw`: Set it to true to obtain data in raw format. The default is false.
 
 ```ruby
 queues = ironmq.queues.all(:page => 1, :per_page => 10)

data/lib/iron_mq/client.rb

@@ -37,9 +37,15 @@ module IronMQ
     end
 
     def queues_list(options = {})
+      is_raw = [options.delete(:raw),
+                options.delete('raw')].compact.first
       response = parse_response(get('', options)) # GET base_url
       # returns list of evaluated queues
-      response.each_with_object([]) { |q_info, ret| ret << Queue.new(self, q_info["name"]) }
+      if is_raw
+        response.map{ |q_info| ResponseBase.new(q_info) }
+      else
+        response.map{ |q_info| Queue.new(self, q_info["name"]) }
+      end
     end
 
     alias_method :list, :queues_list

data/lib/iron_mq/messages.rb

@@ -22,20 +22,18 @@ module IronMQ
     def subscribers(options = {})
       response = call_api_and_parse_response(:get, "/subscribers", {}, false)
 
-      response['subscribers'].each_with_object([]) do |subscriber, ret|
-        ret << Subscriber.new(subscriber, self, options)
-      end
+      response['subscribers'].map { |s| Subscriber.new(s, self, options) }
     end
 
     def delete
-      begin
-        call_api_and_parse_response(:delete)
-      rescue Rest::HttpError => ex
-        if ex.code == 404
-          Rest.logger.info("Delete got 404, safe to ignore.")
-        else
-          raise ex
-        end
+      call_api_and_parse_response(:delete)
+    rescue Rest::HttpError => ex
+      if ex.code == 404
+        Rest.logger.info("Delete got 404, safe to ignore.")
+        # return ResponseBase as normal
+        ResponseBase.new({"msg" => "Deleted"})
+      else
+        raise ex
       end
     end
 

data/lib/iron_mq/queues.rb

@@ -12,14 +12,15 @@ module IronMQ
     end
 
     def info
-      info = raw
-      begin
-        # Do not instantiate response
-        info = call_api_and_parse_response(:get, '', {}, false)
-      rescue Rest::HttpError
+      call_api_and_parse_response(:get)
+    rescue Rest::HttpError => ex
+      if ex.code == 404
+        Rest.logger.info("GET Queue#info return 404, treat as new queue, safe to ignore.")
+        # return ResponseBase as normal
+        ResponseBase.new(raw)
+      else
+        raise ex
       end
-
-      ResponseBase.new(info)
     end
 
     def size
@@ -56,6 +57,14 @@ module IronMQ
     # Backward compatibility, better name is `delete`
     def delete_queue
       call_api_and_parse_response(:delete)
+    rescue Rest::HttpError => ex
+      if ex.code == 404
+        Rest.logger.info("Delete got 404, safe to ignore.")
+        # return ResponseBase as normal
+        ResponseBase.new({"msg" => "Deleted"})
+      else
+        raise ex
+      end
     end
 
     # Backward compatibility
@@ -96,14 +105,10 @@ module IronMQ
                batch = true
                # FIXME: This maybe better to process Array of Objects the same way as for single message.
                #
-               #          payload.each_with_object([]) do |msg, res|
-               #            res << options.merge(:body => msg)
-               #          end
+               #          payload.map { |msg| options.merge(:body => msg) }
                #
                #        For now user must pass objects like `[{:body => msg1}, {:body => msg2}]`
-               payload.each_with_object([]) do |msg, res|
-                 res << msg.merge(options)
-               end
+               payload.map { |msg| msg.merge(options) }
              else
                [ options.merge(:body => payload) ]
              end
@@ -113,16 +118,15 @@ module IronMQ
 
       if instantiate
         n = batch ? 2 : 1
-        msg_ids = res["ids"].map { |id| {'id' => id} }
+        msg_ids = res["ids"].map { |id| {"id" => id} }
 
         process_messages(msg_ids, {:n => n})
       else
         if batch
           # FIXME: Return Array of ResponsBase instead, it seems more clear than raw response
           #
-          #          res["ids"].each_with_object([]) do |id, responses|
-          #            responses << ResponseBase.new({"id" => id, "msg" => res["msg"]})
-          #          end
+          #          res["ids"].map { |id| ResponseBase.new({"id" => id, "msg" => res["msg"]}) }
+          #
           ResponseBase.new(res) # Backward capable
         else
           ResponseBase.new({"id" => res["ids"][0], "msg" => res["msg"]})
@@ -190,8 +194,7 @@ module IronMQ
       #   queue.size
       # etc.
       if args.length == 0
-        res = info.send(meth)
-        res ? res : super
+        self.info[meth.to_s]
       else
         super
       end
@@ -207,13 +210,9 @@ module IronMQ
       multiple = wait_for_multiple?(options)
 
       if messages.is_a?(Array) && messages.size > 0
-        if multiple
-          messages.each_with_object([]) do |m, msgs|
-            msgs << Message.new(self, m)
-          end
-        else
-          Message.new(self, messages[0])
-        end
+        msgs = messages.map { |m| Message.new(self, m) }
+
+        multiple ? msgs : msgs[0]
       else
         multiple ? [] : nil
       end

data/lib/iron_mq/response.rb

@@ -22,7 +22,9 @@ module IronMQ
     private
 
     def stringify_keys(hash)
-      hash.keys.each_with_object({}) { |k, res| res[k.to_s] = hash[k] }
+      hash.keys.each { |k| hash[k.to_s] = hash.delete(k) }
+
+      hash
     end
   end
 

data/lib/iron_mq/subscribers.rb

@@ -13,6 +13,14 @@ module IronMQ
     # `options` was kept for backward compatibility
     def delete(options = {})
       @message.call_api_and_parse_response(:delete, path)
+    rescue Rest::HttpError => ex
+      if ex.code == 404
+        Rest.logger.info("Delete got 404, safe to ignore.")
+        # return ResponseBase as normal
+        ResponseBase.new({"msg" => "Deleted"})
+      else
+        raise ex
+      end
     end
 
     alias_method :acknowledge, :delete

data/lib/iron_mq/version.rb

@@ -1,4 +1,4 @@
 module IronMQ
-  VERSION = "4.0.1"
+  VERSION = "4.0.3"
 end
 

data/lib/iron_mq.rb

@@ -1,4 +1,5 @@
 require File.expand_path('iron_mq/response', File.dirname(__FILE__))
+require File.expand_path('iron_mq/subscribers', File.dirname(__FILE__))
 require File.expand_path('iron_mq/queues', File.dirname(__FILE__))
 require File.expand_path('iron_mq/messages', File.dirname(__FILE__))
 require File.expand_path('iron_mq/client', File.dirname(__FILE__))

data/test/test_base.rb

@@ -32,9 +32,9 @@ class TestBase < Test::Unit::TestCase
     @host = "#{config['host'] || "mq-aws-us-east-1.iron.io"}"
 
     @client = IronMQ::Client.new(@config['iron'])
-
-    Rest.logger.level = Logger::DEBUG # this doesn't work for some reason?
-    IronCore::Logger.logger.level = Logger::DEBUG
+    puts "IronMQ::VERSION = #{IronMQ::VERSION}"
+    #Rest.logger.level = Logger::DEBUG # this doesn't work for some reason?
+    #IronCore::Logger.logger.level = Logger::DEBUG
 
     @queue_name = 'ironmq-ruby-tests' # default queue for tests
   end

data/test/test_iron_mq.rb

@@ -159,9 +159,8 @@ class IronMQTests < TestBase
 
     tries = MAX_TRIES
     while tries > 0
-      sleep 0.5
+      sleep 2
       tries -= 1
-      sleep 1
 
       new_msg = queue.get
       # p new_msg
@@ -180,7 +179,7 @@ class IronMQTests < TestBase
     msg = queue.get
     # p msg
     assert msg
-    assert_equal msg.raw['timeout'], 30
+    assert_equal 30, msg.timeout
 
     msg_nil = queue.get
     # p msg_nil
@@ -188,9 +187,8 @@ class IronMQTests < TestBase
 
     tries = MAX_TRIES
     while tries > 0
-      sleep 0.5
+      sleep 2
       tries -= 1
-      sleep 1
 
       new_msg = queue.get
       next if new_msg.nil?
@@ -205,8 +203,9 @@ class IronMQTests < TestBase
     # timeout on get
     res = queue.post("hello world timeout3!")
     msg = queue.get(:timeout => 30)
+    puts "MESSAGE IS #{msg.inspect}"
     assert msg
-    assert_equal msg.raw['timeout'], 30
+    assert_equal msg.timeout, 30
 
     msg_nil = queue.get
     # p msg_nil
@@ -214,14 +213,13 @@ class IronMQTests < TestBase
 
     tries = MAX_TRIES
     while tries > 0
-      sleep 0.5
+      sleep 2
       tries -= 1
-      sleep 1
 
       new_msg = queue.get
       next if new_msg.nil?
 
-      assert_equal new_msg.id, msg.id
+      assert_equal msg.id, new_msg.id
 
       new_msg.delete
       break
@@ -245,7 +243,11 @@ class IronMQTests < TestBase
     #  q = @client.queues.get(:name => "some_queue_that_does_not_exist")
     #end
     queue = @client.queues.get(:name => "some_queue_that_does_not_exist")
-    assert queue.new? == true
+    assert queue.new?
+
+    # create at least one queue
+    queue.post('create queue message')
+    assert_equal queue.new?, false, "queue must exist on the service after post message to"
 
     res = @client.queues.list
     # puts "res.size: #{res.size}"
@@ -261,6 +263,10 @@ class IronMQTests < TestBase
     # res.each do |q| { p q.name }
 
     assert_equal 0, res.size
+
+    # delete queue on test complete
+    resp = queue.delete_queue
+    assert_equal 200, resp.code, "API must response with HTTP 200 status, but returned HTTP #{resp.code}"
   end
 
   def test_delay
@@ -544,7 +550,7 @@ class IronMQTests < TestBase
     val = "hi mr clean"
     queue.post(val)
 
-    sleep 0.5 # make sure the counter has time to update
+    sleep 2 # make sure the counter has time to update
     assert_equal 1, queue.size
 
     queue.clear
@@ -581,7 +587,7 @@ class IronMQTests < TestBase
     while tries > 0
       tries -= 1
       break if 0 == queue.size
-      sleep 0.5
+      sleep 1
     end
     assert_not_equal tries, 0
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: iron_mq
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 4.0.3
 platform: ruby
 authors:
 - Yury Yantsevich
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-20 00:00:00.000000000 Z
+date: 2013-03-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: iron_core