Add useDefaults option to mockPrompt

[hurrymaplelad]

I wanted to test the default option flow in my generator. mockPrompt didn't make this easy. The useDefaults option makes this easy without breaking backwards compatibilty.

helpers.mockPrompt(generator, { answer: 'foo' }, { useDefaults: true });

See tests for expected results.

[SBoudrias]

You'll have to fix the Travis error about style guide :)

[SBoudrias]

You assert too much in a single test.

Take the mockPrompt call in a beforeEach call (add a describe level -> "when useDefaults: true") and assert that defaults are used, and then in another assert that provided values are taken.

[SBoudrias]

I wonder if we shouldn't just use defaults by default. It'd make sense IMO and wouldn't require an extra option with the related overhead.

[hurrymaplelad]

I'd love for this to be the default! Should we keep the option defaulted to true, then pull it out in a few versions to ease migration, or just go for it?

[SBoudrias]

I'd say just go for it!

[sindresorhus]

👍 for default

[addyosmani]

Let's roll with defaults.

[hurrymaplelad]

Done, squashed.

[coveralls]

Coverage remained the same when pulling 6a235ab on hurrymaplelad:mock_prompt_defaults into 116b321 on yeoman:master.

[SBoudrias]

Perfect! Thanks