Skip to content

Commit

Permalink
test: use fixtures module in test
Browse files Browse the repository at this point in the history
Instead of using common.fixturesDir, uses the fixtures module in
test-http-get-pipeline-problem.

PR-URL: #16117
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Daijiro Wachi <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
  • Loading branch information
NigelKibodeaux authored and MylesBorins committed Nov 21, 2017
1 parent 5257978 commit b667951
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/parallel/test-http-get-pipeline-problem.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
// after http.globalAgent.maxSockets number of files.
// See https://groups.google.com/forum/#!topic/nodejs-dev/V5fB69hFa9o
const common = require('../common');
const fixtures = require('../common/fixtures');
const assert = require('assert');
const http = require('http');
const fs = require('fs');
Expand All @@ -11,7 +12,7 @@ http.globalAgent.maxSockets = 1;

common.refreshTmpDir();

const image = fs.readFileSync(`${common.fixturesDir}/person.jpg`);
const image = fixtures.readSync('/person.jpg');

console.log(`image.length = ${image.length}`);

Expand Down

0 comments on commit b667951

Please sign in to comment.