aruba 1.0.0 → 1.1.0

data/lib/aruba/cucumber/environment.rb

@@ -1,5 +1,5 @@
 Given(/^a mocked home directory$/) do
-  set_environment_variable 'HOME', expand_path('.')
+  set_environment_variable "HOME", expand_path(".")
 end
 
 Given(/^I set the environment variable "(.*)" to "(.*)"/) do |variable, value|
@@ -16,8 +16,8 @@ end
 
 Given(/^I set the environment variables? to:/) do |table|
   table.hashes.each do |row|
-    variable = row['variable'].to_s
-    value = row['value'].to_s
+    variable = row["variable"].to_s
+    value = row["value"].to_s
 
     set_environment_variable(variable, value)
   end
@@ -25,8 +25,8 @@ end
 
 Given(/^I append the values? to the environment variables?:/) do |table|
   table.hashes.each do |row|
-    variable = row['variable'].to_s
-    value = row['value'].to_s
+    variable = row["variable"].to_s
+    value = row["value"].to_s
 
     append_environment_variable(variable, value)
   end
@@ -34,8 +34,8 @@ end
 
 Given(/^I prepend the values? to the environment variables?:/) do |table|
   table.hashes.each do |row|
-    variable = row['variable'].to_s
-    value = row['value'].to_s
+    variable = row["variable"].to_s
+    value = row["value"].to_s
 
     prepend_environment_variable(variable, value)
   end

data/lib/aruba/cucumber/file.rb

@@ -3,11 +3,13 @@ Given(/^I use (?:a|the) fixture(?: named)? "([^"]*)"$/) do |name|
   cd name
 end
 
-Given(/^I copy (?:a|the) (?:file|directory)(?: (?:named|from))? "([^"]*)" to "([^"]*)"$/) do |source, destination|
+Given(/^I copy (?:a|the) (?:file|directory)(?: (?:named|from))? "([^"]*)" to "([^"]*)"$/) \
+  do |source, destination|
   copy source, destination
 end
 
-Given(/^I move (?:a|the) (?:file|directory)(?: (?:named|from))? "([^"]*)" to "([^"]*)"$/) do |source, destination|
+Given(/^I move (?:a|the) (?:file|directory)(?: (?:named|from))? "([^"]*)" to "([^"]*)"$/) \
+  do |source, destination|
   move source, destination
 end
 
@@ -15,7 +17,8 @@ Given(/^(?:a|the|(?:an empty)) directory(?: named)? "([^"]*)"$/) do |dir_name|
   create_directory(dir_name)
 end
 
-Given(/^(?:a|the) directory(?: named)? "([^"]*)" with mode "([^"]*)"$/) do |dir_name, dir_mode|
+Given(/^(?:a|the) directory(?: named)? "([^"]*)" with mode "([^"]*)"$/) \
+  do |dir_name, dir_mode|
   create_directory(dir_name)
   chmod(dir_mode, dir_name)
 end
@@ -25,14 +28,15 @@ Given(/^(?:a|the) file(?: named)? "([^"]*)" with:$/) do |file_name, file_content
 end
 
 Given(/^(?:an|the) executable(?: named)? "([^"]*)" with:$/) do |file_name, file_content|
-  step %(a file named "#{file_name}" with mode "0755" and with:), file_content
+  step %(a file named "#{file_name}" with mode "0o755" and with:), file_content
 end
 
 Given(/^(?:a|the) file(?: named)? "([^"]*)" with "([^"]*)"$/) do |file_name, file_content|
   write_file(file_name, unescape_text(file_content))
 end
 
-Given(/^(?:a|the) file(?: named)? "([^"]*)" with mode "([^"]*)" and with:$/) do |file_name, file_mode, file_content|
+Given(/^(?:a|the) file(?: named)? "([^"]*)" with mode "([^"]*)" and with:$/) \
+  do |file_name, file_mode, file_content|
   write_file(file_name, unescape_text(file_content))
   chmod(file_mode, file_name)
 end
@@ -42,11 +46,12 @@ Given(/^(?:a|the) (\d+) byte file(?: named)? "([^"]*)"$/) do |file_size, file_na
 end
 
 Given(/^(?:an|the) empty file(?: named)? "([^"]*)"$/) do |file_name|
-  write_file(file_name, '')
+  write_file(file_name, "")
 end
 
-Given(/^(?:an|the) empty file(?: named)? "([^"]*)" with mode "([^"]*)"$/) do |file_name, file_mode|
-  write_file(file_name, '')
+Given(/^(?:an|the) empty file(?: named)? "([^"]*)" with mode "([^"]*)"$/) \
+  do |file_name, file_mode|
+  write_file(file_name, "")
   chmod(file_mode, file_name)
 end
 
@@ -54,7 +59,8 @@ When(/^I write to "([^"]*)" with:$/) do |file_name, file_content|
   write_file(file_name, file_content)
 end
 
-When(/^I overwrite(?: (?:a|the) file(?: named)?)? "([^"]*)" with:$/) do |file_name, file_content|
+When(/^I overwrite(?: (?:a|the) file(?: named)?)? "([^"]*)" with:$/) \
+  do |file_name, file_content|
   overwrite_file(file_name, file_content)
 end
 
@@ -62,12 +68,17 @@ When(/^I append to "([^"]*)" with:$/) do |file_name, file_content|
   append_to_file(file_name, file_content)
 end
 
+When(/^I append the following lines to "([^"]*)":$/) do |file_name, file_content|
+  append_lines_to_file(file_name, file_content)
+end
+
 When(/^I append to "([^"]*)" with "([^"]*)"$/) do |file_name, file_content|
   append_to_file(file_name, file_content)
 end
 
-When(/^I remove (?:a|the) (?:file|directory)(?: named)? "([^"]*)"( with full force)?$/) do |name, force_remove|
-  remove(name, force: force_remove.nil? ? false : true)
+When(/^I remove (?:a|the) (?:file|directory)(?: named)? "([^"]*)"( with full force)?$/) \
+  do |name, force_remove|
+  remove(name, force: !force_remove.nil?)
 end
 
 Given(/^(?:a|the) (?:file|directory)(?: named)? "([^"]*)" does not exist$/) do |name|
@@ -88,7 +99,8 @@ Then(/^the following files should (not )?exist:$/) do |negated, files|
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?exist(?: anymore)?$/) do |path, expect_match|
+Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?exist(?: anymore)?$/) \
+  do |path, expect_match|
   if expect_match
     expect(path).not_to be_an_existing_file
   else
@@ -96,7 +108,8 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?exist(?: anymore)?$/) d
   end
 end
 
-Then(/^(?:a|the) directory(?: named)? "([^"]*)" should (not )?exist(?: anymore)?$/) do |path, expect_match|
+Then(/^(?:a|the) directory(?: named)? "([^"]*)" should (not )?exist(?: anymore)?$/) \
+  do |path, expect_match|
   if expect_match
     expect(path).not_to be_an_existing_directory
   else
@@ -112,7 +125,8 @@ Then(/^(?:a|the) file matching %r<(.*?)> should (not )?exist$/) do |pattern, exp
   end
 end
 
-Then(/^(?:a|the) (\d+) byte file(?: named)? "([^"]*)" should (not )?exist$/) do |size, file, negated|
+Then(/^(?:a|the) (\d+) byte file(?: named)? "([^"]*)" should (not )?exist$/) \
+  do |size, file, negated|
   if negated
     expect(file).not_to have_file_size(size)
   else
@@ -130,7 +144,8 @@ Then(/^the following directories should (not )?exist:$/) do |negated, directorie
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain "([^"]*)"$/) do |file, negated, content|
+Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain "([^"]*)"$/) \
+  do |file, negated, content|
   if negated
     expect(file).not_to have_file_content file_content_including(content.chomp)
   else
