Merge pull request #342 from apenney/fix/privileges

Fix an issue with lowercase privileges.

Author: Ashley Penney

Gemfile

@@ -1,6 +1,7 @@
 source 'https://rubygems.org'
 
 group :development, :test do
+  gem 'mime-types', '<2.0',      :require => false
   gem 'rake',                    :require => false
   gem 'rspec-puppet',            :require => false
   gem 'puppetlabs_spec_helper',  :require => false

lib/puppet/type/mysql_grant.rb

@@ -3,9 +3,7 @@
   @doc = "Manage a MySQL user's rights."
   ensurable
 
-  autorequire(:file) do
-    '/root/.my.cnf'
-  end
+  autorequire(:file) { '/root/.my.cnf' }
 
   def initialize(*args)
     super
@@ -38,6 +36,10 @@ def initialize(*args)
 
   newproperty(:privileges, :array_matching => :all) do
     desc 'Privileges for user'
+
+    munge do |value|
+      value.upcase
+    end
   end
 
   newproperty(:table) do

spec/system/types/mysql_grant_spec.rb

@@ -283,5 +283,32 @@ class { 'mysql::server': }
     end
   end
 
+  describe 'lower case privileges' do
+    it 'create ALL privs' do
+      pp = <<-EOS
+      mysql_grant { 'lowercase@localhost/*.*':
+          user       => 'lowercase@localhost',
+          privileges => 'ALL',
+          table      => '*.*',
+      }
+      EOS
+
+      puppet_apply(pp)
+    end
+
+    it 'create lowercase all privs' do
+      pp = <<-EOS
+      mysql_grant { 'lowercase@localhost/*.*':
+          user       => 'lowercase@localhost',
+          privileges => 'all',
+          table      => '*.*',
+      }
+      EOS
+
+      puppet_apply(pp) do |r|
+        r.exit_code.should be_zero
+      end
+    end
+  end
 
 end