Merge pull request #154 from tonymet/issue/147

add 307,308 to redirect assertion #147

Author: keithamus

lib/http.js

@@ -244,13 +244,13 @@ module.exports = function (chai, _) {
    */
 
   Assertion.addProperty('redirect', function() {
-    var redirectCodes = [301, 302, 303]
+    var redirectCodes = [301, 302, 303, 307, 308]
       , status = this._obj.status
       , redirects = this._obj.redirects;
 
     this.assert(
         redirectCodes.indexOf(status) >= 0 || redirects && redirects.length
-      , "expected redirect with 30{1-3} status code but got " + status
+      , "expected redirect with 30X status code but got " + status
       , "expected not to redirect but got " + status + " status"
     );
   });

test/http.js

@@ -177,7 +177,7 @@ describe('assertions', function () {
     var res = { status: 200 };
     res.should.not.redirect;
 
-    [301, 302, 303].forEach(function (status) {
+    [301, 302, 303, 307, 308].forEach(function (status) {
       var res = { status: status };
       res.should.redirect;
     });
@@ -195,7 +195,7 @@ describe('assertions', function () {
     (function () {
       var res = { status: 200 };
       res.should.redirect;
-    }).should.throw('expected redirect with 30{1-3} status code but got 200');
+    }).should.throw('expected redirect with 30X status code but got 200');
 
     (function () {
       var res = { status: 301 };