Skip to content

Commit

Permalink
test: replaced fixturesDir with fixtures module
Browse files Browse the repository at this point in the history
This was an assigned task at Node.js Interactive North America 2017.
This replaced the fixturesDir exported by the common module with
the fixturesDir on the common/fixtures module in the path-makelong test.

PR-URL: #15908
Reviewed-By: Daijiro Wachi <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
  • Loading branch information
mckalexee authored and MylesBorins committed Oct 11, 2017
1 parent 547c284 commit ccecaca
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/parallel/test-path-makelong.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,12 @@

'use strict';
const common = require('../common');
const fixtures = require('../common/fixtures');
const assert = require('assert');
const path = require('path');

if (common.isWindows) {
const file = path.join(common.fixturesDir, 'a.js');
const file = fixtures.path('a.js');
const resolvedFile = path.resolve(file);

assert.strictEqual(`\\\\?\\${resolvedFile}`, path._makeLong(file));
Expand Down

0 comments on commit ccecaca

Please sign in to comment.