Merge pull request #291 from inkblot/fix_issue_290

Add collation with the create statement

Author: Ashley Penney

lib/puppet/provider/mysql_database/mysql.rb

@@ -42,7 +42,7 @@ def self.prefetch(resources)
   end
 
   def create
-    mysql([defaults_file, '-NBe', "create database `#{@resource[:name]}` character set #{@resource[:charset]}"].compact)
+    mysql([defaults_file, '-NBe', "create database `#{@resource[:name]}` character set #{@resource[:charset]} collate #{@resource[:collate]}"].compact)
 
     @property_hash[:ensure]  = :present
     @property_hash[:charset] = @resource[:charset]

spec/unit/puppet/provider/mysql_database/mysql_spec.rb

@@ -55,23 +55,23 @@
   end
 
   describe 'create' do
-    it 'makes a user' do
-      provider.expects(:mysql).with([defaults_file, '-NBe', "create database `#{resource[:name]}` character set #{resource[:charset]}"])
+    it 'makes a database' do
+      provider.expects(:mysql).with([defaults_file, '-NBe', "create database `#{resource[:name]}` character set #{resource[:charset]} collate #{resource[:collate]}"])
       provider.expects(:exists?).returns(true)
       provider.create.should be_true
     end
   end
 
   describe 'destroy' do
-    it 'removes a user if present' do
+    it 'removes a database if present' do
       provider.expects(:mysql).with([defaults_file, '-NBe', "drop database `#{resource[:name]}`"])
       provider.expects(:exists?).returns(false)
       provider.destroy.should be_true
     end
   end
 
   describe 'exists?' do
-    it 'checks if user exists' do
+    it 'checks if database exists' do
       instance.exists?.should be_true
     end
   end