Adds a new `../common/fixtures' module to begin normalizing `test/fixtures` use. Our test code is a bit inconsistent with regards to use of the fixtures directory. Some code uses `path.join()`, some code uses string concats, some other code uses template strings, etc. In mnay cases, significant duplication of code is seen when accessing fixture files, etc. This updates many (but by no means all) of the tests in the test suite to use the new consistent API. There are still many more to update, which would make an excelent Code-n-Learn exercise. Backport-PR-URL: https://github.com/nodejs/node/pull/16265 PR-URL: https://github.com/nodejs/node/pull/14332 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Michaël Zasso <targos@protonmail.com>
28 lines
637 B
JavaScript
28 lines
637 B
JavaScript
/* eslint-disable required-modules */
|
|
'use strict';
|
|
|
|
const path = require('path');
|
|
const fs = require('fs');
|
|
|
|
const fixturesDir = path.join(__dirname, '..', 'fixtures');
|
|
|
|
function fixturesPath(...args) {
|
|
return path.join(fixturesDir, ...args);
|
|
}
|
|
|
|
function readFixtureSync(args, enc) {
|
|
if (Array.isArray(args))
|
|
return fs.readFileSync(fixturesPath(...args), enc);
|
|
return fs.readFileSync(fixturesPath(args), enc);
|
|
}
|
|
|
|
function readFixtureKey(name, enc) {
|
|
return fs.readFileSync(fixturesPath('keys', name), enc);
|
|
}
|
|
|
|
module.exports = {
|
|
fixturesDir,
|
|
path: fixturesPath,
|
|
readSync: readFixtureSync,
|
|
readKey: readFixtureKey
|
|
};
|