rails_best_practices 1.19.0 → 1.19.1

data/spec/rails_best_practices/reviews/protect_mass_assignment_review_spec.rb

@@ -6,17 +6,17 @@ module RailsBestPractices
       let(:runner) { Core::Runner.new(prepares: [Prepares::GemfilePrepare.new, Prepares::ConfigPrepare.new, Prepares::InitializerPrepare.new],
                                       reviews: ProtectMassAssignmentReview.new) }
 
-      it "should protect mass assignment" do
+      it 'should protect mass assignment' do
         content =<<-EOF
         class User < ActiveRecord::Base
         end
         EOF
         runner.review('app/models/user.rb', content)
         expect(runner.errors.size).to eq(1)
-        expect(runner.errors[0].to_s).to eq("app/models/user.rb:1 - protect mass assignment")
+        expect(runner.errors[0].to_s).to eq('app/models/user.rb:1 - protect mass assignment')
       end
 
-      it "should not protect mass assignment if attr_accessible is used with arguments and user set config.active_record.whitelist_attributes" do
+      it 'should not protect mass assignment if attr_accessible is used with arguments and user set config.active_record.whitelist_attributes' do
         content =<<-EOF
         module RailsBestPracticesCom
           class Application < Rails::Application
@@ -34,7 +34,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not protect mass assignment if attr_accessible is used without arguments and user set config.active_record.whitelist_attributes" do
+      it 'should not protect mass assignment if attr_accessible is used without arguments and user set config.active_record.whitelist_attributes' do
         content =<<-EOF
         module RailsBestPracticesCom
           class Application < Rails::Application
@@ -52,7 +52,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not protect mass assignment with attr_protected if user set config.active_record.whitelist_attributes" do
+      it 'should not protect mass assignment with attr_protected if user set config.active_record.whitelist_attributes' do
         content =<<-EOF
         module RailsBestPracticesCom
           class Application < Rails::Application
@@ -70,7 +70,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not protect mass assignment if using devise" do
+      it 'should not protect mass assignment if using devise' do
         content =<<-EOF
         class User < ActiveRecord::Base
           devise :database_authenticatable, :registerable, :confirmable, :recoverable, stretches: 20
@@ -80,7 +80,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not protect mass assignment if using authlogic with configuration" do
+      it 'should not protect mass assignment if using authlogic with configuration' do
         content =<<-EOF
         class User < ActiveRecord::Base
           acts_as_authentic do |c|
@@ -92,7 +92,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not protect mass assignment if using authlogic without configuration" do
+      it 'should not protect mass assignment if using authlogic without configuration' do
         content =<<-EOF
         class User < ActiveRecord::Base
           acts_as_authentic
@@ -102,7 +102,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not protect mass assignment if checking non ActiveRecord::Base inherited model" do
+      it 'should not protect mass assignment if checking non ActiveRecord::Base inherited model' do
         content =<<-EOF
         class User < Person
         end
@@ -111,8 +111,8 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      context "strong_parameters" do
-        it "should not protect mass assignment for strong_parameters" do
+      context 'strong_parameters' do
+        it 'should not protect mass assignment for strong_parameters' do
           content =<<-EOF
           class User < ActiveRecord::Base
             include ActiveModel::ForbiddenAttributesProtection
@@ -122,7 +122,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not protect mass assignment for strong_parameters" do
+        it 'should not protect mass assignment for strong_parameters' do
           content =<<-EOF
           class AR
             ActiveRecord::Base.send(:include, ActiveModel::ForbiddenAttributesProtection)
@@ -138,8 +138,8 @@ module RailsBestPractices
         end
       end
 
-      context "activerecord 4" do
-        it "should not protect mass assignment for activerecord 4" do
+      context 'activerecord 4' do
+        it 'should not protect mass assignment for activerecord 4' do
           content =<<-EOF
   GEM
     remote: https://rubygems.org
