Merge pull request #50 from jhoblitt/bugfix/hub-and-node-on-same-host

acceptance test coverage of hub+node on the same host

manifests/config.pp

@@ -19,20 +19,18 @@
   validate_string($java)
   validate_string($jar_name)
 
-  $init_script = "init-script-${name}"
+  # prog is the 'name' of the init.d script.
+  $prog = "selenium${name}"
 
   case $::osfamily {
     'debian': {
       ensure_packages(['daemon'])
-      Package['daemon'] -> File[$init_script]
+      Package['daemon'] -> File[$prog]
     }
     default : {}
   }
 
-  # prog is the 'name' of the init.d script.
-  $prog = "selenium${name}"
-
-  file { $init_script:
+  file { $prog:
     ensure  => 'file',
     path    => "/etc/init.d/${prog}",
     owner   => 'root',

spec/acceptance/selenium_hub_and_node_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper_acceptance'
+
+describe 'selenium::hub & selenium::node on the same host' do
+  after(:all) do
+    shell "service seleniumhub stop"
+    shell "service seleniumnode stop"
+  end
+
+  describe 'running puppet code' do
+    # Using puppet_apply as a helper
+    it 'should work with no errors' do
+      pp = <<-EOS
+        include java
+        Class['java'] -> Class['selenium::hub']
+        Class['java'] -> Class['selenium::node']
+
+        class { 'selenium::hub': }
+        class { 'selenium::node': }
+      EOS
+
+      # Run it twice and test for idempotency
+      apply_manifest(pp, :catch_failures => true)
+      apply_manifest(pp, :catch_changes => true)
+    end
+  end
+
+  ['seleniumhub', 'seleniumnode'].each do |service|
+    describe service(service) do
+      it { should be_running }
+      it { should be_enabled }
+    end
+  end
+end

spec/unit/defines/selenium_config_spec.rb

@@ -18,7 +18,7 @@
     p.merge!(params) if params
 
     it do
-      should contain_file("init-script-#{title}").with({
+      should contain_file("selenium#{title}").with({
         'ensure' => 'file',
         'path'   => "/etc/init.d/selenium#{title}",
         'owner'  => 'root',
@@ -93,4 +93,14 @@
     end
   end
 
+  context 'for osfamily Debian' do
+    let(:title) { 'server' }
+    let(:facts) {{ :osfamily => 'Debian' }}
+    let :pre_condition do
+      "include selenium"
+    end
+
+    it { should contain_package('daemon').with_ensure('present') }
+  end
+
 end