test: cover deprecation warnings for res.redirect

Signed-off-by: Saiyam Jain <jimisierra2@gmail.com>

Author: saiyam-jain-dev

test/res.redirect.js

@@ -1,5 +1,6 @@
 'use strict'
 
+var assert = require('node:assert');
 var express = require('..');
 var request = require('supertest');
 var utils = require('./support/utils');
@@ -211,4 +212,80 @@ describe('res', function(){
         .end(done)
     })
   })
+
+  // Note: depd reports deprecations once per call site; keep separate res.redirect() calls.
+  describe('when arguments are omitted or invalid', function () {
+    it('should emit deprecation when url is missing', function (done) {
+      var app = express();
+      var warnings = [];
+      function onDeprecation(err) {
+        warnings.push(err.message);
+      }
+      process.on('deprecation', onDeprecation);
+
+      app.use(function (req, res) {
+        res.redirect();
+        res.end();
+      });
+
+      request(app)
+        .get('/')
+        .expect(302)
+        .end(function (err) {
+          process.removeListener('deprecation', onDeprecation);
+          if (err) return done(err);
+          assert.deepStrictEqual(warnings, ['Provide a url argument', 'Url must be a string']);
+          done();
+        });
+    })
+
+    it('should emit deprecation when url is not a string', function (done) {
+      var app = express();
+      var warnings = [];
+      function onDeprecation(err) {
+        warnings.push(err.message);
+      }
+      process.on('deprecation', onDeprecation);
+
+      app.use(function (req, res) {
+        res.redirect(302, 12345);
+        res.end();
+      });
+
+      request(app)
+        .get('/')
+        .expect(302)
+        .end(function (err) {
+          process.removeListener('deprecation', onDeprecation);
+          if (err) return done(err);
+          assert.deepStrictEqual(warnings, ['Url must be a string']);
+          done();
+        });
+    })
+
+    it('should emit deprecation when status is not a number', function (done) {
+      var app = express();
+      var warnings = [];
+      function onDeprecation(err) {
+        warnings.push(err.message);
+      }
+      process.on('deprecation', onDeprecation);
+
+      app.use(function (req, res) {
+        assert.throws(function () {
+          res.redirect('301', 'http://google.com');
+        }, TypeError);
+        res.end();
+      });
+
+      request(app)
+        .get('/')
+        .end(function (err) {
+          process.removeListener('deprecation', onDeprecation);
+          if (err) return done(err);
+          assert.deepStrictEqual(warnings, ['Status must be a number']);
+          done();
+        });
+    })
+  })
 })