test: use defaultHistoryPath instead of path.join

PR-URL: #15969 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
nodejs · Oct 8, 2017 · ab3c84f · ab3c84f
1 parent 37681b5
commit ab3c84f
Showing 1 changed file with 20 additions and 19 deletions.
diff --git a/test/parallel/test-repl-persistent-history.js b/test/parallel/test-repl-persistent-history.js
@@ -51,14 +51,32 @@ ActionStream.prototype.readable = true;
 const UP = { name: 'up' };
 const ENTER = { name: 'enter' };
 const CLEAR = { ctrl: true, name: 'u' };
+
+// File paths
+const fixtures = common.fixturesDir;
+const historyFixturePath = path.join(fixtures, '.node_repl_history');
+const historyPath = path.join(common.tmpDir, '.fixture_copy_repl_history');
+const historyPathFail = path.join(common.tmpDir, '.node_repl\u0000_history');
+const oldHistoryPathObj = path.join(fixtures,
+ 'old-repl-history-file-obj.json');
+const oldHistoryPathFaulty = path.join(fixtures,
+ 'old-repl-history-file-faulty.json');
+const oldHistoryPath = path.join(fixtures, 'old-repl-history-file.json');
+const enoentHistoryPath = path.join(fixtures, 'enoent-repl-history-file.json');
+const emptyHistoryPath = path.join(fixtures, '.empty-repl-history-file');
+const defaultHistoryPath = path.join(common.tmpDir, '.node_repl_history');
+const emptyHiddenHistoryPath = path.join(fixtures,
+ '.empty-hidden-repl-history-file');
+const devNullHistoryPath = path.join(common.tmpDir,
+ '.dev-null-repl-history-file');
 // Common message bits
 const prompt = '> ';
 const replDisabled = '\nPersistent history support disabled. Set the ' +
  'NODE_REPL_HISTORY environment\nvariable to a valid, ' +
  'user-writable path to enable.\n';
 const convertMsg = '\nConverted old JSON repl history to line-separated ' +
  'history.\nThe new repl history file can be found at ' +
- `${path.join(common.tmpDir, '.node_repl_history')}.\n`;
+ `${defaultHistoryPath}.\n`;
 const homedirErr = '\nError: Could not get the home directory.\n' +
  'REPL session history will not be persisted.\n';
 const replFailedRead = '\nError: Could not open history file.\n' +
@@ -71,25 +89,8 @@ const oldHistoryObj = '\nError: The old history file data has to be an Array' +
  '.\nREPL session history will not be persisted.\n';
 const sameHistoryFilePaths = '\nThe old repl history file has the same name ' +
  'and location as the new one i.e., ' +
- path.join(common.tmpDir, '.node_repl_history') +
+ `${defaultHistoryPath}` +
  ' and is empty.\nUsing it as is.\n';
-// File paths
-const fixtures = common.fixturesDir;
-const historyFixturePath = path.join(fixtures, '.node_repl_history');
-const historyPath = path.join(common.tmpDir, '.fixture_copy_repl_history');
-const historyPathFail = path.join(common.tmpDir, '.node_repl\u0000_history');
-const oldHistoryPathObj = path.join(fixtures,
- 'old-repl-history-file-obj.json');
-const oldHistoryPathFaulty = path.join(fixtures,
- 'old-repl-history-file-faulty.json');
-const oldHistoryPath = path.join(fixtures, 'old-repl-history-file.json');
-const enoentHistoryPath = path.join(fixtures, 'enoent-repl-history-file.json');
-const emptyHistoryPath = path.join(fixtures, '.empty-repl-history-file');
-const defaultHistoryPath = path.join(common.tmpDir, '.node_repl_history');
-const emptyHiddenHistoryPath = path.join(fixtures,
- '.empty-hidden-repl-history-file');
-const devNullHistoryPath = path.join(common.tmpDir,
- '.dev-null-repl-history-file');
 
 const tests = [
  {