gearman-ruby 2.0.0 → 3.0.1

data/test/client_test.rb

@@ -0,0 +1,145 @@
+#!/usr/bin/env ruby
+
+$:.unshift('../lib')
+require 'gearman'
+require 'gearman/testlib'
+require 'test/unit'
+
+class TestClient < Test::Unit::TestCase
+
+  def setup
+    @server = FakeJobServer.new(self)
+    @server_b = FakeJobServer.new(self)
+  end
+
+  def teardown
+    @server.stop
+  end
+  ##
+  # Test the get_socket, return_socket, close_socket, and
+  # get_hostport_for_socket methods of Client.
+  def test_sockets
+    client = Gearman::Client.new
+    hostport = "localhost:#{@server.port}"
+    client.job_servers = [hostport]
+
+    # If we get a socket, return it, and request a socket for the same
+    # host, we should get the original one again.
+    origsock = client.get_socket(client.get_job_server)
+    client.return_socket(origsock)
+    assert_equal(origsock, client.get_socket(client.get_job_server))
+    assert_equal(hostport, client.get_hostport_for_socket(origsock))
+
+    # We should get another socket if we call get_socket again.
+    newsock = client.get_socket(client.get_job_server)
+    assert_not_equal(origsock, newsock)
+    assert_equal(hostport, client.get_hostport_for_socket(newsock))
+
+    # They should be closed when we call close_socket, and we should get
+    # a new socket the next time.
+    client.close_socket(origsock)
+    client.close_socket(newsock)
+    assert(origsock.closed?)
+    assert(newsock.closed?)
+    assert_not_equal(origsock, client.get_socket(client.get_job_server))
+  end
+
+  ##
+  # We check that the client does not fail if at least one
+  # server is up.
+  def test_first_connection_server_down_retry
+    client = Gearman::Client.new
+    hostport_good = "localhost:#{@server.port}"
+    hostport_bad = "nonexistent:8080"
+    # TODO: why does this fail?
+    # hostport_bad = "localhost:#{@server.port+x}" for x in (1..5)
+
+    client.job_servers = [hostport_bad,hostport_good]
+
+    first_requested_server = client.get_job_server
+    assert(first_requested_server == hostport_bad)
+
+    begin
+      result = client.get_socket(first_requested_server)
+      assert(false)
+    rescue RuntimeError
+      assert(true)
+    end
+
+    assert(client.bad_servers.size == 1)
+
+    second_requested_server = client.get_job_server
+    assert(second_requested_server == hostport_good)
+    begin
+      client.get_socket(second_requested_server)
+      assert(true)
+    rescue RuntimeError
+      assert(false)
+    end
+  end
+
+  ##
+  # We check that the client raises a fatal exception if server fails
+  # while connected.
+  def test_client_down_if_server_down
+    client = Gearman::Client.new
+    hostport_good = "localhost:#{@server.port}"
+    hostport_good_b = "localhost:#{@server_b.port}"
+
+     client.job_servers = [hostport_good,hostport_good_b]
+
+    # We simulate a gearmand server failure after receiving some data
+    # from the client
+    Thread.new do
+      server_socket = @server.expect_connection
+      @server.expect_anything_and_close_socket(server_socket)
+    end
+
+    taskset = Gearman::TaskSet.new(client)
+
+    task = Gearman::Task.new('sleep', 20)
+    task.on_complete {|d| puts d }
+
+    should_be_true = true
+    begin
+      taskset.add_task(task)
+      should_be_true = false
+    rescue Exception => ex
+    end
+    assert(should_be_true)
+  end
+
+  def test_option_request_exceptions
+    this_server = FakeJobServer.new(self)
+    Thread.new do
+      server_socket = this_server.expect_connection
+      this_server.expect_request(server_socket, "option_req", "exceptions")
+      this_server.send_response(server_socket, :job_created, 'a')
+    end
+    client = Gearman::Client.new
+    hostport = "localhost:#{this_server.port}"
+    client.job_servers = [hostport]
+    client.option_request("exceptions")
+  end
+
+  def test_option_request_bad
+    this_server = FakeJobServer.new(self)
+    Thread.new do
+      server_socket = this_server.expect_connection
+      this_server.expect_request(server_socket, "option_req", "cccceptionsccc")
+      this_server.send_response(server_socket, :exception, 'a')
+    end
+
+    client = Gearman::Client.new
+    hostport = "localhost:#{this_server.port}"
+    client.job_servers = [hostport]
+    begin
+      client.option_request("cccceptionsccc")
+      assert(false)
+    rescue Gearman::ProtocolError
+      assert(true)
+    end
+  end
+
+
+end

