move repo definition to setup_acceptance_node

theforeman · Jan 23, 2025 · 02f7e5e · 02f7e5e
1 parent 2325c9e
commit 02f7e5e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 16 deletions.
diff --git a/spec/acceptance/foreman_cli_plugins_spec.rb b/spec/acceptance/foreman_cli_plugins_spec.rb
@@ -63,11 +63,6 @@ class { 'foreman::cli':
       it_behaves_like 'an idempotent resource' do
         let(:manifest) do
           <<-PUPPET
-          yumrepo { 'katello':
-            baseurl  => "http://yum.theforeman.org/katello/nightly/katello/el${facts['os']['release']['major']}/x86_64/",
-            gpgcheck => 0,
-          }
-
           class { 'foreman::cli':
             foreman_url => 'https://foreman.example.com',
             username    => 'admin',
@@ -77,8 +72,6 @@ class { 'foreman::cli':
           include foreman::cli::katello
           include foreman::cli::virt_who_configure
           include foreman::cli::rh_cloud
-
-          Yumrepo['katello'] -> Class['foreman::cli::katello']
           PUPPET
         end
       end

diff --git a/spec/acceptance/foreman_rh_cloud_spec.rb b/spec/acceptance/foreman_rh_cloud_spec.rb
@@ -6,15 +6,6 @@
   it_behaves_like 'an idempotent resource' do
     let(:manifest) do
       <<-PUPPET
-      yumrepo { 'katello':
-        baseurl  => "http://yum.theforeman.org/katello/nightly/katello/el${facts['os']['release']['major']}/x86_64/",
-        gpgcheck => 0,
-      }
-      yumrepo { 'candlepin':
-        baseurl  => "https://yum.theforeman.org/candlepin/4.4/el${facts['os']['release']['major']}/x86_64/",
-        gpgcheck => 0,
-      }
-
       include foreman
       include foreman::plugin::rh_cloud
       PUPPET

diff --git a/spec/setup_acceptance_node.pp b/spec/setup_acceptance_node.pp
@@ -2,6 +2,17 @@
   repo => 'nightly',
 }
 
+if $facts['os']['family'] == 'RedHat' {
+  yumrepo { 'katello':
+    baseurl  => "http://yum.theforeman.org/katello/nightly/katello/el${facts['os']['release']['major']}/x86_64/",
+    gpgcheck => 0,
+  }
+  yumrepo { 'candlepin':
+    baseurl  => "https://yum.theforeman.org/candlepin/4.4/el${facts['os']['release']['major']}/x86_64/",
+    gpgcheck => 0,
+  }
+}
+
 # Not /etc/foreman because purging removes that
 $directory = '/etc/foreman-certs'
 $certificate = "${directory}/certificate.pem"