From 97a4a43e456b076ee39821b008c014f6d9607065 Mon Sep 17 00:00:00 2001
From: Tim Meusel <tim@bastelfreak.de>
Date: Fri, 10 May 2024 15:50:45 +0200
Subject: [PATCH] drop system-v leftovers

---
 .rubocop_todo.yml                  |  5 ++---
 spec/beaker/host/unix/exec_spec.rb | 29 +----------------------------
 spec/helpers.rb                    | 26 ++++++--------------------
 3 files changed, 9 insertions(+), 51 deletions(-)

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 89b549019..0c44dec59 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2024-05-10 13:25:58 UTC using RuboCop version 1.62.1.
+# on 2024-05-10 14:20:09 UTC using RuboCop version 1.62.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -245,11 +245,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
index 42ae75ae0..130dee65c 100644
--- 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
index 2a846bbdf..e83468a3f 100644
--- a/spec/helpers.rb
+++ b/spec/helpers.rb
@@ -125,24 +125,10 @@ module PlatformHelpers
   FEDORASYSTEMD    = (14..39).to_a.collect! { |i| "fedora-#{i}" }
 
   SYSTEMDPLATFORMS = %w[amazon-2023
-                        el-7
-                        centos-7
-                        redhat-7
-                        oracle-7
-                        scientific-7
-                        eos-7
-                        el-8
-                        centos-8
-                        redhat-8
-                        oracle-8].concat(FEDORASYSTEMD)
-
-  FEDORASYSTEMV    = (1..13).to_a.collect! { |i| "fedora-#{i}" }
-
-  SYSTEMVPLATFORMS = ['el-',
-                      'centos',
-                      'fedora',
-                      'redhat',
-                      'oracle',
-                      'scientific',
-                      'eos',].concat(FEDORASYSTEMV)
+                        el-
+                        centos
+                        redhat
+                        oracle
+                        scientific
+                        eos].concat(FEDORASYSTEMD).concat(DEBIANPLATFORMS)
 end