(#15165) Fix error --tags already defined in puppet

By renaming the offending options. See Puppetlabs bug #15165:
http://projects.puppetlabs.com/issues/15165

This commit amended by Jeff McCune.  Commit message changed to follow
our style guidelines, spec tests updated.
Fix ProgressBar spec failure

Without this patch the following failure is surfacing with MRI 1.9.3

Failures:

  1) ProgressBar ReversedProgressBar Utils InstanceErrorState Installer
     Failure/Error: it { should include("ProgressBar") }
       expected [:ProgressBar, :ReversedProgressBar, :Utils, :InstanceErrorState, :Installer] to include "ProgressBar"
       Diff:
       @@ -1,2 +1,2 @@
       -["ProgressBar"]
       +[:ProgressBar, :ReversedProgressBar, :Utils, :InstanceErrorState, :Installer]
     # ./spec/unit/puppet/cloudpack/progressbar_spec.rb:8:in `block (2 levels) in <top (required)>'

This patch fixes the problem by coercing all of the constants to strings before
comparing them in the spec test.  This approach should be compatible with 1.8
and 1.9.
Rename option names to match convention

The convention is to use hyphens instead of underscores and I prefer
full words instead of abbreviations for clarity.