Always include all Cops by default

I made a change a couple of weeks back to only include Rails cops in
the event we detect railties in the project's Gemfile.lock. I'm
convinced this was the wrong course of action since our default rubocop
configuration file (https://github.com/codeclimate/codeclimate/blob/master/config/rubocop/.rubocop.yml#L1119)
has these mostly disabled out of the box. It makes much more sense to
rely on either a user's existing .rubocop.yml or our default rather
than mucking with this within the engine itself.

Author: jpignata

lib/cc/engine/lockfile_specs.rb

@@ -4,7 +4,6 @@
 require "rubocop/cop/method_complexity_patch"
 require "cc/engine/category_parser"
 require "cc/engine/file_list_resolver"
-require "cc/engine/lockfile_specs"
 require "cc/engine/violation_decorator"
 require "active_support"
 require "active_support/core_ext"
@@ -69,7 +68,7 @@ def rubocop_config_store
 
       def rubocop_team_for_path(path)
         rubocop_config = rubocop_config_store.for(path)
-        RuboCop::Cop::Team.new(cops, rubocop_config)
+        RuboCop::Cop::Team.new(RuboCop::Cop::Cop.all, rubocop_config)
       end
 
       def violation_positions(location)
@@ -120,16 +119,6 @@ def content_body(cop_name)
         )
         File.exist?(path) ? File.read(path) : nil
       end
-
-      def cops
-        specs = LockfileSpecs.new("Gemfile.lock")
-
-        if specs.include?("railties")
-          RuboCop::Cop::Cop.all
-        else
-          RuboCop::Cop::Cop.non_rails
-        end
-      end
     end
   end
 end

lib/cc/engine/rubocop.rb

@@ -110,79 +110,6 @@ def method
         assert !includes_check?(output, "Lint/UselessAssignment")
       end
 
-      it "excludes Rails cops when there is no Gemfile.lock file present" do
-        create_source_file("object.rb", <<-EORUBY)
-          class Object
-            def method
-              target.method
-            end
-          end
-        EORUBY
-
-        assert !includes_check?(run_engine, "Rubocop/Rails/Delegate")
-      end
-
-      it "excludes Rails cops when railties isn't in the Gemfile.lock file" do
-        create_source_file("object.rb", <<-EORUBY)
-          class Object
-            def method
-              target.method
-            end
-          end
-        EORUBY
-
-        create_source_file("Gemfile.lock", <<-EORUBY.strip_heredoc)
-          GEM
-            remote: https://rubygems.org/
-            specs:
-              rake (10.4.2)
-        EORUBY
-
-        assert !includes_check?(run_engine, "Rubocop/Rails/Delegate")
-      end
-
-      it "excludes Rails cops when there is no valid Gemfile.lock file" do
-        create_source_file("object.rb", <<-EORUBY)
-          class Object
-            def method
-              target.method
-            end
-          end
-        EORUBY
-
-        # This will create a parse error as Bundler::LockfileParser looks
-        # specifically for merge conflict artifacts and bails.
-        create_source_file("Gemfile.lock", <<-EORUBY.strip_heredoc)
-          <<<<<<< HEAD
-          =======
-          GEM
-            remote: https://rubygems.org/
-            specs:
-              rake (10.4.2)
-        EORUBY
-
-        assert !includes_check?(run_engine, "Rubocop/Rails/Delegate")
-      end
-
-      it "includes Rails cops when railties is in the Gemfile.lock file" do
-        create_source_file("object.rb", <<-EORUBY)
-          class Object
-            def method
-              target.method
-            end
-          end
-        EORUBY
-
-        create_source_file("Gemfile.lock", <<-EORUBY.strip_heredoc)
-          GEM
-            remote: https://rubygems.org/
-            specs:
-              railties (4.2.4)
-        EORUBY
-
-        assert includes_check?(run_engine, "Rubocop/Rails/Delegate")
-      end
-
       it "layers config exclusions on top of the YAML config" do
         create_source_file("foo.rb", <<-EORUBY)
           def method