chore(test): remove promise support check from tests (#6148)

Promises are supported in all supported Node.js version so the check is unnecessary
expressjs · Nov 12, 2024 · 9e3dbb4 · 9e3dbb4
1 parent b31910c
commit 9e3dbb4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/test/app.route.js b/test/app.route.js
@@ -3,8 +3,6 @@
 var express = require('../');
 var request = require('supertest');
 
-var describePromises = global.Promise ? describe : describe.skip
-
 describe('app.route', function(){
   it('should return a new route', function(done){
     var app = express();
@@ -64,7 +62,7 @@ describe('app.route', function(){
     .expect(404, done);
   });
 
-  describePromises('promise support', function () {
+  describe('promise support', function () {
     it('should pass rejected promise value', function (done) {
       var app = express()
       var route = app.route('/foo')

diff --git a/test/app.router.js b/test/app.router.js
@@ -6,7 +6,6 @@ var express = require('../')
   , assert = require('assert')
   , methods = require('methods');
 
-var describePromises = global.Promise ? describe : describe.skip
 var shouldSkipQuery = require('./support/utils').shouldSkipQuery
 
 describe('app.router', function(){
@@ -963,7 +962,7 @@ describe('app.router', function(){
     })
   })
 
-  describePromises('promise support', function () {
+  describe('promise support', function () {
     it('should pass rejected promise value', function (done) {
       var app = express()
       var router = new express.Router()