aruba 1.0.0 → 1.1.0

data/lib/aruba/matchers/command/have_output_on_stderr.rb

@@ -17,7 +17,9 @@ RSpec::Matchers.define :have_output_on_stderr do |expected|
   match do |actual|
     @old_actual = actual
 
-    raise "Expected #{@old_actual} to respond to #stderr" unless @old_actual.respond_to? :stderr
+    unless @old_actual.respond_to? :stderr
+      raise "Expected #{@old_actual} to respond to #stderr"
+    end
 
     @old_actual.stop
 

data/lib/aruba/matchers/command/have_output_on_stdout.rb

@@ -17,7 +17,9 @@ RSpec::Matchers.define :have_output_on_stdout do |expected|
   match do |actual|
     @old_actual = actual
 
-    raise "Expected #{@old_actual} to respond to #stdout" unless @old_actual.respond_to? :stdout
+    unless @old_actual.respond_to? :stdout
+      raise "Expected #{@old_actual} to respond to #stdout"
+    end
 
     @old_actual.stop
 

data/lib/aruba/matchers/directory.rb

@@ -1 +1 @@
-Aruba.platform.require_matching_files('../directory/**/*.rb', __FILE__)
+Aruba.platform.require_matching_files("../directory/**/*.rb", __FILE__)

data/lib/aruba/matchers/directory/be_an_existing_directory.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method be_an_existing_directory
 #   This matchers checks if <directory> exists in filesystem
@@ -19,7 +19,7 @@ RSpec::Matchers.define :be_an_existing_directory do |_|
   match do |actual|
     stop_all_commands
 
-    raise 'String expected' unless actual.is_a? String
+    raise "String expected" unless actual.is_a? String
 
     directory?(actual)
   end

data/lib/aruba/matchers/directory/have_sub_directory.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method have_sub_directory(sub_directory)
 #   This matchers checks if <directory> has given sub-directory
@@ -23,7 +23,8 @@ require 'rspec/expectations/version'
 #
 #     RSpec.describe do
 #       it { expect('dir1.d').to have_sub_directory(['subdir.1.d', 'subdir.2.d']) }
-#       it { expect(directories).to include a_directory_with_sub_directory(['subdir.1.d', 'subdir.2.d']) }
+#       it { expect(directories)
+#       .  to include a_directory_with_sub_directory(['subdir.1.d', 'subdir.2.d']) }
 #     end
 RSpec::Matchers.define :have_sub_directory do |expected|
   match do |actual|
@@ -40,11 +41,15 @@ RSpec::Matchers.define :have_sub_directory do |expected|
   diffable
 
   failure_message do |actual|
-    format('expected that directory "%s" has the following sub-directories: %s.', actual.join(', '), expected)
+    format('expected that directory "%s" has the following sub-directories: %s.',
+           actual.join(", "),
+           expected)
   end
 
   failure_message_when_negated do |actual|
-    format('expected that directory "%s" does not have the following sub-directories: %s.', actual.join(', '), expected)
+    format('expected that directory "%s" does not have the following sub-directories: %s.',
+           actual.join(", "),
+           expected)
   end
 end
 

data/lib/aruba/matchers/environment.rb

@@ -1 +1 @@
-Aruba.platform.require_matching_files('../matchers/environment/*.rb', __FILE__)
+Aruba.platform.require_matching_files("../matchers/environment/*.rb", __FILE__)

data/lib/aruba/matchers/file.rb

@@ -1 +1 @@
-Aruba.platform.require_matching_files('../file/**/*.rb', __FILE__)
+Aruba.platform.require_matching_files("../file/**/*.rb", __FILE__)

data/lib/aruba/matchers/file/be_a_command_found_in_path.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method be_a_command_found_in_path
 #   This matchers checks if <command> can be found in path

data/lib/aruba/matchers/file/be_an_existing_executable.rb

@@ -1,5 +1,5 @@
-require 'rspec/expectations/version'
-require 'shellwords'
+require "rspec/expectations/version"
+require "shellwords"
 
 # @!method be_an_existing_executable
 #   This matchers checks if <file> exists in filesystem

data/lib/aruba/matchers/file/be_an_existing_file.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method be_an_existing_file
 #   This matchers checks if <file> exists in filesystem
@@ -19,7 +19,7 @@ RSpec::Matchers.define :be_an_existing_file do |_|
   match do |actual|
     stop_all_commands
 
-    raise 'String expected' unless actual.is_a? String
+    raise "String expected" unless actual.is_a? String
 
     file?(actual)
   end

data/lib/aruba/matchers/file/have_file_content.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method have_file_content(content)
 #   This matchers checks if <file> has content. `content` can be a string,

data/lib/aruba/matchers/file/have_file_size.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method have_file_size(size)
 #   This matchers checks if path has file size
@@ -34,11 +34,13 @@ RSpec::Matchers.define :have_file_size do |expected|
   end
 
   failure_message do |_actual|
-    format('expected that file "%s" has size "%s", but has "%s"', @old_actual, @actual, @expected)
+    format('expected that file "%s" has size "%s", but has "%s"',
+           @old_actual, @actual, @expected)
   end
 
   failure_message_when_negated do |_actual|
-    format('expected that file "%s" does not have size "%s", but has "%s"', @old_actual, @actual, @expected)
+    format('expected that file "%s" does not have size "%s", but has "%s"',
+           @old_actual, @actual, @expected)
   end
 end
 

data/lib/aruba/matchers/file/have_same_file_content.rb

@@ -1,5 +1,5 @@
-require 'rspec/expectations/version'
-require 'fileutils'
+require "rspec/expectations/version"
+require "fileutils"
 
 # @!method have_same_file_content_as(file_name)
 #   This matchers checks if <file1> has the same content like <file2>

data/lib/aruba/matchers/path.rb

@@ -1 +1 @@
-Aruba.platform.require_matching_files('../path/**/*.rb', __FILE__)
+Aruba.platform.require_matching_files("../path/**/*.rb", __FILE__)

data/lib/aruba/matchers/path/a_path_matching_pattern.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method a_path_matching_pattern(/pattern/)
 #   This matchers checks if <path> matches pattern. `pattern` can be a string,

data/lib/aruba/matchers/path/be_an_absolute_path.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method be_an_absolute_path
 #   This matchers checks if <path> exists in filesystem

data/lib/aruba/matchers/path/be_an_existing_path.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method be_an_existing_path
 #   This matchers checks if <path> exists in filesystem

data/lib/aruba/matchers/path/have_permissions.rb

@@ -1,4 +1,4 @@
-require 'rspec/expectations/version'
+require "rspec/expectations/version"
 
 # @!method have_permissions(permissions)
 #   This matchers checks if <file> or <directory> has <perm> permissions
@@ -39,10 +39,11 @@ RSpec::Matchers.define :have_permissions do |expected|
     @old_actual = actual
     @actual = permissions(expand_path(@old_actual))
 
-    @expected = if expected.is_a? Integer
+    @expected = case expected
+                when Integer
                   expected.to_s(8)
-                elsif expected.is_a? String
-                  expected.gsub(/^0*/, '')
+                when String
+                  expected.gsub(/^0*/, "")
                 else
                   expected
                 end
@@ -51,11 +52,13 @@ RSpec::Matchers.define :have_permissions do |expected|
   end
 
   failure_message do |_actual|
-    format('expected that path "%s" has permissions "%s", but has "%s".', @old_actual, @expected, @actual)
+    format('expected that path "%s" has permissions "%s", but has "%s".',
+           @old_actual, @expected, @actual)
   end
 
   failure_message_when_negated do |_actual|
-    format('expected that path "%s" does not have permissions "%s", but has "%s".', @old_actual, @expected, @actual)
+    format('expected that path "%s" does not have permissions "%s", but has "%s".',
+           @old_actual, @expected, @actual)
   end
 end
 

data/lib/aruba/matchers/string.rb

@@ -1 +1 @@
-Aruba.platform.require_matching_files('../string/**/*.rb', __FILE__)
+Aruba.platform.require_matching_files("../string/**/*.rb", __FILE__)

data/lib/aruba/matchers/string/include_output_string.rb

@@ -18,7 +18,7 @@
 #     end
 RSpec::Matchers.define :include_output_string do |expected|
   match do |actual|
-    actual.force_encoding('UTF-8')
+    actual.force_encoding("UTF-8")
     @expected = Regexp.new(Regexp.escape(sanitize_text(expected.to_s)), Regexp::MULTILINE)
     @actual   = sanitize_text(actual)
 

data/lib/aruba/matchers/string/match_output_string.rb

@@ -18,7 +18,7 @@
 #     end
 RSpec::Matchers.define :match_output_string do |expected|
   match do |actual|
-    actual.force_encoding('UTF-8')
+    actual.force_encoding("UTF-8")
     @expected = Regexp.new(unescape_text(expected), Regexp::MULTILINE)
     @actual   = sanitize_text(actual)
 

data/lib/aruba/matchers/string/output_string_eq.rb

@@ -18,7 +18,7 @@
 #     end
 RSpec::Matchers.define :output_string_eq do |expected|
   match do |actual|
-    actual.force_encoding('UTF-8')
+    actual.force_encoding("UTF-8")
     @expected = sanitize_text(expected.to_s)
     @actual   = sanitize_text(actual.to_s)
 

data/lib/aruba/platform.rb

@@ -1,5 +1,5 @@
-require 'aruba/platforms/unix_platform'
-require 'aruba/platforms/windows_platform'
+require "aruba/platforms/unix_platform"
+require "aruba/platforms/windows_platform"
 
 # Aruba
 module Aruba

data/lib/aruba/platforms/announcer.rb

@@ -1,7 +1,7 @@
-require 'shellwords'
-require 'aruba/colorizer'
+require "shellwords"
+require "aruba/colorizer"
 
-Aruba::Colorizer.coloring = false if !STDOUT.tty? && !ENV.key?('AUTOTEST')
+Aruba::Colorizer.coloring = false if !$stdout.tty? && !ENV.key?("AUTOTEST")
 
 # Aruba
 module Aruba
@@ -21,7 +21,9 @@ module Aruba
     #
     #   before do
     #     current_example = context.example
-    #     aruba.announcer.activate :my_channel if current_example.metadata[:announce_my_channel]
+    #     if current_example.metadata[:announce_my_channel]
+    #       aruba.announcer.activate :my_channel
+    #     end
     #   end
     #
     #   Aruba.announcer.announce(:my_channel, 'my message')
@@ -79,19 +81,29 @@ module Aruba
       private
 
       def after_init
-        output_format :changed_configuration, proc { |n, v| format('# %s = %s', n, v) }
-        output_format :changed_environment, proc { |n, v| format('$ export %s=%s', n, Shellwords.escape(v)) }
-        output_format :command, '$ %s'
-        output_format :directory, '$ cd %s'
-        output_format :environment, proc { |n, v| format('$ export %s=%s', n, Shellwords.escape(v)) }
-        output_format :full_environment, proc { |h| format("<<-ENVIRONMENT\n%s\nENVIRONMENT", Aruba.platform.simple_table(h)) }
-        output_format :modified_environment, proc { |n, v| format('$ export %s=%s', n, Shellwords.escape(v)) }
+        output_format :changed_configuration, proc { |n, v| format("# %s = %s", n, v) }
+        output_format :changed_environment,
+                      proc { |n, v| format("$ export %s=%s", n, Shellwords.escape(v)) }
+        output_format :command, "$ %s"
+        output_format :directory, "$ cd %s"
+        output_format :environment,
+                      proc { |n, v| format("$ export %s=%s", n, Shellwords.escape(v)) }
+        output_format :full_environment,
+                      proc { |h|
+                        format("<<-ENVIRONMENT\n%s\nENVIRONMENT",
+                               Aruba.platform.simple_table(h))
+                      }
+        output_format :modified_environment,
+                      proc { |n, v| format("$ export %s=%s", n, Shellwords.escape(v)) }
         output_format :stderr, "<<-STDERR\n%s\nSTDERR"
         output_format :stdout, "<<-STDOUT\n%s\nSTDOUT"
         output_format :command_content, "<<-COMMAND\n%s\nCOMMAND"
