verdnatura-mirror
/
loopback
mirror of https://github.com/strongloop/loopback.git
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: fix too strict test assertion 

Rework the test verifying properties of `loopback` to ignore
new express properties added after the test was written.

Ignore "json" and "urlencoded" middleware that was added back
to Express, keep using our wrappers printing a deprecation message.
This commit is contained in:
Miroslav Bajtoš 2017-10-04 10:31:50 +02:00
parent cd8f1775bc
commit 4f928bf965
No known key found for this signature in database
GPG Key ID: 6F2304BA9361C7E3
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
package.json
View File

@ -45,7 +45,7 @@
"depd": "^1.0.0",
"ejs": "^2.3.1",
"errorhandler": "^1.3.4",
"express": "^4.12.2",
"express": "^4.16.2",
"inflection": "^1.6.0",
"isemail": "^1.2.0",
"loopback-connector-remote": "^1.0.3",

7
test/loopback.test.js
View File

@ -117,7 +117,7 @@ describe('loopback', function() {
var actual = Object.getOwnPropertyNames(loopback);
actual.sort();
expect(actual).to.eql(EXPECTED);
expect(actual).to.include.members(EXPECTED);
});
});
@ -530,6 +530,11 @@ describe('loopback', function() {
it('inherits properties from express', function() {
var express = require('express');
for (var i in express) {
// Express added back body-parsing middleware in v4.16.0, see
// https://github.com/expressjs/express/issues/2211
// However, we are keeping them deprecated in 2.x,
// because it's in LTS mode
if (i === 'json' || i === 'urlencoded') continue;
expect(loopback).to.have.property(i, express[i]);
}
});