Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-02-17 23:44:39 -05:00
Code Issues Activity

Updated vhost middleware upstream test

Browse source 
refs 8255bfdfda

- the original project had a commit which fixes a specific test for
  missing host headers
- funnily enough, this was our only missing coverage on this package, so
  we achieve 100% with this
This commit is contained in:
Daniel Lockyer 2022-07-27 22:16:00 +02:00
parent ac4ff8ada4
commit 590bd7bdb8
1 changed files with 13 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
ghost/mw-vhost/test/vhost.test.js
View file

@ -102,7 +102,16 @@ describe('vhost(hostname, server)', function () {
vhosts.push(vhost('tobi.com', tobi));
vhosts.push(vhost('loki.com', loki));
const app = createServer(vhosts);
const server = createServer(vhosts);
const listeners = server.listeners('request');
server.removeAllListeners('request');
listeners.unshift(function (req) {
req.headers.host = undefined;
});
listeners.forEach(function (l) {
server.addListener('request', l);
});
function tobi(req, res) {
res.end('tobi');
@ -111,10 +120,9 @@ describe('vhost(hostname, server)', function () {
res.end('loki');
}
request(app)
request(server)
.get('/')
.unset('Host')
.expect(404, done);
.expect(404, 'no vhost for "undefined"', done);
});
describe('arguments', function () {
@ -277,7 +285,7 @@ function createServer(hostname, server, pretest) {
if (!vhost || err) {
res.statusCode = err ? (err.status || 500) : 404;
res.end(err ? err.message : 'oops');
res.end(err ? err.message : `no vhost for "${req.headers.host}"`);
return;
}