mirror of
https://github.com/zebrajr/express.git
synced 2025-12-06 12:19:51 +01:00
41113599af
Since v5 relies on node >= 18, this is now possible (since v16, v14.18.0 [^1][^2]). It's functionally irrelevant: 1. It's not required for CJS nor ESM (with a few exceptions [^3]) 2. It has no performance promises However, there are upsides to this approach: 1. It brings clear boundaries to what's a built-in and what's an external dependency 2. It reduces the risk of importing unwanted deps where a built-in is expected 3. It's slightly more interoperable with other JS runtimes that provide node compatibility[^4], albeit only during development. Once imported from npm, built-ins are assumed. [^1]:https://nodejs.org/docs/latest-v22.x/api/modules.html#built-in-modules [^2]:https://github.com/nodejs/node/pull/37246 [^3]:https://nodejs.org/api/modules.html#built-in-modules-with-mandatory-node-prefix [^4]:https://docs.deno.com/runtime/fundamentals/node/#using-node's-built-in-modules
84 lines
2.2 KiB
JavaScript
84 lines
2.2 KiB
JavaScript
'use strict'
|
|
|
|
var assert = require('node:assert')
|
|
var express = require('../')
|
|
, fs = require('node:fs');
|
|
var path = require('node:path')
|
|
|
|
function render(path, options, fn) {
|
|
fs.readFile(path, 'utf8', function(err, str){
|
|
if (err) return fn(err);
|
|
str = str.replace('{{user.name}}', options.user.name);
|
|
fn(null, str);
|
|
});
|
|
}
|
|
|
|
describe('app', function(){
|
|
describe('.engine(ext, fn)', function(){
|
|
it('should map a template engine', function(done){
|
|
var app = express();
|
|
|
|
app.set('views', path.join(__dirname, 'fixtures'))
|
|
app.engine('.html', render);
|
|
app.locals.user = { name: 'tobi' };
|
|
|
|
app.render('user.html', function(err, str){
|
|
if (err) return done(err);
|
|
assert.strictEqual(str, '<p>tobi</p>')
|
|
done();
|
|
})
|
|
})
|
|
|
|
it('should throw when the callback is missing', function(){
|
|
var app = express();
|
|
assert.throws(function () {
|
|
app.engine('.html', null);
|
|
}, /callback function required/)
|
|
})
|
|
|
|
it('should work without leading "."', function(done){
|
|
var app = express();
|
|
|
|
app.set('views', path.join(__dirname, 'fixtures'))
|
|
app.engine('html', render);
|
|
app.locals.user = { name: 'tobi' };
|
|
|
|
app.render('user.html', function(err, str){
|
|
if (err) return done(err);
|
|
assert.strictEqual(str, '<p>tobi</p>')
|
|
done();
|
|
})
|
|
})
|
|
|
|
it('should work "view engine" setting', function(done){
|
|
var app = express();
|
|
|
|
app.set('views', path.join(__dirname, 'fixtures'))
|
|
app.engine('html', render);
|
|
app.set('view engine', 'html');
|
|
app.locals.user = { name: 'tobi' };
|
|
|
|
app.render('user', function(err, str){
|
|
if (err) return done(err);
|
|
assert.strictEqual(str, '<p>tobi</p>')
|
|
done();
|
|
})
|
|
})
|
|
|
|
it('should work "view engine" with leading "."', function(done){
|
|
var app = express();
|
|
|
|
app.set('views', path.join(__dirname, 'fixtures'))
|
|
app.engine('.html', render);
|
|
app.set('view engine', '.html');
|
|
app.locals.user = { name: 'tobi' };
|
|
|
|
app.render('user', function(err, str){
|
|
if (err) return done(err);
|
|
assert.strictEqual(str, '<p>tobi</p>')
|
|
done();
|
|
})
|
|
})
|
|
})
|
|
})
|