@@ -138,7 +153,8 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain "([^"]*)"$/) do
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain:$/) do |file, negated, content|
+Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain:$/) \
+  do |file, negated, content|
   if negated
     expect(file).not_to have_file_content file_content_including(content.chomp)
   else
@@ -146,7 +162,8 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain:$/) do |file, n
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain exactly:$/) do |file, negated, content|
+Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain exactly:$/) \
+  do |file, negated, content|
   if negated
     expect(file).not_to have_file_content content
   else
@@ -154,7 +171,8 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?contain exactly:$/) do
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?match %r<([^>]*)>$/) do |file, negated, content|
+Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?match %r<([^>]*)>$/) \
+  do |file, negated, content|
   if negated
     expect(file).not_to have_file_content file_content_matching(content)
   else
@@ -162,7 +180,8 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?match %r<([^>]*)>$/) do
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?match \/([^\/]*)\/$/) do |file, negated, content|
+Then(%r{^(?:a|the) file(?: named)? "([^"]*)" should (not )?match /([^/]*)/$}) \
+  do |file, negated, content|
   if negated
     expect(file).not_to have_file_content file_content_matching(content)
   else
@@ -170,7 +189,8 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?match \/([^\/]*)\/$/) d
   end
 end
 
-Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?be equal to file "([^"]*)"/) do |file, negated, reference_file|
+Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?be equal to file "([^"]*)"/) \
+  do |file, negated, reference_file|
   if negated
     expect(file).not_to have_same_file_content_as(reference_file)
   else
