Handle rate limit responses gracefully

Author: derrickreimer

lib/drip/response.rb

@@ -44,16 +44,20 @@ def parse_meta
     end
 
     def parse_members
-      {}.tap do |members|
-        if body.is_a?(Hash)
-          body.each do |key, value|
-            klass = if value.is_a?(Array)
-              Drip::Collections.find_class(key)
-            else
-              Drip::Resources.find_class(key)
-            end
+      if status == 429
+        body
+      else
+        {}.tap do |members|
+          if body.is_a?(Hash)
+            body.each do |key, value|
+              klass = if value.is_a?(Array)
+                Drip::Collections.find_class(key)
+              else
+                Drip::Resources.find_class(key)
+              end
 
-            members[key] = klass.new(value)
+              members[key] = klass.new(value)
+            end
           end
         end
       end

test/drip/response_test.rb

@@ -121,4 +121,16 @@ class Drip::ResponseTest < Drip::TestCase
       assert_equal 1, @subject.subscribers.count
     end
   end
+
+  context "rate limit response" do
+    setup do
+      @body = {"message"=>"API rate limit exceeded. Please try again in an hour."}
+      @subject = Drip::Response.new(429, @body)
+    end
+
+    should "parse correctly" do
+      assert_equal 429, @subject.status
+      assert_equal @body["message"], @subject.message
+    end
+  end
 end