rails_best_practices 1.19.5 → 1.22.0

data/rails_best_practices.gemspec

@@ -2,42 +2,41 @@
 
 require File.expand_path('lib/rails_best_practices/version', __dir__)
 
-Gem::Specification.new do |s|
-  s.name        = 'rails_best_practices'
-  s.version     = RailsBestPractices::VERSION
-  s.platform    = Gem::Platform::RUBY
-  s.authors     = ['Richard Huang']
-  s.email       = ['flyerhzm@gmail.com']
-  s.homepage    = 'http://rails-bestpractices.com'
-  s.summary     = 'a code metric tool for rails codes.'
-  s.description = 'a code metric tool for rails codes, written in Ruby.'
-  s.license     = 'MIT'
-
-  s.required_ruby_version = '>= 1.9.0'
-  s.required_rubygems_version = '>= 1.3.6'
-
-  s.add_dependency('activesupport')
-  # TODO: add a dependency for Rails >= 3 ?
-  s.add_dependency('code_analyzer', '>= 0.5.1')
-  s.add_dependency('erubis')
-  s.add_dependency('i18n')
-  s.add_dependency('json')
-  s.add_dependency('require_all', '~> 3.0')
-  s.add_dependency('ruby-progressbar')
-
-  s.add_development_dependency('awesome_print')
-  s.add_development_dependency('bundler')
-  s.add_development_dependency('haml')
-  s.add_development_dependency('rake', '< 11.0')
-  s.add_development_dependency('rspec')
-  s.add_development_dependency('slim')
-
-  s.files         = `git ls-files`.split("\n")
-  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
-  s.require_paths = %w[lib assets]
-
-  s.post_install_message = <<~POST_INSTALL_MESSAGE
+Gem::Specification.new do |spec|
+  spec.name        = 'rails_best_practices'
+  spec.version     = RailsBestPractices::VERSION
+  spec.platform    = Gem::Platform::RUBY
+  spec.authors     = ['Richard Huang']
+  spec.email       = ['flyerhzm@gmail.com']
+  spec.homepage    = 'http://rails-bestpractices.com'
+  spec.summary     = 'a code metric tool for rails codes.'
+  spec.description = 'a code metric tool for rails codes, written in Ruby.'
+  spec.license     = 'MIT'
+
+  spec.required_ruby_version = '>= 1.9.0'
+  spec.required_rubygems_version = '>= 1.3.6'
+
+  spec.add_dependency('activesupport')
+  spec.add_dependency('code_analyzer', '>= 0.5.2')
+  spec.add_dependency('erubis')
+  spec.add_dependency('i18n')
+  spec.add_dependency('json')
+  spec.add_dependency('require_all', '~> 3.0')
+  spec.add_dependency('ruby-progressbar')
+
+  spec.add_development_dependency('awesome_print')
+  spec.add_development_dependency('bundler')
+  spec.add_development_dependency('haml')
+  spec.add_development_dependency('rake')
+  spec.add_development_dependency('rspec')
+  spec.add_development_dependency('slim')
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = %w[lib assets]
+
+  spec.post_install_message = <<~POST_INSTALL_MESSAGE
     #{'*' * 80}
 
       rails_best_practices is a code metric tool to check the quality of rails codes.

data/spec/fixtures/lib/rails_best_practices/plugins/reviews/not_use_rails_root_review.rb

@@ -5,8 +5,7 @@ require 'rails_best_practices/reviews/review'
 module RailsBestPractices
   module Plugins
     module Reviews
-      class NotUseRailsRootReview < RailsBestPractices::Reviews::Review
-      end
+      class NotUseRailsRootReview < RailsBestPractices::Reviews::Review; end
     end
   end
 end

data/spec/rails_best_practices/analyzer_spec.rb

@@ -5,39 +5,70 @@ require 'tmpdir'
 
 module RailsBestPractices
   describe Analyzer do
-    subject { Analyzer.new('.') }
+    subject { described_class.new('.') }
 
     describe '::new' do
-      it 'should expand a relative path to an absolute' do
+      it 'expands a relative path to an absolute' do
         expect(subject.path).to eq File.expand_path('.')
       end
     end
 
     describe 'expand_dirs_to_files' do
-      it 'should expand all files in spec directory' do
+      it 'expands all files in spec directory' do
         dir = File.dirname(__FILE__)
         expect(subject.expand_dirs_to_files(dir)).to be_include(dir + '/analyzer_spec.rb')
       end
     end
 
     describe 'file_sort' do
