You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
When running jest through npm script, the Snapshot Summary report outputs wrong help message. e.g. run `npm run test:react -u` to remove it. rather than run `npm run test:react -- -u` to remove it.
If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
make a snapshot
run test
remove original component
run test again
What is the expected behavior?
It should output run `npm run test:react -- -u
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.
Using npm@5node@8Macbook OSX
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.
Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
When running jest through npm script, the
Snapshot Summary
report outputs wrong help message. e.g.run `npm run test:react -u` to remove it.
rather thanrun `npm run test:react -- -u` to remove it.
If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can
yarn install
andyarn test
.What is the expected behavior?
It should output
run `npm run test:react -- -u
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.
Using
npm@5
node@8
Macbook OSX
Additional info
Suspect https://github.com/facebook/jest/blob/master/packages/jest-cli/src/reporters/summary_reporter.js#L147 is the line to fix. Only checking
isYarn
.The text was updated successfully, but these errors were encountered: