Merge pull request #522 from cmurphy/fix_mysql_grant_MODULES-1040

Require title of mysql_grant resource to match form user/table

Author: Ashley Penney

lib/puppet/type/mysql_grant.rb

@@ -24,6 +24,7 @@ def initialize(*args)
     fail('privileges parameter is required.') if self[:ensure] == :present and self[:privileges].nil?
     fail('table parameter is required.') if self[:ensure] == :present and self[:table].nil?
     fail('user parameter is required.') if self[:ensure] == :present and self[:user].nil?
+    fail('name must match user and table parameters') if self[:name] != "#{self[:user]}/#{self[:table]}"
   end
 
   newparam(:name, :namevar => true) do

spec/acceptance/types/mysql_grant_spec.rb

@@ -70,6 +70,21 @@ class { 'mysql::server': }
     end
   end
 
+  describe 'adding privileges with invalid name' do
+    it 'should fail' do
+      pp = <<-EOS
+        mysql_grant { 'test':
+          ensure     => 'present',
+          table      => 'test.*',
+          user       => 'test2@tester',
+          privileges => ['SELECT', 'UPDATE'],
+        }
+      EOS
+
+      expect(apply_manifest(pp, :expect_failures => true).stderr).to match(/name must match user and table parameters/)
+    end
+  end
+
   describe 'adding option' do
     it 'should work without errors' do
       pp = <<-EOS

spec/unit/puppet/type/mysql_grant_spec.rb

@@ -41,4 +41,10 @@
     }.to raise_error(Puppet::Error, 'Title or name must be provided')
   end
 
-end
+  it 'should require the name to match the user and table' do
+    expect {
+      Puppet::Type.type(:mysql_grant).new(:name => 'foo', :privileges => ['ALL', 'PROXY'], :table => ['*.*','@'], :user => 'foo@localhost')
+    }.to raise_error /name must match user and table parameters/
+  end
+
+end