-      it 'should get models first, mailers, helpers and then others' do
-        files = ['app/controllers/users_controller.rb', 'app/mailers/user_mailer.rb', 'app/helpers/users_helper.rb', 'app/models/user.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim', 'lib/user.rb']
-        expect(subject.file_sort(files)).to eq(['app/models/user.rb', 'app/mailers/user_mailer.rb', 'app/helpers/users_helper.rb', 'app/controllers/users_controller.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim', 'lib/user.rb'])
+      it 'gets models first, mailers, helpers and then others' do
+        files = [
+          'app/controllers/users_controller.rb',
+          'app/mailers/user_mailer.rb',
+          'app/helpers/users_helper.rb',
+          'app/models/user.rb',
+          'app/views/users/index.html.haml',
+          'app/views/users/show.html.slim',
+          'lib/user.rb'
+        ]
+        expect(subject.file_sort(files)).to eq(
+          [
+            'app/models/user.rb',
+            'app/mailers/user_mailer.rb',
+            'app/helpers/users_helper.rb',
+            'app/controllers/users_controller.rb',
+            'app/views/users/index.html.haml',
+            'app/views/users/show.html.slim',
+            'lib/user.rb'
+          ]
+        )
       end
     end
 
     describe 'file_ignore' do
       before do
-        @all = ['app/controllers/users_controller.rb', 'app/mailers/user_mailer.rb', 'app/models/user.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim', 'lib/user.rb']
-        @filtered = ['app/controllers/users_controller.rb', 'app/mailers/user_mailer.rb', 'app/models/user.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim']
-      end
-
-      it 'should ignore lib' do
+        @all = [
+          'app/controllers/users_controller.rb',
+          'app/mailers/user_mailer.rb',
+          'app/models/user.rb',
+          'app/views/users/index.html.haml',
+          'app/views/users/show.html.slim',
+          'lib/user.rb'
+        ]
+        @filtered = [
+          'app/controllers/users_controller.rb',
+          'app/mailers/user_mailer.rb',
+          'app/models/user.rb',
+          'app/views/users/index.html.haml',
+          'app/views/users/show.html.slim'
+        ]
+      end
+
+      it 'ignores lib' do
         expect(subject.file_ignore(@all, 'lib/')).to eq(@filtered)
       end
 
-      it 'should ignore regexp patterns' do
+      it 'ignores regexp patterns' do
         expect(subject.file_ignore(@all, /lib/)).to eq(@filtered)
       end
     end
@@ -88,7 +119,7 @@ module RailsBestPractices
     end
 
     describe 'output_terminal_errors' do
-      it 'should output errors in terminal' do
+      it 'outputs errors in terminal' do
         check1 = Reviews::LawOfDemeterReview.new
         check2 = Reviews::UseQueryAttributeReview.new
         runner = Core::Runner.new(reviews: [check1, check2])
@@ -102,12 +133,14 @@ module RailsBestPractices
         subject.output_terminal_errors
         result = $stdout.string
         $stdout = $origin_stdout
-        expect(result).to eq([
-          "\e[31mapp/models/user.rb:10 - law of demeter\e[0m",
-          "\e[31mapp/models/post.rb:100 - use query attribute\e[0m",
-          "\e[32m\nPlease go to https://rails-bestpractices.com to see more useful Rails Best Practices.\e[0m",
-          "\e[31m\nFound 2 warnings.\e[0m"
-        ].join("\n") + "\n")
+        expect(result).to eq(
+          [
+            "\e[31mapp/models/user.rb:10 - law of demeter\e[0m",
+            "\e[31mapp/models/post.rb:100 - use query attribute\e[0m",
+            "\e[32m\nPlease go to https://rails-bestpractices.com to see more useful Rails Best Practices.\e[0m",
+            "\e[31m\nFound 2 warnings.\e[0m"
+          ].join("\n") + "\n"
+        )
       end
     end
 
@@ -115,9 +148,7 @@ module RailsBestPractices
       let(:output_file) { 'rails_best_practices_output.json' }
 
       subject do
-        described_class.new('.',
-                            'format' => 'json',
-                            'output-file' => output_file)
+        described_class.new('.', 'format' => 'json', 'output-file' => output_file)
       end
 
       let(:check1) { Reviews::LawOfDemeterReview.new }
@@ -142,54 +173,54 @@ module RailsBestPractices
     end
 
     describe 'parse_files' do
