-
Notifications
You must be signed in to change notification settings - Fork 369
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
build: Streamline mocha
config
#1905
Conversation
@@ -26,13 +26,16 @@ | |||
"google storage", | |||
"storage" | |||
], | |||
"mocha": { |
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.
What does moving this here vs the command line provide?
@@ -26,13 +26,16 @@ | |||
"google storage", | |||
"storage" | |||
], | |||
"mocha": { |
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.
What does moving this here vs the command line provide?
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.
It allows 2 things:
npx mocha <path-to-test(s)>
without additional flags to type/remember- Easier IDE integration - IDEs no long require additional config in order to run
mocha
tests (which can change over time)
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.
Add, if mocha
is globally installed (npm i -g mocha
) simply typing mocha
in the CLI will run the default tests (same as npx mocha
)
Additional note/context: Mocha merges the config file ( |
* test: disable resumable upload in acl test before (#1903) * build: Streamline `mocha` config (#1905) * build: Streamline `mocha` config * fix: Require `--require ts-node/register` first * refactor: Misc cleanup Co-authored-by: Denis DelGrosso <[email protected]>
* test: disable resumable upload in acl test before (googleapis#1903) * build: Streamline `mocha` config (googleapis#1905) * build: Streamline `mocha` config * fix: Require `--require ts-node/register` first * refactor: Misc cleanup Co-authored-by: Denis DelGrosso <[email protected]>
Adds support for native
mocha
config support - now we can usenpx mocha
without passing additional args/config