drop system-v leftovers

voxpupuli · May 10, 2024 · 9a8a474 · 9a8a474
1 parent 8832094
commit 9a8a474
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 46 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -244,11 +244,10 @@ RSpec/ReceiveMessages:
     - 'spec/beaker/test_suite_spec.rb'
     - 'spec/helpers.rb'
 
-# Offense count: 7
+# Offense count: 4
 RSpec/RepeatedDescription:
   Exclude:
     - 'spec/beaker/cli_spec.rb'
-    - 'spec/beaker/host/unix/exec_spec.rb'
     - 'spec/beaker/host/unix/pkg_spec.rb'
 
 # Offense count: 4

diff --git a/spec/beaker/host/unix/exec_spec.rb b/spec/beaker/host/unix/exec_spec.rb
@@ -82,9 +82,7 @@ def to_s
         let(:ssh_command) { "echo 'PermitUserEnvironment yes' | cat - /etc/ssh/sshd_config > #{directory}/sshd_config.permit" }
         let(:ssh_move) { "mv #{directory}/sshd_config.permit /etc/ssh/sshd_config" }
 
-        platforms = PlatformHelpers::SYSTEMDPLATFORMS + PlatformHelpers::DEBIANPLATFORMS + PlatformHelpers::SYSTEMVPLATFORMS
-
-        platforms.each do |platform|
+        PlatformHelpers::SYSTEMDPLATFORMS.each do |platform|
           it "calls the correct commands for #{platform}" do
             opts['platform'] = platform
             expect(instance).to receive(:exec).twice
@@ -114,31 +112,6 @@ def to_s
           expect { instance.ssh_service_restart }.not_to raise_error
         end
       end
-
-      PlatformHelpers::DEBIANPLATFORMS.each do |platform|
-        it "calls the correct command for #{platform}" do
-          opts['platform'] = platform
-          expect(instance).to receive(:exec)
-          expect(Beaker::Command).to receive(:new).with("service ssh restart")
-          expect { instance.ssh_service_restart }.not_to raise_error
-        end
-      end
-
-      PlatformHelpers::SYSTEMVPLATFORMS.each do |platform|
-        it "calls the correct command for #{platform}" do
-          opts['platform'] = "#{platform}-arch"
-          expect(instance).to receive(:exec)
-          expect(Beaker::Command).to receive(:new).with("/sbin/service sshd restart")
-          expect { instance.ssh_service_restart }.not_to raise_error
-        end
-      end
-
-      it 'raises an error on unsupported platforms' do
-        opts['platform'] = 'notarealthing02-parts-arch'
-        expect do
-          instance.ssh_service_restart
-        end.to raise_error(ArgumentError, /#{opts['platform']}/)
-      end
     end
 
     describe '#prepend_commands' do

diff --git a/spec/helpers.rb b/spec/helpers.rb
@@ -123,20 +123,9 @@ module PlatformHelpers
   DEBIANPLATFORMS = %w[debian ubuntu]
 
   SYSTEMDPLATFORMS = %w[amazon-2023
-                        fedora
-                        el-7
-                        centos-7
-                        redhat-7
-                        oracle-7
-                        scientific-7
-                        el-8
-                        centos-8
-                        redhat-8
-                        oracle-8]
-
-  SYSTEMVPLATFORMS = ['el-',
-                      'centos',
-                      'redhat',
-                      'oracle',
-                      'scientific',]
+                        el-
+                        centos
+                        redhat
+                        oracle
+                        scientific].concat(DEBIANPLATFORMS)
 end