-      it 'should not filter out all files when the path contains "vendor"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "vendor"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'vendor'))
           Dir.mkdir(File.join(random_dir, 'vendor', 'my_project'))
           File.open(File.join(random_dir, 'vendor', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'vendor', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'vendor', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'vendor', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "spec"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "spec"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'spec'))
           Dir.mkdir(File.join(random_dir, 'spec', 'my_project'))
           File.open(File.join(random_dir, 'spec', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'spec', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'spec', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'spec', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "test"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "test"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'test'))
           Dir.mkdir(File.join(random_dir, 'test', 'my_project'))
           File.open(File.join(random_dir, 'test', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'test', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'test', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'test', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "features"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "features"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'test'))
           Dir.mkdir(File.join(random_dir, 'test', 'my_project'))
           File.open(File.join(random_dir, 'test', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'test', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'test', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'test', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "tmp"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "tmp"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'tmp'))
           Dir.mkdir(File.join(random_dir, 'tmp', 'my_project'))
           File.open(File.join(random_dir, 'tmp', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'tmp', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'tmp', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'tmp', 'my_project', 'my_file.rb')
-        }
+        end
       end
     end
   end

data/spec/rails_best_practices/core/check_spec.rb

@@ -4,13 +4,13 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Check do
-    let(:check) { Check.new }
+    let(:check) { described_class.new }
 
     context 'debug' do
-      it 'should be debug mode' do
-        Check.debug
-        expect(Check).to be_debug
-        Check.class_eval { @debug = false }
+      it 'is debug mode' do
+        described_class.debug
+        expect(described_class).to be_debug
+        described_class.class_eval { @debug = false }
       end
     end
   end

data/spec/rails_best_practices/core/checks_loader_spec.rb

@@ -4,17 +4,17 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe ChecksLoader do
-    let(:checks_loader) { ChecksLoader.new(RailsBestPractices::Analyzer::DEFAULT_CONFIG) }
+    let(:checks_loader) { described_class.new(RailsBestPractices::Analyzer::DEFAULT_CONFIG) }
 
     describe 'load_lexicals' do
-      it 'should load lexical checks from the default configuration' do
+      it 'loads lexical checks from the default configuration' do
         lexicals = checks_loader.load_lexicals
         expect(lexicals.map(&:class)).to include(RailsBestPractices::Lexicals::RemoveTrailingWhitespaceCheck)
       end
     end
 
     describe 'load_reviews' do
-      it 'should load the reviews from the default the configuration' do
+      it 'loads the reviews from the default the configuration' do
         reviews = checks_loader.load_reviews
         expect(reviews.map(&:class)).to include(RailsBestPractices::Reviews::AlwaysAddDbIndexReview)
       end

data/spec/rails_best_practices/core/configs_spec.rb

@@ -4,6 +4,6 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Configs do
-    it { should be_a_kind_of Hash }
+    it { is_expected.to be_a_kind_of Hash }
   end
 end

data/spec/rails_best_practices/core/controllers_spec.rb

@@ -4,6 +4,6 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Controllers do
-    it { should be_a_kind_of Klasses }
+    it { is_expected.to be_a_kind_of Klasses }
   end
 end

data/spec/rails_best_practices/core/error_spec.rb

@@ -4,32 +4,32 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Error do
-    it 'should return error with filename, line number and message' do
-      expect(Error.new(
-        filename: 'app/models/user.rb',
-        line_number: '100',
-        message: 'not good',
-        type: 'BogusReview'
-      ).to_s).to eq('app/models/user.rb:100 - not good')
+    it 'returns error with filename, line number and message' do
+      expect(
+        described_class.new(
+          filename: 'app/models/user.rb', line_number: '100', message: 'not good', type: 'BogusReview'
+        ).to_s
+      ).to eq('app/models/user.rb:100 - not good')
     end
 
-    it 'should return short filename' do
+    it 'returns short filename' do
       Runner.base_path = '../rails-bestpractices.com'
-      expect(Error.new(
-        filename: '../rails-bestpractices.com/app/models/user.rb',
-        line_number: '100',
-        message: 'not good',
-        type: 'BogusReview'
-      ).short_filename).to eq('app/models/user.rb')
+      expect(
+        described_class.new(
+          filename: '../rails-bestpractices.com/app/models/user.rb',
+          line_number: '100',
+          message: 'not good',
+          type: 'BogusReview'
+        ).short_filename
+      ).to eq('app/models/user.rb')
     end
 
