rails_best_practices 1.20.0 → 1.20.1

data/spec/rails_best_practices/core_ext/erubis_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 
 describe Erubis::OnlyRuby do
-  subject {
+  subject do
     content = <<-EOF
     <h1>Title</h1>
     <% if current_user %>
@@ -13,14 +13,14 @@ describe Erubis::OnlyRuby do
       Not logged in
     <% end %>
     EOF
-    Erubis::OnlyRuby.new(content).src
-  }
+    described_class.new(content).src
+  end
 
-  it { should_not include('h1') }
-  it { should_not include('Title') }
-  it { should_not include('Not logged in') }
-  it { should include('current_user') }
-  it { should include('if') }
-  it { should include('else') }
-  it { should include('end') }
+  it { is_expected.not_to include('h1') }
+  it { is_expected.not_to include('Title') }
+  it { is_expected.not_to include('Not logged in') }
+  it { is_expected.to include('current_user') }
+  it { is_expected.to include('if') }
+  it { is_expected.to include('else') }
+  it { is_expected.to include('end') }
 end

data/spec/rails_best_practices/lexicals/long_line_check_spec.rb

@@ -5,8 +5,8 @@ require 'spec_helper'
 module RailsBestPractices
   module Lexicals
     describe LongLineCheck do
-      it 'should find long lines' do
-        runner = Core::Runner.new(lexicals: LongLineCheck.new)
+      it 'finds long lines' do
+        runner = Core::Runner.new(lexicals: described_class.new)
         content = <<~EOF
           class User < ActiveRecord::Base
           # 81 Chars
@@ -20,8 +20,8 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(1)
         expect(runner.errors[0].to_s).to eq('app/models/user.rb:3 - line is longer than 80 characters (81 characters)')
       end
-      it 'should find long lines with own max size' do
-        runner = Core::Runner.new(lexicals: LongLineCheck.new('max_line_length' => 90))
+      it 'finds long lines with own max size' do
+        runner = Core::Runner.new(lexicals: described_class.new('max_line_length' => 90))
         content = <<~EOF
           class User < ActiveRecord::Base
           # 91 Chars
@@ -35,16 +35,17 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(1)
         expect(runner.errors[0].to_s).to eq('app/models/user.rb:3 - line is longer than 90 characters (91 characters)')
       end
-      it 'should not check non .rb files' do
-        runner = Core::Runner.new(lexicals: LongLineCheck.new)
-        content = "
+      it 'does not check non .rb files' do
+        runner = Core::Runner.new(lexicals: described_class.new)
+        content =
+          '
 xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
-"
+'
         runner.lexical('app/views/users/index.html.erb', content)
         expect(runner.errors.size).to eq(0)
       end
-      it 'should not check ignored files' do
-        runner = Core::Runner.new(lexicals: LongLineCheck.new(max_line_length: 80, ignored_files: /user/))
+      it 'does not check ignored files' do
+        runner = Core::Runner.new(lexicals: described_class.new(max_line_length: 80, ignored_files: /user/))
         content = <<~EOF
           class User < ActiveRecord::Base
           # 81 Chars

data/spec/rails_best_practices/lexicals/remove_tab_check_spec.rb

@@ -5,9 +5,9 @@ require 'spec_helper'
 module RailsBestPractices
   module Lexicals
     describe RemoveTabCheck do
-      let(:runner) { Core::Runner.new(lexicals: RemoveTabCheck.new) }
+      let(:runner) { Core::Runner.new(lexicals: described_class.new) }
 
-      it 'should remove tab' do
+      it 'removes tab' do
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects
@@ -19,7 +19,7 @@ module RailsBestPractices
         expect(runner.errors[0].to_s).to eq('app/models/user.rb:1 - remove tab, use spaces instead')
       end
 
-      it 'should remove tab with third line' do
+      it 'removes tab with third line' do
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects
@@ -31,7 +31,7 @@ module RailsBestPractices
         expect(runner.errors[0].to_s).to eq('app/models/user.rb:3 - remove tab, use spaces instead')
       end
 
-      it 'should not remove trailing whitespace' do
+      it 'does not remove trailing whitespace' do
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects
@@ -40,8 +40,8 @@ module RailsBestPractices
         runner.lexical('app/models/user.rb', content)
         expect(runner.errors.size).to eq(0)
       end
-      it 'should not check ignored files' do
-        runner = Core::Runner.new(lexicals: RemoveTabCheck.new(ignored_files: /user/))
+      it 'does not check ignored files' do
+        runner = Core::Runner.new(lexicals: described_class.new(ignored_files: /user/))
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects

data/spec/rails_best_practices/lexicals/remove_trailing_whitespace_check_spec.rb

@@ -5,9 +5,9 @@ require 'spec_helper'
 module RailsBestPractices
   module Lexicals
     describe RemoveTrailingWhitespaceCheck do
-      let(:runner) { Core::Runner.new(lexicals: RemoveTrailingWhitespaceCheck.new) }
+      let(:runner) { Core::Runner.new(lexicals: described_class.new) }
 
-      it 'should remove trailing whitespace' do
+      it 'removes trailing whitespace' do
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects
@@ -19,7 +19,7 @@ module RailsBestPractices
         expect(runner.errors[0].to_s).to eq('app/models/user.rb:1 - remove trailing whitespace')
       end
 
-      it 'should remove whitespace with third line' do
+      it 'removes whitespace with third line' do
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects
@@ -31,7 +31,7 @@ module RailsBestPractices
         expect(runner.errors[0].to_s).to eq('app/models/user.rb:3 - remove trailing whitespace')
       end
 
-      it 'should not remove trailing whitespace' do
+      it 'does not remove trailing whitespace' do
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects
@@ -41,8 +41,8 @@ module RailsBestPractices
         expect(runner.errors.size).to eq(0)
       end
 
-      it 'should not check ignored files' do
-        runner = Core::Runner.new(lexicals: RemoveTrailingWhitespaceCheck.new(ignored_files: /user/))
+      it 'does not check ignored files' do
+        runner = Core::Runner.new(lexicals: described_class.new(ignored_files: /user/))
         content = <<-EOF
         class User < ActiveRecord::Base
           has_many :projects

data/spec/rails_best_practices/prepares/config_prepare_spec.rb

@@ -5,10 +5,10 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe ConfigPrepare do
-      let(:runner) { Core::Runner.new(prepares: ConfigPrepare.new) }
+      let(:runner) { Core::Runner.new(prepares: described_class.new) }
 
       context 'configs' do
-        it 'should parse configs' do
+        it 'parses configs' do
           content = <<-EOF
           module RailsBestPracticesCom
             class Application < Rails::Application

data/spec/rails_best_practices/prepares/controller_prepare_spec.rb

@@ -5,10 +5,10 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe ControllerPrepare do
-      let(:runner) { Core::Runner.new(prepares: [ControllerPrepare.new, HelperPrepare.new]) }
+      let(:runner) { Core::Runner.new(prepares: [described_class.new, HelperPrepare.new]) }
 
       context 'methods' do
-        it 'should parse controller methods' do
+        it 'parses controller methods' do
           content = <<-EOF
           class PostsController < ApplicationController
             def index; end
@@ -20,7 +20,7 @@ module RailsBestPractices
           expect(methods.get_methods('PostsController').map(&:method_name)).to eq(%w[index show])
         end
 
-        it 'should parse model methods with access control' do
+        it 'parses model methods with access control' do
           content = <<-EOF
           class PostsController < ApplicationController
             def index; end
@@ -39,7 +39,7 @@ module RailsBestPractices
           expect(methods.get_methods('PostsController', 'private').map(&:method_name)).to eq(['resource'])
         end
 
-        it 'should parse controller methods with module ::' do
+        it 'parses controller methods with module ::' do
           content = <<-EOF
           class Admin::Blog::PostsController < ApplicationController
             def index; end
@@ -51,7 +51,7 @@ module RailsBestPractices
           expect(methods.get_methods('Admin::Blog::PostsController').map(&:method_name)).to eq(%w[index show])
         end
 
-        it 'should parse controller methods with module' do
+        it 'parses controller methods with module' do
           content = <<-EOF
           module Admin
             module Blog
