Merge pull request #876 from DavidS/test-service_manage

(MODULES-3702) add tests proving that service_manage is working

Author: bmjen

spec/acceptance/mysql_server_spec.rb

@@ -51,6 +51,35 @@ class { 'mysql::server':
     it_behaves_like "a idempotent resource"
   end
 
+  describe 'minimal config' do
+    before(:all) do
+      @tmpdir = default.tmpdir('mysql')
+    end
+    let(:pp) do
+      <<-EOS
+        class { 'mysql::server':
+          config_file             => '#{@tmpdir}/my.cnf',
+          includedir              => '#{@tmpdir}/include',
+          manage_config_file      => 'false',
+          override_options        => { 'mysqld' => { 'key_buffer_size' => '32M' }},
+          package_ensure          => 'present',
+          purge_conf_dir          => 'false',
+          remove_default_accounts => 'false',
+          restart                 => 'false',
+          root_group              => 'root',
+          root_password           => 'test',
+          service_enabled         => 'false',
+          service_manage          => 'false',
+          users                   => {},
+          grants                  => {},
+          databases               => {},
+        }
+      EOS
+    end
+
+    it_behaves_like "a idempotent resource"
+  end
+
   describe 'syslog configuration' do
     let(:pp) do
       <<-EOS

spec/classes/mysql_server_spec.rb

@@ -27,6 +27,12 @@
         it { is_expected.to compile.with_all_deps }
       end
 
+      context 'when not managing the service' do
+        let(:params) {{ :service_manage => false }}
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.not_to contain_service('mysqld') }
+      end
+
       context 'mysql::server::install' do
         it 'contains the package by default' do
           is_expected.to contain_package('mysql-server').with({