Merge pull request #1875 from bastelfreak/validate

drop validate_setup method

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2024-05-10 14:44:46 UTC using RuboCop version 1.62.1.
+# on 2024-05-10 15:53:37 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
@@ -155,7 +155,7 @@ RSpec/Eq:
   Exclude:
     - 'spec/beaker/logger_spec.rb'
 
-# Offense count: 232
+# Offense count: 231
 # Configuration parameters: CountAsOne.
 RSpec/ExampleLength:
   Max: 44
@@ -219,7 +219,7 @@ RSpec/MultipleMemoizedHelpers:
 RSpec/NamedSubject:
   Enabled: false
 
-# Offense count: 42
+# Offense count: 41
 # Configuration parameters: AllowedPatterns.
 # AllowedPatterns: ^expect_, ^assert_
 RSpec/NoExpectationExample:

lib/beaker/host/unix.rb

@@ -57,15 +57,6 @@ def skip_set_env?
       end
     end
 
-    # Validates that the host was setup correctly
-    #
-    # @return nil
-    # @raise [ArgumentError] If the host is setup incorrectly,
-    #   this will be raised with the appropriate message
-    def validate_setup
-      nil
-    end
-
     def initialize name, host_hash, options
       super
 

lib/beaker/network_manager.rb

@@ -61,7 +61,6 @@ def provision
         @machines[hypervisor] = [] unless @machines[hypervisor]
         hostless_options[:timesync] = host_hash[:timesync] if host_hash[:timesync] != nil
         host_itself = Beaker::Host.create(name, host_hash, hostless_options)
-        host_itself.validate_setup
         @machines[hypervisor] << host_itself
       end