@@ -178,10 +198,14 @@ Then(/^(?:a|the) file(?: named)? "([^"]*)" should (not )?be equal to file "([^"]
   end
 end
 
-Then(/^(?:a|the) (?:file|directory)(?: named)? "([^"]*)" should( not)? have permissions "([^"]*)"$/) do |path, negated, permissions|
-  if negated
-    expect(path).not_to have_permissions(permissions)
-  else
-    expect(path).to have_permissions(permissions)
-  end
+Then(
+  /^(?:a|the) (?:file|directory)(?: named)? "([^"]*)" should have permissions "([^"]*)"$/
+) do |path, permissions|
+  expect(path).to have_permissions(permissions)
+end
+
+Then(
+  /^(?:a|the) (?:file|directory)(?: named)? "([^"]*)" should not have permissions "([^"]*)"$/
+) do |path, permissions|
+  expect(path).not_to have_permissions(permissions)
 end

data/lib/aruba/cucumber/hooks.rb

@@ -1,6 +1,6 @@
-require 'aruba/aruba_path'
-require 'aruba/api'
-require 'aruba/platform'
+require "aruba/aruba_path"
+require "aruba/api"
+require "aruba/platform"
 World(Aruba::Api)
 
 Around do |_, block|
@@ -9,8 +9,11 @@ end
 
 Before do
   # ... so every change needs to be done later
-  prepend_environment_variable 'PATH', aruba.config.command_search_paths.join(File::PATH_SEPARATOR) + File::PATH_SEPARATOR
-  set_environment_variable 'HOME', aruba.config.home_directory
+  prepend_environment_variable(
+    "PATH",
+    aruba.config.command_search_paths.join(File::PATH_SEPARATOR) + File::PATH_SEPARATOR
+  )
+  set_environment_variable "HOME", aruba.config.home_directory
 end
 
 After do
@@ -18,68 +21,68 @@ After do
   aruba.command_monitor.clear
 end
 
-Before('@no-clobber') do
+Before("@no-clobber") do
   setup_aruba(false)
 end
 
-Before('~@no-clobber') do
+Before("~@no-clobber") do
   setup_aruba
 end
 
-Before('@puts') do
+Before("@puts") do
   aruba.announcer.mode = :puts
 end
 
-Before('@announce-command') do
+Before("@announce-command") do
   aruba.announcer.activate :command
 end
 
-Before('@announce-command-content') do
+Before("@announce-command-content") do
   aruba.announcer.activate :command_content
 end
 
-Before('@announce-command-filesystem-status') do
+Before("@announce-command-filesystem-status") do
   aruba.announcer.activate :command_filesystem_status
 end
 
-Before('@announce-output') do
+Before("@announce-output") do
   aruba.announcer.activate :stdout
   aruba.announcer.activate :stderr
 end
 
-Before('@announce-stdout') do
+Before("@announce-stdout") do
   aruba.announcer.activate :stdout
 end
 
-Before('@announce-stderr') do
+Before("@announce-stderr") do
   aruba.announcer.activate :stderr
 end
 
-Before('@announce-directory') do
+Before("@announce-directory") do
   aruba.announcer.activate :directory
 end
 
-Before('@announce-stop-signal') do
+Before("@announce-stop-signal") do
   aruba.announcer.activate :stop_signal
 end
 
-Before('@announce-full-environment') do
+Before("@announce-full-environment") do
   aruba.announcer.activate :full_environment
 end
 
-Before('@announce-changed-environment') do
+Before("@announce-changed-environment") do
   aruba.announcer.activate :changed_environment
 end
 
-Before('@announce-timeout') do
+Before("@announce-timeout") do
   aruba.announcer.activate :timeout
 end
 
-Before('@announce-wait-time') do
+Before("@announce-wait-time") do
   aruba.announcer.activate :wait_time
 end
 
-Before('@announce') do
+Before("@announce") do
   aruba.announcer.activate :changed_environment
   aruba.announcer.activate :command
   aruba.announcer.activate :directory
@@ -95,30 +98,30 @@ Before('@announce') do
   aruba.announcer.activate :command_filesystem_status
 end
 
-Before('@keep-ansi-escape-sequences') do
+Before("@keep-ansi-escape-sequences") do
   aruba.config.remove_ansi_escape_sequences = false
 end
 
-Before '@mocked-home-directory' do
-  set_environment_variable 'HOME', expand_path('.')
+Before "@mocked-home-directory" do
+  set_environment_variable "HOME", expand_path(".")
 end
 
-Before('@disable-bundler') do
+Before("@disable-bundler") do
   unset_bundler_env_vars
 end
 
-Before('@debug') do
+Before("@debug") do
   aruba.config.command_launcher = :debug
 end
 
-Before('@command-launcher-spawn') do
+Before("@command-launcher-spawn") do
   aruba.config.command_launcher = :spawn
 end
 
-Before('@command-launcher-in-process') do
+Before("@command-launcher-in-process") do
   aruba.config.command_launcher = :in_process
 end
 
-Before('@command-launcher-debug') do
+Before("@command-launcher-debug") do
   aruba.config.command_launcher = :debug
 end

data/lib/aruba/cucumber/testing_frameworks.rb

@@ -1,119 +1,121 @@
 # Cucumber
-Then /^the feature(?:s)? should not(?: all)? pass$/ do
+Then(/^the feature(?:s)? should not(?: all)? pass$/) do
   step 'the output should contain " failed)"'
-  step 'the exit status should be 1'
+  step "the exit status should be 1"
 end
 
 # Cucumber
-Then /^the feature(?:s)? should(?: all)? pass$/ do
+Then(/^the feature(?:s)? should(?: all)? pass$/) do
   step 'the output should not contain " failed)"'
   step 'the output should not contain " undefined)"'
-  step 'the exit status should be 0'
+  step "the exit status should be 0"
 end
 
 # Cucumber
-Then /^the feature(?:s)? should not(?: all)? pass with( regex)?:$/ do |regex, string|
+Then(/^the feature(?:s)? should not(?: all)? pass with( regex)?:$/) do |regex, string|
   step 'the output should contain " failed)"'
-  step 'the exit status should be 1'
+  step "the exit status should be 1"
 
   if regex
     step "the output should match %r<#{string}>"
   else
-    step 'the output should contain:', string
+    step "the output should contain:", string
   end
 end
 
 # Cucumber
-Then /^the feature(?:s)? should(?: all)? pass with( regex)?:$/ do |regex, string|
+Then(/^the feature(?:s)? should(?: all)? pass with( regex)?:$/) do |regex, string|
   step 'the output should not contain " failed)"'
   step 'the output should not contain " undefined)"'
-  step 'the exit status should be 0'
+  step "the exit status should be 0"
 
   if regex
     step "the output should match %r<#{string}>"
   else
-    step 'the output should contain:', string
+    step "the output should contain:", string
   end
 end
 
 # RSpec
-Then /^the spec(?:s)? should not(?: all)? pass(?: with (\d+) failures?)?$/ do |count_failures|
+Then(/^the spec(?:s)? should not(?: all)? pass(?: with (\d+) failures?)?$/) \
+  do |count_failures|
   if count_failures.nil?
     step 'the output should not contain "0 failures"'
   else
     step %(the output should contain "#{count_failures} failures")
   end
 
-  step 'the exit status should be 1'
+  step "the exit status should be 1"
 end
 
 # RSpec
-Then /^the spec(?:s)? should all pass$/ do
+Then(/^the spec(?:s)? should all pass$/) do
   step 'the output should contain "0 failures"'
-  step 'the exit status should be 0'
+  step "the exit status should be 0"
 end
 
 # RSpec
-Then /^the spec(?:s)? should not(?: all)? pass with( regex)?:$/ do |regex, string|
+Then(/^the spec(?:s)? should not(?: all)? pass with( regex)?:$/) do |regex, string|
   step 'the output should not contain "0 failures"'
-  step 'the exit status should be 1'
+  step "the exit status should be 1"
 
   if regex
     step "the output should match %r<#{string}>"
   else
-    step 'the output should contain:', string
+    step "the output should contain:", string
   end
 end
 
 # RSpec
-Then /^the spec(?:s)? should(?: all)? pass with( regex)?:$/ do |regex, string|
+Then(/^the spec(?:s)? should(?: all)? pass with( regex)?:$/) do |regex, string|
   step 'the output should contain "0 failures"'
-  step 'the exit status should be 0'
+  step "the exit status should be 0"
 
   if regex
     step "the output should match %r<#{string}>"
   else
-    step 'the output should contain:', string
+    step "the output should contain:", string
   end
 end
 
 # Minitest
-Then /^the tests(?:s)? should not(?: all)? pass(?: with (\d+) failures?)?$/ do |count_failures|
+Then(/^the tests(?:s)? should not(?: all)? pass(?: with (\d+) failures?)?$/) \
+  do |count_failures|
   if count_failures.nil?
     step 'the output should not contain "0 errors"'
   else
     step %(the output should contain "#{count_failures} errors")
   end
 
-  step 'the exit status should be 1'
+  step "the exit status should be 1"
 end
 
 # Minitest
-Then /^the tests(?:s)? should all pass$/ do
+Then(/^the tests(?:s)? should all pass$/) do
   step 'the output should contain "0 errors"'
-  step 'the exit status should be 0'
+  step "the exit status should be 0"
 end
 
 # Minitest
-Then /^the test(?:s)? should not(?: all)? pass with( regex)?:$/ do |regex, string|
+Then(/^the test(?:s)? should not(?: all)? pass with( regex)?:$/) do |regex, string|
   step 'the output should contain "0 errors"'
-  step 'the exit status should be 1'
+  step "the exit status should be 1"
 
   if regex
     step "the output should match %r<#{string}>"
   else
-    step 'the output should contain:', string
+    step "the output should contain:", string
   end
 end
 
 # Minitest
-Then /^the test(?:s)? should(?: all)? pass with( regex)?:$/ do |regex, string|
+Then(/^the test(?:s)? should(?: all)? pass with( regex)?:$/) do |regex, string|
   step 'the output should not contain "0 errors"'
-  step 'the exit status should be 0'
+  step "the exit status should be 0"
 
   if regex
     step "the output should match %r<#{string}>"
   else
-    step 'the output should contain:', string
+    step "the output should contain:", string
   end
 end