From ab8949ac2e50880bc8adb40080287ad89758d772 Mon Sep 17 00:00:00 2001 From: "mbelshe@chromium.org" Date: Mon, 29 Mar 2010 21:16:54 +0000 Subject: Re-enable the ConnectFailure case. Previously, this test case reported valgrind warnings; the warnings are legit; we were leaking the sessions in this case through two different leaks BUG=29471 TEST=SpdyNetworkTransactionTest.ConnectFailure Review URL: http://codereview.chromium.org/1553005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42995 0039d316-1c4b-4281-b951-d872f2087c98 --- net/spdy/spdy_network_transaction_unittest.cc | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'net/spdy/spdy_network_transaction_unittest.cc') diff --git a/net/spdy/spdy_network_transaction_unittest.cc b/net/spdy/spdy_network_transaction_unittest.cc index 7520b38..377c7a5 100644 --- a/net/spdy/spdy_network_transaction_unittest.cc +++ b/net/spdy/spdy_network_transaction_unittest.cc @@ -809,8 +809,10 @@ class SpdyNetworkTransactionTest : public PlatformTest { TestCompletionCallback callback; - int rv = trans->Start(&request, &callback, log); - EXPECT_EQ(ERR_IO_PENDING, rv); + out.rv = trans->Start(&request, &callback, log); + EXPECT_LT(out.rv, 0); // We expect an IO Pending or some sort of error. + if (out.rv != ERR_IO_PENDING) + return out; out.rv = callback.WaitForResult(); if (out.rv != OK) @@ -822,8 +824,8 @@ class SpdyNetworkTransactionTest : public PlatformTest { out.status_line = response->headers->GetStatusLine(); out.response_info = *response; // Make a copy so we can verify. - rv = ReadTransaction(trans.get(), &out.response_data); - EXPECT_EQ(OK, rv); + out.rv = ReadTransaction(trans.get(), &out.response_data); + EXPECT_EQ(OK, out.rv); // Verify that we consumed all test data. EXPECT_TRUE(data->at_read_eof()); @@ -1804,7 +1806,7 @@ TEST_F(SpdyNetworkTransactionTest, PartialWrite) { EXPECT_EQ("hello!", out.response_data); } -TEST_F(SpdyNetworkTransactionTest, DISABLED_ConnectFailure) { +TEST_F(SpdyNetworkTransactionTest, ConnectFailure) { MockConnect connects[] = { MockConnect(true, ERR_NAME_NOT_RESOLVED), MockConnect(false, ERR_NAME_NOT_RESOLVED), -- cgit v1.1