tie selenium::{install,config,service} together in selenium::server

manifests/config.pp

@@ -18,7 +18,7 @@
 # Joshua Hoblitt <[email protected]>
 #
 #
-class selenium::config inherits selenium::server {
+class selenium::config {
 
   file { '/etc/init.d/selenium':
     ensure  => 'file',

manifests/install.pp

@@ -23,7 +23,7 @@
 class selenium::install(
   $version = '2.35.0',
   $url     = undef,
-) inherits selenium::server {
+) {
   validate_string($version)
   validate_string($url)
 

manifests/server.pp

@@ -11,12 +11,18 @@
   $group        = $selenium::params::group,
   $install_path = $selenium::params::install_path,
 ) inherits selenium::params {
-  validate_string($user)
+  validate_string($user) 
   validate_string($group)
+  validate_string($install_path)
 
   user { $user:
     gid => [$group],
   }
   group { $group: }
 
+  class { 'selenium::install': } ->
+  class { 'selenium::config': } ->
+  class { 'selenium::service': } ->
+  Class[ 'selenium::server' ]
+
 }

manifests/service.pp

@@ -18,7 +18,7 @@
 # Joshua Hoblitt <[email protected]>
 #
 #
-class selenium::service inherits selenium::server {
+class selenium::service {
 
   service { 'selenium':
     ensure     => running,

spec/classes/selenium_server_spec.rb

@@ -2,14 +2,79 @@
 
 describe 'selenium::server', :type => :class do
 
-  describe 'for osfamily RedHat' do
-    let :facts do
-      {
-        :osfamily => 'RedHat',
-      }
+  shared_examples 'server' do |user, group|
+    it do
+      should contain_class('selenium::server')
+      should contain_class('selenium::install')
+      should contain_class('selenium::config')
+      should contain_class('selenium::service')
+      should contain_user(user).with_gid(group)
+      should contain_group(group)
     end
+  end
+
+  context 'for osfamily RedHat' do
+    let(:facts) {{ :osfamily => 'RedHat' }}
+
+    context 'no params' do
+      it_behaves_like 'server', 'selenium', 'selenium'
+    end
+
+    context 'user => foo' do
+      let(:params) {{ :user => 'foo' }}
+
+      it_behaves_like 'server', 'foo', 'selenium'
+    end
+
+    context 'user => []' do
+      let(:params) {{ :user => [] }}
+
+      it 'should fail' do
+        expect {
+          should contain_class('selenium::server')
+        }.to raise_error
+      end
+    end
+
+    context 'group => foo' do
+      let(:params) {{ :group => 'foo' }}
 
-    it { should contain_class('selenium::server') }
+      it_behaves_like 'server', 'selenium', 'foo'
+    end
+
+    context 'group => []' do
+      let(:params) {{ :group => [] }}
+
+      it 'should fail' do
+        expect {
+          should contain_class('selenium::server')
+        }.to raise_error
+      end
+    end
+
+    context 'install_path => /foo/selenium' do
+      let(:params) {{ :install_path => '/foo/selenium' }}
+
+      it_behaves_like 'server', 'selenium', 'selenium'
+
+      it do
+        should contain_file('/foo/selenium').with({
+          'ensure' => 'directory',
+          'owner'  => 'selenium',
+          'group'  => 'selenium',
+        })
+      end
+    end
+
+    context 'install_path => []' do
+      let(:params) {{ :install_path => [] }}
+
+      it 'should fail' do
+        expect {
+          should contain_class('selenium::server')
+        }.to raise_error
+      end
+    end
   end
 
 end