Merge pull request #219 from Mirantis/package-before-service

generic_service requires package first

manifests/generic_service.pp

@@ -49,7 +49,7 @@ define nova::generic_service(
       name    => $service_name,
       ensure  => $service_ensure,
       enable  => $enabled,
-      require => Package['nova-common'],
+      require => [Package['nova-common'], Package[$nova_title]],
     }
   }
 

spec/defines/nova_generic_service_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe 'nova::generic_service' do
+  describe 'package should come before service' do
+    let :pre_condition do
+      'include nova'
+    end
+
+    let :params do
+      {
+        :package_name => 'foo',
+        :service_name => 'food',
+        :enabled => true
+      }
+    end
+ 
+    let :facts do
+      { :osfamily => 'Debian' }
+    end
+
+    let :title do
+      'foo'
+    end
+
+    it { should contain_service('nova-foo').with(
+      'name'    => 'food',
+      'ensure'  => 'running',
+      'enable'  => true,
+      'require' => ['Package[nova-common]', 'Package[nova-foo]']
+    )}
+  end
+end