There were several problems with the scala plugin:

1) scala -version is returned on stderr not stdout so we were never setting this and thus the plugin never ran

2) If sbt or scala weren't installed it threw exceptions

3) we were storing shell outs value in the output value for no real reason

4) if scala failed for some reason, but sbt succeeded we weren't setting the sbt values since were relied on scala[:version] being set

5) the specs incorrectly mocked scala output to stdout

6) the specs that checked to make sure values weren't set didn't actually run the plugin so they would always no find scala's key

While I was here I moved the version key for sbt into it's own key.  This would allow us to add other sbt values in the future if we found them. If we make version to top level value we lose that ability forever.

Add debug logging to hints and improve cloud specs

Allowing a method to receive ec2 or rackspace doesn't mean we detected this.  These are actually duplicates of the "hint file no present" specs we already have.  We just think they're doing something different.

This isn't going to work here

We test this once in the hint spec.  We don't need to expect the windows and linux file paths.  Additionally if you're on a host that actually has a hint it breaks these tests.  The hint spec does it all in a temp dir which is the right way to test it.

Also while I'm in these specs update the format, fix some poor wording, and use shared_examples in a few places to dry things up