Simplify Service attr helper methods

Author: dduugg

Library/Homebrew/service.rb

@@ -109,52 +109,27 @@ def run(command = nil, macos: nil, linux: nil)
 
     sig { params(path: T.nilable(T.any(String, Pathname))).returns(T.nilable(String)) }
     def working_dir(path = nil)
-      case path
-      when nil
-        @working_dir
-      when String, Pathname
-        @working_dir = path.to_s
-      end
+      path ? @working_dir = path.to_s : @working_dir
     end
 
     sig { params(path: T.nilable(T.any(String, Pathname))).returns(T.nilable(String)) }
     def root_dir(path = nil)
-      case path
-      when nil
-        @root_dir
-      when String, Pathname
-        @root_dir = path.to_s
-      end
+      path ? @root_dir = path.to_s : @root_dir
     end
 
     sig { params(path: T.nilable(T.any(String, Pathname))).returns(T.nilable(String)) }
     def input_path(path = nil)
-      case path
-      when nil
-        @input_path
-      when String, Pathname
-        @input_path = path.to_s
-      end
+      path ? @input_path = path.to_s : @input_path
     end
 
     sig { params(path: T.nilable(T.any(String, Pathname))).returns(T.nilable(String)) }
     def log_path(path = nil)
-      case path
-      when nil
-        @log_path
-      when String, Pathname
-        @log_path = path.to_s
-      end
+      path ? @log_path = path.to_s : @log_path
     end
 
     sig { params(path: T.nilable(T.any(String, Pathname))).returns(T.nilable(String)) }
     def error_log_path(path = nil)
-      case path
-      when nil
-        @error_log_path
-      when String, Pathname
-        @error_log_path = path.to_s
-      end
+      path ? @error_log_path = path.to_s : @error_log_path
     end
 
     sig {
@@ -178,12 +153,7 @@ def keep_alive(value = nil)
 
     sig { params(value: T.nilable(T::Boolean)).returns(T::Boolean) }
     def require_root(value = nil)
-      case value
-      when nil
-        @require_root
-      when TrueClass, FalseClass
-        @require_root = value
-      end
+      value.nil? ? @require_root : @require_root = value
     end
 
     # Returns a `Boolean` describing if a service requires root access.
@@ -194,12 +164,7 @@ def requires_root?
 
     sig { params(value: T.nilable(T::Boolean)).returns(T.nilable(T::Boolean)) }
     def run_at_load(value = nil)
-      case value
-      when nil
-        @run_at_load
-      when TrueClass, FalseClass
-        @run_at_load = value
-      end
+      value.nil? ? @run_at_load : @run_at_load = value
     end
 
     sig {
@@ -240,22 +205,12 @@ def keep_alive?
 
     sig { params(value: T.nilable(T::Boolean)).returns(T::Boolean) }
     def launch_only_once(value = nil)
-      case value
-      when nil
-        @launch_only_once
-      when TrueClass, FalseClass
-        @launch_only_once = value
-      end
+      value.nil? ? @launch_only_once : @launch_only_once = value
     end
 
     sig { params(value: T.nilable(Integer)).returns(T.nilable(Integer)) }
     def restart_delay(value = nil)
-      case value
-      when nil
-        @restart_delay
-      when Integer
-        @restart_delay = value
-      end
+      restart_delay ? @restart_delay = value : @restart_delay
     end
 
     sig { params(value: T.nilable(Symbol)).returns(T.nilable(Symbol)) }
@@ -286,22 +241,12 @@ def run_type(value = nil)
 
     sig { params(value: T.nilable(Integer)).returns(T.nilable(Integer)) }
     def interval(value = nil)
-      case value
-      when nil
-        @interval
-      when Integer
-        @interval = value
-      end
+      value ? @interval = value : @interval
     end
 
     sig { params(value: T.nilable(String)).returns(T::Hash[Symbol, T.any(Integer, String)]) }
     def cron(value = nil)
-      case value
-      when nil
-        @cron
-      when String
-        @cron = parse_cron(value)
-      end
+      value ? @cron = parse_cron(value) : @cron
     end
 
     sig { returns(T::Hash[Symbol, T.any(Integer, String)]) }
@@ -352,20 +297,12 @@ def parse_cron(cron_statement)
 
     sig { params(variables: T::Hash[Symbol, String]).returns(T.nilable(T::Hash[Symbol, String])) }
     def environment_variables(variables = {})
-      case variables
-      when Hash
-        @environment_variables = variables.transform_values(&:to_s)
-      end
+      @environment_variables = variables.transform_values(&:to_s)
     end
 
     sig { params(value: T.nilable(T::Boolean)).returns(T::Boolean) }
     def macos_legacy_timers(value = nil)
-      case value
-      when nil
-        @macos_legacy_timers
-      when TrueClass, FalseClass
-        @macos_legacy_timers = value
-      end
+      value ? @macos_legacy_timers = value : @macos_legacy_timers
     end
 
     delegate [:bin, :etc, :libexec, :opt_bin, :opt_libexec, :opt_pkgshare, :opt_prefix, :opt_sbin, :var] => :@formula