geordi 2.12.3 → 3.0.0

data/lib/geordi/cli.rb

@@ -5,11 +5,6 @@ require 'geordi/util'
 
 module Geordi
   class CLI < Thor
-
-    if Geordi::Util.ruby_version <= Gem::Version.new('2.0.0')
-      warn "Deprecation warning: Ruby 1.8.7 and 1.9.3 support will be dropped in Geordi 3.x."
-    end
-
     include Geordi::Interaction
 
     def self.exit_on_failure?
@@ -17,7 +12,7 @@ module Geordi
     end
 
     # load all tasks defined in lib/geordi/commands
-    Dir[File.expand_path '../commands/*.rb', __FILE__].each do |file|
+    Dir[File.expand_path 'commands/*.rb', __dir__].each do |file|
       class_eval File.read(file), file
     end
 

data/lib/geordi/commands/_setup_vnc.rb

@@ -1,4 +1,4 @@
-desc '-setup-vnc', 'Setup VNC for running Selenium tests there', :hide => true
+desc '-setup-vnc', 'Setup VNC for running Selenium tests there', hide: true
 def _setup_vnc
   `clear`
 

data/lib/geordi/commands/apache_site.rb

@@ -10,13 +10,13 @@ def apache_site(*args)
     puts 'Please call: apache-site my-site'
     puts
     puts 'Available sites:'
-    Dir.new(".").each do |file|
+    Dir.new('.').each do |file|
       puts "- #{file}" if file != '.' && file != '..'
     end
     exit
   end
 
-  has_default = File.exists?('default')
-  exec "sudo a2dissite \*; sudo a2ensite #{"default " if has_default}#{site} && sudo apache2ctl restart"
+  has_default = File.exist?('default')
+  exec "sudo a2dissite \*; sudo a2ensite #{'default ' if has_default}#{site} && sudo apache2ctl restart"
   Dir.chdir old_cwd
 end

data/lib/geordi/commands/bundle_install.rb

@@ -1,6 +1,6 @@
-desc 'bundle-install', 'Run bundle install if required', :hide => true
+desc 'bundle-install', 'Run bundle install if required', hide: true
 def bundle_install
-  if File.exists?('Gemfile') and !system('bundle check > /dev/null 2>&1')
+  if File.exist?('Gemfile') && !system('bundle check > /dev/null 2>&1')
     announce 'Bundling'
     Util.system! 'bundle install'
   end

data/lib/geordi/commands/capistrano.rb

@@ -8,7 +8,7 @@ def capistrano(*args)
 
   note 'Found the following deploy targets:'
   puts targets
-  prompt('Continue?', 'n', /y|yes/) or fail 'Cancelled.'
+  prompt('Continue?', 'n', /y|yes/) || raise('Cancelled.')
 
   targets << nil if targets.empty? # default target
   targets.each do |stage|
@@ -17,7 +17,6 @@ def capistrano(*args)
     command = "bundle exec cap #{stage} " + args.join(' ')
     note_cmd command
 
-    Util.system!(command, :fail_message => 'Capistrano failed. Have a look!')
+    Util.system!(command, fail_message: 'Capistrano failed. Have a look!')
   end
-
 end

data/lib/geordi/commands/chromedriver_update.rb

@@ -9,14 +9,5 @@ LONGDESC
 def chromedriver_update
   require 'geordi/chromedriver_updater'
 
-  # Ruby 1.9.3 introduces #capture3 in open3
-  supported_ruby_version = '1.9.2'
-
-  # We do not want to backport this command to Ruby 1.8.7, a user can just use a newer Ruby version to run it. For all
-  # other commands it still is necessary to have a proper Ruby 1.8.7 support.
-  if Gem::Version.new(RUBY_VERSION) < Gem::Version.new(supported_ruby_version)
-    raise("Unsupported ruby version #{RUBY_VERSION}, please use at least #{supported_ruby_version} to run this command!")
-  end
-
   ChromedriverUpdater.new.run
 end

data/lib/geordi/commands/clean.rb

@@ -1,16 +1,14 @@
 desc 'clean', 'Remove unneeded files from the current directory'
 def clean
-
   announce 'Removing tempfiles'
-  for pattern in %w[ webrat-* capybara-* tmp/webrat-* tmp/capybara-* tmp/rtex/* log/*.log ]
+  %w[webrat-* capybara-* tmp/webrat-* tmp/capybara-* tmp/rtex/* log/*.log].each do |pattern|
     note pattern
     puts `rm -vfR #{pattern}`
   end
 
   announce 'Finding recursively and removing backup files'
-  for pattern in %w[ *~ ]
+  %w[*~].each do |pattern|
     note pattern
     `find . -name #{pattern} -exec rm {} ';'`
   end
-
 end

data/lib/geordi/commands/commit.rb

@@ -21,4 +21,3 @@ Unsupported Ruby Version #{RUBY_VERSION}. `geordi commit` requires Ruby 2.1+.
 
   Gitpt.new.run(git_args)
 end
-

data/lib/geordi/commands/console.rb

@@ -6,9 +6,9 @@ Open a Rails console on `staging`: `geordi console staging`
 LONGDESC
 
 
-option :select_server, :default => false, :type => :boolean, :aliases => '-s'
+option :select_server, default: false, type: :boolean, aliases: '-s'
 
-def console(target = 'development', *args)
+def console(target = 'development', *_args)
   require 'geordi/remote'
 
   if target == 'development'

data/lib/geordi/commands/create_database_yml.rb

@@ -1,9 +1,9 @@
-desc 'create-database-yml', '[sic]', :hide => true
+desc 'create-database-yml', '[sic]', hide: true
 def create_database_yml
   real_yml = 'config/database.yml'
   sample_yml = 'config/database.sample.yml'
 
-  if File.exists?(sample_yml) and not File.exists?(real_yml)
+  if File.exist?(sample_yml) && !File.exist?(real_yml)
     announce 'Creating ' + real_yml
 
     sample = File.read(sample_yml)

data/lib/geordi/commands/create_databases.rb

@@ -1,11 +1,11 @@
-desc 'create-databases', 'Create all databases', :hide => true
+desc 'create-databases', 'Create all databases', hide: true
 def create_databases
   invoke_cmd 'create_database_yml'
   invoke_cmd 'bundle_install'
 
   announce 'Creating databases'
 
-  if File.exists?('config/database.yml')
+  if File.exist?('config/database.yml')
     command = 'bundle exec rake db:create:all'
     command << ' parallel:create' if Util.file_containing?('Gemfile', /parallel_tests/)
 

data/lib/geordi/commands/cucumber.rb

@@ -17,30 +17,30 @@ or `-d`.
 e.g. `--format pretty`.
 LONGDESC
 
-option :modified, :aliases => '-m', :type => :boolean,
-  :desc => 'Run all modified features'
-option :containing, :aliases => '-c', :banner => 'STRING',
-  :desc => 'Run all features that contain STRING'
-option :verbose, :aliases => '-v', :type => :boolean,
-  :desc => 'Print the testrun command'
-option :debug, :aliases => '-d', :type => :boolean,
-  :desc => 'Run with `-f pretty -b` which helps hunting down bugs'
-option :rerun, :aliases => '-r', :type => :numeric, :default => 0,
-  :desc => 'Rerun features up to N times while failing'
+option :modified, aliases: '-m', type: :boolean,
+  desc: 'Run all modified features'
+option :containing, aliases: '-c', banner: 'STRING',
+  desc: 'Run all features that contain STRING'
+option :verbose, aliases: '-v', type: :boolean,
+  desc: 'Print the testrun command'
+option :debug, aliases: '-d', type: :boolean,
+  desc: 'Run with `-f pretty -b` which helps hunting down bugs'
+option :rerun, aliases: '-r', type: :numeric, default: 0,
+  desc: 'Rerun features up to N times while failing'
 
 def cucumber(*args)
   if args.empty?
     # This is not testable as there is no way to stub `git` :(
     if options.modified?
-      modified_features = `git status --short`.split($/).map do |line|
+      modified_features = `git status --short`.split($INPUT_RECORD_SEPARATOR).map do |line|
         indicators = line.slice!(0..2) # Remove leading indicators
-        line if line.include?('.feature') and not indicators.include?('D')
+        line if line.include?('.feature') && indicators.exclude?('D')
       end.compact
       args = modified_features
     end
 
     if options.containing
-      matching_features = `grep -lri '#{options.containing}' --include=*.feature features/`.split($/)
+      matching_features = `grep -lri '#{options.containing}' --include=*.feature features/`.split($INPUT_RECORD_SEPARATOR)
       args = matching_features.uniq
     end
   end
@@ -64,31 +64,31 @@ def cucumber(*args)
         files.join(' ')
       end
 
-      solo_tag_usages = `grep -r '@solo' #{ solo_files }`.split("\n")
+      solo_tag_usages = `grep -r '@solo' #{solo_files}`.split("\n")
 
       if solo_tag_usages.any?
         solo_cmd_opts = cmd_opts.dup
         solo_cmd_opts << '--tags' << '@solo'
 
         announce 'Running @solo features'
-        solo_success = Geordi::Cucumber.new.run files, solo_cmd_opts, :verbose => options.verbose, :parallel => false
-        solo_success or fail 'Features failed.'
+        solo_success = Geordi::Cucumber.new.run files, solo_cmd_opts, verbose: options.verbose, parallel: false
+        solo_success || raise('Features failed.')
       end
     end
 
     # Parallel run of all given features + reruns ##############################
     announce 'Running features'
-    normal_run_successful = Geordi::Cucumber.new.run(files, cmd_opts, :verbose => options.verbose)
+    normal_run_successful = Geordi::Cucumber.new.run(files, cmd_opts, verbose: options.verbose)
 
     unless normal_run_successful
       cmd_opts << '--profile' << 'rerun'
 
       # Reruns
       (options.rerun + 1).times do |i|
-        fail 'Features failed.' if (i == options.rerun) # All reruns done?
+        raise 'Features failed.' if i == options.rerun # All reruns done?
 
-        announce "Rerun ##{ i + 1 } of #{ options.rerun }"
-        break if Geordi::Cucumber.new.run([], cmd_opts, :verbose => options.verbose, :parallel => false)
+        announce "Rerun ##{i + 1} of #{options.rerun}"
+        break if Geordi::Cucumber.new.run([], cmd_opts, verbose: options.verbose, parallel: false)
       end
     end
 

data/lib/geordi/commands/delete_dumps.rb

@@ -12,22 +12,22 @@ LONGDESC
 
 def delete_dumps(dump_directory = nil)
   deletable_dumps = []
-  if dump_directory.nil?
-    dump_directories = [
+  dump_directories = if dump_directory.nil?
+    [
       File.join(Dir.home, 'dumps'),
-      Dir.pwd
+      Dir.pwd,
     ]
   else
-    dump_directories = [dump_directory]
+    [dump_directory]
   end
   announce 'Looking for *.dump in ' << dump_directories.join(',')
   dump_directories.each do |d|
-    d2 = File.expand_path(d)
-    unless File.directory? File.realdirpath(d2)
-      warn "Directory #{d2} does not exist"
+    d_2 = File.expand_path(d)
+    unless File.directory? File.realdirpath(d_2)
+      warn "Directory #{d_2} does not exist"
       next
     end
-    deletable_dumps.concat(Dir.glob("#{d2}/**/*.dump"))
+    deletable_dumps.concat(Dir.glob("#{d_2}/**/*.dump"))
   end
   if deletable_dumps.empty?
     success 'No dumps to delete' if deletable_dumps.empty?
@@ -37,7 +37,7 @@ def delete_dumps(dump_directory = nil)
   note 'The following dumps can be deleted:'
   puts
   puts deletable_dumps
-  prompt 'Delete those dumps', 'n', /y|yes/ or fail 'Cancelled.'
+  prompt('Delete those dumps', 'n', /y|yes/) || raise('Cancelled.')
   deletable_dumps.each do |dump|
     File.delete dump unless File.directory? dump
   end

data/lib/geordi/commands/deploy.rb

@@ -30,15 +30,15 @@ instead of `cap deploy:migrations`. You can force using `deploy` by passing the
 -M option: `geordi deploy -M staging`.
 LONGDESC
 
-option :no_migrations, :aliases => '-M', :type => :boolean,
-  :desc => 'Run cap deploy instead of cap deploy:migrations'
-option :current_branch, :aliases => '-c', :type => :boolean,
-  :desc => 'Set DEPLOY_BRANCH to the current branch during deploy'
+option :no_migrations, aliases: '-M', type: :boolean,
+  desc: 'Run cap deploy instead of cap deploy:migrations'
+option :current_branch, aliases: '-c', type: :boolean,
+  desc: 'Set DEPLOY_BRANCH to the current branch during deploy'
 
 def deploy(target_stage = nil)
   # Set/Infer default values
-  branch_stage_map = { 'master' => 'staging', 'production' => 'production'}
-  if target_stage and not Util.deploy_targets.include? target_stage
+  branch_stage_map = { 'master' => 'staging', 'production' => 'production' }
+  if target_stage && (Util.deploy_targets.exclude? target_stage)
     # Target stage autocompletion from available stages
     target_stage = Util.deploy_targets.find { |t| t.start_with? target_stage }
     target_stage || warn('Given deployment stage not found')
@@ -48,7 +48,7 @@ def deploy(target_stage = nil)
   target_stage ||= prompt 'Deployment stage:', branch_stage_map.fetch(Util.current_branch, 'staging')
   if options.current_branch
     stage_file = "config/deploy/#{target_stage}.rb"
-    Util.file_containing? stage_file, 'DEPLOY_BRANCH' or fail <<-ERROR
+    Util.file_containing?(stage_file, 'DEPLOY_BRANCH') || raise(<<-ERROR)
 To deploy from the current branch, configure #{stage_file} to respect the
 environment variable DEPLOY_BRANCH. Example:
 
@@ -67,9 +67,9 @@ set :branch, ENV['DEPLOY_BRANCH'] || 'master'
 
   announce "Checking whether your #{source_branch} branch is ready" ############
   Util.system! "git checkout #{source_branch}"
-  if `git status -s | wc -l`.strip != '0' and not Util.testing?
+  if (`git status -s | wc -l`.strip != '0') && !Util.testing?
     warn "Your #{source_branch} branch holds uncommitted changes."
-    prompt('Continue anyway?', 'n', /y|yes/) or fail 'Cancelled.'
+    prompt('Continue anyway?', 'n', /y|yes/) || raise('Cancelled.')
   else
     note 'All good.'
   end
@@ -100,17 +100,16 @@ set :branch, ENV['DEPLOY_BRANCH'] || 'master'
     capistrano_call = "DEPLOY_BRANCH=#{source_branch} #{capistrano_call}" if options.current_branch
 
     if git_call.any?
-      Util.system! git_call.join(' && '), :show_cmd => true
+      Util.system! git_call.join(' && '), show_cmd: true
     end
 
     invoke_cmd 'bundle_install'
 
-    Util.system! capistrano_call, :show_cmd => true
+    Util.system! capistrano_call, show_cmd: true
 
     success 'Deployment complete.'
   else
     Util.system! "git checkout #{source_branch}"
-    fail 'Deployment cancelled.'
+    raise 'Deployment cancelled.'
   end
-
 end

data/lib/geordi/commands/drop_databases.rb

@@ -17,18 +17,18 @@ Geordi will ask for confirmation before actually dropping databases and will
 offer to edit the whitelist instead.
 LONGDESC
 
-option :postgres_only, :aliases => '-P', :type => :boolean,
-  :desc => 'Only clean Postgres', :default => false
-option :mysql_only, :aliases => '-M', :type => :boolean,
-  :desc => 'Only clean MySQL/MariaDB', :default => false
-option :postgres, :banner => 'PORT_OR_SOCKET',
-  :desc => 'Use Postgres port or socket'
-option :mysql, :banner => 'PORT_OR_SOCKET',
-  :desc => 'Use MySQL/MariaDB port or socket'
+option :postgres_only, aliases: '-P', type: :boolean,
+  desc: 'Only clean Postgres', default: false
+option :mysql_only, aliases: '-M', type: :boolean,
+  desc: 'Only clean MySQL/MariaDB', default: false
+option :postgres, banner: 'PORT_OR_SOCKET',
+  desc: 'Use Postgres port or socket'
+option :mysql, banner: 'PORT_OR_SOCKET',
+  desc: 'Use MySQL/MariaDB port or socket'
 
 def drop_databases
   require 'geordi/db_cleaner'
-  fail '-P and -M are mutually exclusive' if options.postgres_only and options.mysql_only
+  raise '-P and -M are mutually exclusive' if options.postgres_only && options.mysql_only
   mysql_flags = nil
   postgres_flags = nil
 
@@ -38,7 +38,7 @@ def drop_databases
       mysql_flags = "--port=#{mysql_port} --protocol=TCP"
     rescue AttributeError
       unless File.exist? options.mysql
-        fail "Path #{options.mysql} is not a valid MySQL socket"
+        raise "Path #{options.mysql} is not a valid MySQL socket"
       end
       mysql_flags = "--socket=#{options.mysql}"
     end
@@ -48,13 +48,11 @@ def drop_databases
     postgres_flags = "--port=#{options.postgres}"
   end
 
-  extra_flags = {'mysql' => mysql_flags,
-                 'postgres' => postgres_flags
-  }
+  extra_flags = { 'mysql' => mysql_flags,
+                 'postgres' => postgres_flags }
   cleaner = DBCleaner.new(extra_flags)
   cleaner.clean_mysql unless options.postgres_only
   cleaner.clean_postgres unless options.mysql_only
 
   success 'Done.'
 end
-

data/lib/geordi/commands/dump.rb

@@ -20,18 +20,18 @@ dump into the development database after downloading it.
     geordi dump staging -l
 DESC
 
-option :load, :aliases => ['-l'], :type => :string, :desc => 'Load a dump'
-option :select_server, :default => false, :type => :boolean, :aliases => '-s'
+option :load, aliases: ['-l'], type: :string, desc: 'Load a dump'
+option :select_server, default: false, type: :boolean, aliases: '-s'
 
-def dump(target = nil, *args)
+def dump(target = nil, *_args)
   require 'geordi/dump_loader'
   require 'geordi/remote'
 
   if target.nil?
     if options.load
       # validate load option
-      fail 'Missing a dump file.' if options.load == 'load'
-      File.exists?(options.load) or fail 'Could not find the given dump file: ' + options.load
+      raise 'Missing a dump file.' if options.load == 'load'
+      File.exist?(options.load) || raise('Could not find the given dump file: ' + options.load)
 
       loader = DumpLoader.new(options.load)
 
@@ -59,5 +59,4 @@ def dump(target = nil, *args)
       success "Your #{loader.config['database']} database has now the data of #{target}."
     end
   end
-
 end

data/lib/geordi/commands/eurest.rb

@@ -1,4 +1,4 @@
-desc 'eurest', 'Open the current Eurest cantina menu', :hide => true
+desc 'eurest', 'Open the current Eurest cantina menu', hide: true
 def eurest
-  Util.system! %{file="Speiseplan_KW`date +%V`.pdf" && wget -O/tmp/$file http://www.eurest-extranet.de/eurest/export/sites/default/sigma-technopark/de/downloads/$file && xdg-open /tmp/$file}
+  Util.system! %(file="Speiseplan_KW`date +%V`.pdf" && wget -O/tmp/$file http://www.eurest-extranet.de/eurest/export/sites/default/sigma-technopark/de/downloads/$file && xdg-open /tmp/$file)
 end

data/lib/geordi/commands/firefox.rb

@@ -10,12 +10,12 @@ Useful when you need Firefox for Selenium or the VNC set up, but can't use the
 This command is aliased `chrome` for users running Selenium in Chrome.
 LONGDESC
 
-option :setup, :banner => 'FIREFOX_VERSION',
-  :desc => 'Install a special test runner Firefox with the given version'
+option :setup, banner: 'FIREFOX_VERSION',
+  desc: 'Install a special test runner Firefox with the given version'
 
 def firefox(*command)
   if options.setup
-    fail 'Firefox version required (e.g. --setup 24.0)' if options.setup == 'setup'
+    raise 'Firefox version required (e.g. --setup 24.0)' if options.setup == 'setup'
 
     require 'geordi/firefox_for_selenium'
     Geordi::FirefoxForSelenium.install(options.setup)