Raise a real exception on no data

GitHub: fix GH-25

Reported by Herwin. Thanks!!!

Author: kou

lib/xmlrpc/client.rb

@@ -526,8 +526,12 @@ def do_rpc(request, async=false)
         end
       end
 
+      if data.nil?
+        raise "No data"
+      end
+
       expected = resp["Content-Length"] || "<unknown>"
-      if data.nil? or data.bytesize == 0
+      if data.bytesize == 0
         raise "Wrong size. Was #{data.bytesize}, should be #{expected}"
       end
 

test/test_client.rb

@@ -299,6 +299,18 @@ def test_bad_content_type
       assert_equal expected, resp
     end
 
+    def test_no_data
+      responses = {
+        '/foo' => [ Fake::Response.new(nil, [['Content-Type', 'text/xml']]) ]
+      }
+
+      client = fake_client(responses).new2 'http://example.org/foo'
+
+      assert_raise(RuntimeError.new("No data")) do
+        client.call('wp.getUsersBlogs', 'tlo', 'omg')
+      end
+    end
+
     def test_i8_tag
       fh = read('blog.xml').gsub(/string/, 'i8')