Support an option to add minitest-handler to run list

[jtimberman]

To facilitate running tests under minitest-chef handler automatically, the default suite should include minitest-handler in the run list if a command-line option to the init command is specified. Currently:

suites:
- name: default
  run_list:
  - recipe[rng-tools]
  attributes: {}

Proposed:

suites:
- name: default
  run_list:
  - recipe[minitest-handler]
  - recipe[rng-tools]
  attributes: {}

[fnichol]

Heh, I wonder if this is truly a common happy path for most users currently--it hasn't been for me yet. Good news is that the init task is just a starting point. If we went down this path we'd also have to add these cookbooks to the Berksfile as well.

[jtimberman]

Perhaps this could just be an extension of the converter I want per #19?

[fnichol]

@jtimberman that would be worth a spike, it would save someone (like you, say) a lot of time in a migration.

[jtimberman]

Yeah I might just build a gem to do it out of band of this :).

[sethvargo]

@jtimberman do you still see value in this?

[damm]

I could see this as a kitchen init --with-minitest-handler and have it set up a basic minitest suite for you in your cookbook.

[jtimberman]

For a future version perhaps but not required for 1.0. I'd like to see the scaffolding tests created too.

[damm]

I have no opinion on versioning; but scaffolding the tests would make it a lot easier for starters.

[sethvargo]

Okay. I'm going to mark this as 2.0. We can push it into 1.1 or something, but it's currently marked 1.0 and I don't think it should block the 1.0 release 😄

[lamont-granquist]

@sethvargo should this go into chefdk generators?

[sethvargo]

👍 @lamont-granquist

[juliandunn]

If we're being prescriptive with people in ChefDK, should we be telling them to use ServerSpec and not do this feature?

[sethvargo]

👍

[jtimberman]

Down with this and up with server spec.