test: use const instead of var in test-require-json.js · nodejs/node@e9d4665 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 634
• Pull requests 592
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit e9d4665

Browse filesBrowse the repository at this point in the historyBrowse files

sarahmeyer

authored and

addaleax

committed

test: use const instead of var in test-require-json.js

PR-URL: #9904 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent f4b6b9f commit e9d4665
Copy full SHA for e9d4665

1 file changed

+5-5Lines changed: 5 additions & 5 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-require-json.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-require-json.js‎

Copy file name to clipboard

+5-5Lines changed: 5 additions & 5 deletions

Original file line number	Diff line number	Diff line change
@@ -1,12 +1,12 @@
1	1	'use strict';
2		- var common = require('../common');
3		- var path = require('path');
4		- var assert = require('assert');
	2	+ const common = require('../common');
	3	+ const path = require('path');
	4	+ const assert = require('assert');
5	5
6	6	try {
7	7	require(path.join(common.fixturesDir, 'invalid.json'));
8	8	} catch (err) {
9		- var re = /test[/\\]fixtures[/\\]invalid.json: Unexpected string/;
10		- var i = err.message.match(re);
	9	+ const re = /test[/\\]fixtures[/\\]invalid.json: Unexpected string/;
	10	+ const i = err.message.match(re);
11	11	assert.notStrictEqual(null, i, 'require() json error should include path');
12	12	}

0 commit comments

Comments

0 (0)