test: prevent extraneous HOSTNAME substitution in test-runner-output · nodejs/node@103034b · GitHub

nodejs / node Public

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

• Code
• Issues 632
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 103034b

Browse filesBrowse the repository at this point in the historyBrowse files

Renegade334

authored and

aduh95

committed

test: prevent extraneous HOSTNAME substitution in test-runner-output

PR-URL: #58076 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

1 parent 1eec170 commit 103034b
Copy full SHA for 103034b

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-runner-output.mjs

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-runner-output.mjs‎

Copy file name to clipboardExpand all lines: test/parallel/test-runner-output.mjs

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -37,7 +37,7 @@ function replaceJunitDuration(str) {
37	37	return str
38	38	.replaceAll(/time="[0-9.]+"/g, 'time="*"')
39	39	.replaceAll(/duration_ms [0-9.]+/g, 'duration_ms *')
40		- .replaceAll(hostname(), 'HOSTNAME')
	40	+ .replaceAll(`hostname="${hostname()}"`, 'hostname="HOSTNAME"')
41	41	.replace(stackTraceBasePath, '$3');
42	42	}
43	43

0 commit comments

Comments

0 (0)