add version defaults for Debian, Ubuntu and RedHat (clones)

data/common.yaml

@@ -0,0 +1,3 @@
+---
+
+{}

data/os/Debian/11.yaml

@@ -0,0 +1,3 @@
+---
+
+nginx::nginx_version: '1.18.0'

data/os/Debian/12.yaml

@@ -0,0 +1,3 @@
+---
+
+nginx::nginx_version: '1.22.1'

data/os/RedHat/8.yaml

@@ -0,0 +1,3 @@
+---
+
+nginx::nginx_version: '1.14.1'

data/os/RedHat/9.yaml

@@ -0,0 +1,3 @@
+---
+
+nginx::nginx_version: '1.20.1'

data/os/Ubuntu/20.04.yaml

@@ -0,0 +1,3 @@
+---
+
+nginx::nginx_version: '1.18.0'

data/os/Ubuntu/22.04.yaml

@@ -0,0 +1,3 @@
+---
+
+nginx::nginx_version: '1.18.0'

hiera.yaml

@@ -0,0 +1,21 @@
+---
+version: 5
+
+defaults:  # Used for any hierarchy level that omits these keys.
+  datadir: data         # This path is relative to hiera.yaml's directory.
+  data_hash: yaml_data  # Use the built-in YAML backend.
+
+hierarchy:
+  - name: "osfamily/major release"
+    paths:
+        # Used to distinguish between Debian and Ubuntu
+      - "os/%{facts.os.name}/%{facts.os.release.major}.yaml"
+      - "os/%{facts.os.family}/%{facts.os.release.major}.yaml"
+        # Used for Solaris
+      - "os/%{facts.os.family}/%{facts.kernelrelease}.yaml"
+  - name: "osfamily"
+    paths:
+      - "os/%{facts.os.name}.yaml"
+      - "os/%{facts.os.family}.yaml"
+  - name: 'common'
+    path: 'common.yaml'

spec/defines/resource_server_spec.rb

@@ -688,19 +688,19 @@
             end
 
             context 'with fact nginx_version=1.14.1' do
-              let(:facts) { facts.merge(nginx_version: '1.14.1') }
+              let(:pre_condition) { 'class {"nginx": nginx_version => "1.14.1"}' }
 
               it { is_expected.to contain_concat__fragment("#{title}-ssl-header").with_content(%r{  ssl on;}) }
             end
 
             context 'with fact nginx_version=1.15.1' do
-              let(:facts) { facts.merge(nginx_version: '1.15.1') }
+              let(:pre_condition) { 'class {"nginx": nginx_version => "1.15.1"}' }
 
               it { is_expected.to contain_concat__fragment("#{title}-ssl-header").without_content(%r{  ssl on;}) }
             end
 
             context 'http2 on with fact nginx_version=1.25.1' do
-              let(:facts) { facts.merge(nginx_version: '1.25.1') }
+              let(:pre_condition) { 'class {"nginx": nginx_version => "1.25.1"}' }
               let :params do
                 default_params.merge(
                   http2: 'on',
@@ -714,7 +714,7 @@
             end
 
             context 'with fact nginx_version=1.25.1' do
-              let(:facts) { facts.merge(nginx_version: '1.25.1') }
+              let(:pre_condition) { 'class {"nginx": nginx_version => "1.25.1"}' }
 
               it { is_expected.to contain_concat__fragment("#{title}-ssl-header").with_content(%r{^\s+http2\s+off;}) }
             end
@@ -1472,7 +1472,7 @@
 
             # TODO: implement test after this can be tested
             # msg = %r{nginx: ssl must be true if listen_port is the same as ssl_port}
             it 'Testing for warnings not yet implemented in classes'
           end
 
           context 'when listen_port != ssl_port' do