Skip to content

test: more useful assert messaging for child-process-execSync#15972

Closed
ericljpemberton wants to merge 1 commit intonodejs:masterfrom
ericljpemberton:testChildProcessExecSync
Closed

test: more useful assert messaging for child-process-execSync#15972
ericljpemberton wants to merge 1 commit intonodejs:masterfrom
ericljpemberton:testChildProcessExecSync

Conversation

@ericljpemberton
Copy link
Contributor

@ericljpemberton ericljpemberton commented Oct 6, 2017

This is a commit for Node Interactive. I changed some basic assertion messages to hopefully provide better insights into errors.

Checklist
Affected core subsystem(s)

tests

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

child_process Issues and PRs related to the child_process subsystem. code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants