Allow cookbook uploads without dependency resolution. Add spec.

[chrisroberts]

No description provided.

[sethvargo]

I would personally prefer if this were :no_dependencies (--no-dependencies). I think it's far better to be specific. --nodeps isn't very informative. You already have the "short-form" -D.

[chrisroberts]

@reset @sethvargo updated option name and added custom class/exit code in the branch this pull request was based on if it would be useful.

[reset]

@chrisroberts reverted this initial PR.

ode/berkshelf (refactor-nodeps ✘)✹ ᐅ bundle exec bin/berks upload -D
Using artifact (0.9.8)
Using build-essential (0.7.0)
/Users/reset/code/berkshelf/lib/berkshelf/berksfile.rb:512:in `upload': Failed to locate requested cookbook(s). (RuntimeError)
  - Cookbooks uploaded without dependencies must be defined within Berksfile.
    from /Users/reset/code/berkshelf/lib/berkshelf/cli.rb:230:in `upload'
    from /Users/reset/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/thor-0.16.0/lib/thor/task.rb:27:in `run'
    from /Users/reset/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/thor-0.16.0/lib/thor/invocation.rb:120:in `invoke_task'
    from /Users/reset/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/thor-0.16.0/lib/thor.rb:275:in `dispatch'
    from /Users/reset/code/berkshelf/lib/berkshelf/cli.rb:10:in `dispatch'
    from /Users/reset/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/thor-0.16.0/lib/thor/base.rb:425:in `start'
    from bin/berks:6:in `<main>'

This error message is not clear enough to why the upload failed. The expected behavior from the description of the CLI flag is that the explicitly defined cookbooks in the Berksfile (in this case, artifact and build-essential) would be uploaded but their dependencies would be skipped.

[reset]

we can clean this up, but please take a look at the surrounding code and match the style.

• no single letter variables
• no parentheses around parameters to control statements (if, then, else)
• do/end on multiline blocks

[reset]

prefer the skip-dependencies flag/option to nodeps