-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: improved local test running #1441
Conversation
There is one problem with the cleanup of our global agent and |
refs mochajs/mocha#5149 |
Fixed 586cd0a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Add aliases to your terminal: | ||
|
||
```sh | ||
node bin/add-test-aliases.js bash|zsh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
node bin/add-test-aliases.js bash
worked for me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I verified in zsh.
node bin/add-test-aliases.js zsh
worked fine for me!
- node script to add aliases - integrate mocha watch - removed redundant sh helper scripts
Co-authored-by: Arya <[email protected]>
0ac6390
to
b929cff
Compare
mocha --watch
bin/run-tests.sh
.See CONTRIBUTING.md.