cucumber · mvz · May 31, 2024 · May 31, 2024 · May 31, 2024 · May 31, 2024
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -72,11 +72,6 @@ Style/StringLiterals:
 Style/SymbolArray:
   EnforcedStyle: brackets
 
-# Code is currently broken with frozen string literal comments applied.
-# TODO: Enable and fix build
-Style/FrozenStringLiteralComment:
-  Enabled: false
-
 # Be relatively lenient with line length
 Layout/LineLength:
   Max: 94

diff --git a/.simplecov b/.simplecov
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 SimpleCov.configure do
   # Activate branch coverage
   enable_coverage :branch

diff --git a/Gemfile b/Gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 # Use dependencies from gemspec

diff --git a/Rakefile b/Rakefile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $LOAD_PATH << File.expand_path(__dir__)
 
 require "aruba/platform"

diff --git a/aruba.gemspec b/aruba.gemspec
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative "lib/aruba/version"
 
 Gem::Specification.new do |spec|

diff --git a/bin/console b/bin/console
@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH.unshift File.expand_path("../lib", __dir__)
 

diff --git a/exe/aruba b/exe/aruba
@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH << File.expand_path("../lib", __dir__)
 

diff --git a/features/step_definitions/hooks.rb b/features/step_definitions/hooks.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "cucumber/platform"
 
 Before "@requires-python" do

diff --git a/features/support/aruba.rb b/features/support/aruba.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/cucumber"
 
 Aruba.configure do |config|

diff --git a/features/support/env.rb b/features/support/env.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $LOAD_PATH.unshift File.expand_path("../../lib", __dir__)
 
 # Has to be the first file required so that all other files show coverage information

diff --git a/features/support/jruby.rb b/features/support/jruby.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 if RUBY_PLATFORM == "java"
   Before do
     @aruba_timeout_seconds = 15

diff --git a/features/support/simplecov_setup.rb b/features/support/simplecov_setup.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @note this file is loaded in env.rb to setup simplecov using RUBYOPTs for
 # child processes and @in-process
 unless RUBY_PLATFORM.include?("java")

diff --git a/features/support/timing.rb b/features/support/timing.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # From https://itshouldbeuseful.wordpress.com/2010/11/10/find-your-slowest-running-cucumber-features/
 
 scenario_times = {}

diff --git a/fixtures/cli-app/Gemfile b/fixtures/cli-app/Gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 # Use dependencies from gemspec

diff --git a/fixtures/cli-app/Rakefile b/fixtures/cli-app/Rakefile
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "bundler/gem_tasks"
diff --git a/fixtures/cli-app/bin/aruba-test-cli b/fixtures/cli-app/bin/aruba-test-cli
@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH << File.expand_path("../lib", __dir__)
 require "cli/app"

diff --git a/fixtures/cli-app/cli-app.gemspec b/fixtures/cli-app/cli-app.gemspec
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "cli/app/version"

diff --git a/fixtures/cli-app/features/support/aruba.rb b/fixtures/cli-app/features/support/aruba.rb
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "aruba/cucumber"
diff --git a/fixtures/cli-app/features/support/env.rb b/fixtures/cli-app/features/support/env.rb
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require_relative "aruba"
diff --git a/fixtures/cli-app/lib/cli/app.rb b/fixtures/cli-app/lib/cli/app.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "cli/app/version"
 
 Dir.glob(File.expand_path("**/*.rb", __dir__)).each { |f| require_relative f }

diff --git a/fixtures/cli-app/lib/cli/app/suppress_simple_cov_output.rb b/fixtures/cli-app/lib/cli/app/suppress_simple_cov_output.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SimpleCov
   module Formatter
     class HTMLFormatter

diff --git a/fixtures/cli-app/lib/cli/app/version.rb b/fixtures/cli-app/lib/cli/app/version.rb
@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Cli
   module App
-    VERSION = "0.1.0".freeze
+    VERSION = "0.1.0"
   end
 end
diff --git a/fixtures/cli-app/script/console b/fixtures/cli-app/script/console
@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "bundler/setup"
 require "cli/app"

