Fix nova and selinux unit tests

The unit tests jobs are failing because of missing pre conditions for
the new shared class introduced by
Ib233689fdcdda391596d01a21f77bd8e1672ae04.  Additionally this change
moved some classes around so that the tests are now failing due to
duplicate class declarations for nova::compute::libvirt::services. This
change moves the include that pulls in the declaration first prior to
the include that exists in tripleo::profile::base::nova::libvirt.

The selinux test was also failing due to a type issue with the fact
being used (boolean vs string)

Change-Id: I5bd4b61d6008820729d58f7743e7e61955dd6f51
Closes-Bug: #1707034

1 files changed, 16 insertions, 5 deletions

diff --git a/spec/classes/tripleo_profile_base_nova_compute_libvirt_spec.rb b/spec/classes/tripleo_profile_base_nova_compute_libvirt_spec.rb
index db9e77e..0a9ae80 100644
--- a/spec/classes/tripleo_profile_base_nova_compute_libvirt_spec.rb
+++ b/spec/classes/tripleo_profile_base_nova_compute_libvirt_spec.rb
@@ -22,12 +22,21 @@ describe 'tripleo::profile::base::nova::compute::libvirt' do
     context 'with step less than 4' do
       let(:params) { { :step => 1, } }
 
+      let(:pre_condition) do
+        <<-eos
+        class { '::tripleo::profile::base::nova::compute_libvirt_shared':
+          step => #{params[:step]},
+        }
+eos
+      end
+
+
       it {
         is_expected.to contain_class('tripleo::profile::base::nova::compute::libvirt')
         is_expected.to_not contain_class('tripleo::profile::base::nova')
         is_expected.to_not contain_class('tripleo::profile::base::nova::compute')
-        is_expected.to_not contain_class('nova::compute::libvirt')
-        is_expected.to_not contain_class('nova::compute::libvirt::qemu')
+        is_expected.to_not contain_class('tripleo::profile::base::nova::migration::client')
+        is_expected.to contain_class('tripleo::profile::base::nova::compute_libvirt_shared')
       }
     end
 
@@ -38,6 +47,9 @@ describe 'tripleo::profile::base::nova::compute::libvirt' do
           step => #{params[:step]},
           oslomsg_rpc_hosts => [ '127.0.0.1' ],
         }
+        class { '::tripleo::profile::base::nova::compute_libvirt_shared':
+          step => #{params[:step]},
+        }
         class { '::tripleo::profile::base::nova::compute':
           step => #{params[:step]},
         }
@@ -54,10 +66,9 @@ eos
 
       it {
         is_expected.to contain_class('tripleo::profile::base::nova::compute::libvirt')
-        is_expected.to contain_class('tripleo::profile::base::nova')
         is_expected.to contain_class('tripleo::profile::base::nova::compute')
-        is_expected.to contain_class('nova::compute::libvirt')
-        is_expected.to contain_class('nova::compute::libvirt::qemu')
+        is_expected.to contain_class('tripleo::profile::base::nova::migration::client')
+        is_expected.to contain_class('tripleo::profile::base::nova::compute_libvirt_shared')
       }
     end
   end