@@ -75,7 +75,9 @@ module RailsBestPractices
             EOF
             runner.prepare('app/controllers/posts_controller.rb', content)
             methods = Prepares.controller_methods
-            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(%w[index show new create edit update destroy])
+            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(
+              %w[index show new create edit update destroy]
+            )
           end
 
           it 'extend inherited_resources with actions' do
@@ -97,7 +99,9 @@ module RailsBestPractices
             EOF
             runner.prepare('app/controllers/posts_controller.rb', content)
             methods = Prepares.controller_methods
-            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(%w[index new create edit update destroy])
+            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(
+              %w[index new create edit update destroy]
+            )
           end
 
           it 'extend inherited_resources with all actions with no arguments' do
@@ -108,7 +112,9 @@ module RailsBestPractices
             EOF
             runner.prepare('app/controllers/posts_controller.rb', content)
             methods = Prepares.controller_methods
-            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(%w[index show new create edit update destroy])
+            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(
+              %w[index show new create edit update destroy]
+            )
           end
 
           it 'DSL inherit_resources' do
@@ -119,13 +125,15 @@ module RailsBestPractices
             EOF
             runner.prepare('app/controllers/posts_controller.rb', content)
             methods = Prepares.controller_methods
-            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(%w[index show new create edit update destroy])
+            expect(methods.get_methods('PostsController').map(&:method_name)).to eq(
+              %w[index show new create edit update destroy]
+            )
           end
         end
       end
 
       context 'helpers' do
-        it 'should add helper descendant' do
+        it 'adds helper descendant' do
           content = <<-EOF
           module PostsHelper
           end

data/spec/rails_best_practices/prepares/gemfile_prepare_spec.rb

@@ -5,10 +5,10 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe GemfilePrepare do
-      let(:runner) { Core::Runner.new(prepares: GemfilePrepare.new) }
+      let(:runner) { Core::Runner.new(prepares: described_class.new) }
 
       context 'gemfile' do
-        it 'should parse gems' do
+        it 'parses gems' do
           content = <<~EOF
             GEM
               remote: https://rubygems.org/

data/spec/rails_best_practices/prepares/helper_prepare_spec.rb

@@ -5,10 +5,10 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe HelperPrepare do
-      let(:runner) { Core::Runner.new(prepares: HelperPrepare.new) }
+      let(:runner) { Core::Runner.new(prepares: described_class.new) }
 
       context 'methods' do
-        it 'should parse helper methods' do
+        it 'parses helper methods' do
           content = <<-EOF
           module PostsHelper
             def used; end
@@ -20,7 +20,7 @@ module RailsBestPractices
           expect(methods.get_methods('PostsHelper').map(&:method_name)).to eq(%w[used unused])
         end
 
-        it 'should parse helpers' do
+        it 'parses helpers' do
           content = <<-EOF
           module PostsHelper
           end

data/spec/rails_best_practices/prepares/initializer_prepare_spec.rb

@@ -5,10 +5,10 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe InitializerPrepare do
-      let(:runner) { Core::Runner.new(prepares: InitializerPrepare.new) }
+      let(:runner) { Core::Runner.new(prepares: described_class.new) }
 
       context 'initializers' do
-        it 'should set include_forbidden_attributes_protection config' do
+        it 'sets include_forbidden_attributes_protection config' do
           content = <<-EOF
           class AR
             ActiveRecord::Base.send(:include, ActiveModel::ForbiddenAttributesProtection)
@@ -19,7 +19,7 @@ module RailsBestPractices
           expect(configs['railsbp.include_forbidden_attributes_protection']).to eq('true')
         end
 
-        it 'should not set include_forbidden_attributes_protection config' do
+        it 'does not set include_forbidden_attributes_protection config' do
           content = <<-EOF
           class AR
           end

data/spec/rails_best_practices/prepares/mailer_prepare_spec.rb

@@ -5,9 +5,9 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe MailerPrepare do
-      let(:runner) { Core::Runner.new(prepares: MailerPrepare.new) }
+      let(:runner) { Core::Runner.new(prepares: described_class.new) }
 
-      it 'should parse mailer names' do
+      it 'parses mailer names' do
         content = <<-EOF
         class ProjectMailer < ActionMailer::Base
         end

