Added manage_service option

manifests/init.pp

@@ -1,9 +1,11 @@
-class puppetserver(
-  $version = 'present',
-  $enable  = true,
-  $start   = true,
-  $package = 'puppetserver',
-  $config = {},
+class puppetserver (
+  $version        = 'present',
+  $enable         = true,
+  $start          = true,
+  $package        = 'puppetserver',
+  $service        = 'puppetserver',
+  $manage_service = true,
+  $config         = {},
 ) {
   anchor { 'puppetserver::begin': } ->
   class { '::puppetserver::install': } ->

manifests/service.pp

@@ -1,10 +1,13 @@
 class puppetserver::service {
-  $ensure = $::puppetserver::start ? {
-    true    => running,
-    default => stopped,
-  }
-  service { 'puppetserver':
-    ensure => $ensure,
-    enable => $::puppetserver::enable,
+  if $::puppetserver::manage_service {
+    $ensure = $::puppetserver::start ? {
+      true    => running,
+      default => stopped,
+    }
+
+    service { $::puppetserver::service:
+      ensure => $ensure,
+      enable => $::puppetserver::enable,
+    }
   }
 }

spec/classes/puppetserver_spec.rb

@@ -10,6 +10,9 @@
       context 'with defaults for all parameters' do
         it { is_expected.to compile.with_all_deps }
         it { is_expected.to contain_class('puppetserver') }
+        it { is_expected.to contain_class('puppetserver::install') }
+        it { is_expected.to contain_class('puppetserver::config') }
+        it { is_expected.to contain_class('puppetserver::service') }
       end
     end
   end

spec/classes/service_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe 'puppetserver::service' do
+  on_supported_os.each do |os, facts|
+    context "on #{os}" do
+      let(:facts) do
+        facts
+      end
+
+      context 'with defaults for all parameters' do
+        let(:pre_condition) do
+          "class { 'puppetserver': }"
+        end
+
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.to contain_class('puppetserver::service') }
+        it { is_expected.to contain_service('puppetserver').with('ensure' => 'running') }
+        it { is_expected.to contain_service('puppetserver').with('enable' => true) }
+      end
+
+      context 'with custom service name' do
+        let(:pre_condition) do
+          "class { 'puppetserver':
+            service => 'foo',
+          }"
+        end
+
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.to contain_class('puppetserver::service') }
+        it { is_expected.to contain_service('foo').with('ensure' => 'running') }
+        it { is_expected.to contain_service('foo').with('enable' => true) }
+      end
+
+      context 'with service disabled' do
+        let(:pre_condition) do
+          "class { 'puppetserver':
+            enable => false,
+          }"
+        end
+
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.to contain_class('puppetserver::service') }
+        it { is_expected.to contain_service('puppetserver').with('ensure' => 'running') }
+        it { is_expected.to contain_service('puppetserver').with('enable' => false) }
+      end
+
+      context 'with service stopped' do
+        let(:pre_condition) do
+          "class { 'puppetserver':
+            start => false,
+          }"
+        end
+
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.to contain_class('puppetserver::service') }
+        it { is_expected.to contain_service('puppetserver').with('ensure' => 'stopped') }
+        it { is_expected.to contain_service('puppetserver').with('enable' => true) }
+      end
+
+      context 'with service not managed' do
+        let(:pre_condition) do
+          "class { 'puppetserver':
+            manage_service => false,
+          }"
+        end
+
+        it { is_expected.to compile.with_all_deps }
+        it { is_expected.to contain_class('puppetserver::service') }
+        it { is_expected.not_to contain_service('puppetserver') }
+      end
+    end
+  end
+end