Skip to content

test: migrate util message tests from Python to JS#62705

Open
Vansh1811 wants to merge 9 commits intonodejs:mainfrom
Vansh1811:test-migrate-util-message-to-js
Open

test: migrate util message tests from Python to JS#62705
Vansh1811 wants to merge 9 commits intonodejs:mainfrom
Vansh1811:test-migrate-util-message-to-js

Conversation

@Vansh1811
Copy link
Copy Markdown

Migrates test/message/util_inspect_error and test/message/util-inspect-error-cause tests from the Python-based test runner to JavaScript, following the established pattern from #47498.

Changes:

  • Moves test/message/util_inspect_error.js to test/fixtures/util/util_inspect_error.js (updates require path)
  • Renames test/message/util_inspect_error.out to test/fixtures/util/util_inspect_error.snapshot
  • Moves test/message/util-inspect-error-cause.js to test/fixtures/util/util-inspect-error-cause.js (updates require path)
  • Renames test/message/util-inspect-error-cause.out to test/fixtures/util/util-inspect-error-cause.snapshot
  • Adds test/parallel/test-node-output-util.mjs as the JS test runner using assertSnapshot.spawnAndAssert
  • Removes the old .js and .out files from test/message/

Closes #47707 (partial - migrates util tests)

Ref: #47707

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

message tests can be migrated from python to JS

2 participants