data/spec/rails_best_practices/prepares/model_prepare_spec.rb

@@ -5,7 +5,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Prepares
     describe ModelPrepare do
-      let(:runner) { Core::Runner.new(prepares: ModelPrepare.new) }
+      let(:runner) { Core::Runner.new(prepares: described_class.new) }
 
       context 'models' do
         it 'class_name with modules ::' do
@@ -32,7 +32,7 @@ module RailsBestPractices
       end
 
       context 'associations' do
-        it 'should parse model associations' do
+        it 'parses model associations' do
           content = <<-EOF
           class Project < ActiveRecord::Base
             belongs_to              :portfolio
@@ -43,14 +43,22 @@ module RailsBestPractices
           EOF
           runner.prepare('app/models/project.rb', content)
           model_associations = Prepares.model_associations
-          expect(model_associations.get_association('Project', 'portfolio')).to eq('meta' => 'belongs_to', 'class_name' => 'Portfolio')
-          expect(model_associations.get_association('Project', 'project_manager')).to eq('meta' => 'has_one', 'class_name' => 'ProjectManager')
-          expect(model_associations.get_association('Project', 'milestones')).to eq('meta' => 'has_many', 'class_name' => 'Milestone')
-          expect(model_associations.get_association('Project', 'categories')).to eq('meta' => 'has_and_belongs_to_many', 'class_name' => 'Category')
+          expect(model_associations.get_association('Project', 'portfolio')).to eq(
+            'meta' => 'belongs_to', 'class_name' => 'Portfolio'
+          )
+          expect(model_associations.get_association('Project', 'project_manager')).to eq(
+            'meta' => 'has_one', 'class_name' => 'ProjectManager'
+          )
+          expect(model_associations.get_association('Project', 'milestones')).to eq(
+            'meta' => 'has_many', 'class_name' => 'Milestone'
+          )
+          expect(model_associations.get_association('Project', 'categories')).to eq(
+            'meta' => 'has_and_belongs_to_many', 'class_name' => 'Category'
+          )
         end
 
         context 'with class_name option' do
-          it 'should parse belongs_to' do
+          it 'parses belongs_to' do
             content = <<-EOF
             class Post < ActiveRecord::Base
               belongs_to :author, "class_name" => "Person"
@@ -58,10 +66,12 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/post.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Post', 'author')).to eq('meta' => 'belongs_to', 'class_name' => 'Person')
+            expect(model_associations.get_association('Post', 'author')).to eq(
+              'meta' => 'belongs_to', 'class_name' => 'Person'
+            )
           end
 
-          it 'should parse has_one' do
+          it 'parses has_one' do
             content = <<-EOF
             class Project < ActiveRecord::Base
               has_one :project_manager, "class_name" => "Person"
@@ -69,10 +79,12 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/post.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Project', 'project_manager')).to eq('meta' => 'has_one', 'class_name' => 'Person')
+            expect(model_associations.get_association('Project', 'project_manager')).to eq(
+              'meta' => 'has_one', 'class_name' => 'Person'
+            )
           end
 
-          it 'should parse has_many' do
+          it 'parses has_many' do
             content = <<-EOF
             class Project < ActiveRecord::Base
               has_many :people, "class_name" => "Person"
@@ -80,10 +92,12 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/project.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Project', 'people')).to eq('meta' => 'has_many', 'class_name' => 'Person')
+            expect(model_associations.get_association('Project', 'people')).to eq(
+              'meta' => 'has_many', 'class_name' => 'Person'
+            )
           end
 
-          it 'should parse has_and_belongs_to_many' do
+          it 'parses has_and_belongs_to_many' do
             content = <<-EOF
             class Citizen < ActiveRecord::Base
               has_and_belongs_to_many :nations, "class_name" => "Country"
@@ -91,11 +105,13 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/citizen.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Citizen', 'nations')).to eq('meta' => 'has_and_belongs_to_many', 'class_name' => 'Country')
+            expect(model_associations.get_association('Citizen', 'nations')).to eq(
+              'meta' => 'has_and_belongs_to_many', 'class_name' => 'Country'
+            )
           end
 
           context 'namespace' do
