Merge pull request #286 from alekbarszczewski/redirects-fix

Fixes issue with redirects

lib/test.js

@@ -119,7 +119,7 @@ Test.prototype.end = function(fn){
   var end = Request.prototype.end;
 
   end.call(this, function(err, res){
-    if (server) return server.close(assert);
+    if (server && server._handle) return server.close(assert);
 
     assert();
 

test/supertest.js

@@ -161,6 +161,28 @@ describe('request(app)', function(){
     .expect(302, done);
   });
 
+  it('should handle redirects', function(done){
+    var app = express();
+
+    app.get('/login', function (req, res){
+      res.end('Login')
+    });
+
+    app.get('/', function(req, res){
+      res.redirect('/login');
+    });
+
+    request(app)
+    .get('/')
+    .redirects(1)
+    .end(function (err, res) {
+      should.exist(res)
+      res.status.should.be.equal(200)
+      res.text.should.be.equal('Login')
+      done()
+    })
+  });
+
   it('should handle socket errors', function(done) {
     var app = express();
 
@@ -1030,4 +1052,4 @@ describe("request.get(url).query(vals) works as expected", function(){
       done();
     });
   });
-});
+});