nginx: add service resource to nginx_site resource

nginx: cookstyle fixes
Removes specific Orb version pin

Author: zmaupin

.circleci/config.yml

@@ -8,7 +8,7 @@ lint_and_unit: &lint_and_unit
 version: 2.1
 
 orbs:
-  kitchen: sous-chefs/kitchen@2.0.2
+  kitchen: sous-chefs/kitchen@2
 
 workflows:
   kitchen:

CHANGELOG.md

@@ -4,6 +4,10 @@ This file is used to list changes made in each version of the nginx cookbook.
 
 ## Unreleased
 
+## 10.0.2 (2019-09-24)
+
+- Bug Fix: Add missing service resource in `nginx_site` resource. [Issue #505](https://github.com/sous-chefs/nginx/issues/505))
+
 ## 10.0.1 (2019-08-04)
 
 - Maintenance: Add contributing.md file for cookbook health metrics

metadata.rb

@@ -6,8 +6,7 @@
 chef_version     '>= 14'
 license          'Apache-2.0'
 description      'Installs and configures nginx'
-long_description IO.read(File.join(File.dirname(__FILE__), 'README.md'))
-version          '10.0.1'
+version          '10.0.2'
 
 depends  'ohai', '~> 5.2'
 

resources/site.rb

@@ -29,6 +29,11 @@
     not_if   { site_enabled?(new_resource.site_name) }
     only_if  { site_available?(new_resource.site_name) }
   end
+
+  service 'nginx' do
+    supports status: true, restart: true, reload: true
+    action   :nothing
+  end
 end
 
 action :disable do
@@ -47,4 +52,9 @@
       notifies :reload, 'service[nginx]'
     end
   end
+
+  service 'nginx' do
+    supports status: true, restart: true, reload: true
+    action   :nothing
+  end
 end

spec/unit/libraries/helpers_spec.rb

@@ -181,13 +181,13 @@ class DummyClass < Chef::Node
         context 'with version 8' do
           let(:platform_version) { '8' }
 
-          it { expect(subject.passenger_packages).to eq ['ruby-dev', 'libcurl4-gnutls-dev', 'passenger'] }
+          it { expect(subject.passenger_packages).to eq %w(ruby-dev libcurl4-gnutls-dev passenger) }
         end
 
         context 'with version 9' do
           let(:platform_version) { '9' }
 
-          it { expect(subject.passenger_packages).to eq ['ruby-dev', 'libcurl4-gnutls-dev', 'libnginx-mod-http-passenger'] }
+          it { expect(subject.passenger_packages).to eq %w(ruby-dev libcurl4-gnutls-dev libnginx-mod-http-passenger) }
         end
       end
 
@@ -197,13 +197,13 @@ class DummyClass < Chef::Node
         context 'with version 16.04' do
           let(:platform_version) { '16.04' }
 
-          it { expect(subject.passenger_packages).to eq ['ruby-dev', 'libcurl4-gnutls-dev', 'passenger'] }
+          it { expect(subject.passenger_packages).to eq %w(ruby-dev libcurl4-gnutls-dev passenger) }
         end
 
         context 'with version 18.04' do
           let(:platform_version) { '18.04' }
 
-          it { expect(subject.passenger_packages).to eq ['ruby-dev', 'libcurl4-gnutls-dev', 'libnginx-mod-http-passenger'] }
+          it { expect(subject.passenger_packages).to eq %w(ruby-dev libcurl4-gnutls-dev libnginx-mod-http-passenger) }
         end
       end
     end