fix spec errors in jenkins_repo_el_spec.rb

Author: bastelfreak

spec/classes/jenkins_repo_el_spec.rb

@@ -1,40 +1,46 @@
 require 'spec_helper'
 
 describe 'jenkins', type: :class do
-  # Switching OS Family to prevent duplicate declaration
-  let(:facts) do
-    {
-      osfamily: 'Redhat',
-      operatingsystem: 'CentOS',
-      operatingsystemrelease: '6.7',
-      operatingsystemmajrelease: '6'
-    }
-  end
-
-  context 'repo::el' do
-    describe 'default' do
-      it { is_expected.to contain_yumrepo('jenkins').with_baseurl('https://pkg.jenkins.io/redhat-stable/') }
-      it { is_expected.to contain_yumrepo('jenkins').with_proxy(nil) }
-    end
-
-    describe 'lts = true' do
-      let(:params) { { lts: true } }
-
-      it { is_expected.to contain_yumrepo('jenkins').with_baseurl('https://pkg.jenkins.io/redhat-stable/') }
-      it { is_expected.to contain_yumrepo('jenkins').with_proxy(nil) }
-    end
-
-    describe 'lts = false' do
-      let(:params) { { lts: false } }
-
-      it { is_expected.to contain_yumrepo('jenkins').with_proxy(nil) }
-      it { is_expected.to contain_yumrepo('jenkins').with_baseurl('https://pkg.jenkins.io/redhat/') }
-    end
-
-    describe 'repo_proxy is set' do
-      let(:params) { { repo_proxy: 'http://proxy:8080/' } }
-
-      it { is_expected.to contain_yumrepo('jenkins').with_proxy('http://proxy:8080/') }
+  on_supported_os.each do |os, facts|
+    next unless facts[:os]['family'] == 'RedHat'
+
+    context "on #{os} " do
+      systemd_fact = case facts[:operatingsystemmajrelease]
+                     when '6'
+                       { systemd: false }
+                     else
+                       { systemd: true }
+                     end
+      let :facts do
+        facts.merge(systemd_fact)
+      end
+
+      context 'repo::el' do
+        describe 'default' do
+          it { is_expected.to contain_yumrepo('jenkins').with_baseurl('https://pkg.jenkins.io/redhat-stable/') }
+          it { is_expected.to contain_yumrepo('jenkins').with_proxy(nil) }
+        end
+
+        describe 'lts = true' do
+          let(:params) { { lts: true } }
+
+          it { is_expected.to contain_yumrepo('jenkins').with_baseurl('https://pkg.jenkins.io/redhat-stable/') }
+          it { is_expected.to contain_yumrepo('jenkins').with_proxy(nil) }
+        end
+
+        describe 'lts = false' do
+          let(:params) { { lts: false } }
+
+          it { is_expected.to contain_yumrepo('jenkins').with_proxy(nil) }
+          it { is_expected.to contain_yumrepo('jenkins').with_baseurl('https://pkg.jenkins.io/redhat/') }
+        end
+
+        describe 'repo_proxy is set' do
+          let(:params) { { repo_proxy: 'http://proxy:8080/' } }
+
+          it { is_expected.to contain_yumrepo('jenkins').with_proxy('http://proxy:8080/') }
+        end
+      end
     end
   end
 end