Skip to content

Commit a8d9ccf

Browse files
Trotttargos
authored andcommitted
test: remove common.projectDir
common.projectDir is used in one test, so it's not so common. Remove from common module to the one test file that needs it. PR-URL: #17781 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Timothy Gu <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
1 parent fd8523f commit a8d9ccf

File tree

3 files changed

+1
-7
lines changed

3 files changed

+1
-7
lines changed

test/common/README.md

-5
Original file line numberDiff line numberDiff line change
@@ -353,11 +353,6 @@ original state after calling [`common.hijackStdOut()`][].
353353

354354
Path to the 'root' directory. either `/` or `c:\\` (windows)
355355

356-
### projectDir
357-
* [&lt;String>]
358-
359-
Path to the project directory.
360-
361356
### skip(msg)
362357
* `msg` [&lt;string>]
363358

test/common/index.js

-1
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,6 @@ exports.enoughTestCpu = Array.isArray(cpus) &&
6565
(cpus.length > 1 || cpus[0].speed > 999);
6666

6767
exports.rootDir = exports.isWindows ? 'c:\\' : '/';
68-
exports.projectDir = path.resolve(__dirname, '..', '..');
6968

7069
exports.buildType = process.config.target_defaults.default_configuration;
7170

test/doctool/test-make-doc.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ const assert = require('assert');
1111
const fs = require('fs');
1212
const path = require('path');
1313

14-
const apiPath = path.resolve(common.projectDir, 'out', 'doc', 'api');
14+
const apiPath = path.resolve(__dirname, '..', '..', 'out', 'doc', 'api');
1515
const docs = fs.readdirSync(apiPath);
1616
assert.ok(docs.includes('_toc.html'));
1717

0 commit comments

Comments
 (0)