@@ -155,7 +155,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should protect mass assignment for activerecord 3" do
+        it 'should protect mass assignment for activerecord 3' do
           content =<<-EOF
   GEM
     remote: https://rubygems.org
@@ -172,7 +172,7 @@ module RailsBestPractices
         end
       end
 
-      it "should not check ignored files" do
+      it 'should not check ignored files' do
         runner = Core::Runner.new(prepares: [Prepares::GemfilePrepare.new, Prepares::ConfigPrepare.new, Prepares::InitializerPrepare.new],
                                   reviews: ProtectMassAssignmentReview.new(ignored_files: /app\/models\/user\.rb/))
         content =<<-EOF

data/spec/rails_best_practices/reviews/remove_empty_helpers_review_spec.rb

@@ -5,17 +5,17 @@ module RailsBestPractices
     describe RemoveEmptyHelpersReview do
       let(:runner) { Core::Runner.new(reviews: RemoveEmptyHelpersReview.new) }
 
-      it "should remove empty helpers" do
+      it 'should remove empty helpers' do
         content =<<-EOF
         module PostsHelper
         end
         EOF
         runner.review('app/helpers/posts_helper.rb', content)
         expect(runner.errors.size).to eq(1)
-        expect(runner.errors[0].to_s).to eq("app/helpers/posts_helper.rb:1 - remove empty helpers")
+        expect(runner.errors[0].to_s).to eq('app/helpers/posts_helper.rb:1 - remove empty helpers')
       end
 
-      it "should not remove empty helpers" do
+      it 'should not remove empty helpers' do
         content =<<-EOF
         module PostsHelper
           def post_link(post)
@@ -27,7 +27,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not remove empty application_helper" do
+      it 'should not remove empty application_helper' do
         content =<<-EOF
         module ApplicationHelper
         end
@@ -36,7 +36,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not check ignored files" do
+      it 'should not check ignored files' do
         runner = Core::Runner.new(reviews: RemoveEmptyHelpersReview.new(ignored_files: /posts_helper/))
         content =<<-EOF
         module PostsHelper

data/spec/rails_best_practices/reviews/remove_unused_methods_in_controllers_review_spec.rb

@@ -5,11 +5,11 @@ module RailsBestPractices
     describe RemoveUnusedMethodsInControllersReview do
       let(:runner) { Core::Runner.new(
         prepares: [Prepares::ControllerPrepare.new, Prepares::RoutePrepare.new],
-        reviews: RemoveUnusedMethodsInControllersReview.new({'except_methods' => ["ExceptableController#*"]})
+        reviews: RemoveUnusedMethodsInControllersReview.new({'except_methods' => ['ExceptableController#*']})
       ) }
 
-      context "private/protected" do
-        it "should remove unused methods" do
+      context 'private/protected' do
+        it 'should remove unused methods' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts do
@@ -36,11 +36,11 @@ module RailsBestPractices
           runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(2)
-          expect(runner.errors[0].to_s).to eq("app/controllers/posts_controller.rb:6 - remove unused methods (PostsController#load_post)")
-          expect(runner.errors[1].to_s).to eq("app/controllers/posts_controller.rb:8 - remove unused methods (PostsController#load_user)")
+          expect(runner.errors[0].to_s).to eq('app/controllers/posts_controller.rb:6 - remove unused methods (PostsController#load_post)')
+          expect(runner.errors[1].to_s).to eq('app/controllers/posts_controller.rb:8 - remove unused methods (PostsController#load_user)')
         end
 
-        it "should not remove unused methods for before_filter" do
+        it 'should not remove unused methods for before_filter' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts
@@ -62,7 +62,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods for around_filter" do
+        it 'should not remove unused methods for around_filter' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts
@@ -87,7 +87,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods for layout" do
+        it 'should not remove unused methods for layout' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts
@@ -109,7 +109,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove inherited_resources methods" do
+        it 'should not remove inherited_resources methods' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts
@@ -132,8 +132,8 @@ module RailsBestPractices
         end
       end
 