-            it 'should parse with namespace' do
+            it 'parses with namespace' do
               content = <<-EOF
               class Community < ActiveRecord::Base
                 has_many :members
@@ -110,11 +126,15 @@ module RailsBestPractices
               runner.prepare('app/models/community/member.rb', content)
               runner.after_prepare
               model_associations = Prepares.model_associations
-              expect(model_associations.get_association('Community', 'members')).to eq('meta' => 'has_many', 'class_name' => 'Community::Member')
-              expect(model_associations.get_association('Community::Member', 'community')).to eq('meta' => 'belongs_to', 'class_name' => 'Community')
+              expect(model_associations.get_association('Community', 'members')).to eq(
+                'meta' => 'has_many', 'class_name' => 'Community::Member'
+              )
+              expect(model_associations.get_association('Community::Member', 'community')).to eq(
+                'meta' => 'belongs_to', 'class_name' => 'Community'
+              )
             end
 
-            it 'should parse without namespace' do
+            it 'parses without namespace' do
               content = <<-EOF
               class Community::Member::Rating < ActiveRecord::Base
                 belongs_to :member
@@ -129,14 +149,18 @@ module RailsBestPractices
               runner.prepare('app/models/community/member.rb', content)
               runner.after_prepare
               model_associations = Prepares.model_associations
-              expect(model_associations.get_association('Community::Member::Rating', 'member')).to eq('meta' => 'belongs_to', 'class_name' => 'Community::Member')
-              expect(model_associations.get_association('Community::Member', 'ratings')).to eq('meta' => 'has_many', 'class_name' => 'Community::Member::Rating')
+              expect(model_associations.get_association('Community::Member::Rating', 'member')).to eq(
+                'meta' => 'belongs_to', 'class_name' => 'Community::Member'
+              )
+              expect(model_associations.get_association('Community::Member', 'ratings')).to eq(
+                'meta' => 'has_many', 'class_name' => 'Community::Member::Rating'
+              )
             end
           end
         end
 
         context 'mongoid embeds' do
-          it 'should parse embeds_many' do
+          it 'parses embeds_many' do
             content = <<-EOF
             class Person
               include Mongoid::Document
@@ -145,10 +169,12 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/person.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Person', 'addresses')).to eq('meta' => 'embeds_many', 'class_name' => 'Address')
+            expect(model_associations.get_association('Person', 'addresses')).to eq(
+              'meta' => 'embeds_many', 'class_name' => 'Address'
+            )
           end
 
-          it 'should parse embeds_one' do
+          it 'parses embeds_one' do
             content = <<-EOF
             class Lush
               include Mongoid::Document
@@ -157,10 +183,12 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/lush.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Lush', 'whiskey')).to eq('meta' => 'embeds_one', 'class_name' => 'Drink')
+            expect(model_associations.get_association('Lush', 'whiskey')).to eq(
+              'meta' => 'embeds_one', 'class_name' => 'Drink'
+            )
           end
 
-          it 'should parse embedded_in' do
+          it 'parses embedded_in' do
             content = <<-EOF
             class Drink
               include Mongoid::Document
@@ -169,12 +197,14 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/drink.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Drink', 'alcoholic')).to eq('meta' => 'embedded_in', 'class_name' => 'Lush')
+            expect(model_associations.get_association('Drink', 'alcoholic')).to eq(
+              'meta' => 'embedded_in', 'class_name' => 'Lush'
+            )
           end
         end
 
         context 'mongomapper many/one' do
-          it 'should parse one' do
+          it 'parses one' do
             content = <<-EOF
             class Employee
               include MongoMapper::Document
@@ -183,10 +213,12 @@ module RailsBestPractices
             EOF
             runner.prepare('app/models/employee.rb', content)
             model_associations = Prepares.model_associations
-            expect(model_associations.get_association('Employee', 'desk')).to eq('meta' => 'one', 'class_name' => 'Desk')
+            expect(model_associations.get_association('Employee', 'desk')).to eq(
+              'meta' => 'one', 'class_name' => 'Desk'
+            )
           end
 
-          it 'should parse many' do
+          it 'parses many' do
             content = <<-EOF
             class Tree
               include MongoMapper::Document
