Remove debugger, byebug, and web-console from 4.2

debugger and byebug cannot be present in Appraisals because they only
work on specific Ruby versions, and we test against a range of Ruby
versions. Hence, they can't be present in the Rails application that
gets generated in acceptance tests, either.

Also, we don't really need web-console to be there, it's just an extra
dependency.

Appraisals

@@ -120,8 +120,6 @@ if Gem::Requirement.new('> 1.9.2').satisfied_by?(ruby_version)
     gem 'jbuilder', '~> 2.0'
     gem 'sdoc', '~> 0.4.0'
     gem 'bcrypt', '~> 3.1.7'
-    gem 'byebug'
-    gem 'web-console', '~> 2.0'
     gem 'spring'
     gem 'protected_attributes', "~> 1.0.6"
   end

gemfiles/4.2.gemfile

@@ -34,5 +34,3 @@ gem "minitest-reporters"
 gem "sass-rails", "~> 5.0"
 gem "jbuilder", "~> 2.0"
 gem "bcrypt", "~> 3.1.7"
-gem "byebug"
-gem "web-console", "~> 2.0"

gemfiles/4.2.gemfile.lock

@@ -51,15 +51,9 @@ GEM
       cucumber (>= 1.1.1)
       rspec-expectations (>= 2.7.0)
     bcrypt (3.1.9)
-    binding_of_caller (0.7.2)
-      debug_inspector (>= 0.0.1)
     bourne (1.6.0)
       mocha (~> 1.1)
     builder (3.2.2)
-    byebug (3.5.1)
-      columnize (~> 0.8)
-      debugger-linecache (~> 1.2)
-      slop (~> 3.6)
     childprocess (0.5.5)
       ffi (~> 1.0, >= 1.0.11)
     coderay (1.1.0)
@@ -70,15 +64,12 @@ GEM
       coffee-script-source
       execjs
     coffee-script-source (1.8.0)
-    columnize (0.9.0)
     cucumber (1.3.17)
       builder (>= 2.1.2)
       diff-lcs (>= 1.1.3)
       gherkin (~> 2.12)
       multi_json (>= 1.7.5, < 2.0)
       multi_test (>= 0.1.1)
-    debug_inspector (0.0.2)
-    debugger-linecache (1.2.0)
     diff-lcs (1.2.5)
     erubis (2.7.0)
     execjs (2.2.2)
@@ -215,11 +206,6 @@ GEM
       execjs (>= 0.3.0)
       json (>= 1.8.0)
     watchr (0.7)
-    web-console (2.0.0)
-      activemodel (~> 4.0)
-      binding_of_caller (>= 0.7.2)
-      railties (~> 4.0)
-      sprockets-rails (>= 2.0, < 4.0)
     yajl-ruby (1.1.0)
     yard (0.8.7.6)
 
@@ -235,7 +221,6 @@ DEPENDENCIES
   bcrypt (~> 3.1.7)
   bourne (~> 1.3)
   bundler (~> 1.1)
-  byebug
   coffee-rails (~> 4.1.0)
   jbuilder (~> 2.0)
   jdbc-sqlite3
@@ -259,5 +244,4 @@ DEPENDENCIES
   turbolinks
   uglifier (>= 1.3.0)
   watchr
-  web-console (~> 2.0)
   yard

spec/support/acceptance/helpers/step_helpers.rb

@@ -85,6 +85,9 @@ module AcceptanceTests
         bundle.remove_gem 'turn'
         bundle.remove_gem 'coffee-rails'
         bundle.remove_gem 'uglifier'
+        bundle.remove_gem 'debugger'
+        bundle.remove_gem 'byebug'
+        bundle.remove_gem 'web-console'
       end
     end
 

spec/support/unit/rails_application.rb

@@ -12,7 +12,10 @@ module UnitTests
     def create
       fs.clean
       generate
-      fs.within_project { install_gems }
+      fs.within_project do
+        install_gems
+        remove_unwanted_gems
+      end
     end
 
     def load
@@ -75,17 +78,13 @@ module UnitTests
     def fix_available_locales_warning
       # See here for more on this:
       # http://stackoverflow.com/questions/20361428/rails-i18n-validation-deprecation-warning
-
-      filename = 'config/application.rb'
-
-      lines = fs.read(filename).split("\n")
-      lines.insert(-3, <<EOT)
+      fs.transform('config/application.rb') do |lines|
+        lines.insert(-3, <<-EOT)
 if I18n.respond_to?(:enforce_available_locales=)
   I18n.enforce_available_locales = false
 end
-EOT
-
-      fs.write(filename, lines.join("\n"))
+        EOT
+      end
     end
 
     def load_environment
@@ -101,7 +100,13 @@ EOT
       bundle.install_gems
     end
 
-    private
+    def remove_unwanted_gems
+      bundle.updating do
+        bundle.remove_gem 'debugger'
+        bundle.remove_gem 'byebug'
+        bundle.remove_gem 'web-console'
+      end
+    end
 
     def run_command!(*args)
       Tests::CommandRunner.run!(*args)