-        output_format :stop_signal, proc { |p, s| format('Command will be stopped with `kill -%s %s`', s, p) }
-        output_format :timeout, '# %s-timeout: %s seconds'
-        output_format :wait_time, '# %s: %s seconds'
+        output_format :stop_signal,
+                      proc { |p, s|
+                        format("Command will be stopped with `kill -%s %s`", s, p)
+                      }
+        output_format :timeout, "# %s-timeout: %s seconds"
+        output_format :wait_time, "# %s: %s seconds"
         output_format :command_filesystem_status,
                       proc { |status|
                         format("<<-COMMAND FILESYSTEM STATUS\n%s\nCOMMAND FILESYSTEM STATUS",
@@ -99,8 +111,8 @@ module Aruba
                       }
       end
 
-      def output_format(channel, string = '%s', &block)
-        output_formats[channel.to_sym] = if block_given?
+      def output_format(channel, string = "%s", &block)
+        output_formats[channel.to_sym] = if block
                                            block
                                          elsif string.is_a?(Proc)
                                            string
@@ -168,7 +180,7 @@ module Aruba
         the_output_format = if output_formats.key? channel
                               output_formats[channel]
                             else
-                              proc { |v| format('%s', v) }
+                              proc { |v| format("%s", v) }
                             end
 
         return unless activated?(channel)

data/lib/aruba/platforms/aruba_file_creator.rb

@@ -18,7 +18,9 @@ module Aruba
       # @param [Boolean] check_presence (false)
       #   Check if file exist
       def call(path, content, check_presence = false)
-        fail "Expected #{path} to be present" if check_presence && !Aruba.platform.file?(path)
+        if check_presence && !Aruba.platform.file?(path)
+          raise "Expected #{path} to be present"
+        end
 
         Aruba.platform.mkdir(File.dirname(path))
 

data/lib/aruba/platforms/aruba_fixed_size_file_creator.rb

@@ -22,11 +22,16 @@ module Aruba
       # @param [Boolean] check_presence (false)
       #   Check if file exist
       def call(path, size, check_presence)
-        fail "Expected #{path} to be present" if check_presence && !Aruba.platform.file?(path)
+        if check_presence && !Aruba.platform.file?(path)
+          raise "Expected #{path} to be present"
+        end
 
         Aruba.platform.mkdir(File.dirname(path))
 
-        File.open(path, 'wb') { |f| f.seek(size - 1); f.write("\0") }
+        File.open(path, "wb") do |f|
+          f.seek(size - 1)
+          f.write("\0")
+        end
 
         self
       end

data/lib/aruba/platforms/aruba_logger.rb

@@ -1,4 +1,4 @@
-require 'logger'
+require "logger"
 
 # Aruba
 module Aruba
@@ -47,20 +47,21 @@ module Aruba
     def logger
       l = ::Logger.new($stderr)
 
-      case mode
-      when :debug
-        l.level = ::Logger::DEBUG
+      if mode == :debug
         format_debug(l)
-      when :silent
-        l.level = 9_999
-      when :info
-        l.level = ::Logger::INFO
-        format_standard(l)
       else
-        l.level = ::Logger::INFO
         format_standard(l)
       end
 
+      l.level = case mode
+                when :debug
+                  ::Logger::DEBUG
+                when :silent
+                  9_999
+                else
+                  ::Logger::INFO
+                end
+
       l
     end
 

data/lib/aruba/platforms/command_monitor.rb

@@ -1,4 +1,4 @@
-require 'aruba/errors'
+require "aruba/errors"
 
 # Aruba
 module Aruba
@@ -12,7 +12,7 @@ module Aruba
 
     public
 
-    attr_reader :registered_commands, :last_command_started
+    attr_reader :registered_commands, :last_command_started, :last_command_stopped
 
     class DefaultLastCommandStopped
       def nil?
@@ -20,7 +20,7 @@ module Aruba
       end
 
       def method_missing(*)
-        fail NoCommandHasBeenStoppedError, 'No last command stopped available'
+        raise NoCommandHasBeenStoppedError, "No last command stopped available"
       end
 
       def respond_to_missing?(*)
@@ -34,7 +34,7 @@ module Aruba
       end
 
       def method_missing(*)
-        fail NoCommandHasBeenStartedError, 'No last command started available'
+        raise NoCommandHasBeenStartedError, "No last command started available"
       end
 
       def respond_to_missing?(*)
@@ -52,8 +52,6 @@ module Aruba
       raise ArgumentError, e.message
     end
 
-    attr_reader :last_command_stopped
-
     # Set last command started
     #
     # @param [String] cmd
@@ -78,7 +76,7 @@ module Aruba
       cmd = cmd.commandline if cmd.respond_to? :commandline
       command = registered_commands.reverse.find { |c| c.commandline == cmd }
 
-      fail CommandNotFoundError, "No command named '#{cmd}' has been started" if command.nil?
+      raise CommandNotFoundError, "No command named '#{cmd}' has been started" if command.nil?
 
       command
     end
@@ -98,7 +96,7 @@ module Aruba
     def all_stdout
       registered_commands.each(&:stop)
 
-      registered_commands.each_with_object('') { |e, a| a << e.stdout }
+      registered_commands.each_with_object("") { |e, a| a << e.stdout }
     end
 
     # Get stderr of all commands
@@ -108,7 +106,7 @@ module Aruba
     def all_stderr
       registered_commands.each(&:stop)
 
-      registered_commands.each_with_object('') { |e, a| a << e.stderr }
+      registered_commands.each_with_object("") { |e, a| a << e.stderr }
     end
 
     # Get stderr and stdout of all commands