prettierPath only works via CLI option, not config

[exogen]

🐛 Bug Report

When specifying the prettierPath option in a config file or JSON object, it is ignored. Jest prints the warning:

    Jest: Inline Snapshots requires Prettier.
    Please ensure "prettier" is installed in your project.

By putting a console.log in the module at the specified prettierPath, it is apparent that Jest never even attempts to import the supplied path. The value that eventually makes it to getPrettier() in the jest-jasmine2 package is null.

Using the same value with --prettierPath via the CLI works.

To Reproduce

Specify the prettierPath option in a config file or --config JSON object.

Expected behavior

The provided path will be used for Prettier.

Link to repl or repo (highly encouraged)

Please provide either a repl.it demo or a minimal repository on GitHub.

Issues without a reproduction link are likely to stall.

Run npx envinfo --preset jest

Paste the results here:

npx: installed 1 in 2.496s

  System:
    OS: macOS 10.14.1
    CPU: (4) x64 Intel(R) Core(TM) i5-4278U CPU @ 2.60GHz
  Binaries:
    Node: 8.9.4 - ~/.nvm/versions/node/v8.9.4/bin/node
    Yarn: 1.12.3 - /usr/local/bin/yarn
    npm: 6.4.1 - ~/.nvm/versions/node/v8.9.4/bin/npm
  npmPackages:
    jest: ^23.6.0 => 23.6.0

[grosto]

It seems that argv has a default value for prettierPath and it overrides config file's prettierPath. I could do a PR, unless I am missing something and the default value is intentional.

[rickhanlonii]

Hmm, @azz is that intentional?

[SimenB]

I refuse to believe this is intentional (if it is I'll argue against it), so PR very much welcome 🙂

[azz]

Definately not intentional

[github-actions]

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.