fix ssl mode test that assume the stream will end after the response is completed

Author: beenotung

test/client-test.js

@@ -112,7 +112,7 @@ describe('SPDY Client', function () {
           assert.strictEqual(res.statusCode, 200)
           assert.strictEqual(res.headers.post, 'headers')
 
-          fixtures.expectData(res, 'post body', done)
+          fixtures.expectData(res, 'post body', done, plain)
         })
 
         agent._spdyState.socket.once(plain ? 'connect' : 'secureConnect',

test/fixtures.js

@@ -52,13 +52,21 @@ exports.keys = {
         '-----END CERTIFICATE-----'
 }
 
-function expectData (stream, expected, callback) {
+function expectData (stream, expected, callback, plain) {
   var actual = ''
 
   stream.on('data', function (chunk) {
     actual += chunk
+    // early terminate the test because the stream in ssl mode will not end
+    // this allows streaming further request with the same network resource
+    if (!plain && actual === expected) {
+      callback()
+    }
   })
   stream.on('end', function () {
+    if (!plain) {
+      return
+    }
     assert.strictEqual(actual, expected)
     callback()
   })

test/server-test.js

@@ -360,7 +360,7 @@ describe('SPDY Server', function () {
         stream.on('response', function (status, headers) {
           assert.strictEqual(status, 200)
 
-          fixtures.expectData(stream, 'hello world, what\'s up?', done)
+          fixtures.expectData(stream, 'hello world, what\'s up?', done, plain)
         })
 
         stream.end()