Skip to content
This repository has been archived by the owner on Aug 31, 2018. It is now read-only.

Commit

Permalink
test: use fixtures module over fixturesDir
Browse files Browse the repository at this point in the history
clean up code by using fixtures helper module
instead of fixturesDir directly.

PR-URL: nodejs/node#15847
Reviewed-By: Ryan Graham <r.m.graham@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
JamesNimlos authored and addaleax committed Oct 15, 2017
1 parent f0cf222 commit b03faa5
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions test/sequential/test-deprecation-flags.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,20 +20,21 @@
// USE OR OTHER DEALINGS IN THE SOFTWARE.

'use strict';
const common = require('../common');
require('../common');
const fixtures = require('../common/fixtures');
const assert = require('assert');
const execFile = require('child_process').execFile;
const depmod = require.resolve(`${common.fixturesDir}/deprecated.js`);
const depmod = fixtures.path('deprecated.js');
const node = process.execPath;

const depUserlandFunction =
require.resolve(`${common.fixturesDir}/deprecated-userland-function.js`);
fixtures.path('deprecated-userland-function.js');

const depUserlandClass =
require.resolve(`${common.fixturesDir}/deprecated-userland-class.js`);
fixtures.path('deprecated-userland-class.js');

const depUserlandSubClass =
require.resolve(`${common.fixturesDir}/deprecated-userland-subclass.js`);
fixtures.path('deprecated-userland-subclass.js');

const normal = [depmod];
const noDep = ['--no-deprecation', depmod];
Expand Down

0 comments on commit b03faa5

Please sign in to comment.