Commit 7040ec9
build: update test-wpt-report to use NODE instead of OUT_NODE
PR-URL: #61024
Refs: https://openjs-foundation.slack.com/archives/C03BJP63CH0/p1765400746909869
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>1 parent 23fb3a6 commit 7040ec9
1 file changed
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -655,11 +655,15 @@ test-test426: all ## Run the Web Platform Tests. | |||
| 655 | 655 | test-wpt: all ## Run the Web Platform Tests. | |
| 656 | 656 | $(PYTHON) tools/test.py $(PARALLEL_ARGS) wpt | |
| 657 | 657 | ||
| 658 | + # https://github.com/nodejs/node/blob/a00d95c73dcac8fc2b316238fb978a7d5aa650c6/.github/workflows/daily-wpt-fyi.yml#L71-L97 | ||
| 659 | + # This uses NODE instead of OUT_NODE and if changes are to be made they'd need to be made on all | ||
| 660 | + # non-EOL release lines' since the wpt.fyi job checks out the repository in a state of a given release | ||
| 661 | + # to ensure future changes to the WPT runner don't need to be backported. | ||
| 658 | 662 | .PHONY: test-wpt-report | |
| 659 | 663 | test-wpt-report: ## Run the Web Platform Tests and generate a report. | |
| 660 | 664 | $(RM) -r out/wpt | |
| 661 | 665 | mkdir -p out/wpt | |
| 662 | - -WPT_REPORT=1 $(PYTHON) tools/test.py --shell $(OUT_NODE) $(PARALLEL_ARGS) wpt | ||
| 666 | + -WPT_REPORT=1 $(PYTHON) tools/test.py --shell $(NODE) $(PARALLEL_ARGS) wpt | ||
| 663 | 667 | $(NODE) "$$PWD/tools/merge-wpt-reports.mjs" | |
| 664 | 668 | ||
| 665 | 669 | .PHONY: test-internet | |
0 commit comments