test: print instruction for creating missing snapshot in assertSnapshot · nodejs/node@e70e974 · GitHub

nodejs / node Public

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

• Code
• Issues 637
• Pull requests 761
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit e70e974

Browse filesBrowse the repository at this point in the historyBrowse files

rluvaton

authored and

RafaelGSS

committed

test: print instruction for creating missing snapshot in assertSnapshot

PR-URL: #48914 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

1 parent f23b2a3 commit e70e974
Copy full SHA for e70e974

1 file changed

+11-1Lines changed: 11 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/common
  • assertSnapshot.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/common/assertSnapshot.js‎

Copy file name to clipboardExpand all lines: test/common/assertSnapshot.js

+11-1Lines changed: 11 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -34,7 +34,17 @@ async function assertSnapshot(actual, filename = process.argv[1]) {
34	34	if (process.env.NODE_REGENERATE_SNAPSHOTS) {
35	35	await fs.writeFile(snapshot, actual);
36	36	} else {
37		- const expected = await fs.readFile(snapshot, 'utf8');
	37	+ let expected;
	38	+ try {
	39	+ expected = await fs.readFile(snapshot, 'utf8');
	40	+ } catch (e) {
	41	+ if (e.code === 'ENOENT') {
	42	+ console.log(
	43	+ 'Snapshot file does not exist. You can create a new one by running the test with NODE_REGENERATE_SNAPSHOTS=1',
	44	+ );
	45	+ }
	46	+ throw e;
	47	+ }
38	48	assert.strictEqual(actual, replaceWindowsLineEndings(expected));
39	49	}
40	50	}

0 commit comments

Comments

0 (0)