diff --git a/fixtures/cli-app/spec/spec_helper.rb b/fixtures/cli-app/spec/spec_helper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $LOAD_PATH.unshift File.expand_path("../lib", __dir__)
 
 require "cli/app"

diff --git a/fixtures/cli-app/spec/support/aruba.rb b/fixtures/cli-app/spec/support/aruba.rb
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "aruba/rspec"
diff --git a/fixtures/empty-app/Rakefile b/fixtures/empty-app/Rakefile
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "bundler/gem_tasks"
diff --git a/fixtures/empty-app/cli-app.gemspec b/fixtures/empty-app/cli-app.gemspec
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "cli/app/version"

diff --git a/fixtures/empty-app/lib/cli/app.rb b/fixtures/empty-app/lib/cli/app.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "cli/app/version"
 
 module Cli

diff --git a/fixtures/empty-app/lib/cli/app/version.rb b/fixtures/empty-app/lib/cli/app/version.rb
@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Cli
   module App
-    VERSION = "0.1.0".freeze
+    VERSION = "0.1.0"
   end
 end
diff --git a/fixtures/getting-started-app/Gemfile b/fixtures/getting-started-app/Gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 gem "aruba", path: File.expand_path("../../..", __dir__)

diff --git a/fixtures/getting-started-app/features/support/env.rb b/fixtures/getting-started-app/features/support/env.rb
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "aruba/cucumber"
diff --git a/gemfiles/cucumber_8.gemfile b/gemfiles/cucumber_8.gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file was generated by Appraisal
 
 source "https://rubygems.org"

diff --git a/gemfiles/cucumber_9.gemfile b/gemfiles/cucumber_9.gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file was generated by Appraisal
 
 source "https://rubygems.org"

diff --git a/lib/aruba.rb b/lib/aruba.rb
@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "aruba/api"
diff --git a/lib/aruba/api.rb b/lib/aruba/api.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "rspec/expectations"
 require "shellwords"
 

diff --git a/lib/aruba/api/bundler.rb b/lib/aruba/api/bundler.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/api/environment"
 require "bundler"
 

diff --git a/lib/aruba/api/commands.rb b/lib/aruba/api/commands.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pathname"
 
 require "aruba/platform"

diff --git a/lib/aruba/api/core.rb b/lib/aruba/api/core.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "rspec/expectations"
 require "aruba/runtime"
 require "aruba/errors"

diff --git a/lib/aruba/api/environment.rb b/lib/aruba/api/environment.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/platform"
 
 # Aruba

diff --git a/lib/aruba/api/filesystem.rb b/lib/aruba/api/filesystem.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pathname"
 
 require "aruba/platform"

diff --git a/lib/aruba/api/text.rb b/lib/aruba/api/text.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Aruba
 module Aruba
   # Api

diff --git a/lib/aruba/aruba_logger.rb b/lib/aruba/aruba_logger.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "logger"
 
 # Aruba

diff --git a/lib/aruba/aruba_path.rb b/lib/aruba/aruba_path.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pathname"
 
 # Aruba

diff --git a/lib/aruba/basic_configuration.rb b/lib/aruba/basic_configuration.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "contracts"
 require "aruba/basic_configuration/option"
 require "aruba/in_config_wrapper"

diff --git a/lib/aruba/basic_configuration/option.rb b/lib/aruba/basic_configuration/option.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Aruba
 module Aruba
   # Basic Configuration

diff --git a/lib/aruba/cli.rb b/lib/aruba/cli.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "thor"
 require "aruba/console"
 require "aruba/initializer"

diff --git a/lib/aruba/colorizer.rb b/lib/aruba/colorizer.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Aruba
 module Aruba
   # Simple colorizer class. Only supports the color cyan

diff --git a/lib/aruba/command.rb b/lib/aruba/command.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "delegate"
 require "aruba/processes/spawn_process"
 require "aruba/processes/in_process"

diff --git a/lib/aruba/command_monitor.rb b/lib/aruba/command_monitor.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/errors"
 
 # Aruba
@@ -96,7 +98,7 @@ def clear
     def all_stdout
       registered_commands.each(&:stop)
 
-      registered_commands.each_with_object("") { |e, a| a << e.stdout }
+      registered_commands.map(&:stdout).join
     end
 
     # Get stderr of all commands
@@ -106,7 +108,7 @@ def all_stdout
     def all_stderr
       registered_commands.each(&:stop)
 
-      registered_commands.each_with_object("") { |e, a| a << e.stderr }
+      registered_commands.map(&:stderr).join
     end
 
     # Get stderr and stdout of all commands

diff --git a/lib/aruba/config/jruby.rb b/lib/aruba/config/jruby.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "rbconfig"
 
 # ideas taken from: http://blog.headius.com/2010/03/jruby-startup-time-tips.html

diff --git a/lib/aruba/config_wrapper.rb b/lib/aruba/config_wrapper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Aruba
   # This wraps the current runtime configuration of aruba.
   # If an option is changed, it notifies the event queue.

diff --git a/lib/aruba/configuration.rb b/lib/aruba/configuration.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "contracts"
 
 require "aruba/version"

diff --git a/lib/aruba/console.rb b/lib/aruba/console.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "irb"
 
 require "aruba/api"

diff --git a/lib/aruba/console/help.rb b/lib/aruba/console/help.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/api"
 
 # Aruba

diff --git a/lib/aruba/contracts/absolute_path.rb b/lib/aruba/contracts/absolute_path.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/platform"
 
 # Aruba

diff --git a/lib/aruba/contracts/enum.rb b/lib/aruba/contracts/enum.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "contracts"
 
 # Aruba

diff --git a/lib/aruba/contracts/is_power_of_two.rb b/lib/aruba/contracts/is_power_of_two.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/aruba_path"
 
 # Aruba

diff --git a/lib/aruba/contracts/relative_path.rb b/lib/aruba/contracts/relative_path.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/platform"
 
 # Aruba

diff --git a/lib/aruba/cucumber.rb b/lib/aruba/cucumber.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/version"
 
 require "aruba/api"

diff --git a/lib/aruba/cucumber/command.rb b/lib/aruba/cucumber/command.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "aruba/generators/script_file"
 
 When(/^I run `([^`]*)`$/) do |cmd|
@@ -88,7 +90,7 @@
 When "I wait for output/stdout to contain:" do |expected|
   Timeout.timeout(aruba.config.exit_timeout) do
     loop do
-      output = last_command_started.public_send :stdout, wait_for_io: 0
+      output = last_command_started.stdout wait_for_io: 0
 
       output   = sanitize_text(output)
       expected = sanitize_text(expected)
@@ -103,7 +105,7 @@
 When "I wait for output/stdout to contain {string}" do |expected|
   Timeout.timeout(aruba.config.exit_timeout) do
     loop do
-      output = last_command_started.public_send :stdout, wait_for_io: 0
+      output = last_command_started.stdout wait_for_io: 0
 
       output   = sanitize_text(output)
       expected = sanitize_text(expected)

diff --git a/lib/aruba/cucumber/environment.rb b/lib/aruba/cucumber/environment.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Given(/^a mocked home directory$/) do
   set_environment_variable "HOME", expand_path(".")
 end
Original file line number	Diff line number	Diff line change
		@@ -1 +1,3 @@
		# frozen_string_literal: true

		require "bundler/gem_tasks"
Original file line number	Diff line number	Diff line change
		@@ -1 +1,3 @@
		# frozen_string_literal: true

		require "aruba/cucumber"
Original file line number	Diff line number	Diff line change
		@@ -1 +1,3 @@
		# frozen_string_literal: true

		require_relative "aruba"
Original file line number	Diff line number	Diff line change
		@@ -1 +1,3 @@
		# frozen_string_literal: true

		require "aruba/rspec"
Original file line number	Diff line number	Diff line change
		@@ -1 +1,3 @@
		# frozen_string_literal: true

		require "aruba/api"