data/test/mock_client_test.rb

@@ -0,0 +1,629 @@
+#!/usr/bin/env ruby
+
+$:.unshift('../lib')
+require 'gearman'
+require 'gearman/testlib'
+require 'test/unit'
+require 'thread'
+
+Thread.abort_on_exception = true
+
+class TestClient < Test::Unit::TestCase
+  ##
+  # Do a simple test of the functionality of the client code.
+  def test_client
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock, res1, res2 = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('add', '5 2') }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job, "add\000\0005 2") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+
+    # Create a second task.  It should use the same socket as the first.
+    c.exec { task2 = Gearman::Task.new('add', '10 5') }
+    c.exec { task2.on_complete {|d| res2 = d.to_i } }
+    c.exec { taskset.add_task(task2) }
+
+    # Return the response to the first job before the handle for the
+    # second.
+    s.exec { server.send_response(sock, :work_complete, "a\0007") }
+    s.exec { server.expect_request(sock, :submit_job, "add\000\00010 5") }
+    s.exec { server.send_response(sock, :job_created, 'b') }
+
+    # After the client waits on the taskset, send the response to the
+    # second job.
+    c.exec { taskset.wait }
+    s.exec { server.send_response(sock, :work_complete, "b\00015") }
+    c.wait
+    s.wait
+
+    # Check that we got the right answers.
+    assert_equal(7, res1)
+    assert_equal(15, res2)
+  end
+
+  ##
+  # Tests that the high priority option can be set in a job request
+  def test_client_submit_priority_high
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock, res1, res2 = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('add', '5 2', { :priority => :high }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job_high, "add\000\0005 2") }
+  end
+
+  ##
+  # Tests that the low priority option can be set in a job request
+  def test_client_submit_priority_low
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock, res1, res2 = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('add', '5 2', { :priority => :low }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job_low, "add\000\0005 2") }
+  end
+
+
+  ##
+  # Check that the client sends a correct background job request
+  def test_client_submit_background
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock, res1, res2 = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('add', '5 2', { :background => :true }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job_bg, "add\000\0005 2") }
+  end
+
+  ##
+  # Check that the client sends a correct background job with high priority request
+  def test_client_submit_background
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock, res1, res2 = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('add', '5 2', { :background => :true, :priority => :high }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job_high_bg, "add\000\0005 2") }
+  end
+
+  ##
+  # Check that the client sends a correct background job with low priority request
+  def test_client_submit_background
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock, res1, res2 = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('add', '5 2', { :background => :true, :priority => :low }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job_low_bg, "add\000\0005 2") }
+  end
+  
+  ##
+  # Test Client#do_task.
+  def test_do_task
+    server = FakeJobServer.new(self)
+    client, sock, res = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { res = client.do_task('add', '5 2').to_i }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job, "add\000\0005 2") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+    s.exec { server.send_response(sock, :work_complete, "a\0007") }
+    c.wait
+    s.wait
+
+    assert_equal(7, res)
+
+    c.exec { res = client.do_task('add', '1 2') }
+    s.exec { server.expect_request(sock, :submit_job, "add\000\0001 2") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+    s.exec { server.send_response(sock, :work_fail, 'a') }
+    c.wait
+    s.wait
+
+    assert_equal(nil, res)
+  end
+
+  ##
+  # Test that Gearman::Task's callback's get called when they should.
+  def test_callbacks
+    server = FakeJobServer.new(self)
+    client, task, taskset, sock = nil
+    failed, retries, num, den, warning = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    task = Gearman::Task.new('foo', 'bar', { :retry_count => 3 })
+    task.on_fail { failed = true }
+    task.on_retry {|r| retries = r }
+    task.on_status {|n,d| num = n.to_i; den = d.to_i }
+    task.on_warning {|msg| warning = msg }
+    received_data = ""
+    task.on_data {|data| received_data << data }
+
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    # Send 4 failures back to the client.
+    c.exec { taskset.wait }
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000bar") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+    s.exec { server.send_response(sock, :work_fail, 'a') }
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000bar") }
+    s.exec { server.send_response(sock, :job_created, 'b') }
+    s.exec { server.send_response(sock, :work_fail, 'b') }
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000bar") }
+    s.exec { server.send_response(sock, :job_created, 'c') }
+    s.exec { server.send_response(sock, :work_status, "c\0001\0002") }
+    s.exec { server.send_response(sock, :work_fail, 'c') }
+    s.exec { server.send_response(sock, :job_created, 'd') }
+    s.exec { server.send_response(sock, :work_data, "d\000data chunk 1") }
+    s.exec { server.send_response(sock, :work_data, "d\000data chunk 2") }
+    s.exec { server.send_response(sock, :work_warning, "d\000warning") }
+    s.exec { server.send_response(sock, :work_fail, 'd') }
+    c.wait
+    s.wait
+
+    assert_equal(true, failed)
+    assert_equal(3, retries)
+    assert_equal(1, num)
+    assert_equal(2, den)
+    assert_equal("data chunk 1data chunk 2", received_data)
+    assert_equal("warning", warning)
+  end
+
+  def test_failure
+    server = FakeJobServer.new(self)
+    client, task1, task2, taskset, sock = nil
+    res1, res2, fail1, fail2, setres = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task1 = Gearman::Task.new('func1', 'a') }
+    c.exec { task1.on_complete {|d| res1 = d } }
+    c.exec { task1.on_fail { fail1 = true } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task1) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job, "func1\000\000a") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+
+    c.exec { task2 = Gearman::Task.new('func2', 'b') }
+    c.exec { task2.on_complete {|d| res2 = d } }
+    c.exec { task2.on_fail { fail2 = true } }
+    c.exec { taskset.add_task(task2) }
+
+    s.exec { server.expect_request(sock, :submit_job, "func2\000\000b") }
+    s.exec { server.send_response(sock, :job_created, 'b') }
+
+    s.exec { server.send_response(sock, :work_complete, "a\000a1") }
+    s.exec { server.send_response(sock, :work_fail, "b") }
+
+    c.exec { setres = taskset.wait }
+    c.wait
+    s.wait
+
+    assert_equal('a1', res1)
+    assert_equal(nil, res2)
+    assert_equal(nil, fail1)
+    assert_equal(true, fail2)
+    assert_equal(false, setres)
+  end
+
+  def test_exception
+    server = FakeJobServer.new(self)
+    client, task, taskset, sock = nil
+    res,exception, setres = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { task = Gearman::Task.new('func2', 'b') }
+    c.exec { task.on_complete {|d| res = d } }
+    c.exec { task.on_exception {|message| exception=message; false } }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task) }
+    s.exec { sock = server.expect_connection }
+
+    s.exec { server.expect_request(sock, :submit_job, "func2\000\000b") }
+    s.exec { server.send_response(sock, :job_created, 'b') }
+
+    s.exec { server.send_response(sock, :work_exception, "b\0exceptionmsg") }
+
+    c.exec { setres = taskset.wait }
+    c.wait
+    s.wait
+
+    assert_equal(nil, res)
+    assert_equal('exceptionmsg',exception)
+  end
+
+  ##
+  # Test that user-supplied uniq values are handled correctly.
+  def test_uniq
+    server1 = FakeJobServer.new(self)
+    server2 = FakeJobServer.new(self)
+    client = nil
+    sock1, sock2 = nil
+    taskset = nil
+    task1, task2, task3, task4 = nil
+    res1, res2, res3, res4 = nil
+    hostport1 = "localhost:#{server1.port}"
+    hostport2 = "localhost:#{server2.port}"
+
+    s1 = TestScript.new
+    s2 = TestScript.new
+    c = TestScript.new
+
+    server1_thread = Thread.new { s1.loop_forever }.run
+    server2_thread = Thread.new { s2.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new }
+    c.exec { client.job_servers = [hostport1, hostport2] }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+
+    # Submit a task with uniq key 'u' to the first server.
+    c.exec { client.test_hostport = hostport1 }
+    c.exec { task1 = Gearman::Task.new('func1', 'arg', { :uniq => 'u' }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset.add_task(task1) }
+
+    s1.exec { sock1 = server1.expect_connection }
+    s1.wait
+
+    s1.exec { server1.expect_request(
+      sock1, :submit_job, "func1\000#{'u'.hash}\000arg") }
+    s1.exec { server1.send_response(sock1, :job_created, 'a') }
+
+    # If we submit a second task with the same key, it should get sent to
+    # the same server.
+    c.exec { client.test_hostport = hostport2 }
+    c.exec { task2 = Gearman::Task.new('func1', 'arg2', { :uniq => 'u' }) }
+    c.exec { task2.on_complete {|d| res2 = d.to_i } }
+    c.exec { taskset.add_task(task2) }
+
+    s1.exec { server1.expect_request(
+      sock1, :submit_job, "func1\000#{'u'.hash}\000arg2") }
+    s1.exec { server1.send_response(sock1, :job_created, 'a') }
+
+    # When we create a task with key 'a', it should go to the second
+    # server.
+    c.exec { task3 = Gearman::Task.new('func1', 'arg', { :uniq => 'a' }) }
+    c.exec { task3.on_complete {|d| res3 = d.to_i } }
+    c.exec { taskset.add_task(task3) }
+
+    s2.exec { sock2 = server2.expect_connection }
+    s2.wait
+
+    s2.exec { server2.expect_request(
+      sock2, :submit_job, "func1\000#{'a'.hash}\000arg") }
+    s2.exec { server2.send_response(sock2, :job_created, 'b') }
+
+    # If we tell the client to use the first server again and create
+    # another job with no uniq key, it should go back to the first server.
+    c.exec { client.test_hostport = hostport1 }
+    c.exec { task4 = Gearman::Task.new('func1', 'arg') }
+    c.exec { task4.on_complete {|d| res4 = d.to_i } }
+    c.exec { taskset.add_task(task4) }
+
+    s1.exec { server1.expect_request(
+      sock1, :submit_job, "func1\000\000arg") }
+    s1.exec { server1.send_response(sock1, :job_created, 'c') }
+
+    # Send back responses for all the handles we've handed out and make
+    # sure that we got what we expected.
+    c.exec { taskset.wait }
+    s1.exec { server1.send_response(sock1, :work_complete, "a\0001") }
+    s2.exec { server2.send_response(sock2, :work_complete, "b\0002") }
+    s1.exec { server1.send_response(sock1, :work_complete, "c\0003") }
+
+    c.wait
+    s1.wait
+    s2.wait
+
+    assert_equal(1, res1)
+    assert_equal(1, res2)
+    assert_equal(2, res3)
+    assert_equal(3, res4)
+
+    c.wait
+    s1.wait
+    s2.wait
+  end
+
+  ##
+  # Test that '-' uniq values work correctly.
+  def test_uniq_dash
+    server1 = FakeJobServer.new(self)
+    server2 = FakeJobServer.new(self)
+    client, taskset, sock1, sock2 = nil
+    task1, task2, task3 = nil
+    res1, res2, res3 = nil
+    hostport1 = "localhost:#{server1.port}"
+    hostport2 = "localhost:#{server2.port}"
+
+    s1 = TestScript.new
+    s2 = TestScript.new
+    c = TestScript.new
+
+    server1_thread = Thread.new { s1.loop_forever }.run
+    server2_thread = Thread.new { s2.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new }
+    c.exec { client.job_servers = [hostport1, hostport2] }
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+
+    # The first task uses uniq = '-' with the argument 'arg'.
+    c.exec { client.test_hostport = hostport1 }
+    c.exec { task1 = Gearman::Task.new('func1', 'arg', { :uniq => '-' }) }
+    c.exec { task1.on_complete {|d| res1 = d.to_i } }
+    c.exec { taskset.add_task(task1) }
+
+    s1.exec { sock1 = server1.expect_connection }
+    s1.wait
+
+    s1.exec { server1.expect_request(
+      sock1, :submit_job, "func1\000#{'arg'.hash}\000arg") }
+    s1.exec { server1.send_response(sock1, :job_created, 'a') }
+
+    # The second task uses the same arg, so it should be merged with the
+    # first by the server (and also be executed on the first server, even
+    # though we've changed the client to use the second by default).
+    c.exec { client.test_hostport = hostport2 }
+    c.exec { task2 = Gearman::Task.new('func1', 'arg', { :uniq => '-' }) }
+    c.exec { task2.on_complete {|d| res2 = d.to_i } }
+    c.exec { taskset.add_task(task2) }
+
+    s1.exec { server1.expect_request(
+      sock1, :submit_job, "func1\000#{'arg'.hash}\000arg") }
+    s1.exec { server1.send_response(sock1, :job_created, 'a') }
+
+    # The third task uses 'arg2', so it should not be merged and instead
+    # run on the second server.
+    c.exec { task3 = Gearman::Task.new('func1', 'arg2', { :uniq => '-' }) }
+    c.exec { task3.on_complete {|d| res3 = d.to_i } }
+    c.exec { taskset.add_task(task3) }
+
+    s2.exec { sock2 = server2.expect_connection }
+    s2.wait
+
+    s2.exec { server2.expect_request(
+      sock2, :submit_job, "func1\000#{'arg2'.hash}\000arg2") }
+    s2.exec { server2.send_response(sock2, :job_created, 'b') }
+
+    # Send back results for the two handles that we've handed out.
+    c.exec { taskset.wait }
+    s1.exec { server1.send_response(sock1, :work_complete, "a\0001") }
+    s2.exec { server2.send_response(sock2, :work_complete, "b\0002") }
+
+    c.wait
+    s1.wait
+    s2.wait
+
+    assert_equal(1, res1)
+    assert_equal(1, res2)
+    assert_equal(2, res3)
+  end
+
+  ##
+  # Test that NUL bytes in returned data are preserved.
+  def test_nuls_in_data
+    server = FakeJobServer.new(self)
+    client, sock, res = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    c.exec { res = client.do_task('foo', nil) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+    s.exec { server.send_response(sock, :work_complete, "a\0001\0002\0003") }
+    c.wait
+    s.wait
+
+    assert_equal("1\0002\0003", res)
+  end
+
+  ##
+  # Test that clients time out when the server sends a partial packet and
+  # then hangs.
+  def test_read_timeouts
+    server = FakeJobServer.new(self)
+    client, sock, task, taskset, res = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+
+    # First, create a new task.  The server claims to be sending back a
+    # packet with 1 byte of data, but actually sends an empty packet.  The
+    # client should time out after 0.1 sec.
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { task = Gearman::Task.new('foo', 'bar') }
+    c.exec { client.task_create_timeout_sec = 0.1 }
+    c.exec { res = taskset.add_task(task) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000bar") }
+    s.exec { server.send_response(sock, :job_created, '', 1) }
+    c.wait
+    s.wait
+
+    assert_equal(false, res)
+
+    # Now create a task, but only return a partial packet for
+    # work_complete.  The client should again time out after 0.1 sec.
+    c.exec { res = taskset.add_task(task) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000bar") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+    c.exec { res = taskset.wait(0.1) }
+    s.exec { server.send_response(sock, :work_complete, "a\000", 3) }
+    c.wait
+    s.wait
+
+    assert_equal(false, res)
+  end
+  
+  ##
+  # Tests partial data responses (work_data)
+  def test_chunked_data
+    server = FakeJobServer.new(self)
+    client, sock, task, taskset, res = nil
+
+    s = TestScript.new
+    c = TestScript.new
+
+    server_thread = Thread.new { s.loop_forever }.run
+    client_thread = Thread.new { c.loop_forever }.run
+
+    c.exec { client = Gearman::Client.new("localhost:#{server.port}") }
+    task = Gearman::Task.new('foo', 'bar', { :retry_count => 3 })
+    received_data = ""
+    task.on_data {|data| received_data << data }
+    task.on_complete {|data| received_data << data}
+
+    c.exec { taskset = Gearman::TaskSet.new(client) }
+    c.exec { taskset.add_task(task) }
+    s.exec { sock = server.expect_connection }
+    s.wait
+
+    c.exec { taskset.wait }
+    s.exec { server.expect_request(sock, :submit_job, "foo\000\000bar") }
+    s.exec { server.send_response(sock, :job_created, 'a') }
+    s.exec { server.send_response(sock, :work_data, "a\000data chunk 1\n") }
+    s.exec { server.send_response(sock, :work_data, "a\000data chunk 2\n") }
+    s.exec { server.send_response(sock, :work_complete, "a\000data complete") }
+    c.wait
+    s.wait
+
+    assert_equal("data chunk 1\ndata chunk 2\ndata complete", received_data)
+  end
+end