Fix error when no domain is set (#223)

Fixes #222

Author: silug

CHANGELOG.md

@@ -1,3 +1,7 @@
+### 1.34.1 / 2024-12-17
+* Fixed:
+  * Error when no domain is set (#222)
+
 ### 1.34.0 / 2024-09-17
 * Fixed:
   * Legacy fact usage

lib/simp/beaker_helpers.rb

@@ -723,10 +723,10 @@ def linux_errata( suts )
       os_info = fact_on(sut, 'os')
 
       # Make sure we have a domain on our host
-      current_domain = fact_on(sut, 'networking.domain').strip
+      current_domain = fact_on(sut, 'networking.domain')&.strip
       hostname = fact_on(sut, 'networking.hostname').strip
 
-      if current_domain.empty?
+      if current_domain.nil? || current_domain.empty?
         new_fqdn = hostname + '.beaker.test'
 
         safe_sed(sut, 's/#{hostname}.*/#{new_fqdn} #{hostname}/', '/etc/hosts')
@@ -744,9 +744,8 @@ def linux_errata( suts )
         end
       end
 
-      if fact_on(sut, 'networking.domain').strip.empty?
-        fail("Error: hosts must have an FQDN, got domain='#{current_domain}'")
-      end
+      current_domain = fact_on(sut, 'networking.domain')&.strip
+      fail("Error: hosts must have an FQDN, got domain='#{current_domain}'") if current_domain.nil? || current_domain.empty?
 
       # This may not exist in docker so just skip the whole thing
       if sut.file_exist?('/etc/ssh')

lib/simp/beaker_helpers/version.rb

@@ -3,5 +3,5 @@
 module Simp; end
 
 module Simp::BeakerHelpers
-  VERSION = '1.34.0'
+  VERSION = '1.34.1'
 end