Throw a fatal error when 'strainer test' is called on a chef_repo with no arguments

Author: sethvargo

lib/strainer/runner.rb

@@ -65,7 +65,9 @@ def run!
       end
 
       # Move the logfile back over
-      FileUtils.mv(Strainer.logfile_path, Strainer.sandbox_path.join('strainer.out'))
+      if File.exist?(Strainer.sandbox_path.join('strainer.out'))
+        FileUtils.mv(Strainer.logfile_path, Strainer.sandbox_path.join('strainer.out'))
+      end
 
       if @report.values.collect(&:values).flatten.all?
         Strainer.ui.say "Strainer marked build OK"

lib/strainer/sandbox.rb

@@ -37,8 +37,15 @@ def initialize(cookbook_names, options = {})
       @options = options
 
       if chef_repo?
-        @cookbooks = load_cookbooks(cookbook_names)
+        Strainer.ui.debug "Detected as a chef repo"
+        if cookbook_names.empty?
+          Strainer.ui.fatal "You must specify one or more cookbook names to `strainer test`!"
+          exit(false)
+        else
+          @cookbooks = load_cookbooks(cookbook_names)
+        end
       elsif cookbook_repo?
+        Strainer.ui.debug "Detected as a cookbook repo"
         unless cookbook_names.empty?
           Strainer.ui.warn "Supply specific cookbooks to a cookbook_repo is not supported. Use `strainer test` with no arguments instead."
         end