Merge pull request #62 from yyuu/batch-metrics-should-return-response

`Dogapi::Client#batch_metrics` should return response from API

lib/dogapi/facade.rb

@@ -94,7 +94,7 @@ def batch_metrics()
       @metric_svc.switch_to_batched
       begin
         yield
-        @metric_svc.flush_buffer
+        @metric_svc.flush_buffer # flush_buffer should returns the response from last API call
       ensure
         @metric_svc.switch_to_single
       end

lib/dogapi/v1/metric.rb

@@ -35,11 +35,13 @@ def submit_to_api(metric, points, scope, options = {})
       def submit_to_buffer(metric, points, scope, options = {})
         payload = self.make_metric_payload(metric, points, scope, options)
         @buffer << payload
+        return 200, {}
       end
 
       def flush_buffer()
-        self.upload(@buffer)
+        payload = @buffer
         @buffer = nil
+        self.upload(payload)
       end
 
       alias :submit :submit_to_api

spec/facade_spec.rb

@@ -17,6 +17,7 @@
       @service.instance_variable_set(:@uploaded, Array.new)
       def @service.upload payload
         @uploaded << payload
+        return 200, {}
       end
     end
 
@@ -49,10 +50,11 @@ def @service.upload payload
     end
 
     it "emit_points can be batched" do
-      @dogmock.batch_metrics do
+      code, resp = @dogmock.batch_metrics do
         @dogmock.emit_point("metric.name", 1, :type => 'counter')
         @dogmock.emit_point("othermetric.name", 2, :type => 'counter')
       end
+      expect(code).to eq 200
       # Verify that we uploaded what we expected
       uploaded =  @service.instance_variable_get(:@uploaded)
       expect(uploaded.length).to eq 1