-      context "public" do
-        it "should remove unused methods" do
+      context 'public' do
+        it 'should remove unused methods' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts
@@ -150,10 +150,10 @@ module RailsBestPractices
           runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/controllers/posts_controller.rb:3 - remove unused methods (PostsController#list)")
+          expect(runner.errors[0].to_s).to eq('app/controllers/posts_controller.rb:3 - remove unused methods (PostsController#list)')
         end
 
-        it "should not remove inline routes" do
+        it 'should not remove inline routes' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :posts, only: :none do
@@ -174,7 +174,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods if all actions are used in route" do
+        it 'should not remove unused methods if all actions are used in route' do
           content =<<-EOF
           ActionController::Routing::Routes.draw do |map|
             map.connect 'internal/:action/*whatever', controller: "internal"
@@ -194,7 +194,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods if they are except_methods" do
+        it 'should not remove unused methods if they are except_methods' do
           content =<<-EOF
           class ExceptableController < ApplicationController
             def list; end
@@ -207,8 +207,8 @@ module RailsBestPractices
         end
       end
 
-      context "assignment" do
-        it "should not remove unused methods if call in base class" do
+      context 'assignment' do
+        it 'should not remove unused methods if call in base class' do
           content =<<-EOF
           RailsBestPracticesCom::Application.routes.draw do
             resources :user, only: :show do; end
@@ -236,8 +236,8 @@ module RailsBestPractices
         end
       end
 
-      context "helper_method" do
-        it "should remove unused methods if helper method is not called" do
+      context 'helper_method' do
+        it 'should remove unused methods if helper method is not called' do
           content = <<-EOF
           class PostsController < ApplicationController
             helper_method :helper_post
@@ -249,10 +249,10 @@ module RailsBestPractices
           runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/controllers/posts_controller.rb:4 - remove unused methods (PostsController#helper_post)")
+          expect(runner.errors[0].to_s).to eq('app/controllers/posts_controller.rb:4 - remove unused methods (PostsController#helper_post)')
         end
 
-        it "should not remove unused methods if call helper method in views" do
+        it 'should not remove unused methods if call helper method in views' do
           content = <<-EOF
           class PostsController < ApplicationController
             helper_method :helper_post
@@ -270,7 +270,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods if call helper method in helpers" do
+        it 'should not remove unused methods if call helper method in helpers' do
           content = <<-EOF
           class PostsController < ApplicationController
             helper_method :helper_post
@@ -293,8 +293,8 @@ module RailsBestPractices
         end
       end
 
-      context "delegate to: :controller" do
-        it "should remove unused methods if delegate method is not called" do
+      context 'delegate to: :controller' do
+        it 'should remove unused methods if delegate method is not called' do
           content = <<-EOF
           class PostsController < ApplicationController
             protected
@@ -311,10 +311,10 @@ module RailsBestPractices
           runner.review('app/helpers/posts_helper.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/controllers/posts_controller.rb:3 - remove unused methods (PostsController#helper_post)")
+          expect(runner.errors[0].to_s).to eq('app/controllers/posts_controller.rb:3 - remove unused methods (PostsController#helper_post)')
         end
 
-        it "should remove unused methods if delegate method is called" do
+        it 'should remove unused methods if delegate method is called' do
           content = <<-EOF
           class PostsController < ApplicationController
             protected
@@ -338,8 +338,8 @@ module RailsBestPractices
         end
       end
 
-      context "cells" do
-        it "should remove unused methods" do
+      context 'cells' do
+        it 'should remove unused methods' do
           content =<<-EOF
           class PostsCell < Cell::Rails
             def list; end
@@ -349,10 +349,10 @@ module RailsBestPractices
           runner.review('app/cells/posts_cell.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/cells/posts_cell.rb:2 - remove unused methods (PostsCell#list)")
+          expect(runner.errors[0].to_s).to eq('app/cells/posts_cell.rb:2 - remove unused methods (PostsCell#list)')
         end
 
-        it "should not remove unused methods if render_cell" do
+        it 'should not remove unused methods if render_cell' do
           content =<<-EOF
           class PostsCell < Cell::Rails
             def list; end
@@ -370,7 +370,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods if render with state" do
+        it 'should not remove unused methods if render with state' do
           content =<<-EOF
           class PostsCell < Cell::Rails
             def list
@@ -393,7 +393,7 @@ module RailsBestPractices
         end
       end
 
-      it "should not remove unused methods" do
+      it 'should not remove unused methods' do
         route_content =<<-EOF
         RailsBestPracticesCom::Application.routes.draw do
           namespace :admin do
@@ -430,7 +430,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not check ignored files" do
+      it 'should not check ignored files' do
         runner = Core::Runner.new(prepares: [Prepares::ControllerPrepare.new, Prepares::RoutePrepare.new],
                                   reviews: RemoveUnusedMethodsInControllersReview.new(ignored_files: /posts_controller/, except_methods: []))
         content =<<-EOF

data/spec/rails_best_practices/reviews/remove_unused_methods_in_helpers_review_spec.rb

@@ -8,7 +8,7 @@ module RailsBestPractices
         reviews: RemoveUnusedMethodsInHelpersReview.new(except_methods: [])
       ) }
 
-      it "should remove unused methods" do
+      it 'should remove unused methods' do
         content =<<-EOF
         module PostsHelper
           def unused; end
@@ -18,10 +18,10 @@ module RailsBestPractices
         runner.review('app/helpers/posts_helper.rb', content)
         runner.after_review
         expect(runner.errors.size).to eq(1)
-        expect(runner.errors[0].to_s).to eq("app/helpers/posts_helper.rb:2 - remove unused methods (PostsHelper#unused)")
+        expect(runner.errors[0].to_s).to eq('app/helpers/posts_helper.rb:2 - remove unused methods (PostsHelper#unused)')
       end
 
-      it "should not remove unused methods if called on views" do
+      it 'should not remove unused methods if called on views' do
         content =<<-EOF
         module PostsHelper
           def used?(post); end
@@ -38,7 +38,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not remove unused methods if called on helpers" do
+      it 'should not remove unused methods if called on helpers' do
         content =<<-EOF
         module PostsHelper
           def used?(post)
@@ -59,7 +59,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not remove unused methods if called on controllers" do
+      it 'should not remove unused methods if called on controllers' do
         helper_content =<<-EOF
         module PostsHelper
           def used?(post); end
@@ -84,7 +84,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not remove unused methods if called in descendant controllers" do
+      it 'should not remove unused methods if called in descendant controllers' do
         application_helper_content =<<-EOF
         module ApplicationHelper
           def admin?; end
@@ -114,7 +114,7 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-       it "should not check ignored files" do
+       it 'should not check ignored files' do
         runner = Core::Runner.new(prepares: [Prepares::ControllerPrepare.new, Prepares::HelperPrepare.new],
                                   reviews: RemoveUnusedMethodsInHelpersReview.new(ignored_files: /posts_helper/, except_methods: []))
 

data/spec/rails_best_practices/reviews/remove_unused_methods_in_models_review_spec.rb

@@ -5,11 +5,11 @@ module RailsBestPractices
     describe RemoveUnusedMethodsInModelsReview do
       let(:runner) { Core::Runner.new(
         prepares: Prepares::ModelPrepare.new,
-        reviews: RemoveUnusedMethodsInModelsReview.new({'except_methods' => ["*#set_cache"]})
+        reviews: RemoveUnusedMethodsInModelsReview.new({'except_methods' => ['*#set_cache']})
       ) }
 
-      context "private" do
-        it "should remove unused methods" do
+      context 'private' do
+        it 'should remove unused methods' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def find; end
@@ -29,10 +29,10 @@ module RailsBestPractices
           runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/models/post.rb:4 - remove unused methods (Post#find_by_sql)")
+          expect(runner.errors[0].to_s).to eq('app/models/post.rb:4 - remove unused methods (Post#find_by_sql)')
         end
 
-        it "should not remove unused methods with except_methods" do
+        it 'should not remove unused methods with except_methods' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def set_cache; end
@@ -44,7 +44,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods with var_ref" do
+        it 'should not remove unused methods with var_ref' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def find
@@ -68,7 +68,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods with callback" do
+        it 'should not remove unused methods with callback' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             after_save :expire_cache
@@ -82,7 +82,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with command" do
+        it 'should not remove unused method with command' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def fetch
@@ -106,7 +106,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with call" do
+        it 'should not remove unused method with call' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def conditions
@@ -130,7 +130,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with message" do
+        it 'should not remove unused method with message' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def save
@@ -156,7 +156,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with validation condition" do
+        it 'should not remove unused method with validation condition' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             validates_uniqueness_of :login, if: :email_blank?
@@ -170,7 +170,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with aasm" do
+        it 'should not remove unused method with aasm' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             aasm_state :accepted, enter: [:update_datetime]
@@ -184,7 +184,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with initialize" do
+        it 'should not remove unused method with initialize' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             private
@@ -198,8 +198,8 @@ module RailsBestPractices
         end
       end
 
-      context "public" do
-        it "should remove unused methods" do
+      context 'public' do
+        it 'should remove unused methods' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def fetch; end
@@ -208,10 +208,10 @@ module RailsBestPractices
           runner.prepare('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/models/post.rb:2 - remove unused methods (Post#fetch)")
+          expect(runner.errors[0].to_s).to eq('app/models/post.rb:2 - remove unused methods (Post#fetch)')
         end
 
-        it "should not remove unused methods" do
+        it 'should not remove unused methods' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def fetch; end
@@ -230,7 +230,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods for attribute assignment" do
+        it 'should not remove unused methods for attribute assignment' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def user=(user); end
@@ -242,7 +242,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods for try" do
+        it 'should not remove unused methods for try' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def find(user_id); end
@@ -262,7 +262,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods for send" do
+        it 'should not remove unused methods for send' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def find(user_id); end
@@ -282,7 +282,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should remove unused methods for send string_embexpre" do
+        it 'should remove unused methods for send string_embexpre' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def find_first; end
@@ -303,7 +303,7 @@ module RailsBestPractices
           expect(runner.errors.size).to eq(1)
         end
 
-        it "should remove unused methods for send variable" do
+        it 'should remove unused methods for send variable' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def first; end
@@ -325,16 +325,16 @@ module RailsBestPractices
         end
       end
 
-      context "protected" do
-        it "should not remove unused methods" do
+      context 'protected' do
+        it 'should not remove unused methods' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             protected
             def test; end
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def test
@@ -345,10 +345,10 @@ module RailsBestPractices
           runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/models/post.rb:3 - remove unused methods (Post#test)")
+          expect(runner.errors[0].to_s).to eq('app/models/post.rb:3 - remove unused methods (Post#test)')
         end
 
-        it "should not remove unused methods" do
+        it 'should not remove unused methods' do
           post_content =<<-EOF
           class Post < ActiveRecord::Base
             protected
@@ -362,10 +362,10 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.prepare("app/models/post.rb", post_content)
-          runner.prepare("app/models/blog_post.rb", blog_post_content)
-          runner.review("app/models/post.rb", post_content)
-          runner.review("app/models/blog_post.rb", blog_post_content)
+          runner.prepare('app/models/post.rb', post_content)
+          runner.prepare('app/models/blog_post.rb', blog_post_content)
+          runner.review('app/models/post.rb', post_content)
+          runner.review('app/models/blog_post.rb', blog_post_content)
           content =<<-EOF
           class BlogPostsController < ApplicationController
             def play
@@ -379,15 +379,15 @@ module RailsBestPractices
         end
       end
 
-      context "named_scope" do
-        it "should not remove unused named_scope" do
+      context 'named_scope' do
+        it 'should not remove unused named_scope' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             named_scope :active, conditions: {active: true}
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def index
@@ -395,34 +395,34 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.review("app/controllers/posts_controller.rb", content)
+          runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should remove unused named_scope" do
+        it 'should remove unused named_scope' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             named_scope :active, conditions: {active: true}
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/models/post.rb:2 - remove unused methods (Post#active)")
+          expect(runner.errors[0].to_s).to eq('app/models/post.rb:2 - remove unused methods (Post#active)')
         end
       end
 
-      context "scope" do
-        it "should not remove unused scope" do
+      context 'scope' do
+        it 'should not remove unused scope' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             scope :active, where(active: true)
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def index
@@ -430,35 +430,35 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.review("app/controllers/posts_controller.rb", content)
+          runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should remove unused named_scope" do
+        it 'should remove unused named_scope' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             scope :active, where(active: true)
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(1)
-          expect(runner.errors[0].to_s).to eq("app/models/post.rb:2 - remove unused methods (Post#active)")
+          expect(runner.errors[0].to_s).to eq('app/models/post.rb:2 - remove unused methods (Post#active)')
         end
       end
 
-      context "alias" do
-        it "should not remove unused method with alias" do
+      context 'alias' do
+        it 'should not remove unused method with alias' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def old; end
             alias new old
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def show
@@ -466,20 +466,20 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.review("app/controllers/posts_controller.rb", content)
+          runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-         it "should not remove unused method with symbol alias" do
+         it 'should not remove unused method with symbol alias' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def old; end
             alias :new :old
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def show
@@ -487,20 +487,20 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.review("app/controllers/posts_controller.rb", content)
+          runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with alias_method" do
+        it 'should not remove unused method with alias_method' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def old; end
             alias_method :new, :old
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def show
@@ -508,20 +508,20 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.review("app/controllers/posts_controller.rb", content)
+          runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method with alias_method_chain" do
+        it 'should not remove unused method with alias_method_chain' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def method_with_feature; end
             alias_method_chain :method, :feature
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           content =<<-EOF
           class PostsController < ApplicationController
             def show
@@ -529,14 +529,14 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.review("app/controllers/posts_controller.rb", content)
+          runner.review('app/controllers/posts_controller.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
       end
 
-      context "methods hash" do
-        it "should not remove unused method with methods hash" do
+      context 'methods hash' do
+        it 'should not remove unused method with methods hash' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def to_xml(options = {})
@@ -546,15 +546,15 @@ module RailsBestPractices
             def is_discussion_conversation; end
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
       end
 
-      context "callbacks" do
-        it "should not remove unused method" do
+      context 'callbacks' do
+        it 'should not remove unused method' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             before_save :init_columns
@@ -565,15 +565,15 @@ module RailsBestPractices
               def remove_dependencies; end
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
       end
 
-      context "validates" do
-        it "should not remove unused method" do
+      context 'validates' do
+        it 'should not remove unused method' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             validate :valid_birth_date
@@ -582,13 +582,13 @@ module RailsBestPractices
               def valid_birth_date; end
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method for validate_on_create and validate_on_update" do
+        it 'should not remove unused method for validate_on_create and validate_on_update' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             validate_on_create :valid_email
@@ -599,13 +599,13 @@ module RailsBestPractices
               def valid_birth_date; end
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused methods for to_param" do
+        it 'should not remove unused methods for to_param' do
           content =<<-EOF
           class Post < ActiveRecord::Base
             def to_param
@@ -613,83 +613,83 @@ module RailsBestPractices
             end
           end
           EOF
-          runner.prepare("app/models/post.rb", content)
-          runner.review("app/models/post.rb", content)
+          runner.prepare('app/models/post.rb', content)
+          runner.review('app/models/post.rb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
       end
 
-      context "helper method" do
-        it "should not remove unused method for coommand_call collection_select" do
+      context 'helper method' do
+        it 'should not remove unused method for coommand_call collection_select' do
           content =<<-EOF
           class Category < ActiveRecord::Base
             def indented_name; end
           end
           EOF
-          runner.prepare("app/models/category.rb", content)
-          runner.review("app/models/category.rb", content)
+          runner.prepare('app/models/category.rb', content)
+          runner.review('app/models/category.rb', content)
           content =<<-EOF
           <%= f.collection_select :parent_id, Category.all_hierarchic(except: @category), :id, :indented_name, {include_blank: true} %>
           EOF
-          runner.review("app/views/categories/_form.html.erb", content)
+          runner.review('app/views/categories/_form.html.erb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method for command collection_select" do
+        it 'should not remove unused method for command collection_select' do
           content =<<-EOF
           class Category < ActiveRecord::Base
             def indented_name; end
           end
           EOF
-          runner.prepare("app/models/category.rb", content)
-          runner.review("app/models/category.rb", content)
+          runner.prepare('app/models/category.rb', content)
+          runner.review('app/models/category.rb', content)
           content =<<-EOF
           <%= collection_select :category, :parent_id, Category.all_hierarchic(except: @category), :id, :indented_name, {include_blank: true} %>
           EOF
-          runner.review("app/views/categories/_form.html.erb", content)
+          runner.review('app/views/categories/_form.html.erb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
 
-        it "should not remove unused method for options_from_collection_for_select" do
+        it 'should not remove unused method for options_from_collection_for_select' do
           content =<<-EOF
           class Category < ActiveRecord::Base
             def indented_name; end
           end
           EOF
-          runner.prepare("app/models/category.rb", content)
-          runner.review("app/models/category.rb", content)
+          runner.prepare('app/models/category.rb', content)
+          runner.review('app/models/category.rb', content)
           content =<<-EOF
           <%= select_tag 'category', options_from_collection_for_select(Category.all_hierachic(except: @category), :id, :indented_name) %>
           EOF
-          runner.review("app/views/categories/_form.html.erb", content)
+          runner.review('app/views/categories/_form.html.erb', content)
           runner.after_review
           expect(runner.errors.size).to eq(0)
         end
       end
 
-      it "should not remove unused methods for rabl view" do
+      it 'should not remove unused methods for rabl view' do
         content =<<-EOF
         class User
           def first_name; end
           def last_name; end
         end
         EOF
-        runner.prepare("app/models/user.rb", content)
-        runner.review("app/models/user.rb", content)
+        runner.prepare('app/models/user.rb', content)
+        runner.review('app/models/user.rb', content)
         content =<<-EOF
         node :full_name do |u|
           u.first_name + " " + u.last_name
         end
         EOF
-        runner.review("app/views/users/show.json.rabl", content)
+        runner.review('app/views/users/show.json.rabl', content)
         runner.after_review
         expect(runner.errors.size).to eq(0)
       end
 
-      it "should not skip :call as call message" do
+      it 'should not skip :call as call message' do
         content =<<-EOF
         module DateRange
           RANGES = lambda {
@@ -700,11 +700,11 @@ module RailsBestPractices
           }[]
         end
         EOF
-        runner.prepare("app/mixins/date_range.rb", content)
-        runner.review("app/mixins/date_range.rb", content)
+        runner.prepare('app/mixins/date_range.rb', content)
+        runner.review('app/mixins/date_range.rb', content)
       end
 
-      it "should not check ignored files" do
+      it 'should not check ignored files' do
         runner = Core::Runner.new(prepares: Prepares::ModelPrepare.new,
                                   reviews: RemoveUnusedMethodsInModelsReview.new(except_methods: [], ignored_files: /post/))