Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix --passWithNoTests #4639

Merged
merged 3 commits into from
Oct 9, 2017
Merged

Fix --passWithNoTests #4639

merged 3 commits into from
Oct 9, 2017

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Oct 9, 2017

Summary
Ref #4630. /cc @EnoahNetzach

Test plan
Green CI running the test which was previously skipped due to bad name

@@ -464,6 +464,7 @@ export default function normalize(options: InitialOptions, argv: Argv) {
case 'notify':
case 'onlyChanged':
case 'outputFile':
case 'passWithNoTests':
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a weird requirement to get CLI flags to pass through

@SimenB SimenB force-pushed the fix-pass-with-no-tests branch from edf8614 to b9dd62e Compare October 9, 2017 20:00
@cpojer cpojer merged commit 6dbc014 into jestjs:master Oct 9, 2017
@SimenB SimenB deleted the fix-pass-with-no-tests branch October 9, 2017 20:26
@codecov-io
Copy link

Codecov Report

Merging #4639 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4639   +/-   ##
=======================================
  Coverage   57.12%   57.12%           
=======================================
  Files         194      194           
  Lines        6566     6566           
  Branches        3        3           
=======================================
  Hits         3751     3751           
  Misses       2815     2815
Impacted Files Coverage Δ
packages/jest-config/src/normalize.js 78.14% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 412dd61...b9dd62e. Read the comment docs.

@github-actions
Copy link

This pull request 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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 13, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants