From 9e3dbb43744620b8b490c61c9269a168e0c0e3ae Mon Sep 17 00:00:00 2001 From: Phillip Barta Date: Tue, 12 Nov 2024 16:30:34 +0100 Subject: [PATCH] chore(test): remove promise support check from tests (#6148) Promises are supported in all supported Node.js version so the check is unnecessary --- test/app.route.js | 4 +--- test/app.router.js | 3 +-- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/test/app.route.js b/test/app.route.js index a0c8696e..03ae1293 100644 --- 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 index 1db40042..bdf4011a 100644 --- 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()