@@ -201,7 +233,7 @@ module RailsBestPractices
       end
 
       context 'methods' do
-        it 'should parse model methods' do
+        it 'parses model methods' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             def save; end
@@ -213,7 +245,7 @@ module RailsBestPractices
           expect(methods.get_methods('Post').map(&:method_name)).to eq(%w[save find])
         end
 
-        it 'should parse model methods with access control' do
+        it 'parses model methods with access control' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             def save; end
@@ -232,7 +264,7 @@ module RailsBestPractices
           expect(methods.get_methods('Post', 'private').map(&:method_name)).to eq(['find_by_sql'])
         end
 
-        it 'should parse model methods with module ::' do
+        it 'parses model methods with module ::' do
           content = <<-EOF
           class Admin::Blog::Post < ActiveRecord::Base
             def save; end
@@ -244,7 +276,7 @@ module RailsBestPractices
           expect(methods.get_methods('Admin::Blog::Post').map(&:method_name)).to eq(%w[save find])
         end
 
-        it 'should parse model methods with module' do
+        it 'parses model methods with module' do
           content = <<-EOF
           module Admin
             module Blog
@@ -260,7 +292,7 @@ module RailsBestPractices
           expect(methods.get_methods('Admin::Blog::Post').map(&:method_name)).to eq(%w[save find])
         end
 
-        it 'should not add methods from module' do
+        it 'does not add methods from module' do
           content = <<-EOF
             class Model < ActiveRecord::Base
             end
@@ -279,7 +311,7 @@ module RailsBestPractices
       end
 
       context 'scope' do
-        it 'should treat named_scope as method' do
+        it 'treats named_scope as method' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             named_scope :active, conditions: {active: true}
@@ -290,7 +322,7 @@ module RailsBestPractices
           expect(methods.get_methods('Post').map(&:method_name)).to eq(['active'])
         end
 
-        it 'should treat scope as method' do
+        it 'treats scope as method' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             scope :active, where(active: true)
@@ -303,7 +335,7 @@ module RailsBestPractices
       end
 
       context 'alias' do
-        it 'should treat alias as method' do
+        it 'treats alias as method' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             alias :new :old
@@ -314,7 +346,7 @@ module RailsBestPractices
           expect(methods.get_methods('Post').map(&:method_name)).to eq(['new'])
         end
 
-        it 'should treat alias_method as method' do
+        it 'treats alias_method as method' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             alias_method :new, :old
@@ -325,7 +357,7 @@ module RailsBestPractices
           expect(methods.get_methods('Post').map(&:method_name)).to eq(['new'])
         end
 
-        it 'should treat alias_method_chain as method' do
+        it 'treats alias_method_chain as method' do
           content = <<-EOF
           class Post < ActiveRecord::Base
             alias_method_chain :method, :feature
@@ -338,7 +370,7 @@ module RailsBestPractices
       end
 
       context 'attributes' do
-        it 'should parse mongoid field' do
+        it 'parses mongoid field' do
           content = <<-EOF
           class Post
             include Mongoid::Document
@@ -358,7 +390,7 @@ module RailsBestPractices
           expect(model_attributes.get_attribute_type('Post', 'active')).to eq('Boolean')
         end
 
-        it 'should parse mongomapper field' do
+        it 'parses mongomapper field' do
           content = <<-EOF
           class Post
             include MongoMapper::Document
@@ -382,13 +414,17 @@ module RailsBestPractices
       end
 
       context 'no error' do
-        it 'should raised for finder_sql option' do
+        it 'raiseds for finder_sql option' do
           content = <<-EOF
           class EventSubscription < ActiveRecord::Base
             has_many :event_notification_template, finder_sql: ?
           end
           EOF
-          content = content.sub('?', '\'SELECT event_notification_templates.* from event_notification_templates where event_type_id=#{event_type_id} and delivery_method_id=#{delivery_method_id}\'')
+          content =
+            content.sub(
+              '?',
+              '\'SELECT event_notification_templates.* from event_notification_templates where event_type_id=#{event_type_id} and delivery_method_id=#{delivery_method_id}\''
+            )
           expect { runner.prepare('app/models/event_subscription.rb', content) }.not_to raise_error
         end
       end