npm / init-package-json / 125

95%

master: 95%

push

travis-ci

fix: package.json saving optional deps

When saving over existing files that contains optionalDependencies it
should avoid also saving that dep to dependencies, respecting our docs
recommendation on it:

    Entries in optionalDependencies will override entries of the same
    name in dependencies, so it's usually best to only put in one place.

This patches this UX problem by adding an extra check that will avoid
adding a dependency to the package.json `dependencies` object in case
that package is already listed under `optionalDependencies`.

Relates to: https://github.com/npm/arborist/pull/221
Fixes: https://github.com/npm/cli/issues/724

PR-URL: https://github.com/npm/init-package-json/pull/109
Credit: @ruyadorno
Close: #109
Reviewed-by: @nlf, @wraithgar

53 of 65 branches covered (81.54%)

8 of 8 new or added lines in 1 file covered. (100.0%)

84 of 88 relevant lines covered (95.45%)

15.1 hits per line