tty-command 0.8.0 → 0.10.1

data/lib/tty-command.rb

@@ -1,3 +1 @@
-# encoding: utf-8
-
-require 'tty/command'
+require_relative "tty/command"

data/lib/tty/command.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
-require 'rbconfig'
-
-require_relative 'command/cmd'
-require_relative 'command/exit_error'
-require_relative 'command/dry_runner'
-require_relative 'command/process_runner'
-require_relative 'command/printers/null'
-require_relative 'command/printers/pretty'
-require_relative 'command/printers/progress'
-require_relative 'command/printers/quiet'
-require_relative 'command/version'
+require "rbconfig"
+
+require_relative "command/cmd"
+require_relative "command/exit_error"
+require_relative "command/dry_runner"
+require_relative "command/process_runner"
+require_relative "command/printers/null"
+require_relative "command/printers/pretty"
+require_relative "command/printers/progress"
+require_relative "command/printers/quiet"
+require_relative "command/version"
 
 module TTY
   class Command
@@ -19,9 +19,9 @@ module TTY
     TimeoutExceeded = Class.new(StandardError)
 
     # Path to the current Ruby
-    RUBY = ENV['RUBY'] || ::File.join(
-      RbConfig::CONFIG['bindir'],
-      RbConfig::CONFIG['ruby_install_name'] + RbConfig::CONFIG['EXEEXT']
+    RUBY = ENV["RUBY"] || ::File.join(
+      RbConfig::CONFIG["bindir"],
+      RbConfig::CONFIG["ruby_install_name"] + RbConfig::CONFIG["EXEEXT"]
     )
 
     WIN_PLATFORMS = /cygwin|mswin|mingw|bccwin|wince|emx/.freeze
@@ -35,7 +35,7 @@ module TTY
     end
 
     def self.windows?
-      !!(RbConfig::CONFIG['host_os'] =~ WIN_PLATFORMS)
+      !!(RbConfig::CONFIG["host_os"] =~ WIN_PLATFORMS)
     end
 
     attr_reader :printer
@@ -128,7 +128,7 @@ module TTY
     def wait(*args)
       pattern = args.pop
       unless pattern
-        raise ArgumentError, 'Please provide condition to wait for'
+        raise ArgumentError, "Please provide condition to wait for"
       end
 
       run(*args) do |out, _|
@@ -204,7 +204,7 @@ module TTY
     #
     # @api private
     def find_printer_class(name)
-      const_name = name.to_s.split('_').map(&:capitalize).join.to_sym
+      const_name = name.to_s.split("_").map(&:capitalize).join.to_sym
       if const_name.empty? || !TTY::Command::Printers.const_defined?(const_name)
         raise ArgumentError, %(Unknown printer type "#{name}")
       end

data/lib/tty/command/child_process.rb

@@ -1,9 +1,8 @@
-# encoding: utf-8
 # frozen_string_literal: true
 
-require 'tempfile'
-require 'securerandom'
-require 'io/console'
+require "tempfile"
+require "securerandom"
+require "io/console"
 
 module TTY
   class Command
@@ -28,12 +27,12 @@ module TTY
         verbose = cmd.options[:verbose]
 
         pty = try_loading_pty(verbose) if pty
-        require('pty') if pty # load within this scope
+        require("pty") if pty # load within this scope
 
         # Create pipes
-        in_rd,  in_wr  = pty ? PTY.open : IO.pipe('utf-8') # reading
-        out_rd, out_wr = pty ? PTY.open : IO.pipe('utf-8') # writing
-        err_rd, err_wr = pty ? PTY.open : IO.pipe('utf-8') # error
+        in_rd,  in_wr  = pty ? PTY.open : IO.pipe("utf-8") # reading
+        out_rd, out_wr = pty ? PTY.open : IO.pipe("utf-8") # writing
+        err_rd, err_wr = pty ? PTY.open : IO.pipe("utf-8") # error
         in_wr.sync = true
 
         if binmode
@@ -133,7 +132,7 @@ module TTY
         key   = fd_to_process_key(spawn_key)
         value = spawn_value
 
-        if key.to_s == 'in'
+        if key.to_s == "in"
           value = convert_to_fd(spawn_value)
         end
 
@@ -196,7 +195,7 @@ module TTY
           return object
         end
 
-        tmp = ::Tempfile.new(::SecureRandom.uuid.split('-')[0])
+        tmp = ::Tempfile.new(::SecureRandom.uuid.split("-")[0])
         content = try_reading(object)
         tmp.write(content)
         tmp.rewind

data/lib/tty/command/cmd.rb

@@ -1,10 +1,13 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'securerandom'
-require 'shellwords'
+require "securerandom"
+require "shellwords"
 
 module TTY
   class Command
+    # Encapsulates the executed command
+    #
+    # @api private
     class Cmd
       # A string command name, or shell program
       # @api public
@@ -33,14 +36,15 @@ module TTY
         if env_or_cmd.respond_to?(:to_hash)
           @env = env_or_cmd
           unless command = args.shift
-            raise ArgumentError, 'Cmd requires command argument'
+            raise ArgumentError, "Cmd requires command argument"
           end
         else
           command = env_or_cmd
         end
 
         if args.empty? && cmd = command.to_s
-          raise ArgumentError, 'No command provided' if cmd.empty?
+          raise ArgumentError, "No command provided" if cmd.empty?
+
           @command = sanitize(cmd)
           @argv = []
         else
@@ -55,7 +59,7 @@ module TTY
         @env ||= {}
         @options = opts
 
-        @uuid = SecureRandom.uuid.split('-')[0]
+        @uuid = SecureRandom.uuid.split("-")[0]
         @only_output_on_error = opts.fetch(:only_output_on_error) { false }
         freeze
       end
@@ -63,8 +67,8 @@ module TTY
       # Extend command options if keys don't already exist
       #
       # @api public
-      def update(**options)
-        @options.update(options.update(@options))
+      def update(options)
+        @options.update(options.merge(@options))
       end
 
       # The shell environment variables
@@ -79,32 +83,37 @@ module TTY
           converted_key = key.is_a?(Symbol) ? key.to_s.upcase : key.to_s
           escaped_val = val.to_s.gsub(/"/, '\"')
           %(#{converted_key}="#{escaped_val}")
-        end.join(' ')
+        end.join(" ")
       end
 
       def evars(value, &block)
         return (value || block) unless environment.any?
-        %(( export #{environment_string} ; %s )) % [value || block.call]
+
+        "( export #{environment_string} ; #{value || block.call} )"
       end
 
       def umask(value)
         return value unless options[:umask]
+
         %(umask #{options[:umask]} && %s) % [value]
       end
 
       def chdir(value)
         return value unless options[:chdir]
-        %(cd #{options[:chdir]} && %s) % [value]
+
+        %(cd #{Shellwords.escape(options[:chdir])} && #{value})
       end
 
       def user(value)
         return value unless options[:user]
-        vars = environment.any? ? "#{environment_string} " : ''
+
+        vars = environment.any? ? "#{environment_string} " : ""
         %(sudo -u #{options[:user]} #{vars}-- sh -c '%s') % [value]
       end
 
       def group(value)
         return value unless options[:group]
+
         %(sg #{options[:group]} -c \\\"%s\\\") % [value]
       end
 
@@ -123,15 +132,15 @@ module TTY
 
       # @api public
       def to_s
-        [command.to_s, *Array(argv)].join(' ')
+        [command.to_s, *Array(argv)].join(" ")
       end
 
       # @api public
       def to_hash
         {
           command: command,
-          argv:    argv,
-          uuid:    uuid
+          argv: argv,
+          uuid: uuid
         }
       end
 

data/lib/tty/command/dry_runner.rb

@@ -1,7 +1,6 @@
-# encoding: utf-8
 # frozen_string_literal: true
 
-require_relative 'result'
+require_relative "result"
 
 module TTY
   class Command
@@ -18,10 +17,10 @@ module TTY
       # @api public
       def run!(*)
         cmd.to_command
-        message = "#{@printer.decorate('(dry run)', :blue)} " +
+        message = "#{@printer.decorate("(dry run)", :blue)} " +
                   @printer.decorate(cmd.to_command, :yellow, :bold)
         @printer.write(cmd, message, cmd.uuid)
-        Result.new(0, '', '')
+        Result.new(0, "", "")
       end
     end # DryRunner
   end # Command

data/lib/tty/command/exit_error.rb

@@ -1,4 +1,3 @@
-# encoding: utf-8
 # frozen_string_literal: true
 
 module TTY
@@ -25,7 +24,7 @@ module TTY
       end
 
       def extract_output(value)
-        (value || '').strip.empty? ? 'Nothing written' : value.strip
+        (value || "").strip.empty? ? "Nothing written" : value.strip
       end
     end # ExitError
   end # Command

data/lib/tty/command/printers/abstract.rb

@@ -1,6 +1,4 @@
-# encoding: utf-8
-
-require 'pastel'
+require "pastel"
 
 module TTY
   class Command
@@ -22,11 +20,11 @@ module TTY
         def initialize(output, options = {})
           @output  = output
           @options = options
-          @enabled = options.fetch(:color) { true }
-          @color   = ::Pastel.new(output: output, enabled: @enabled)
+          @enabled = options.fetch(:color, true)
+          @color   = ::Pastel.new(enabled: @enabled)
 
-          @out_data = ''
-          @err_data = ''
+          @out_data = ""
+          @err_data = ""
         end
 
         def print_command_start(cmd, *args)
@@ -34,15 +32,15 @@ module TTY
         end
 
         def print_command_out_data(cmd, *args)
-          write(args.join(' '))
+          write(args.join(" "))
         end
 
         def print_command_err_data(cmd, *args)
-          write(args.join(' '))
+          write(args.join(" "))
         end
 
         def print_command_exit(cmd, *args)
-          write(args.join(' '))
+          write(args.join(" "))
         end
 
         def write(cmd, message)

data/lib/tty/command/printers/null.rb

@@ -1,7 +1,4 @@
-# encoding: utf-8
-# frozen_string_literal: true
-
-require_relative 'abstract'
+require_relative "abstract"
 
 module TTY
   class Command

data/lib/tty/command/printers/pretty.rb

@@ -1,53 +1,54 @@
-# encoding: utf-8
-# frozen_string_literal: true
-
-require 'pastel'
-
-require_relative 'abstract'
+require_relative "abstract"
 
 module TTY
   class Command
     module Printers
       class Pretty < Abstract
-        TIME_FORMAT = "%5.3f %s".freeze
+        TIME_FORMAT = "%5.3f %s"
+
+        def initialize(*)
+          super
+          @uuid = options.fetch(:uuid, true)
+        end
 
         def print_command_start(cmd, *args)
           message = ["Running #{decorate(cmd.to_command, :yellow, :bold)}"]
-          message << args.map(&:chomp).join(' ') unless args.empty?
-          write(cmd, message.join, cmd.uuid)
+          message << args.map(&:chomp).join(" ") unless args.empty?
+          write(cmd, message.join)
         end
 
         def print_command_out_data(cmd, *args)
-          message = args.map(&:chomp).join(' ')
-          write(cmd, "\t#{message}", cmd.uuid, out_data)
+          message = args.map(&:chomp).join(" ")
+          write(cmd, "\t#{message}", out_data)
         end
 
         def print_command_err_data(cmd, *args)
-          message = args.map(&:chomp).join(' ')
-          write(cmd, "\t" + decorate(message, :red), cmd.uuid, err_data)
+          message = args.map(&:chomp).join(" ")
+          write(cmd, "\t" + decorate(message, :red), err_data)
         end
 
         def print_command_exit(cmd, status, runtime, *args)
-          unless !cmd.only_output_on_error || status.zero?
+          if cmd.only_output_on_error && !status.zero?
             output << out_data
             output << err_data
           end
 
-          runtime = TIME_FORMAT % [runtime, pluralize(runtime, 'second')]
+          runtime = TIME_FORMAT % [runtime, pluralize(runtime, "second")]
           message = ["Finished in #{runtime}"]
           message << " with exit status #{status}" if status
           message << " (#{success_or_failure(status)})"
-          write(cmd, message.join, cmd.uuid)
+          write(cmd, message.join)
         end
 
         # Write message out to output
         #
         # @api private
-        def write(cmd, message, uuid = nil, data = nil)
-          uuid_needed = options.fetch(:uuid) { true }
+        def write(cmd, message, data = nil)
+          cmd_set_uuid = cmd.options.fetch(:uuid, true)
+          uuid_needed = cmd.options[:uuid].nil? ? @uuid : cmd_set_uuid
           out = []
           if uuid_needed
-            out << "[#{decorate(uuid, :green)}] " unless uuid.nil?
+            out << "[#{decorate(cmd.uuid, :green)}] " unless cmd.uuid.nil?
           end
           out << "#{message}\n"
           target = (cmd.only_output_on_error && !data.nil?) ? data : output
@@ -60,15 +61,15 @@ module TTY
         #
         # @api private
         def pluralize(count, word)
-          "#{word}#{'s' unless count.to_f == 1}"
+          "#{word}#{'s' unless count.to_i == 1}"
         end
 
         # @api private
         def success_or_failure(status)
           if status == 0
-            decorate('successful', :green, :bold)
+            decorate("successful", :green, :bold)
           else
-            decorate('failed', :red, :bold)
+            decorate("failed", :red, :bold)
           end
         end
       end # Pretty

data/lib/tty/command/printers/progress.rb

@@ -1,14 +1,9 @@
-# encoding: utf-8
-# frozen_string_literal: true
-
-require 'pastel'
-require_relative 'abstract'
+require_relative "abstract"
 
 module TTY
   class Command
     module Printers
       class Progress < Abstract
-
         def print_command_exit(cmd, status, runtime, *args)
           output.print(success_or_failure(status))
         end
@@ -21,9 +16,9 @@ module TTY
         # @api private
         def success_or_failure(status)
           if status == 0
-            decorate('.', :green)
+            decorate(".", :green)
           else
-            decorate('F', :red)
+            decorate("F", :red)
           end
         end
       end # Progress

data/lib/tty/command/printers/quiet.rb

@@ -1,23 +1,19 @@
-# encoding: utf-8
-
-require_relative 'abstract'
+require_relative "abstract"
 
 module TTY
   class Command
     module Printers
       class Quiet < Abstract
-        attr_reader :output, :options
-
         def print_command_start(cmd)
           # quiet
         end
 
         def print_command_out_data(cmd, *args)
-          write(cmd, args.join(' '), out_data)
+          write(cmd, args.join(" "), out_data)
         end
 
         def print_command_err_data(cmd, *args)
-          write(cmd, args.join(' '), err_data)
+          write(cmd, args.join(" "), err_data)
         end
 
         def print_command_exit(cmd, status, *args)