(QENG-608) Check_for_package method should not use which command

- properly differentiate between packages and commands
- check_for_package updated to check for the given package name using
  the correct package manager per OS
- added check_for_command, which uses 'which' to determine if a command
  is currently available
- added spec test coverage for check_for_package
(QENG-64) Fix ordering issue when deploying the frictionless agent

Previously, the puppet agent was not stopped on the master until all
nodes were deployed. This could cause a situation where the background
agent would be running when we attempt to deploy the frictionless
repo. In this situation, the agent run we kick off to deploy the repo
would fail. This causes the entire beaker process to bail.

We now stop the background agent immediately, preventing this
condition from occuring.
Add noop argument
Spec for the noops flag
Adding yarddoc for :noop option
Fixing spec
Merge pull request #298 from branan/qeng_64

(QENG-64) Fix ordering issue when deploying the frictionless agent
Merge pull request #309 from petems/add_noops_option

Add noops option
(MAINT) Bugfixes in host.exec and logger.

* Honor the :silent option in host.exec
* Many logging functions seem to try to pass positional parameters in to
  optionally_color but mistakenly flattened the list into a single argument.
  This made it impossible to choose whether or not to add a newline.
* Remove call to logger.notify with bogus arity. Seems like "args" probably
  becomes an empty array which made this method call okay previously. However,
  if there are no arguments in "*args" then no args get passed down to
  optionally_color, thus the arity of the original "*args" is preserved.

Signed-off-by: Wayne <wayne@puppetlabs.com>
(MAINT) Make retry_command output easier on the eyes.

Silent all the "on" commands by passing silent => true to the underlying
host.exec in the options map. Instead, handle console/logfile output for
retry_command manually. Should significantly reduce logfile size for
applications that make extensive use of retry_command, not to mention making
stdout more readable.

Signed-off-by: Wayne <wayne@puppetlabs.com>
Merge pull request #300 from waynr/feature/logging-improvements

Feature/logging improvements
(gh-305) Start docker containers using privileged mode
Merge pull request #311 from treydock/docker_privileged_fix

(gh-305) Start docker containers using privileged mode
(QENG-714/QENG-686) use all junit tags available for beaker xml output...

... provide local xslt for viewing beaker xml output

- switch over to nokogiri (cleaner, easier)
- add logs to system-out tag
- add tag for time that each test suite takes to execute
- add property tags for each beaker option
- add junit.xsl stylesheet to interpret test results
Merge pull request #293 from anodelman/junit

(QENG-714) use all junit tags available for beaker xml output
GH-312 -  Value needs to be nil if it has no value
GH-312 Fix spec to reflect that value is nil
Merge pull request #229 from anodelman/pkg-vs-cmd

(QENG-608) Check_for_package method should not use which command
GH-312 - Change test to check command line
Merge pull request #313 from petems/fix_noop_to_nil

Fix noop to nil