Fixes #258, handles undefined Response object rather than throwing an…

lib/test.js

@@ -268,4 +268,3 @@ function error(msg, expected, actual) {
   err.showDiff = true;
   return err;
 }
-

test/supertest.js

@@ -270,6 +270,46 @@ describe('request(app)', function(){
         done();
       });
     });
+
+    it('should handle an undefined Response', function (done) {
+      var app = express();
+
+      app.get('/', function(req, res){
+        setTimeout( function () {
+          res.end();
+        }, 20);
+      });
+
+      var s = app.listen(function(){
+        var url = 'http://localhost:' + s.address().port;
+        request(url)
+        .get('/')
+        .timeout(1)
+        .expect(200, function (err) {
+          err.should.be.an.instanceof(Error);
+          return done();
+        });
+      });
+    });
+
+    it('should handle error returned when server goes down', function (done) {
+      var app = express();
+
+      app.get('/', function(req, res){
+        res.end();
+      });
+
+      var s = app.listen(function(){
+        var url = 'http://localhost:' + s.address().port;
+        s.close();
+        request(url)
+        .get('/')
+        .expect(200, function (err) {
+          err.should.be.an.instanceof(Error);
+          return done();
+        });
+      });
+    });
   });
 
   describe('.expect(status[, fn])', function(){
@@ -988,4 +1028,4 @@ describe("request.get(url).query(vals) works as expected", function(){
       done();
     });
   });
-});
+});