Fix unit tests

puppetlabs · Jul 26, 2022 · 3762396 · 3762396
1 parent 95718fb
commit 3762396
Showing 1 changed file with 37 additions and 0 deletions.
diff --git a/spec/classes/service_spec.rb b/spec/classes/service_spec.rb
@@ -4,6 +4,7 @@
   let(:facts) do
     {
       :kernel           => 'Linux',
+      :osfamily         => 'Debian',
       :os               => {
         :family => "Debian",
         :name    => 'Ubuntu',
@@ -21,6 +22,11 @@
   end
 
   context 'with controller => true and container_runtime => cri_containerd and manage_etcd => true' do
+    let(:pre_condition) { 'include kubernetes; include kubernetes::config::kubeadm; include kubernetes::repos
+                           class { "kubernetes::packages":
+                                    container_runtime => "cri-containerd"
+                                 }' 
+                        }
     let(:params) do
       {
         'kubernetes_version' => '1.10.2',
@@ -44,6 +50,14 @@
   end
 
   context 'with controller => true and container_runtime => docker and manage_docker => true and manage_etcd => false' do
+    let(:pre_condition) { 
+      'include kubernetes
+       include kubernetes::config::kubeadm
+       include kubernetes::repos
+       class { "kubernetes::packages":
+                container_runtime => "docker"
+             }' 
+            }
     let(:params) do
       {
         'kubernetes_version' => '1.10.2',
@@ -62,6 +76,13 @@
   end
 
   context 'with controller => true and container_runtime => docker and manage_docker => false and manage_etcd => true' do
+    let(:pre_condition) {'include kubernetes; include kubernetes::config::kubeadm; include kubernetes::repos
+                          class { "kubernetes::packages":
+                          container_runtime => "docker",
+                          manage_docker => false,
+                          manage_etcd   => false,
+                                }' 
+                         }
     let(:params) do
         {
             'kubernetes_version' => '1.10.2',
@@ -81,6 +102,10 @@
   end
 
   context 'with os.family => RedHat' do
+    let(:pre_condition) {'include kubernetes; include kubernetes::config::kubeadm; include kubernetes::repos
+                          class { "kubernetes::packages":    
+                                }' 
+                         }
     let(:facts) do
       super().merge({ :os => { :family => 'RedHat' }})
     end
@@ -89,6 +114,10 @@
   end
 
   context 'with version => 1.10 and cloud_provider => aws and cloud_config => undef' do
+    let(:pre_condition) {'include kubernetes; include kubernetes::config::kubeadm; include kubernetes::repos
+                          class { "kubernetes::packages":    
+                                 }'
+                         }
     let(:params) do
         {
             'kubernetes_version' => '1.10.2',
@@ -109,6 +138,10 @@
   end
 
   context 'with version => 1.10 and cloud_provider => openstack and cloud_config => /etc/kubernetes/cloud.conf' do
+    let(:pre_condition) {'include kubernetes; include kubernetes::config::kubeadm; include kubernetes::repos
+                         class { "kubernetes::packages":    
+                                }'
+                         }
     let(:params) do
         {
             'kubernetes_version' => '1.10.2',
@@ -129,6 +162,10 @@
   end
 
   context 'with version => 1.12 and cloud_provider => aws' do
+    let(:pre_condition) {'include kubernetes; include kubernetes::config::kubeadm; include kubernetes::repos
+                          class { "kubernetes::packages":    
+                                 }'
+                         }
     let(:params) do
         {
             'kubernetes_version' => '1.12.3',