Skip to content

Commit 9e3dbb4

Browse files
authored
chore(test): remove promise support check from tests (#6148)
Promises are supported in all supported Node.js version so the check is unnecessary
1 parent b31910c commit 9e3dbb4

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

test/app.route.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33
var express = require('../');
44
var request = require('supertest');
55

6-
var describePromises = global.Promise ? describe : describe.skip
7-
86
describe('app.route', function(){
97
it('should return a new route', function(done){
108
var app = express();
@@ -64,7 +62,7 @@ describe('app.route', function(){
6462
.expect(404, done);
6563
});
6664

67-
describePromises('promise support', function () {
65+
describe('promise support', function () {
6866
it('should pass rejected promise value', function (done) {
6967
var app = express()
7068
var route = app.route('/foo')

test/app.router.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ var express = require('../')
66
, assert = require('assert')
77
, methods = require('methods');
88

9-
var describePromises = global.Promise ? describe : describe.skip
109
var shouldSkipQuery = require('./support/utils').shouldSkipQuery
1110

1211
describe('app.router', function(){
@@ -963,7 +962,7 @@ describe('app.router', function(){
963962
})
964963
})
965964

966-
describePromises('promise support', function () {
965+
describe('promise support', function () {
967966
it('should pass rejected promise value', function (done) {
968967
var app = express()
969968
var router = new express.Router()

0 commit comments

Comments
 (0)