-    it 'should return first line number' do
-      expect(Error.new(
-        filename: 'app/models/user.rb',
-        line_number: '50,70,100',
-        message: 'not good',
-        type: 'BogusReview'
-      ).first_line_number).to eq('50')
+    it 'returns first line number' do
+      expect(
+        described_class.new(
+          filename: 'app/models/user.rb', line_number: '50,70,100', message: 'not good', type: 'BogusReview'
+        ).first_line_number
+      ).to eq('50')
     end
   end
 end

data/spec/rails_best_practices/core/except_methods_spec.rb

@@ -10,7 +10,7 @@ module RailsBestPractices::Core
       let(:except_method) { '*#*' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql true
+        expect(described_class.matches(method, except_method)).to eql true
       end
     end
 
@@ -18,7 +18,7 @@ module RailsBestPractices::Core
       let(:except_method) { '*#approve' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql true
+        expect(described_class.matches(method, except_method)).to eql true
       end
     end
 
@@ -26,7 +26,7 @@ module RailsBestPractices::Core
       let(:except_method) { '*#disapprove' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql false
+        expect(described_class.matches(method, except_method)).to eql false
       end
     end
 
@@ -34,7 +34,7 @@ module RailsBestPractices::Core
       let(:except_method) { 'BlogPost#*' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql true
+        expect(described_class.matches(method, except_method)).to eql true
       end
     end
 
@@ -42,7 +42,7 @@ module RailsBestPractices::Core
       let(:except_method) { 'User#*' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql false
+        expect(described_class.matches(method, except_method)).to eql false
       end
     end
 
@@ -50,7 +50,7 @@ module RailsBestPractices::Core
       let(:except_method) { 'BlogPost#approve' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql true
+        expect(described_class.matches(method, except_method)).to eql true
       end
     end
 
@@ -58,7 +58,7 @@ module RailsBestPractices::Core
       let(:except_method) { 'User#disapprove' }
 
       it 'matches' do
-        expect(Check::Exceptable.matches(method, except_method)).to eql false
+        expect(described_class.matches(method, except_method)).to eql false
       end
     end
   end

data/spec/rails_best_practices/core/gems_spec.rb

@@ -4,15 +4,15 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Gems do
-    it { should be_a_kind_of Array }
+    it { is_expected.to be_a_kind_of Array }
 
-    let(:gems) { Gems.new }
+    let(:gems) { described_class.new }
     before do
       gems << Gem.new('rails', '4.0.0')
       gems << Gem.new('mysql2', '0.2.0')
     end
 
-    context '#has_gem?' do
+    describe '#has_gem?' do
       it 'has rails gem' do
         expect(gems).to be_has_gem 'rails'
       end
@@ -22,7 +22,7 @@ module RailsBestPractices::Core
       end
     end
 
-    context '#gem_version' do
+    describe '#gem_version' do
       it 'gets rails version' do
         expect(gems.gem_version('rails')).to eq '4.0.0'
       end

data/spec/rails_best_practices/core/helpers_spec.rb

@@ -4,6 +4,6 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Helpers do
-    it { should be_a_kind_of Modules }
+    it { is_expected.to be_a_kind_of Modules }
   end
 end

data/spec/rails_best_practices/core/klasses_spec.rb

@@ -4,10 +4,10 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Klasses do
-    it { should be_a_kind_of Array }
+    it { is_expected.to be_a_kind_of Array }
 
     context 'Klass' do
-      context '#class_name' do
+      describe '#class_name' do
         it 'gets class name without module' do
           klass = Klass.new('BlogPost', 'Post', [])
           expect(klass.class_name).to eq('BlogPost')
@@ -19,7 +19,7 @@ module RailsBestPractices::Core
         end
       end
 
-      context '#extend_class_name' do
+      describe '#extend_class_name' do
         it 'gets extend class name without module' do
           klass = Klass.new('BlogPost', 'Post', [])
           expect(klass.extend_class_name).to eq('Post')

data/spec/rails_best_practices/core/mailers_spec.rb

@@ -4,6 +4,6 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Mailers do
-    it { should be_a_kind_of Klasses }
+    it { is_expected.to be_a_kind_of Klasses }
   end
 end

data/spec/rails_best_practices/core/methods_spec.rb

@@ -4,9 +4,9 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Methods do
-    let(:methods) { Methods.new }
+    let(:methods) { described_class.new }
 
-    before :each do
+    before do
       methods.add_method('Post', 'create')
       methods.add_method('Post', 'destroy')
       methods.add_method('Post', 'save_or_update', {}, 'protected')
@@ -14,7 +14,7 @@ module RailsBestPractices::Core
       methods.add_method('Comment', 'create')
     end
 
-    it 'should get_methods' do
+    it 'get_methodses' do
       expect(methods.get_methods('Post').map(&:method_name)).to eq(%w[create destroy save_or_update find_by_sql])
       expect(methods.get_methods('Post', 'public').map(&:method_name)).to eq(%w[create destroy])
       expect(methods.get_methods('Post', 'protected').map(&:method_name)).to eq(['save_or_update'])
@@ -22,7 +22,7 @@ module RailsBestPractices::Core
       expect(methods.get_methods('Comment').map(&:method_name)).to eq(['create'])
     end
 
-    it 'should has_method?' do
+    it 'has_method?s' do
       expect(methods).to be_has_method('Post', 'create', 'public')
       expect(methods).to be_has_method('Post', 'destroy', 'public')
       expect(methods).not_to be_has_method('Post', 'save_or_update', 'public')
@@ -32,12 +32,12 @@ module RailsBestPractices::Core
       expect(methods).not_to be_has_method('Comment', 'destroy')
     end
 
-    it 'should get_method' do
+    it 'get_methods' do
       expect(methods.get_method('Post', 'create', 'public')).not_to be_nil
       expect(methods.get_method('Post', 'create', 'protected')).to be_nil
     end
 
-    it 'should get_all_unused_methods' do
+    it 'get_all_unused_methodses' do
       methods.get_method('Comment', 'create').mark_used
       expect(methods.get_all_unused_methods('public').map(&:method_name)).to eq(%w[create destroy])
       expect(methods.get_all_unused_methods('protected').map(&:method_name)).to eq(['save_or_update'])

data/spec/rails_best_practices/core/model_associations_spec.rb

@@ -4,20 +4,24 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe ModelAssociations do
-    let(:model_associations) { ModelAssociations.new }
+    let(:model_associations) { described_class.new }
 
-    before :each do
+    before do
       model_associations.add_association('Project', 'project_manager', 'belongs_to')
       model_associations.add_association('Project', 'people', 'has_many', 'Person')
     end
 
-    it 'should get model associations' do
-      expect(model_associations.get_association('Project', 'project_manager')).to eq('meta' => 'belongs_to', 'class_name' => 'ProjectManager')
-      expect(model_associations.get_association('Project', 'people')).to eq('meta' => 'has_many', 'class_name' => 'Person')
+    it 'gets model associations' do
+      expect(model_associations.get_association('Project', 'project_manager')).to eq(
+        'meta' => 'belongs_to', 'class_name' => 'ProjectManager'
+      )
+      expect(model_associations.get_association('Project', 'people')).to eq(
+        'meta' => 'has_many', 'class_name' => 'Person'
+      )
       expect(model_associations.get_association('Project', 'unknown')).to be_nil
     end
 
-    it 'should check is model associatiosn' do
+    it 'checks is model associatiosn' do
       expect(model_associations.is_association?('Project', 'project_manager')).to eq true
       expect(model_associations.is_association?('Project', 'people')).to eq true
       expect(model_associations.is_association?('Project', 'unknown')).to eq false

data/spec/rails_best_practices/core/model_attributes_spec.rb

@@ -4,20 +4,20 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe ModelAttributes do
-    let(:model_attributes) { ModelAttributes.new }
+    let(:model_attributes) { described_class.new }
 
-    before :each do
+    before do
       model_attributes.add_attribute('Post', 'title', :string)
       model_attributes.add_attribute('Post', 'user_id', :integer)
     end
 
-    it 'should get model attributes' do
+    it 'gets model attributes' do
       expect(model_attributes.get_attribute_type('Post', 'title')).to eq(:string)
       expect(model_attributes.get_attribute_type('Post', 'user_id')).to eq(:integer)
       expect(model_attributes.get_attribute_type('Post', 'unknonw')).to be_nil
     end
 
-    it 'should check is model attributes' do
+    it 'checks is model attributes' do
       expect(model_attributes.is_attribute?('Post', 'title')).to be true
       expect(model_attributes.is_attribute?('Post', 'user_id')).to be true
       expect(model_attributes.is_attribute?('Post', 'unknonw')).to be false

data/spec/rails_best_practices/core/models_spec.rb

@@ -4,6 +4,6 @@ require 'spec_helper'
 
 module RailsBestPractices::Core
   describe Models do
-    it { should be_a_kind_of Klasses }
+    it { is_expected.to be_a_kind_of Klasses }
   end
 end