Return {} for a response with no body instead of raising TypeError

lib/courrier/email/result.rb

@@ -19,7 +19,7 @@ def success? = @success
       private
 
       def parse_body(body)
-        return {} if @response.nil?
+        return {} if body.nil? || body.empty?
 
         begin
           JSON.parse(body)

lib/courrier/subscriber/result.rb

@@ -17,7 +17,7 @@ def success? = @success
       private
 
       def parsed(body)
-        return {} if @response.nil?
+        return {} if body.nil? || body.empty?
 
         begin
           JSON.parse(body)

test/courrier/email/result_test.rb

@@ -31,6 +31,22 @@ def test_initialize_with_error
     assert_equal({}, result.data)
   end
 
+  def test_initialize_with_no_content_response
+    response = Data.define(:code, :body).new(code: "204", body: nil)
+    result = Courrier::Email::Result.new(response: response)
+
+    assert result.success?
+    assert_equal({}, result.data)
+  end
+
+  def test_initialize_with_empty_body
+    response = Data.define(:code, :body).new(code: "202", body: "")
+    result = Courrier::Email::Result.new(response: response)
+
+    assert result.success?
+    assert_equal({}, result.data)
+  end
+
   def test_success_predicate_method
     result = Courrier::Email::Result.new(
       response: Data.define(:code, :body).new(code: "200", body: "{}")

test/courrier/subscriber/result_test.rb

@@ -33,6 +33,22 @@ def test_initialize_with_error
     assert_equal({}, result.data)
   end
 
+  def test_initialize_with_no_content_response
+    response = Data.define(:code, :body).new(code: "204", body: nil)
+    result = Courrier::Subscriber::Result.new(response: response)
+
+    assert result.success?
+    assert_equal({}, result.data)
+  end
+
+  def test_initialize_with_empty_body
+    response = Data.define(:code, :body).new(code: "200", body: "")
+    result = Courrier::Subscriber::Result.new(response: response)
+
+    assert result.success?
+    assert_equal({}, result.data)
+  end
+
   def test_success_predicate_method
     result = Courrier::Subscriber::Result.new(
       response: Data.define(:code, :body).new(code: "200", body: "{}")