search

jestjs / jest

Delightful JavaScript Testing.
https://jestjs.io
MIT License
44.16k stars 6.45k forks source link

Consider some way to have a `warn` state next to `pass` and `fail` #5167

Open Haroenv opened 6 years ago

Haroenv commented 6 years ago

Do you want to request a feature or report a bug?

feature

What is the current behavior?

an external runner can only pass or fail

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.

N/A

What is the expected behavior?

Have some third state, which is warnings. These warnings should not fail the build (maybe unless you have some config to disallow warnings), but still show the output like a failing test.

See https://github.com/jest-community/jest-runner-eslint/issues/21 and https://github.com/keplersj/jest-runner-tslint/commit/9bd862fdca07a78316e05e5d7d17a45060907abc

cc @keplersj, @ljharb

Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.

N/A

github-actions[bot] commented 2 years ago

This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.

ljharb commented 2 years ago

bump, i guess

SimenB commented 2 years ago

PR welcome! #6996 can be used as example adding a new result type (although we won't need support in circus/jasmine for a warn like we did todo, so a bit smaller than that one was)

github-actions[bot] commented 1 year ago

This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.

ljharb commented 1 year ago

bump

github-actions[bot] commented 7 months ago

This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.

ljharb commented 7 months ago

bump